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 Closed Issues

1 - 20 Older ›

	Subject	Owner	Reviewers	Comments	Last updated
3011383002	[Telemetry] Explicitly disable ModalPermissionPrompts on Android (Closed)	perezju	nednguyen, Timothy Loh		37 minutes
3008373002	Only return stats for the most recent unsignaled audio stream. (Closed)	Taylor Brandstetter	the sun	4	4 hours, 18 minutes
3016583002	Remove Windows MSR power monitoring code (Closed)	charliea	nednguyen		5 hours, 49 minutes
3011963002	Remove backwards compatibilty header for Optional (Closed)	kwiberg-webrtc	mbonadei		5 hours, 56 minutes
3018583002	Revert of [devil] Update chromium build target deps. (Closed)	jbudorick	rnephew (Reviews Here), aluo, aluo1		6 hours, 6 minutes
3002623002	[WIP] Port media metrics to TBMv2 (Closed)	johnchen			7 hours, 1 minute
2804633003	Add base::FeatureParam<> struct (Closed)	sfiera	gab, Alexei Svitkine (slow)	22	8 hours, 38 minutes
2573573004	[stubs] Enable graph verification for builtins. (Closed)	Igor Sheludko	epertoso, Nico	2	9 hours, 59 minutes
2481183002	Generate ADMX template for Chrome OS policies (Closed)	ljusten (tachyonic)	pastarmovj		10 hours, 12 minutes
3015643002	Remove Mac powermetrics power monitoring code (Closed)	charliea	nednguyen		10 hours, 57 minutes
3015633002	[Telemetry] Include debugging info when foreground tab is missing (Closed)	perezju	nednguyen		11 hours, 58 minutes
2448983002	offsetParent() fails to consider continuations when searching for the parent. (Closed)	Karl Øygard			12 hours, 10 minutes
3012153002	Add leakDetectionMetric for tracing (Closed)	yuzuchan	haraken, keishi, benjhayden	55	18 hours, 7 minutes
2534413004	Made varied number of cells in each row based on row's requirement. (Closed)	a.suchit	a.suchit2, eae, dgrogan, mstensho (USE GERRIT), suchit.agrawal, Julien - ping for review	20	20 hours, 22 minutes
3013223002	[pinpoint] Narrower chart bounds. (Closed)	dtu	shatch		1 day, 2 hours
3011133002	Remove the support of fallback from DTLS to SDES. (Closed)	Zhi Huang	Taylor Brandstetter	2	1 day, 4 hours
3018563002	Remove Linux MSR power monitoring code (Closed)	charliea	nednguyen		1 day, 4 hours
3012363002	Log the detailed results for each metric for diagnosing and debugging purpose. (Closed)	Lei Lei	benjhayden		1 day, 4 hours
3013743002	Dashboard - Populate bug_id with uri parameter if available. (Closed)	shatch	sullivan		1 day, 5 hours
3012413002	[pinpoint] Increase default repeat count from 12 to 15. (Closed)	dtu	perezju, shatch, sullivan		1 day, 5 hours

1 - 20 Older ›