Bug 1290680 - Netmonitor mousemove throws TypeError: this._baseNode.contains is not a function. r=jdescottes, a=ritu
authorJarda Snajdr <jsnajdr@gmail.com>
Mon, 15 Aug 2016 16:02:57 +0200
changeset 349895 c0a227502cb4297c7fecbb9eff3f3c9e769fab95
parent 349894 a353fc4095ca9a489b4ef728607a42d630a20477
child 349896 f249312855c246c677e4c46456666ea0fbf8c627
push id1230
push userjlund@mozilla.com
push dateMon, 31 Oct 2016 18:13:35 +0000
treeherdermozilla-release@5e06e3766db2 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjdescottes, ritu
bugs1290680
milestone50.0a2
Bug 1290680 - Netmonitor mousemove throws TypeError: this._baseNode.contains is not a function. r=jdescottes, a=ritu MozReview-Commit-ID: GHJl6nfdEyO
devtools/client/netmonitor/netmonitor-view.js
--- a/devtools/client/netmonitor/netmonitor-view.js
+++ b/devtools/client/netmonitor/netmonitor-view.js
@@ -461,26 +461,27 @@ function RequestsMenuView() {
 
 RequestsMenuView.prototype = Heritage.extend(WidgetMethods, {
   /**
    * Initialization function, called when the network monitor is started.
    */
   initialize: function () {
     dumpn("Initializing the RequestsMenuView");
 
-    this.widget = new SideMenuWidget($("#requests-menu-contents"));
+    let widgetParentEl = $("#requests-menu-contents");
+    this.widget = new SideMenuWidget(widgetParentEl);
     this._splitter = $("#network-inspector-view-splitter");
     this._summary = $("#requests-menu-network-summary-button");
     this._summary.setAttribute("label", L10N.getStr("networkMenu.empty"));
     this.userInputTimer = Cc["@mozilla.org/timer;1"]
       .createInstance(Ci.nsITimer);
 
     // Create a tooltip for the newly appended network request item.
     this.tooltip = new HTMLTooltip(NetMonitorController._toolbox, { type: "arrow" });
-    this.tooltip.startTogglingOnHover(this.widget, this._onHover, {
+    this.tooltip.startTogglingOnHover(widgetParentEl, this._onHover, {
       toggleDelay: REQUESTS_TOOLTIP_TOGGLE_DELAY,
       interactive: true
     });
     $("#requests-menu-contents").addEventListener("scroll", this._onScroll, true);
 
     Prefs.filters.forEach(type => this.filterOn(type));
     this.sortContents(this._byTiming);