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
2492183002	Add struct_traits and typemap for blink::WebGamepad	ke.he	leonhsl, yzshen1, blundell	1	0 minutes
2499843002	Changed fprintf to LOG in GraphicsLayer and PaintLayer logging functions.	bokan	pdr.		0 minutes
2492773002	[sensors][CrOS/Linux] Implementation of motion sensors for CrOS/Linux platforms	maksims	Reilly Grant, shalamov, Mikhail	32	0 minutes
2465253011	Fastpath some spread-call desugaring.	petermarshall	Benedikt Meurer, caitp1, caitp, Yang	10	0 minutes
2463823002	[Bookmark suggestions] Record last visit dates for bookmarks on desktop	jkrcal	Marc Treib, Avi, tschumann, markusheintz_		0 minutes
2486763002	Add support to audioproc_f for running the residual echo detector and producing an echo likelihood …	ivoc	hlundin-webrtc, peah-webrtc	18	0 minutes
2501573002	Update V8 to version 5.6.308.	v8-autoroll	Hablich, Dan Ehrenberg, vogelheim, machenbach (slow)		0 minutes
2490643002	[counters] Implement off-isolate RuntimeCallStats for the Preparser	Camillo Bruni	fmeawad, vogelheim	10	0 minutes
2491293002	Make FlexFEC packets paceable through RTPSender.	brandtr	danilchap, stefan-webrtc (holmer)	30	1 minute
2462513003	PlzNavigate: Add missing Upgrade-Insecure-Requests header.	arthursonzogni	clamy, Mike West	2	2 minutes
2485353004	Let the subresource_filter::RulesetService know about browser start-up.	engedy	waffles, Lei Zhang (slow), pkalinnikov	3	2 minutes
2486923008	Allow url bar hiding/showing when scrolling just the visual viewport.	bokan	aelias		2 minutes
2485783002	PlzNavigate: send searchable form data to the browser side on form submission.	yzshen1	Tom Sepez, clamy, sky, yhirano	15	2 minutes
2501533002	remove unused LayoutUnit::ceilToFloat function	inlinechan	Ian Vollick		4 minutes
2496133003	[test] Move stray variables into namespaces.	Franzi			4 minutes
2453143004	Make FakeChromeUserManager a subclass of ChromeUserManager	yoshiki			5 minutes
2487483004	Only treat possible eval calls going through 'with' as special.	rmcilroy	Toon Verwaest, Michael Starzinger	5	7 minutes
2422643002	Windows install_static refactor.	grt (UTC plus 1)	Sigurður Ásgeirsson, bradnelson, penny, Nico, robertshield_slow_reviews, Derek Schuff, Paweł Hajdan Jr., ananta	65	7 minutes
2451853002	Uniform and precise source positions for inlining	Tobias Tebbi	Benedikt Meurer, Michael Starzinger, alph, Yang, vogelheim	77	8 minutes
2502603002	Remove sorting and deduplication for exceptions in a site-list if it is not an ALL_SITES site-list.	dullweber	msramek		10 minutes

1 - 20 Older ›