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

Issue 667823 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Nov 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug
Team-Security-UX



Sign in to add a comment

Links on SB & SSL interstitials not correct

Project Member Reported by emilyschechter@chromium.org, Nov 22 2016

Issue description

On both the Safe Browsing and SSL interstitials:

"Automatically report" should link to the whitepaper (https://www.google.com/chrome/browser/privacy/whitepaper.html?hl=en-US#malware)
"Privacy policy" should link to the to the privacy policy (https://www.google.com/chrome/browser/privacy/)

This does not appear to be working correctly on my favorite SB test site ianfette.org, nor on badssl.com test sites.

On the new (Scout) interstitials, see  Issue 667385  -- the whitepaper should be linked to "system information and page content" and the privacy policy should be linked to "Privacy policy" still.
 

Comment 1 Deleted

Privacy policy should link to the SB section of privacy policy: https://www.google.com/chrome/browser/privacy/#safe-browsing-policies
Status: Started (was: Assigned)
Project Member

Comment 4 by bugdroid1@chromium.org, Nov 25 2016

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

commit 762fb54cd325ae2f557a3a8f9199daa480f8f8b7
Author: jialiul <jialiul@chromium.org>
Date: Fri Nov 25 20:43:12 2016

Fix privacy whitepaper link on safe browsing blocking page and captive
portal bloking page (whitepaper links on other interstitials are fine).
Link privacy whitepaper to "system information and page content" instead
of "Automatically send" on Scout opt-in string.

BUG= 667823 , 667385 

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

[modify] https://crrev.com/762fb54cd325ae2f557a3a8f9199daa480f8f8b7/chrome/app/generated_resources.grd
[modify] https://crrev.com/762fb54cd325ae2f557a3a8f9199daa480f8f8b7/chrome/browser/safe_browsing/safe_browsing_blocking_page.cc
[modify] https://crrev.com/762fb54cd325ae2f557a3a8f9199daa480f8f8b7/chrome/browser/ssl/captive_portal_blocking_page.cc

Status: Fixed (was: Started)
Labels: Merge-Request-56
Request merge into M56 based on UX review. 

Comment 7 by dimu@chromium.org, Nov 30 2016

Labels: -Merge-Request-56 Merge-Approved-56 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M56 (branch: 2924)
Project Member

Comment 8 by bugdroid1@chromium.org, Nov 30 2016

Labels: -merge-approved-56 merge-merged-2924
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ff1ef44692a1b71e31d366790ad8cd3abf3615ca

commit ff1ef44692a1b71e31d366790ad8cd3abf3615ca
Author: Jialiu Lin <jialiul@chromium.org>
Date: Wed Nov 30 22:52:48 2016

Fix privacy whitepaper link on safe browsing blocking page and captive
portal bloking page (whitepaper links on other interstitials are fine).
Link privacy whitepaper to "system information and page content" instead
of "Automatically send" on Scout opt-in string.

BUG= 667823 , 667385 

Review-Url: https://codereview.chromium.org/2526543002
Cr-Commit-Position: refs/heads/master@{#434546}
(cherry picked from commit 762fb54cd325ae2f557a3a8f9199daa480f8f8b7)

Review URL: https://codereview.chromium.org/2540243003 .

Cr-Commit-Position: refs/branch-heads/2924@{#219}
Cr-Branched-From: 3a87aecc31cd1ffe751dd72c04e5a96a1fc8108a-refs/heads/master@{#433059}

[modify] https://crrev.com/ff1ef44692a1b71e31d366790ad8cd3abf3615ca/chrome/app/generated_resources.grd
[modify] https://crrev.com/ff1ef44692a1b71e31d366790ad8cd3abf3615ca/chrome/browser/safe_browsing/safe_browsing_blocking_page.cc
[modify] https://crrev.com/ff1ef44692a1b71e31d366790ad8cd3abf3615ca/chrome/browser/ssl/captive_portal_blocking_page.cc

Sign in to add a comment