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
2940443003	ProfileIOData: Clear cert_transparency_verifier_'s observer in ~ProfileIOData. (Closed)	mmenke	Eran Messeri		7 minutes
2935033002	Roll WebRTC 18538:18559 (15 commits) (Closed)	hbos_chromium			12 minutes
2936763004	Revert of chromium.webkit: Run Layout Tests on Swarming (on Linux bots). (Closed)	mithro	Dirk Pranke, mcgreevy		12 minutes
2939553003	Update V8 to version 6.1.152. (Closed)	v8-autoroll	kozy, Michael Hablich (OOO), vogelheim, Michael Achenbach		16 minutes
2930303002	[ObjC ARC] Converts ios/chrome/browser/signin:test_support to ARC. (Closed)	marq (ping after 24h)	stkhapugin	1	19 minutes
2939433002	[ObjC ARC] Converts ios/chrome/browser/ui/sync:sync to ARC. (Closed)	marq (ping after 24h)	stkhapugin, msarda		28 minutes
2930273003	[ObjC ARC] Converts components/signin/ios/browser:unit_tests to ARC. (Closed)	marq (ping after 24h)	stkhapugin, msarda		29 minutes
2935733002	[ObjC ARC] Converts ios/chrome/browser/signin:unit_tests to ARC. (Closed)	marq (ping after 24h)	stkhapugin, msarda		29 minutes
2939443003	Restore WebRtcVideoFrameAdapter native behavior (Closed)	magjed_chromium	emircan		32 minutes
2932353002	[ObjC ARC] Converts components/dom_distiller/ios:ios to ARC. (Closed)	marq (ping after 24h)	stkhapugin, noyau (Ping after 24h)	2	33 minutes
2913143002	Delete old include file webrtc/video_frame.h. (Closed)	nisse-webrtc	magjed_webrtc, kwiberg-webrtc		38 minutes
2918403009	[Dice] Parse the Dice response header (Closed)	droger	Randy Smith (Not in Mondays), msarda	31	40 minutes
2933903002	Make the narrow version of the Android Incognito NTP follow the spec (Closed)	msramek (slow)	Michael van Ouwerkerk	6	45 minutes
2773413002	Add SafeMin(), SafeMax(), and SafeClamp(), which accept args of different types (Closed)	kwiberg-webrtc			47 minutes
2937693002	[ios] Disables WindowLocationTest.WindowLocationSetToDOMString on device. (Closed)	rohitrao (ping after 24h)	mrefaat, *sdefresne		53 minutes
2873873003	Expose connection subtype from NetworkChangeNotifier (Closed)	tbansal1		2	1 hour
2926713007	Delete unused method FilesystemInterface::GetFileTime. (Closed)	nisse-webrtc	Taylor Brandstetter		1 hour, 6 minutes
2935043002	Update kernel/visitor_test expectations (Closed)	Johnni Winther	ahe		1 hour, 7 minutes
2929873002	Shifting layer position for sticky element to avoid passing unnessary variable to cc (Closed)	yigu	flackr, smcgruer_OOO, chrishtr, trchen	22	1 hour, 8 minutes
2741713002	Web MIDI: implement TaskService (Closed)	Takashi Toyoshima	yhirano, tzik	24	1 hour, 12 minutes

1 - 20 Older ›