Bug 1354778 - Remove expired telemetry probe PLUGIN_BLOCKED_FOR_STABILITY. r?bsmedberg draft
authorChris Peterson <cpeterson@mozilla.com>
Wed, 05 Apr 2017 15:26:52 -0700
changeset 560135 0964406d809772018cc23dfa84d89bc537763542
parent 560134 6849816d9223d20196f019e22c46ab07028b089b
child 623618 70abaafca8f96165283df9d5b6241c7bfce86689
push id53336
push usercpeterson@mozilla.com
push dateTue, 11 Apr 2017 06:12:28 +0000
reviewersbsmedberg
bugs1354778
milestone55.0a1
Bug 1354778 - Remove expired telemetry probe PLUGIN_BLOCKED_FOR_STABILITY. r?bsmedberg Firefox 51 users with telemetry enabled blocked about 4.35M SWFs for stability. The block count has been in the single-digit millions for the last few releases. We don't expect these SWFs to go away, so we can stop tracking this telemetry and later replace this plugin blocklist with a more general solution. MozReview-Commit-ID: KF1jIBTG5m4
dom/base/nsObjectLoadingContent.cpp
toolkit/components/telemetry/Histograms.json
--- a/dom/base/nsObjectLoadingContent.cpp
+++ b/dom/base/nsObjectLoadingContent.cpp
@@ -1006,20 +1006,20 @@ nsObjectLoadingContent::OnStartRequest(n
     if (console) {
       nsCOMPtr<nsIURI> uri;
       chan->GetURI(getter_AddRefs(uri));
       nsString message = NS_LITERAL_STRING("Blocking ") +
         NS_ConvertASCIItoUTF16(uri->GetSpecOrDefault().get()) +
         NS_LITERAL_STRING(" since it was found on an internal Firefox blocklist.");
       console->LogStringMessage(message.get());
     }
-    Telemetry::Accumulate(Telemetry::PLUGIN_BLOCKED_FOR_STABILITY, 1);
     mContentBlockingEnabled = true;
     return NS_ERROR_FAILURE;
   }
+
   if (status == NS_ERROR_TRACKING_URI) {
     mContentBlockingEnabled = true;
     return NS_ERROR_FAILURE;
   }
 
   if (!success) {
     LOG(("OBJLC [%p]: OnStartRequest: Request failed\n", this));
     // If the request fails, we still call LoadObject() to handle fallback
--- a/toolkit/components/telemetry/Histograms.json
+++ b/toolkit/components/telemetry/Histograms.json
@@ -10726,23 +10726,16 @@
   "MEDIA_DECODER_BACKEND_USED": {
     "alert_emails": ["danderson@mozilla.com"],
     "bug_numbers": [1259695],
     "expires_in_version": "never",
     "kind": "enumerated",
     "n_values": 10,
     "description": "Media decoder backend (0=WMF Software, 1=DXVA2D3D9, 2=DXVA2D3D11)"
   },
-  "PLUGIN_BLOCKED_FOR_STABILITY": {
-    "alert_emails": ["cpeterson@mozilla.com"],
-    "expires_in_version": "52",
-    "kind": "count",
-    "bug_numbers": [1237198],
-    "description": "Count plugins blocked for stability"
-  },
   "IPC_MESSAGE_SIZE": {
     "alert_emails": ["wmccloskey@mozilla.com"],
     "bug_numbers": [1260908],
     "expires_in_version": "55",
     "kind": "exponential",
     "high": 8000000,
     "n_buckets": 50,
     "keyed": true,