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 421 - 440 Older ›

	Subject	Owner	Reviewers	Comments	Last updated
501002	Allow StackwalkerARM to scan much farther to find caller of context frame (Closed)	Ted Mielczarek	jimb, Michael Krebs	2	13 years, 8 months
499003	Add assertion on initialization sequence. (Closed)	qsr	Mark Mentovai		13 years, 8 months
499002	Fix unused variable warning in optimized build (fix proveded by Matthew Riley) (Closed)	Ivan Penkov	Ted Mielczarek, Mark Mentovai, Jess		13 years, 8 months
488002	Cleaning up google-breakpad source code of signed-unsigned comparison warnings (Closed)	Ivan Penkov	Ted Mielczarek, Mark Mentovai	6	13 years, 8 months
487002	Add optional file size limit for minidumps (Closed)	Michael Krebs	Mark Mentovai, Ted Mielczarek, Ben Chan, Ivan Penkov	2	13 years, 8 months
492002	Fix assertion failure in WriteMappings() for zero modules (Closed)	Michael Krebs	Ted Mielczarek		13 years, 8 months
496002	Fix typo from r1079. (Closed)	Lei Zhang (chromium)	digit		13 years, 8 months
495002	Linux: Avoid truncating the vendor id. (Closed)	Lei Zhang (chromium)	digit		13 years, 9 months
486002	Minor cleanup in src/client/linux/minidump_writer/minidump_writer_unittest.cc. (Closed)	Lei Zhang (chromium)	Ted Mielczarek	2	13 years, 9 months
490002	Building fails with gcc 4.1 due to unsupported -Werror= switch	davi.arnaut	Mark Mentovai, Ted Mielczarek	4	13 years, 9 months
413002	Allow processing dumps with missing stack memory for some threads (Closed)	Ted Mielczarek	Mark Mentovai, Michael Krebs	1	13 years, 9 months
494002	Wrong %ebp after skipping a frame for which the instruction pointer is not in a known module. (Closed)	Ivan Penkov	Ted Mielczarek, Mark Mentovai	2	13 years, 9 months
491002	Not checking properly the return value of SuspendThread on Windows (Closed)	Ivan Penkov	ted mielczarek; siyangxie, siyangxie		13 years, 9 months
493002	Fix a compile warning in stack_frame_symbolizer.cc (Closed)	SiyangXie	Ivan Penkov		13 years, 9 months
489002	Fixing Windows client unit tests. They were broken in r1034 due to gMock and gTest upgrade. (Closed)	Ivan Penkov	Ted Mielczarek, Mark Mentovai	12	13 years, 9 months
483002	Fix some compiler errors in exception_handler_unittest by casting like crazy (Closed)	Ted Mielczarek	digit, jimb	1	13 years, 9 months
485002	Merge r1065: Don't bail if a thread's stack pointer is invalid (Closed)	Lei Zhang (chromium)	Michael Krebs, Mark Mentovai		13 years, 9 months
484002	Check-in stack_frame_symbolizer.[h\|cc] that were missing in r1068.	SiyangXie	ivanpe.google, Ted Mielczarek, Ivan Penkov		13 years, 10 months
459002	Refactor the logic of resolving source line info into helper class (Closed)	SiyangXie	Ivan Penkov, Ted Mielczarek, Mark Mentovai	78	13 years, 10 months
481002	Make Linux signal handler more robust. (Closed)	digit	Ted Mielczarek, Mark Mentovai, cjhopman	10	13 years, 10 months

« Newest ‹ Newer 421 - 440 Older ›