Issue 9303006: Convert use of int ms to TimeDelta in scattered chromium files.

Keyboard Shortcuts

	File
u :	up to issue
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

	Issue
u :	up to list of issues
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

Issue 9303006: Convert use of int ms to TimeDelta in scattered chromium files. (Closed)

Created:
8 years, 10 months ago by Ted Vessenes

Modified:
8 years, 10 months ago

Reviewers:
brettw

CC:
chromium-reviews, jam, darin-cc_chromium.org, stuartmorgan+watch_chromium.org

Base URL:
http://git.chromium.org/chromium/src.git@master

Visibility:
Public.

More Reviews

Description

Convert use of int ms to TimeDelta in scattered chromium files. It's not clear to me who needs to review the change to third_party/leveldatabase/env_chromium.cc, or even if it's handled through the standard CL interface for the rest of Chromium. But my best guess was that you could either approve it or tell me who to contact. Let me know if I need to go about this patch submission a different way. R=brettw@chromium.org BUG=108171 TEST= Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=119771

Patch Set 1 #

Created: 8 years, 10 months ago

Download [raw] [tar.bz2]

	Unified diffs	Side-by-side diffs	Stats (+5 lines, -4 lines)			Patch
M	chrome/browser/plugin_download_helper_unittest.cc	View	1 chunk	+1 line, -1 line	0 comments	Download
M	chrome/common/service_process_util_posix.cc	View	1 chunk	+1 line, -1 line	0 comments	Download
M	ppapi/proxy/ppb_message_loop_proxy.cc	View	1 chunk	+2 lines, -1 line	0 comments	Download
M	third_party/leveldatabase/env_chromium.cc	View	1 chunk	+1 line, -1 line	0 comments	Download

Messages

Total messages: 4 (0 generated)

Expand Messages | Collapse Messages

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/tedvessenes@gmail.com/9303006/1

8 years, 10 months ago (2012-01-30 21:24:29 UTC) #3

Change committed as 119771

Expand Messages | Collapse Messages