From b10712de8733ef8e8b963c0d020682d7a2590c3a Mon Sep 17 00:00:00 2001 From: adeshkp Date: Sun, 21 Apr 2019 13:02:52 -0400 Subject: Remove SecurityUI telemetry. --- .../pki/resources/content/exceptionDialog.js | 20 --- security/manager/ssl/moz.build | 1 - security/manager/ssl/nsISecurityUITelemetry.idl | 150 --------------------- 3 files changed, 171 deletions(-) delete mode 100644 security/manager/ssl/nsISecurityUITelemetry.idl (limited to 'security/manager') diff --git a/security/manager/pki/resources/content/exceptionDialog.js b/security/manager/pki/resources/content/exceptionDialog.js index 0ca24a614..c106cdbf5 100644 --- a/security/manager/pki/resources/content/exceptionDialog.js +++ b/security/manager/pki/resources/content/exceptionDialog.js @@ -12,8 +12,6 @@ var gCert; var gChecking; var gBroken; var gNeedReset; -var gSecHistogram; -var gNsISecTel; Components.utils.import("resource://gre/modules/PrivateBrowsingUtils.jsm"); @@ -49,10 +47,6 @@ function initExceptionDialog() { gDialog = document.documentElement; gBundleBrand = document.getElementById("brand_bundle"); gPKIBundle = document.getElementById("pippki_bundle"); - gSecHistogram = Components.classes["@mozilla.org/base/telemetry;1"]. - getService(Components.interfaces.nsITelemetry). - getHistogramById("SECURITY_UI"); - gNsISecTel = Components.interfaces.nsISecurityUITelemetry; var brandName = gBundleBrand.getString("brandShortName"); setText("warningText", gPKIBundle.getFormattedString("addExceptionBrandedWarning2", [brandName])); @@ -190,7 +184,6 @@ function updateCertStatus() { var shortDesc3, longDesc3; var use2 = false; var use3 = false; - let bucketId = gNsISecTel.WARNING_BAD_CERT_TOP_ADD_EXCEPTION_BASE; if (gCert) { if (gBroken) { var mms = "addExceptionDomainMismatchShort"; @@ -201,13 +194,11 @@ function updateCertStatus() { var utl = "addExceptionUnverifiedOrBadSignatureLong2"; var use1 = false; if (gSSLStatus.isDomainMismatch) { - bucketId += gNsISecTel.WARNING_BAD_CERT_TOP_ADD_EXCEPTION_FLAG_DOMAIN; use1 = true; shortDesc = mms; longDesc = mml; } if (gSSLStatus.isNotValidAtThisTime) { - bucketId += gNsISecTel.WARNING_BAD_CERT_TOP_ADD_EXCEPTION_FLAG_TIME; if (!use1) { use1 = true; shortDesc = exs; @@ -220,7 +211,6 @@ function updateCertStatus() { } } if (gSSLStatus.isUntrusted) { - bucketId += gNsISecTel.WARNING_BAD_CERT_TOP_ADD_EXCEPTION_FLAG_UNTRUSTED; if (!use1) { use1 = true; shortDesc = uts; @@ -235,7 +225,6 @@ function updateCertStatus() { longDesc3 = utl; } } - gSecHistogram.add(bucketId); // In these cases, we do want to enable the "Add Exception" button gDialog.getButton("extra1").disabled = false; @@ -307,7 +296,6 @@ function updateCertStatus() { * Handle user request to display certificate details */ function viewCertButtonClick() { - gSecHistogram.add(gNsISecTel.WARNING_BAD_CERT_TOP_CLICK_VIEW_CERT); if (gCert) { viewCertHelper(this, gCert); } @@ -324,27 +312,19 @@ function addException() { var overrideService = Components.classes["@mozilla.org/security/certoverride;1"] .getService(Components.interfaces.nsICertOverrideService); var flags = 0; - let confirmBucketId = gNsISecTel.WARNING_BAD_CERT_TOP_CONFIRM_ADD_EXCEPTION_BASE; if (gSSLStatus.isUntrusted) { flags |= overrideService.ERROR_UNTRUSTED; - confirmBucketId += gNsISecTel.WARNING_BAD_CERT_TOP_CONFIRM_ADD_EXCEPTION_FLAG_UNTRUSTED; } if (gSSLStatus.isDomainMismatch) { flags |= overrideService.ERROR_MISMATCH; - confirmBucketId += gNsISecTel.WARNING_BAD_CERT_TOP_CONFIRM_ADD_EXCEPTION_FLAG_DOMAIN; } if (gSSLStatus.isNotValidAtThisTime) { flags |= overrideService.ERROR_TIME; - confirmBucketId += gNsISecTel.WARNING_BAD_CERT_TOP_CONFIRM_ADD_EXCEPTION_FLAG_TIME; } var permanentCheckbox = document.getElementById("permanent"); var shouldStorePermanently = permanentCheckbox.checked && !inPrivateBrowsingMode(); - if (!permanentCheckbox.checked) { - gSecHistogram.add(gNsISecTel.WARNING_BAD_CERT_TOP_DONT_REMEMBER_EXCEPTION); - } - gSecHistogram.add(confirmBucketId); var uri = getURI(); overrideService.rememberValidityOverride( uri.asciiHost, uri.port, diff --git a/security/manager/ssl/moz.build b/security/manager/ssl/moz.build index 8c8d25dcc..94d766a0b 100644 --- a/security/manager/ssl/moz.build +++ b/security/manager/ssl/moz.build @@ -32,7 +32,6 @@ XPIDL_SOURCES += [ 'nsIPKCS11Slot.idl', 'nsIProtectedAuthThread.idl', 'nsISecretDecoderRing.idl', - 'nsISecurityUITelemetry.idl', 'nsISiteSecurityService.idl', 'nsISSLStatus.idl', 'nsISSLStatusProvider.idl', diff --git a/security/manager/ssl/nsISecurityUITelemetry.idl b/security/manager/ssl/nsISecurityUITelemetry.idl deleted file mode 100644 index 05d3473ad..000000000 --- a/security/manager/ssl/nsISecurityUITelemetry.idl +++ /dev/null @@ -1,150 +0,0 @@ -/* -*- Mode: C++; tab-width: 2; indent-tabs-mode: nil; c-basic-offset: 2 -*- - * - * This Source Code Form is subject to the terms of the Mozilla Public - * License, v. 2.0. If a copy of the MPL was not distributed with this - * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ - -#include "nsISupports.idl" - -[scriptable, uuid(5d1acf82-223a-46fb-a8f3-a1b16e2ceb04)] - -interface nsISecurityUITelemetry : nsISupports { - -/* - * Addon installation warnings - */ - -// Firefox prevented this site from asking you to install addon -const uint32_t WARNING_ADDON_ASKING_PREVENTED = 1; -// User clicks through and allows site to ask to install addons -const uint32_t WARNING_ADDON_ASKING_PREVENTED_CLICK_THROUGH = 2; -// Are you sure you want to install this addon? Only install addons you trust -const uint32_t WARNING_CONFIRM_ADDON_INSTALL = 3; -// User clicked she is sure after waiting 3secs -const uint32_t WARNING_CONFIRM_ADDON_INSTALL_CLICK_THROUGH = 4; - - -/* - * modal dialogs/warnings - */ - -// removed WARNING_ENTERING_SECURE_SITE = 5; -// removed WARNING_ENTERING_WEAK_SITE = 6; -// removed WARNING_LEAVING_SECURE_SITE = 7; -// removed WARNING_MIXED_CONTENT = 8; - -// For confirmation dialogs, the clickthrough constant needs to be 1 -// more than the dialog constant so that -// WARNING_CONFIRM_ + 1 == WARNING_CONFIRM__CLICK_THROUGH -const uint32_t WARNING_CONFIRM_POST_TO_INSECURE_FROM_SECURE = 9; -const uint32_t WARNING_CONFIRM_POST_TO_INSECURE_FROM_SECURE_CLICK_THROUGH = 10; -// removed WARNING_CONFIRM_POST_TO_INSECURE_FROM_INSECURE = 11; -// removed WARNING_CONFIRM_POST_TO_INSECURE_FROM_INSECURE_CLICK_THROUGH = 12; - -/* - * Phishing / Malware page warnings - * deprecated: we use the _TOP and _FRAME versions below - */ - -const uint32_t WARNING_MALWARE_PAGE = 13; -const uint32_t WARNING_MALWARE_PAGE_WHY_BLOCKED = 14; -const uint32_t WARNING_MALWARE_PAGE_GET_ME_OUT_OF_HERE = 15; -const uint32_t WARNING_MALWARE_PAGE_IGNORE_WARNING = 16; - -const uint32_t WARNING_PHISHING_PAGE = 17; -const uint32_t WARNING_PHISHING_PAGE_WHY_BLOCKED = 18; -const uint32_t WARNING_PHISHING_PAGE_GET_ME_OUT_OF_HERE = 19; -const uint32_t WARNING_PHISHING_PAGE_IGNORE_WARNING = 20; - -/* - * SSL Error dialogs - * deprecated: we use the _TOP versions below - */ - -const uint32_t WARNING_BAD_CERT = 21; -const uint32_t WARNING_BAD_CERT_STS = 22; -const uint32_t WARNING_BAD_CERT_CLICK_ADD_EXCEPTION = 23; -const uint32_t WARNING_BAD_CERT_CLICK_VIEW_CERT = 24; -const uint32_t WARNING_BAD_CERT_DONT_REMEMBER_EXCEPTION = 25; -const uint32_t WARNING_BAD_CERT_GET_ME_OUT_OF_HERE = 27; -const uint32_t WARNING_BAD_CERT_UNDERSTAND_RISKS = 28; -const uint32_t WARNING_BAD_CERT_TECHINICAL_DETAILS = 29; - -/* - * Note that if we add more possibilities in the warning dialogs, - * it is a new experiment and we shouldn't reuse these buckets. - */ -const uint32_t WARNING_BAD_CERT_ADD_EXCEPTION_BASE = 30; -const uint32_t WARNING_BAD_CERT_ADD_EXCEPTION_FLAG_UNTRUSTED = 1; -const uint32_t WARNING_BAD_CERT_ADD_EXCEPTION_FLAG_DOMAIN = 2; -const uint32_t WARNING_BAD_CERT_ADD_EXCEPTION_FLAG_TIME = 4; - -const uint32_t WARNING_BAD_CERT_CONFIRM_ADD_EXCEPTION_BASE = 38; -const uint32_t WARNING_BAD_CERT_CONFIRM_ADD_EXCEPTION_FLAG_UNTRUSTED = 1; -const uint32_t WARNING_BAD_CERT_CONFIRM_ADD_EXCEPTION_FLAG_DOMAIN = 2; -const uint32_t WARNING_BAD_CERT_CONFIRM_ADD_EXCEPTION_FLAG_TIME = 4; -// This uses up buckets till 45 - -/* - * Geolocation Popup Telemetry - */ - -const uint32_t WARNING_GEOLOCATION_REQUEST = 46; -const uint32_t WARNING_GEOLOCATION_REQUEST_SHARE_LOCATION = 47; -const uint32_t WARNING_GEOLOCATION_REQUEST_ALWAYS_SHARE = 48; -const uint32_t WARNING_GEOLOCATION_REQUEST_NEVER_SHARE = 49; -// It would be nice to measure the two cases of user clicking -// "not now", and user closing the popup. This is currently not implemented. - -const uint32_t WARNING_MALWARE_PAGE_TOP = 52; -const uint32_t WARNING_MALWARE_PAGE_TOP_WHY_BLOCKED = 53; -const uint32_t WARNING_MALWARE_PAGE_TOP_GET_ME_OUT_OF_HERE = 54; -const uint32_t WARNING_MALWARE_PAGE_TOP_IGNORE_WARNING = 55; - -const uint32_t WARNING_PHISHING_PAGE_TOP = 56; -const uint32_t WARNING_PHISHING_PAGE_TOP_WHY_BLOCKED = 57; -const uint32_t WARNING_PHISHING_PAGE_TOP_GET_ME_OUT_OF_HERE = 58; -const uint32_t WARNING_PHISHING_PAGE_TOP_IGNORE_WARNING = 59; - -const uint32_t WARNING_MALWARE_PAGE_FRAME = 60; -const uint32_t WARNING_MALWARE_PAGE_FRAME_WHY_BLOCKED = 61; -const uint32_t WARNING_MALWARE_PAGE_FRAME_GET_ME_OUT_OF_HERE = 62; -const uint32_t WARNING_MALWARE_PAGE_FRAME_IGNORE_WARNING = 63; - -const uint32_t WARNING_PHISHING_PAGE_FRAME = 64; -const uint32_t WARNING_PHISHING_PAGE_FRAME_WHY_BLOCKED = 65; -const uint32_t WARNING_PHISHING_PAGE_FRAME_GET_ME_OUT_OF_HERE = 66; -const uint32_t WARNING_PHISHING_PAGE_FRAME_IGNORE_WARNING = 67; - -const uint32_t WARNING_BAD_CERT_TOP = 68; -const uint32_t WARNING_BAD_CERT_TOP_STS = 69; -const uint32_t WARNING_BAD_CERT_TOP_CLICK_ADD_EXCEPTION = 70; -const uint32_t WARNING_BAD_CERT_TOP_CLICK_VIEW_CERT = 71; -const uint32_t WARNING_BAD_CERT_TOP_DONT_REMEMBER_EXCEPTION = 72; -const uint32_t WARNING_BAD_CERT_TOP_GET_ME_OUT_OF_HERE = 73; -const uint32_t WARNING_BAD_CERT_TOP_UNDERSTAND_RISKS = 74; -// removed WARNING_BAD_CERT_TOP_TECHNICAL_DETAILS = 75; - -const uint32_t WARNING_BAD_CERT_TOP_ADD_EXCEPTION_BASE = 76; -const uint32_t WARNING_BAD_CERT_TOP_ADD_EXCEPTION_FLAG_UNTRUSTED = 1; -const uint32_t WARNING_BAD_CERT_TOP_ADD_EXCEPTION_FLAG_DOMAIN = 2; -const uint32_t WARNING_BAD_CERT_TOP_ADD_EXCEPTION_FLAG_TIME = 4; - -const uint32_t WARNING_BAD_CERT_TOP_CONFIRM_ADD_EXCEPTION_BASE = 84; -const uint32_t WARNING_BAD_CERT_TOP_CONFIRM_ADD_EXCEPTION_FLAG_UNTRUSTED = 1; -const uint32_t WARNING_BAD_CERT_TOP_CONFIRM_ADD_EXCEPTION_FLAG_DOMAIN = 2; -const uint32_t WARNING_BAD_CERT_TOP_CONFIRM_ADD_EXCEPTION_FLAG_TIME = 4; - -// Another Safe Browsing list (like malware & phishing above) -const uint32_t WARNING_UNWANTED_PAGE_TOP = 92; -const uint32_t WARNING_UNWANTED_PAGE_TOP_WHY_BLOCKED = 93; -const uint32_t WARNING_UNWANTED_PAGE_TOP_GET_ME_OUT_OF_HERE = 94; -const uint32_t WARNING_UNWANTED_PAGE_TOP_IGNORE_WARNING = 95; -const uint32_t WARNING_UNWANTED_PAGE_FRAME = 96; -const uint32_t WARNING_UNWANTED_PAGE_FRAME_WHY_BLOCKED = 97; -const uint32_t WARNING_UNWANTED_PAGE_FRAME_GET_ME_OUT_OF_HERE = 98; -const uint32_t WARNING_UNWANTED_PAGE_FRAME_IGNORE_WARNING = 99; - -// This uses up buckets till 99 (including) -// We only have buckets up to 100. -}; -- cgit v1.2.3