New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 640673 link

Starred by 4 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Sep 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 2
Type: Bug



Sign in to add a comment

Fix strings in download recovery dialogs

Project Member Reported by jialiul@chromium.org, Aug 24 2016

Issue description

Download recovery dialog is the last chance chrome discourages users from landing dangerous files to their file system, thus the information presented here should be accurate and consistent with the download warning previously shown to users.
However, some of the strings in this warning dialog have been mismatched or not accurate for a long time, e.g.  https://crbug.com/607298 ,  https://crbug.com/415377  and etc.

Detailed plan for fixing these issues can be found here:
https://docs.google.com/document/d/1M9AvvXafVRSNquKGhzjAqo3Pl7sSATuEECA78pfkpmA/edit#
 
Cc: jialiul@chromium.org
 Issue 607298  has been merged into this issue.
Cc: f...@chromium.org vakh@chromium.org moheeb@chromium.org
 Issue 415377  has been merged into this issue.

Comment 3 by vakh@chromium.org, Aug 26 2016

Labels: SafeBrowsing-Triaged
Project Member

Comment 4 by bugdroid1@chromium.org, Sep 13 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/f857966ed00a2e7703f98f901d28ae3677592341

commit f857966ed00a2e7703f98f901d28ae3677592341
Author: jialiul <jialiul@chromium.org>
Date: Tue Sep 13 19:43:12 2016

Improve download recovery flow:

(1) refine strings in download recovery dialog
(2) remove all message leads, and keep title and message body to simply
    previously lengthy text in download recovery dialog
(3) refine description strings that used to explain the reason why download
    is blocked in chrome://downloads (text in orange)
(4) If download danger type is DOWNLOAD_DANGER_TYPE_DANGEROUS_FILE, when user
    try to recover a blocked download, directly proceed instead of showing
    download danger prompt.
go/fix_download_recovery  for more detail

BUG= 640673 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2314903003
Cr-Commit-Position: refs/heads/master@{#418340}

[modify] https://crrev.com/f857966ed00a2e7703f98f901d28ae3677592341/chrome/app/chromium_strings.grd
[modify] https://crrev.com/f857966ed00a2e7703f98f901d28ae3677592341/chrome/app/generated_resources.grd
[modify] https://crrev.com/f857966ed00a2e7703f98f901d28ae3677592341/chrome/app/google_chrome_strings.grd
[modify] https://crrev.com/f857966ed00a2e7703f98f901d28ae3677592341/chrome/browser/download/download_danger_prompt.h
[modify] https://crrev.com/f857966ed00a2e7703f98f901d28ae3677592341/chrome/browser/resources/md_downloads/crisper.js
[modify] https://crrev.com/f857966ed00a2e7703f98f901d28ae3677592341/chrome/browser/resources/md_downloads/item.js
[modify] https://crrev.com/f857966ed00a2e7703f98f901d28ae3677592341/chrome/browser/ui/cocoa/download/download_danger_prompt_impl.cc
[modify] https://crrev.com/f857966ed00a2e7703f98f901d28ae3677592341/chrome/browser/ui/views/download/download_danger_prompt_views.cc
[modify] https://crrev.com/f857966ed00a2e7703f98f901d28ae3677592341/chrome/browser/ui/webui/md_downloads/md_downloads_dom_handler.cc
[modify] https://crrev.com/f857966ed00a2e7703f98f901d28ae3677592341/chrome/browser/ui/webui/md_downloads/md_downloads_dom_handler.h
[modify] https://crrev.com/f857966ed00a2e7703f98f901d28ae3677592341/chrome/browser/ui/webui/md_downloads/md_downloads_dom_handler_unittest.cc
[modify] https://crrev.com/f857966ed00a2e7703f98f901d28ae3677592341/chrome/browser/ui/webui/md_downloads/md_downloads_ui.cc

Status: Fixed (was: Started)
To confirm, which Chrome version did this fix go into?
according to git find-releases,  it will be in 55.0.2860.0. So M55. 
Blocking: 645521
Blocking: -645521
Components: -Security>UX
Labels: Team-Security-UX
Security>UX component is deprecated in favor of the Team-Security-UX label

Comment 11 Deleted

Sign in to add a comment