Backout bug 977730 due to bustage [CLOSED TREE]
authorAsaf Romano <aromano@mozilla.com>
Mon, 31 Mar 2014 21:55:15 +0300
changeset 176224 6b01c16172c267c00e52b31280f788afb9162caf
parent 176223 af839c309305a2721c82ca43b20fa61c1c1dc782
child 176225 08a9cf1f40b5c1bf6dab07400384738d91dda472
push id6055
push useraromano@mozilla.com
push dateMon, 31 Mar 2014 18:55:28 +0000
treeherderfx-team@6b01c16172c2 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs977730
milestone31.0a1
Backout bug 977730 due to bustage [CLOSED TREE]
browser/base/content/browser.js
toolkit/components/telemetry/Histograms.json
--- a/browser/base/content/browser.js
+++ b/browser/base/content/browser.js
@@ -3990,22 +3990,20 @@ var CombinedStopReload = {
   }
 };
 
 var TabsProgressListener = {
   onStateChange: function (aBrowser, aWebProgress, aRequest, aStateFlags, aStatus) {
     // Collect telemetry data about tab load times.
     if (aWebProgress.isTopLevel) {
       if (aStateFlags & Ci.nsIWebProgressListener.STATE_IS_WINDOW) {
-        if (aStateFlags & Ci.nsIWebProgressListener.STATE_START) {
+        if (aStateFlags & Ci.nsIWebProgressListener.STATE_START)
           TelemetryStopwatch.start("FX_PAGE_LOAD_MS", aBrowser);
-          Services.telemetry.getHistogramById("FX_TOTAL_TOP_VISITS").add(true);
-        } else if (aStateFlags & Ci.nsIWebProgressListener.STATE_STOP) {
+        else if (aStateFlags & Ci.nsIWebProgressListener.STATE_STOP)
           TelemetryStopwatch.finish("FX_PAGE_LOAD_MS", aBrowser);
-        }
       } else if (aStateFlags & Ci.nsIWebProgressListener.STATE_STOP &&
                  aStatus == Cr.NS_BINDING_ABORTED) {
         TelemetryStopwatch.cancel("FX_PAGE_LOAD_MS", aBrowser);
       }
     }
 
     // Attach a listener to watch for "click" events bubbling up from error
     // pages and other similar pages (like about:newtab). This lets us fix bugs
--- a/toolkit/components/telemetry/Histograms.json
+++ b/toolkit/components/telemetry/Histograms.json
@@ -3250,21 +3250,16 @@
   "FX_PAGE_LOAD_MS": {
     "expires_in_version": "never",
     "kind": "exponential",
     "high": "10000",
     "n_buckets": 20,
     "extended_statistics_ok": true,
     "description": "Firefox: Time taken to load a page (ms)"
   },
-  "FX_TOTAL_TOP_VISITS": {
-    "expires_in_version": "never",
-    "kind": "boolean",
-    "description": "Count the number of times a new top page was starting to load"
-  },
   "FX_THUMBNAILS_CAPTURE_TIME_MS": {
     "expires_in_version": "never",
     "kind": "exponential",
     "high": "500",
     "n_buckets": 15,
     "extended_statistics_ok": true,
     "description": "THUMBNAILS: Time (ms) it takes to capture a thumbnail"
   },