Code Review

Keyboard Shortcuts

	File
u :	up to issue
m :	publish + mail comments
M :	edit review message
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line
<Enter> :	respond to / edit current comment
d :	mark current comment as done

	Issue
u :	up to list of issues
m :	publish + mail comments
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue
# :	close issue

	Comment/message editing
<Ctrl> + s or <Ctrl> + Enter :	save comment
<Esc> :	cancel edit

Recent Open Issues

1 - 20 Older ›

	Subject	Owner	Reviewers	Comments	Last updated
2975693002	Add AwVariationsSeedFetchService and refactory VariationsSeedFetcher	yiyuny	paulmiller, sgurun, gsennton, Alexei Svitkine (slow)	94	1 minute
2944633002	Use SkPngEncoder in gfx jpeg_codec	liyuqian	Elliot Glaysher, scroggo, dcheng, msarett, msarett1, scroggo_chromium	68	2 minutes
2968003003	Stop silently accepting unsupported flags in test binaries	oprypin_webrtc			3 minutes
2973213002	Add documentation on authoring metrics.	tdresser (OOO until July 17)	sullivan	1	4 minutes
2972123002	Click to Open PDF: Open PDF once View PDF is clicked.	amberwon	Lei Zhang (OOO), tommycli, dcheng, lfg	40	5 minutes
2971783002	Skeleton for showing "Show all saved passwords row" for Linux/CrOs/Windows platforms	melandory	vasilii, Evan Stade, boliu, Moe	73	10 minutes
2978593002	Add validateOrigin() to the support library.	Benoit L	Yusuf	4	10 minutes
2970653004	Reimplemeted "Test and fix for huge bwe drop after alr state"	tschumi	holmer, terelius, ossu, stefan-webrtc, philipel	24	13 minutes
2955843002	[LayoutNG] Make floats push line boxes down if no break opportunities can fit	kojii	ikilpatrick		22 minutes
2742563002	Add initial code for ChromeOS MidiManager implementation	pmalani	Takashi Toyoshima, Adam Goode	56	26 minutes
2949263003	Remove call to GetBlockingPool in RLZ	Roger Tawa OOO till Jul 10th	gab (in-out til july17)		39 minutes
2841483003	Added informal generic method syntax and generic function type specs.	eernst	Lasse Reichstein Nielsen, floitsch	110	41 minutes
2983483002	Check function type equivalence	Johnni Winther	Siggi Cherem (dart-lang)		42 minutes
2981583002	Reland of Injectable Obj-C video codecs	andersc	magjed_webrtc, jtt_webrtc, tkchin_webrtc		43 minutes
2565323003	Move gif image decoder to SkCodec	cblume	ccameron, scroggo, msarett, msarett1, f(malita), scroggo_chromium, Noel Gordon	397	45 minutes
2975133003	[DO NOT SUBMIT] Test downstream ios/web tests passing	stkhapugin			46 minutes
2983473002	Android Tango depth camera capture support.	aleksandar.stojiljkovic			48 minutes
2951093002	cc: Implement eviction from SW image decode cache.	sohan	vmpstr, ericrk	21	50 minutes
2980753002	Remove MOCK_METHODs from FakeChromeIdentityService.	stkhapugin	bzanotti, msarda		1 hour, 1 minute
2977033002	Mixed Licenses Enrollment	Denis Kuznetsov (DE-MUC)	emaxx		1 hour, 9 minutes

1 - 20 Older ›