Bug 1516408 - Remove deprecated backward compatibility code in webide;r=ochameau
authorJulian Descottes <jdescottes@mozilla.com>
Thu, 03 Jan 2019 10:18:00 +0000
changeset 509489 30ca625d8d6eb4c2a3760ba235f38ee1dc93b4f9
parent 509488 1b13a38c51e5640aa716d8d367c44e101c3835e3
child 509490 6102219fb8f84277ba8366dbe6e4fa9f4e75786b
push id10547
push userffxbld-merge
push dateMon, 21 Jan 2019 13:03:58 +0000
treeherdermozilla-beta@24ec1916bffe [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersochameau
bugs1516408
milestone66.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1516408 - Remove deprecated backward compatibility code in webide;r=ochameau Depends on D15303 Differential Revision: https://phabricator.services.mozilla.com/D15304
devtools/client/webide/modules/app-manager.js
--- a/devtools/client/webide/modules/app-manager.js
+++ b/devtools/client/webide/modules/app-manager.js
@@ -247,32 +247,22 @@ var AppManager = exports.AppManager = {
     }
     return this.getTarget().then(target => {
       target.activeTab.reload();
     }, console.error);
   },
 
   getTarget: function() {
     if (this.selectedProject.type == "mainProcess") {
-      // Fx >=39 exposes a ParentProcessTargetActor to debug the main process
-      if (this.connection.client.mainRoot.traits.allowChromeProcess) {
-        return this.connection.client.mainRoot.getMainProcess()
-                   .then(front => {
-                     return TargetFactory.forRemoteTab({
-                       activeTab: front,
-                       client: this.connection.client,
-                       chrome: true,
-                     });
-                   });
-      }
-      // Fx <39 exposes chrome target actors on the root actor
-      return TargetFactory.forRemoteTab({
-          form: this._rootForm,
+      return this.connection.client.mainRoot.getMainProcess().then(front => {
+        return TargetFactory.forRemoteTab({
+          activeTab: front,
           client: this.connection.client,
           chrome: true,
+        });
       });
     }
 
     if (this.selectedProject.type == "tab") {
       return this.tabStore.getTargetForTab();
     }
 
     const app = this._getProjectFront(this.selectedProject);