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

Issue 22612003: Warn when extensions suggest conflicting download filenames (Closed)

Created:
7 years, 4 months ago by benjhayden
Modified:
7 years, 4 months ago
CC:
chromium-reviews, dbeam+watch-options_chromium.org, arv+watch_chromium.org
Visibility:
Public.

Description

Warn when extensions suggest conflicting download filenames R=asargent@chromium.org, scheib@chromium.org BUG=269712 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=216882

Patch Set 1 : @r216682 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+195 lines, -7 lines) Patch
M chrome/browser/extensions/api/downloads/downloads_api.h View 2 chunks +16 lines, -0 lines 0 comments Download
M chrome/browser/extensions/api/downloads/downloads_api.cc View 14 chunks +78 lines, -7 lines 0 comments Download
M chrome/browser/extensions/api/downloads/downloads_api_browsertest.cc View 1 chunk +70 lines, -0 lines 0 comments Download
M chrome/browser/extensions/extension_warning_set.h View 3 chunks +12 lines, -0 lines 0 comments Download
M chrome/browser/extensions/extension_warning_set.cc View 2 chunks +19 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
benjhayden
PTAL
7 years, 4 months ago (2013-08-09 19:53:54 UTC) #1
asargent_no_longer_on_chrome
lgtm
7 years, 4 months ago (2013-08-09 20:00:47 UTC) #2
scheib
lgtm
7 years, 4 months ago (2013-08-09 20:25:33 UTC) #3
benjhayden
7 years, 4 months ago (2013-08-10 18:14:59 UTC) #4
Message was sent while issue was closed.
Committed patchset #1 manually as r216882 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698