Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(166)

Issue 220025: linux: port some of the download manager unit test (Closed)

Created:
11 years, 3 months ago by Evan Martin
Modified:
9 years, 7 months ago
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

linux: port some of the download manager unit test Mostly lots of #ifdefs tracking platform behaviors. I tried to make the logic correct for Mac, too, but I didn't enable Mac here. BUG=21632 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=27260

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+137 lines, -28 lines) Patch
M chrome/browser/download/download_manager_unittest.cc View 13 chunks +137 lines, -23 lines 0 comments Download
M chrome/chrome.gyp View 1 chunk +0 lines, -5 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Evan Martin
11 years, 3 months ago (2009-09-23 22:49:57 UTC) #1
Evan Martin
+abarth for security thoughts +estade fyi To answer Paul's question for abarth's benefit: this is ...
11 years, 3 months ago (2009-09-23 23:02:28 UTC) #2
abarth-chromium
This is all fine from a security point of view. I'll let others comment on ...
11 years, 3 months ago (2009-09-24 00:15:49 UTC) #3
Paul Godavari
11 years, 3 months ago (2009-09-24 01:29:50 UTC) #4
LGTM (add BUG=21632)

Powered by Google App Engine
This is Rietveld 408576698