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

Issue 7070001: Change "Save" to "Keep" in dangerous download confirmation prompt. (Closed)

Created:
9 years, 7 months ago by mattm
Modified:
9 years, 6 months ago
CC:
chromium-reviews, pam+watch_chromium.org
Visibility:
Public.

Description

Change "Save" to "Keep" in dangerous download confirmation prompt. BUG=80338 TEST=manual Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=87435

Patch Set 1 #

Patch Set 2 : update prompt strings too #

Patch Set 3 : update from tech writer recommendations #

Patch Set 4 : update again #

Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -9 lines) Patch
M chrome/app/generated_resources.grd View 1 2 3 2 chunks +4 lines, -4 lines 0 comments Download
M chrome/browser/ui/cocoa/download/download_item_controller.mm View 2 chunks +3 lines, -2 lines 0 comments Download
M chrome/browser/ui/gtk/download/download_item_gtk.cc View 1 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/ui/views/download/download_item_view.cc View 1 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/ui/webui/downloads_ui.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 8 (0 generated)
mattm
9 years, 7 months ago (2011-05-26 01:08:28 UTC) #1
Peter Kasting
It seems like the associated prompting strings also need some tweaking. "Keep" is an odd ...
9 years, 7 months ago (2011-05-26 01:21:31 UTC) #2
mattm
On 2011/05/26 01:21:31, Peter Kasting wrote: > It seems like the associated prompting strings also ...
9 years, 7 months ago (2011-05-26 23:27:57 UTC) #3
Peter Kasting
LGTM. Probably want to double-check with a tech writer.
9 years, 7 months ago (2011-05-26 23:30:06 UTC) #4
mattm
On 2011/05/26 23:30:06, Peter Kasting wrote: > LGTM. Probably want to double-check with a tech ...
9 years, 7 months ago (2011-05-28 00:29:19 UTC) #5
Peter Kasting
On 2011/05/28 00:29:19, mattm wrote: > On 2011/05/26 23:30:06, Peter Kasting wrote: > > LGTM. ...
9 years, 7 months ago (2011-05-28 01:14:57 UTC) #6
mattm
On 2011/05/28 01:14:57, Peter Kasting wrote: > On 2011/05/28 00:29:19, mattm wrote: > > On ...
9 years, 6 months ago (2011-06-01 04:45:23 UTC) #7
commit-bot: I haz the power
9 years, 6 months ago (2011-06-01 05:35:13 UTC) #8
Change committed as 87435

Powered by Google App Engine
This is Rietveld 408576698