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

Unified Diff: chrome/browser/resources/safe_browsing/safe_browsing_v3.js

Issue 339503004: Add the extended reporting checkbox to the malware interstitial v3 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Enable relevant test Created 6 years, 6 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/safe_browsing/safe_browsing_v3.js
diff --git a/chrome/browser/resources/safe_browsing/safe_browsing_v3.js b/chrome/browser/resources/safe_browsing/safe_browsing_v3.js
index 519a5824b94bd040146f5b331f38b392eebe42ff..a3bf4eb9c5a94900b9690fd6dc940e2423687628 100644
--- a/chrome/browser/resources/safe_browsing/safe_browsing_v3.js
+++ b/chrome/browser/resources/safe_browsing/safe_browsing_v3.js
@@ -16,3 +16,19 @@ var SB_CMD_TAKE_ME_BACK = 'takeMeBack';
// Other constants defined in safe_browsing_blocking_page.cc.
var SB_BOX_CHECKED = 'boxchecked';
var SB_DISPLAY_CHECK_BOX = 'displaycheckbox';
+
+// This sets up the Extended Safe Browsing Reporting opt-in.
+function setupCheckbox() {
+ if (loadTimeData.getBoolean('ssl')) return;
+ if (loadTimeData.getBoolean('phishing')) return;
+ if (!loadTimeData.getBoolean(SB_DISPLAY_CHECK_BOX)) return;
+
+ $('optin-label').innerHTML = loadTimeData.getString('optinText');
+ $('optin-checkbox').checked = loadTimeData.getBoolean(SB_BOX_CHECKED);
+ $('malware-optin').classList.remove('hidden');
+
+ $('optin-checkbox').addEventListener('click', function() {
+ sendCommand(
+ $('optin-checkbox').checked ? SB_CMD_DO_REPORT : SB_CMD_DONT_REPORT);
+ });
+}
« no previous file with comments | « no previous file | chrome/browser/resources/ssl/interstitial_v2.css » ('j') | chrome/browser/resources/ssl/interstitial_v2.css » ('J')

Powered by Google App Engine
This is Rietveld 408576698