DevTools: Run devtools tests in release mode

third_party/WebKit/Source/devtools/BUILD.gn

 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import("//build/config/features.gni")
 import("//third_party/WebKit/Source/core/core.gni")
 
 gypi_values = exec_script("//build/gypi_to_gn.py",
                           [ rebase_path("devtools.gypi") ],
                           "scope",
@@ skipping 54 matching lines @@
 
 # FIXME: Fix the generate_devtools_grd.py script to accept trailing slashes.
 resources_out_dir_no_slash = "$root_out_dir/resources/inspector"
 resources_out_dir = resources_out_dir_no_slash + "/"
 
 generated_scripts = [
   resources_out_dir + "InspectorBackendCommands.js",
   resources_out_dir + "SupportedCSSProperties.js",
 ]
 
+generated_entry_files = [
+  resources_out_dir + "inspector.html",
+  resources_out_dir + "inspector.js",
+  resources_out_dir + "toolbox.html",
+  resources_out_dir + "toolbox.js",
+]
+
+generated_workers = [
+  resources_out_dir + "formatter_worker.js",
+  resources_out_dir + "heap_snapshot_worker.js",
+  resources_out_dir + "temp_storage_shared_worker.js",
+]
+
+generated_modules = [

[pfeldman, 2016/08/30 21:07:40]

generated_bundled_modules

[chenwilliam, 2016/08/30 22:21:21]

Done.

+  resources_out_dir + "accessibility/accessibility_module.js",
+  resources_out_dir + "animation/animation_module.js",
+  resources_out_dir + "audits/audits_module.js",
+  resources_out_dir + "components_lazy/components_lazy_module.js",
+  resources_out_dir + "console/console_module.js",
+  resources_out_dir + "devices/devices_module.js",
+  resources_out_dir + "diff/diff_module.js",
+  resources_out_dir + "elements/elements_module.js",
+  resources_out_dir + "es_tree/es_tree_module.js",
+  resources_out_dir + "layers/layers_module.js",
+  resources_out_dir + "network/network_module.js",
+  resources_out_dir + "profiler/profiler_module.js",
+  resources_out_dir + "resources/resources_module.js",
+  resources_out_dir + "sass/sass_module.js",
+  resources_out_dir + "security/security_module.js",
+  resources_out_dir + "settings/settings_module.js",
+  resources_out_dir + "snippets/snippets_module.js",
+  resources_out_dir + "source_frame/source_frame_module.js",
+  resources_out_dir + "sources/sources_module.js",
+  resources_out_dir + "text_editor/text_editor_module.js",
+  resources_out_dir + "timeline_model/timeline_model_module.js",
+  resources_out_dir + "timeline/timeline_module.js",
+  resources_out_dir + "ui_lazy/ui_lazy_module.js",
+]
+
 #-------------------------------------------------------------------------------
 
 visibility = [ "//third_party/WebKit/*" ]
 
 group("devtools_frontend_resources") {
   public_deps = [
-    ":build_applications",
+    ":build_release_devtools",
     ":copy_embedder_scripts",
     ":copy_emulated_devices_images",
     ":copy_inspector_images",
     ":devtools_extension_api",
     ":frontend_protocol_sources",
     ":supported_css_properties",
   ]
+  if (debug_devtools) {
+    public_deps += [ ":build_debug_devtools" ]
+  }
 }
 
 copy("copy_embedder_scripts") {
   sources = gypi_values.devtools_embedder_scripts
   outputs = [
     resources_out_dir + "{{source_file_part}}",
   ]
 }
 
 copy("copy_inspector_images") {
@@ skipping 12 matching lines @@
 
 action("generate_devtools_grd") {
   script = "scripts/generate_devtools_grd.py"
 
   deps = [
     ":devtools_frontend_resources",
   ]
   inputs = gypi_values.devtools_image_files + all_devtools_files
   inputs += gypi_values.devtools_embedder_scripts
 
-  if (debug_devtools) {
-    # Debug: all files are picked as-is.
-    generated_files = generated_scripts + [
-                        resources_out_dir + "inspector.html",
-                        resources_out_dir + "toolbox.html",
-                      ]
-
-    # Use a response file since the static files can be too long for the
-    # command line. The args here will be added to the command line.
-    static_files = all_devtools_files + [ "front_end/Runtime.js" ]
-    response_file_contents = rebase_path(static_files, root_build_dir)

[pfeldman, 2016/08/29 20:57:22]

Why is this no longer needed?

[chenwilliam, 2016/08/29 22:45:33]

I don't think we need the grd file in debug_devtools mode because when we're
running the browser, it uses "resources/inspector/debug" and when we're running
tests it's using either "resources/inspector" or "resources/inspector/debug"
(depending on the driver flag). 

If the grd is used, it should be using the files from "out/Release/gen/devtools"
since those are the resources referenced by relative path from
devtools_resources.grd. I manually checked to see if removing this conditional
broke anything, and it seemed OK.

-    static_files_args = [
-      "--static_files_args",
-      "{{response_file_name}}",
-    ]
-  } else {
-    # Release: pick compiled non-remote modules and concatenated app files.
-    generated_files = [
-      resources_out_dir + "inspector.html",
-      resources_out_dir + "inspector.js",
-      resources_out_dir + "toolbox.html",
-      resources_out_dir + "toolbox.js",
-      resources_out_dir + "formatter_worker.js",
-      resources_out_dir + "heap_snapshot_worker.js",
-      resources_out_dir + "temp_storage_shared_worker.js",
-      resources_out_dir + "accessibility/accessibility_module.js",
-      resources_out_dir + "audits/audits_module.js",
-      resources_out_dir + "animation/animation_module.js",
-      resources_out_dir + "components_lazy/components_lazy_module.js",
-      resources_out_dir + "console/console_module.js",
-      resources_out_dir + "devices/devices_module.js",
-      resources_out_dir + "diff/diff_module.js",
-      resources_out_dir + "elements/elements_module.js",
-      resources_out_dir + "es_tree/es_tree_module.js",
-      resources_out_dir + "layers/layers_module.js",
-      resources_out_dir + "network/network_module.js",
-      resources_out_dir + "profiler/profiler_module.js",
-      resources_out_dir + "resources/resources_module.js",
-      resources_out_dir + "sass/sass_module.js",
-      resources_out_dir + "security/security_module.js",
-      resources_out_dir + "settings/settings_module.js",
-      resources_out_dir + "snippets/snippets_module.js",
-      resources_out_dir + "source_frame/source_frame_module.js",
-      resources_out_dir + "sources/sources_module.js",
-      resources_out_dir + "text_editor/text_editor_module.js",
-      resources_out_dir + "timeline_model/timeline_model_module.js",
-      resources_out_dir + "timeline/timeline_module.js",
-      resources_out_dir + "ui_lazy/ui_lazy_module.js",
-      resources_out_dir + "devtools_extension_api.js",
-    ]
-    static_files_args = []  # Nothing needed for this
-  }
+  generated_files =
+      generated_entry_files + generated_workers + generated_modules +
+      [ resources_out_dir + "devtools_extension_api.js" ]
 
   images_path = "front_end/Images"
 
   inputs += generated_files
 
   outfile = "$root_gen_dir/devtools/devtools_resources.grd"
   outputs = [
     outfile,
   ]
 
   relative_path_dirs = [
     resources_out_dir_no_slash,
     "front_end",
   ]
 
   args = rebase_path(generated_files, root_build_dir) +
          rebase_path(generated_files, root_build_dir) +
          rebase_path(gypi_values.devtools_embedder_scripts, root_build_dir) +
-         static_files_args + [ "--relative_path_dirs" ] +
+         [ "--relative_path_dirs" ] +
          rebase_path(relative_path_dirs, root_build_dir) +
          [
            "--images",
            rebase_path(images_path, root_build_dir),
            "--output",
            rebase_path(outfile, root_build_dir),
          ]
 }
 
 action("devtools_extension_api") {
@@ skipping 34 matching lines @@
   outputs = [
     resources_out_dir + "InspectorBackendCommands.js",
   ]
 
   args = rebase_path(inputs, root_build_dir) + [
            "--output_js_dir",
            rebase_path(resources_out_dir, root_build_dir),
          ]
 }
 
-action("build_applications") {
+action("build_release_devtools") {
   script = "scripts/build_applications.py"
+
+  deps = [
+    ":frontend_protocol_sources",
+    ":supported_css_properties",
+  ]
+
   helper_scripts = [
     "scripts/modular_build.py",
     "scripts/concatenate_application_code.py",
     "scripts/rjsmin.py",
   ]
 
   inputs = helper_scripts + all_devtools_files + generated_scripts + [
              "front_end/inspector.html",
              "front_end/toolbox.html",
            ]
 
-  outputs = [
-    resources_out_dir + "inspector.html",
-    resources_out_dir + "toolbox.html",
+  generated_remote_modules = [

[pfeldman, 2016/08/30 21:07:40]

^^ generated_remote_modules

[chenwilliam, 2016/08/30 22:21:21]

Done.

+    resources_out_dir + "cm_modes/cm_modes_module.js",
+    resources_out_dir + "emulated_devices/emulated_devices_module.js",
+    resources_out_dir + "gonzales/gonzales_module.js",
+    resources_out_dir + "screencast/screencast_module.js",
   ]
 
-  deps = [
-    ":frontend_protocol_sources",
-    ":supported_css_properties",
-  ]
+  outputs = generated_entry_files + generated_workers + generated_modules +
+            generated_remote_modules
 
-  if (debug_devtools) {
-    deps += [ ":copy_debug_non_modules" ]
-
-    debug_mode = "1"
-  } else {
-    outputs += [
-      resources_out_dir + "inspector.js",
-      resources_out_dir + "toolbox.js",
-      resources_out_dir + "formatter_worker.js",
-      resources_out_dir + "heap_snapshot_worker.js",
-      resources_out_dir + "temp_storage_shared_worker.js",
-      resources_out_dir + "accessibility/accessibility_module.js",
-      resources_out_dir + "animation/animation_module.js",
-      resources_out_dir + "audits/audits_module.js",
-      resources_out_dir + "cm_modes/cm_modes_module.js",
-      resources_out_dir + "components_lazy/components_lazy_module.js",
-      resources_out_dir + "console/console_module.js",
-      resources_out_dir + "devices/devices_module.js",
-      resources_out_dir + "diff/diff_module.js",
-      resources_out_dir + "elements/elements_module.js",
-      resources_out_dir + "emulated_devices/emulated_devices_module.js",
-      resources_out_dir + "es_tree/es_tree_module.js",
-      resources_out_dir + "gonzales/gonzales_module.js",
-      resources_out_dir + "layers/layers_module.js",
-      resources_out_dir + "network/network_module.js",
-      resources_out_dir + "profiler/profiler_module.js",
-      resources_out_dir + "resources/resources_module.js",
-      resources_out_dir + "sass/sass_module.js",
-      resources_out_dir + "screencast/screencast_module.js",
-      resources_out_dir + "security/security_module.js",
-      resources_out_dir + "settings/settings_module.js",
-      resources_out_dir + "snippets/snippets_module.js",
-      resources_out_dir + "source_frame/source_frame_module.js",
-      resources_out_dir + "sources/sources_module.js",
-      resources_out_dir + "text_editor/text_editor_module.js",
-      resources_out_dir + "timeline_model/timeline_model_module.js",
-      resources_out_dir + "timeline/timeline_module.js",
-      resources_out_dir + "ui_lazy/ui_lazy_module.js",
-    ]
-
-    debug_mode = "0"
-  }
+  debug_mode = "0"
 
   args = [
     "inspector",
     "toolbox",
     "formatter_worker",
     "heap_snapshot_worker",
     "temp_storage_shared_worker",
     "--input_path",
     rebase_path("front_end", root_build_dir),
     "--output_path",
     rebase_path(resources_out_dir, root_build_dir),
     "--debug",
     debug_mode,
   ]
 }
 
 if (debug_devtools) {
-  # Debug: copy non-module directories and core into resources_out_dir as-is.
-  group("copy_debug_non_modules") {
+  resources_out_debug_dir = "$root_out_dir/resources/inspector/debug/"

[pfeldman, 2016/08/29 20:57:22]

I was hoping that we could minimize the diff via using the new approach.

[chenwilliam, 2016/08/29 22:45:33]

I had to add some boilerplate but I'm not sure how to simplify it further? One
of the things I had to add was copying the images because previously the tests
would just not have the images available (which was OK, since none of the tests
relied on the images to pass).

+
+  action("build_debug_devtools") {
+    script = "scripts/build_applications.py"
+
+    deps = [
+      ":copy_debug_files",
+    ]
+
+    inputs = all_devtools_files + [
+               "front_end/inspector.html",
+               "front_end/toolbox.html",
+             ]
+
+    outputs = [
+      resources_out_debug_dir + "inspector.html",
+      resources_out_debug_dir + "toolbox.html",
+    ]
+
+    debug_mode = "1"
+
+    args = [
+      "inspector",
+      "toolbox",
+      "formatter_worker",
+      "heap_snapshot_worker",
+      "temp_storage_shared_worker",
+      "--input_path",
+      rebase_path("front_end", root_build_dir),
+      "--output_path",
+      rebase_path(resources_out_debug_dir, root_build_dir),
+      "--debug",
+      debug_mode,

[pfeldman, 2016/08/30 21:07:40]

It seems that this flag defines the root procedure that takes place in the
build_applications.py. We could split that file into
build_release_applications.py and build_debug_applications.py. And get rid of
the debug_mode switch.

+    ]
+  }
+
+  group("copy_debug_files") {
     public_deps = [
       ":copy_acorn_js_files",
       ":copy_codemirror_files",
+      ":copy_emulated_devices_images_debug",
+      ":copy_generated_scripts",
+      ":copy_inspector_images_debug",
       ":copy_runtime_core",
     ]
   }
 
   copy("copy_runtime_core") {
     sources = gypi_values.devtools_core_base_files
     outputs = [
-      resources_out_dir + "/{{source_file_part}}",
+      resources_out_debug_dir + "/{{source_file_part}}",
     ]
   }
 
   copy("copy_acorn_js_files") {
     sources = gypi_values.devtools_acorn_files
     outputs = [
-      resources_out_dir + "acorn/{{source_file_part}}",
+      resources_out_debug_dir + "acorn/{{source_file_part}}",
     ]
   }
 
   copy("copy_codemirror_files") {
     sources =
         gypi_values.devtools_cm_js_files + gypi_values.devtools_cm_css_files
     outputs = [
-      resources_out_dir + "cm/{{source_file_part}}",
+      resources_out_debug_dir + "cm/{{source_file_part}}",
+    ]
+  }
+
+  copy("copy_generated_scripts") {
+    deps = [
+      ":frontend_protocol_sources",
+      ":supported_css_properties",
+    ]
+    sources = generated_scripts
+    outputs = [
+      resources_out_debug_dir + "/{{source_file_part}}",
+    ]
+  }
+  copy("copy_inspector_images_debug") {
+    sources = gypi_values.devtools_image_files
+    outputs = [
+      resources_out_debug_dir + "Images/{{source_file_part}}",
+    ]
+  }
+
+  copy("copy_emulated_devices_images_debug") {
+    sources = gypi_values.devtools_emulated_devices_images
+    outputs = [
+      resources_out_debug_dir + "emulated_devices/{{source_file_part}}",
     ]
   }
 }