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

Side by Side Diff: chrome/browser/resources/security_warnings/safe_browsing.js

Issue 600153004: Refactor SafeBrowsing and SSL interstitials to have a type variable (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // Must match the commands handled by SafeBrowsingBlockingPage::CommandReceived. 5 // Must match the commands handled by SafeBrowsingBlockingPage::CommandReceived.
6 var SB_CMD_DO_REPORT = 'doReport'; 6 var SB_CMD_DO_REPORT = 'doReport';
7 var SB_CMD_DONT_REPORT = 'dontReport'; 7 var SB_CMD_DONT_REPORT = 'dontReport';
8 var SB_CMD_EXPANDED_SEE_MORE = 'expandedSeeMore'; 8 var SB_CMD_EXPANDED_SEE_MORE = 'expandedSeeMore';
9 var SB_CMD_LEARN_MORE_2 = 'learnMore2'; 9 var SB_CMD_LEARN_MORE_2 = 'learnMore2';
10 var SB_CMD_PROCEED = 'proceed'; 10 var SB_CMD_PROCEED = 'proceed';
11 var SB_CMD_SHOW_DIAGNOSTIC = 'showDiagnostic'; 11 var SB_CMD_SHOW_DIAGNOSTIC = 'showDiagnostic';
12 var SB_CMD_SHOW_PRIVACY = 'showPrivacy'; 12 var SB_CMD_SHOW_PRIVACY = 'showPrivacy';
13 var SB_CMD_TAKE_ME_BACK = 'takeMeBack'; 13 var SB_CMD_TAKE_ME_BACK = 'takeMeBack';
14 14
15 // Other constants defined in safe_browsing_blocking_page.cc. 15 // Other constants defined in safe_browsing_blocking_page.cc.
16 var SB_BOX_CHECKED = 'boxchecked'; 16 var SB_BOX_CHECKED = 'boxchecked';
17 var SB_DISPLAY_CHECK_BOX = 'displaycheckbox'; 17 var SB_DISPLAY_CHECK_BOX = 'displaycheckbox';
18 18
19 // This sets up the Extended Safe Browsing Reporting opt-in. 19 // This sets up the Extended Safe Browsing Reporting opt-in.
20 function setupCheckbox() { 20 function setupCheckbox() {
21 if (loadTimeData.getBoolean('ssl') || loadTimeData.getBoolean('phishing') || 21 if (loadTimeData.getString('type') != 'SAFEBROWSING' ||
22 loadTimeData.getBoolean('phishing') ||
22 !loadTimeData.getBoolean(SB_DISPLAY_CHECK_BOX)) { 23 !loadTimeData.getBoolean(SB_DISPLAY_CHECK_BOX)) {
23 return; 24 return;
24 } 25 }
25 26
26 $('opt-in-label').innerHTML = loadTimeData.getString('optInLink'); 27 $('opt-in-label').innerHTML = loadTimeData.getString('optInLink');
27 $('opt-in-checkbox').checked = loadTimeData.getBoolean(SB_BOX_CHECKED); 28 $('opt-in-checkbox').checked = loadTimeData.getBoolean(SB_BOX_CHECKED);
28 $('malware-opt-in').classList.remove('hidden'); 29 $('malware-opt-in').classList.remove('hidden');
29 30
30 $('opt-in-checkbox').addEventListener('click', function() { 31 $('opt-in-checkbox').addEventListener('click', function() {
31 sendCommand( 32 sendCommand(
32 $('opt-in-checkbox').checked ? SB_CMD_DO_REPORT : SB_CMD_DONT_REPORT); 33 $('opt-in-checkbox').checked ? SB_CMD_DO_REPORT : SB_CMD_DONT_REPORT);
33 }); 34 });
34 } 35 }
OLDNEW
« no previous file with comments | « chrome/browser/resources/security_warnings/interstitial_v2.js ('k') | chrome/browser/resources/security_warnings/ssl.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698