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 Issues

1 - 20 Older ›

	Subject	Owner	Reviewers	Comments	Last updated
2771373003	Add most missing blink files to .gn files	Nico	haraken, tzik		0 minutes
2773283002	media: Simplify CdmHostFile(s)	xhwang_slow			3 minutes
2763803002	BrowserAccessibility should own a AXPlatformNode.	dougt	dmazzoni	1	3 minutes
2776053002	render non-markdown files as text	vapier	Dirk Pranke (slow until 28th)		3 minutes
2774203002	Migrate about:flags messages to const char*	vabr (Chromium)			3 minutes
2762233004	Fix autofill popup controller key press callback registration	vabr (Chromium)	Mathieu, EhsanK	8	9 minutes
2773323002	Make flags resources naming consistent	vabr (Chromium)			10 minutes
2763413010	[DevTools] Remove typed events	dgozman	pfeldman		16 minutes
2774583002	ui: Reduce logging in ui::ClientNativePixmapDmaBuf.	reveman	dnicoara, Daniele Castagna	5	20 minutes
2768143002	[wip] paintopbuffer + paintrecord	enne			24 minutes
2749633004	Track telemetry benchmark cycle time (Closed)	martiniss	nednguyen, sullivan	15	1 hour
2750483007	Only show a beforeunload dialog if a frame has been interacted with.	Avi			1 hour, 1 minute
2778633002	Roll chromium_revision 4f16f0c98f..600bab5603 (459568:459671)	buildbot			1 hour, 4 minutes
2774193002	[Offline pages] Ensuring native init when OBT run though JobScheduler (Closed)	fgorski	nyquist		1 hour, 7 minutes
2777703002	Introduce SelectSettings algorithm for MediaStream video tracks.	Guido Urdaneta			1 hour, 8 minutes
2779453002	[Payments] Return the preferred payment method name to the merchant (Closed)	Mathieu	ಠ_ಠ slow - w3c	2	1 hour, 10 minutes
2774183002	Add a few missing files (including a test) to base/BUILD.gn (Closed)	Nico	Mark Mentovai		1 hour, 13 minutes
2776873002	WIP: Attempt to fix 0 byte file issue.	lazyboy			1 hour, 14 minutes
2775093002	Remove unused ExceptionStatePlaceholder.cpp (Closed)	Nico	haraken		1 hour, 29 minutes
2779483002	Revert of [Chromoting] Send DesktopFrame::capturer_id() to client through WebrtcVideoStream (Closed)	Nico	Sergey Ulanov, Hzj_jie		1 hour, 37 minutes

1 - 20 Older ›