Commit 8f5539d3 authored by Marco Bonardo's avatar Marco Bonardo
Browse files

Bug 723165 - Telemetry for time needed to show the Bookmarks toolbar. 

r=dietrich
parent 695167de

browser/components/places/content/browserPlacesViews.js

+5 −0
Original line number Diff line number Diff line
@@ -41,6 +41,7 @@ 
 * ***** END LICENSE BLOCK ***** */



Components.utils.import("resource://gre/modules/XPCOMUtils.jsm");

Components.utils.import("resource://gre/modules/Services.jsm");



/**

 * The base view implements everything that's common to the toolbar and
@@ -808,6 +809,7 @@ PlacesViewBase.prototype = { 
};



function PlacesToolbar(aPlace) {

  let startTime = Date.now();

  // Add some smart getters for our elements.

  let thisView = this;

  [
@@ -836,6 +838,9 @@ function PlacesToolbar(aPlace) { 
  this._addEventListeners(window, ["resize", "unload"], false);



  PlacesViewBase.call(this, aPlace);



  Services.telemetry.getHistogramById("FX_BOOKMARKS_TOOLBAR_INIT_MS")

                    .add(Date.now() - startTime);

}



PlacesToolbar.prototype = {

toolkit/components/telemetry/TelemetryHistograms.h

+1 −0
Original line number Diff line number Diff line
@@ -324,6 +324,7 @@ HISTOGRAM_BOOLEAN(FX_CONTEXT_SEARCH_AND_TAB_SELECT, "Firefox: Background tab was 
HISTOGRAM_BOOLEAN(FX_KEYWORD_URL_USERSET, "Firefox: keyword.URL has a user-set value")

HISTOGRAM(FX_IDENTITY_POPUP_OPEN_MS, 1, 1000, 10, EXPONENTIAL, "Firefox: Time taken by the identity popup to open in milliseconds")

HISTOGRAM(FX_APP_MENU_OPEN_MS, 1, 1000, 10, EXPONENTIAL, "Firefox: Time taken by the app-menu opening in milliseconds")

HISTOGRAM(FX_BOOKMARKS_TOOLBAR_INIT_MS, 50, 5000, 10, EXPONENTIAL, "Firefox: Time to initialize the bookmarks toolbar view (ms)")



/**

 * Thumbnail Service telemetry.