Bug 979536 - Part 5: Fix reopening graphics tools. r=vporof
authorJ. Ryan Stinnett <jryans@gmail.com>
Wed, 24 Sep 2014 08:49:00 +0200
changeset 222640 d90133c40a646f81103d826d1611164da0f7d949
parent 222639 a7ec293d1f733cb9b4ad216f1156c17009486eb4
child 222641 e9e56750ca5b4a9ac8dbcedf253e298545fe9da9
child 222678 f191b431571c54d96c8ba73ea51ac02c14095e7f
push id7107
push userraliiev@mozilla.com
push dateMon, 13 Oct 2014 17:43:31 +0000
treeherdermozilla-aurora@b4b34e0acc75 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersvporof
bugs979536
milestone35.0a1
Bug 979536 - Part 5: Fix reopening graphics tools. r=vporof
browser/devtools/canvasdebugger/panel.js
browser/devtools/shadereditor/panel.js
--- a/browser/devtools/canvasdebugger/panel.js
+++ b/browser/devtools/canvasdebugger/panel.js
@@ -61,12 +61,14 @@ CanvasDebuggerPanel.prototype = {
 
   destroy: function() {
     // Make sure this panel is not already destroyed.
     if (this._destroyer) {
       return this._destroyer;
     }
 
     return this._destroyer = this.panelWin.shutdownCanvasDebugger().then(() => {
+      // Destroy front to ensure packet handler is removed from client
+      this.panelWin.gFront.destroy();
       this.emit("destroyed");
     });
   }
 };
--- a/browser/devtools/shadereditor/panel.js
+++ b/browser/devtools/shadereditor/panel.js
@@ -61,12 +61,14 @@ ShaderEditorPanel.prototype = {
 
   destroy: function() {
     // Make sure this panel is not already destroyed.
     if (this._destroyer) {
       return this._destroyer;
     }
 
     return this._destroyer = this.panelWin.shutdownShaderEditor().then(() => {
+      // Destroy front to ensure packet handler is removed from client
+      this.panelWin.gFront.destroy();
       this.emit("destroyed");
     });
   }
 };