Change security warning links to point to localized help center article

Change security warning links to point to localized help center article

BUG=402934
Committed: https://crrev.com/e5d8ace7ccebe3c2e4c93f9151f9f482c982dc69
Cr-Commit-Position: refs/heads/master@{#292508}

[felt, 2014-08-28 03:21:22]

felt@chromium.org changed reviewers:
+ mattm@chromium.org, palmer@chromium.org

[felt, 2014-08-28 03:21:58]

palmer: pls review c/b/ssl/ssl_blocking_page.cc
mattm: pls review c/b/safe_browsing/*

[palmer, 2014-08-28 18:46:20]

lgtm

[mattm, 2014-08-28 21:07:59]

https://codereview.chromium.org/515803002/diff/1/chrome/browser/safe_browsing...
File chrome/browser/safe_browsing/download_protection_service.cc (right):

https://codereview.chromium.org/515803002/diff/1/chrome/browser/safe_browsing...
chrome/browser/safe_browsing/download_protection_service.cc:963: GURL
learn_more_url(chrome::kDownloadScanningLearnMoreURL);
This one doesn't seem to work, I think the redirect strips the hl param.

[felt, 2014-08-28 21:54:23]

https://codereview.chromium.org/515803002/diff/1/chrome/browser/safe_browsing...
File chrome/browser/safe_browsing/download_protection_service.cc (right):

https://codereview.chromium.org/515803002/diff/1/chrome/browser/safe_browsing...
chrome/browser/safe_browsing/download_protection_service.cc:963: GURL
learn_more_url(chrome::kDownloadScanningLearnMoreURL);
On 2014/08/28 21:07:58, mattm wrote:
> This one doesn't seem to work, I think the redirect strips the hl param.

I filed a help center bug to get the redirect to stop stripping the param.

[mattm, 2014-08-28 22:05:22]

On 2014/08/28 21:54:23, felt wrote:
>
https://codereview.chromium.org/515803002/diff/1/chrome/browser/safe_browsing...
> File chrome/browser/safe_browsing/download_protection_service.cc (right):
> 
>
https://codereview.chromium.org/515803002/diff/1/chrome/browser/safe_browsing...
> chrome/browser/safe_browsing/download_protection_service.cc:963: GURL
> learn_more_url(chrome::kDownloadScanningLearnMoreURL);
> On 2014/08/28 21:07:58, mattm wrote:
> > This one doesn't seem to work, I think the redirect strips the hl param.
> 
> I filed a help center bug to get the redirect to stop stripping the param.

sg, CL lgtm if you want go ahead with it in the meantime.

[felt, 2014-08-28 22:09:24]

The CQ bit was checked by felt@chromium.org

[commit-bot: I haz the power, 2014-08-28 22:11:11]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/felt@chromium.org/515803002/1

[commit-bot: I haz the power, 2014-08-28 23:51:41]

Committed patchset #1 (id:1) as 97f4265a3900cfc61fba8012e29de0430e5a67f7

[commit-bot: I haz the power, 2014-09-10 03:03:54]

Patchset 1 (id:??) landed as
https://crrev.com/e5d8ace7ccebe3c2e4c93f9151f9f482c982dc69
Cr-Commit-Position: refs/heads/master@{#292508}