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 :	save comment
<Esc> :	cancel edit

Recent Issues

« Newest ‹ Newer 401 - 420 Older ›

	Subject	Owner	Reviewers	Comments	Last updated
518002	Add API to find out if there is crash reports to send to the server. (Closed)	qsr	Mark Mentovai	2	12 years, 9 months
523002	Remove a call to new[] from WriteDSODebugStream (Closed)	Ted Mielczarek	Mark Mentovai, Lei Zhang (chromium)		12 years, 9 months
385001	Include the compilation directory for FILE entries, making them absolute instead of relative	Ryan Sleevi	Mark Mentovai, Ted Mielczarek, eroman, jimb, Marc-Antoine Ruel	20	12 years, 9 months
516003	Fix remaining processor/scoped_ptr.h references for reals. (Closed)	Lei Zhang (chromium)	Ted Mielczarek		12 years, 9 months
516002	Fix remaining processor/scoped_ptr.h references after r1096. (Closed)	Lei Zhang (chromium)	Ted Mielczarek		12 years, 9 months
512002	Make DumpSymbols methods take a vector of debug_dirs (Closed)	Ted Mielczarek	jimb, Mark Mentovai, Lei Zhang (chromium)	2	12 years, 9 months
515002	Remove <(library) usage from gyp files. (Closed)	Lei Zhang (chromium)	tfarina		12 years, 9 months
510002	Provide a ReadSymbolData function from dump_symbols.cc (Closed)	Ted Mielczarek	jimb, Mark Mentovai, Lei Zhang (chromium)	1	12 years, 9 months
511002	Don't make MmapWrapper's destructor assert (Closed)	Ted Mielczarek	Mark Mentovai		12 years, 9 months
509003	Fix some warnings in stack_frame_symbolizer (Closed)	Ted Mielczarek	siyangxie		12 years, 9 months
509002	Move scoped_ptr.h to common (Closed)	Ted Mielczarek	Mark Mentovai	1	12 years, 9 months
505002	Updating BreakpadController to allow start and stop being nop. (Closed)	qsr	Mark Mentovai, glider, qsr1	2	12 years, 9 months
508002	DWARF reader doesn't support length for high_pc	mattdr	jimb	4	12 years, 10 months
497002	minidump_stackwalk: Show argument registers for ARM stack traces	Michael Krebs	jimb, Ted Mielczarek		12 years, 10 months
506002	Fix typo. (Closed)	qsr	Mark Mentovai		12 years, 10 months
502003	Remove dead code	rafael.espindola	Mark Mentovai, jimb	1	12 years, 10 months
504002	Fix minidump size limit used for MinidumpSizeLimit unittest (Closed)	Michael Krebs	Ivan Penkov		12 years, 11 months
503002	Mac/iOS Breakpad client: exceptions off, no libstdc++ (Closed)	Mark Mentovai	qsr		12 years, 11 months
500002	Google-breakpad processor doesn't recognize and use context records generated by ::RtlCaptureContext (Closed)	Ivan Penkov	Mark Mentovai, Jess, Ted Mielczarek	9	12 years, 11 months
502002	mac: Explicitly include unistd.h for getpagesize().	thakis	Mark Mentovai	1	12 years, 11 months

« Newest ‹ Newer 401 - 420 Older ›