DevTools: bake the screencast module into the remote debugging configurations of the inspector app.

Bug: 819449
Change-Id: I492b98d296a2646a79c34c21d6c2a265dff7cc13
Reviewed-on: https://chromium-review.googlesource.com/953144Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Commit-Queue: Pavel Feldman <pfeldman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#542656}

content/shell/browser/shell_devtools_frontend.cc

@@ -19,8 +19,8 @@ namespace content {
 namespace {
 static GURL GetFrontendURL() {
   int port = ShellDevToolsManagerDelegate::GetHttpHandlerPort();
-  return GURL(
-      base::StringPrintf("http://127.0.0.1:%d/devtools/inspector.html", port));
+  return GURL(base::StringPrintf(
+      "http://127.0.0.1:%d/devtools/devtools_app.html", port));
 }
 }  // namespace
 

third_party/WebKit/Source/devtools/BUILD.gn

@@ -1048,7 +1048,6 @@ generated_remote_modules = [
   "$resources_out_dir/cm_modes/cm_modes_module.js",
   "$resources_out_dir/emulated_devices/emulated_devices_module.js",
   "$resources_out_dir/product_registry_impl/product_registry_impl_module.js",
-  "$resources_out_dir/screencast/screencast_module.js",
 ]
 
 devtools_applications = [

third_party/WebKit/Source/devtools/front_end/inspector.json

 {
   "modules" : [
-    { "name": "screencast", "type": "remote" }
+    { "name": "screencast", "type": "autostart" }
   ],
   "extends": "devtools_app",
   "has_html": true

third_party/WebKit/Source/devtools/front_end/main/Main.js

@@ -212,19 +212,8 @@ Main.Main = class {
     this._registerForwardedShortcuts();
     this._registerMessageSinkListener();
 
-    // Pick first app we could instantiate (for test harness).
-    for (const extension of self.runtime.extensions(Common.AppProvider)) {
-      try {
-        const instance = await extension.instance();
-        if (instance) {
-          this._showAppUI(instance);
-          break;
-        }
-      } catch (e) {
-        console.error(e);
-      }
-    }
     Main.Main.timeEnd('Main._createAppUI');
+    this._showAppUI(await self.runtime.extension(Common.AppProvider).instance());
   }
 
   /**