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

‹ Newer 21 - 40 Older ›

	Subject	Owner	Reviewers	Comments	Last updated
2954873002	Add DVLOG_IS_ON() in preparation for adding dvlog_always_on in chromium project.	hiroh	haraken, Yuki		47 minutes
2954893002	Enter a description of the change.	mcgreevy			48 minutes
2941883003	[ServiceWorker] Fetch event should return integrity value	xiaofengzhang	shimazu	12	53 minutes
2957543002	Implement ConsumeShorthandViaLonghand to parse shorthand by calling longhand ParseSingleValue	Jia	meade_UTC10, Bugs Nash, rjwright		56 minutes
2947383002	Put attribute before function name instead of after, as required by GCC	kwiberg-webrtc	kjellander_webrtc		1 hour
2945433003	Add ClampedNumeric templates	jschuh			1 hour, 2 minutes
2946833003	Update list of Feature Policy features	raymes	iclelland, pfeldman	2	1 hour, 7 minutes
2767643002	Enable the Java wrapper to load different .so library.	Zhi Huang	magjed_webrtc, Taylor Brandstetter	4	1 hour, 10 minutes
2940313002	Move PerformanceTests/Editing/page-up-with-many-lines.html to DOM/	yoichio	hayato, yosin_UTC9		1 hour, 12 minutes
2938163002	Store base::Value in ContentSettingPatternSource	tbansal1	raymes	11	1 hour, 13 minutes
2770003002	Set the current context to the function's context when entering to LAP.	Yuki	jochen, Toon Verwaest	11	1 hour, 16 minutes
2955513002	Dynamically load packages for dartdevc tests in test.dart.	Bob Nystrom	vsm, zra, Bill Hesse	9	1 hour, 19 minutes
2956593002	cl format	tapted			1 hour, 19 minutes
2959593002	Update SCT serialization format in Expect-CT reports	estark	mattm		1 hour, 37 minutes
2951353003	Get rid of live granularity behavior from selection	yosin_UTC9	yoichio, Xiaocheng	2	1 hour, 38 minutes
2962453002	Move AX Role predicates into ax_role_properties.h.	dougt			1 hour, 47 minutes
2951703002	[cr-action-menu] Fix anchoring to offscreen elements.	calamity	dpapad	7	1 hour, 47 minutes
2943573002	Make NGInlineItemsBuilder return info of collapsed white spaces	Xiaocheng	yoichio, kojii, yosin_UTC9, eae	26	1 hour, 57 minutes
2955783002	[milo] fix tests on windows.	iannucci	hinoka		2 hours
2962473002	Make Position::LastPositionInNode() to take const Node& instead of Node*	yosin_UTC9			2 hours, 4 minutes

‹ Newer 21 - 40 Older ›