DevTools: implement release note behind an experiment

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/public/public_features.gni")
 import("//third_party/WebKit/Source/core/core.gni")
 
 all_devtools_files = [
   "front_end/accessibility/AccessibilityModel.js",
@@ skipping 228 matching lines @@
   "front_end/heap_snapshot_model/HeapSnapshotModel.js",
   "front_end/heap_snapshot_model/module.json",
   "front_end/heap_snapshot_worker.js",
   "front_end/heap_snapshot_worker.json",
   "front_end/heap_snapshot_worker/AllocationProfile.js",
   "front_end/heap_snapshot_worker/HeapSnapshot.js",
   "front_end/heap_snapshot_worker/HeapSnapshotLoader.js",
   "front_end/heap_snapshot_worker/HeapSnapshotWorker.js",
   "front_end/heap_snapshot_worker/HeapSnapshotWorkerDispatcher.js",
   "front_end/heap_snapshot_worker/module.json",
+  "front_end/help/module.json",
+  "front_end/help/releaseNote.css",
+  "front_end/help/ReleaseNote.js",

[alph, 2017/02/21 23:19:42]

nit: ReleaseNotes

[chenwilliam, 2017/02/22 02:36:38]

It seems more consistent to keep it singular everywhere? If not, I can rename
it.

+  "front_end/help/ReleaseNoteText.js",
   "front_end/host/InspectorFrontendHost.js",
   "front_end/host/InspectorFrontendHostAPI.js",
   "front_end/host/module.json",
   "front_end/host/Platform.js",
   "front_end/host/ResourceLoader.js",
   "front_end/host/UserMetrics.js",
   "front_end/inline_editor/bezierEditor.css",
   "front_end/inline_editor/BezierEditor.js",
   "front_end/inline_editor/bezierSwatch.css",
   "front_end/inline_editor/BezierUI.js",
@@ skipping 555 matching lines @@
   "$resources_out_dir/color_picker/color_picker_module.js",
   "$resources_out_dir/console/console_module.js",
   "$resources_out_dir/cookie_table/cookie_table_module.js",
   "$resources_out_dir/css_tracker/css_tracker_module.js",
   "$resources_out_dir/data_grid/data_grid_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/event_listeners/event_listeners_module.js",
   "$resources_out_dir/heap_snapshot_model/heap_snapshot_model_module.js",
+  "$resources_out_dir/help/help_module.js",

[alph, 2017/02/21 23:19:42]

Do you need this?

[chenwilliam, 2017/02/22 02:36:38]

Removed. I forgot to remove this when I made this module auto-start.

   "$resources_out_dir/inline_editor/inline_editor_module.js",
   "$resources_out_dir/layer_viewer/layer_viewer_module.js",
   "$resources_out_dir/layers/layers_module.js",
   "$resources_out_dir/network/network_module.js",
   "$resources_out_dir/perf_ui/perf_ui_module.js",
   "$resources_out_dir/profiler/profiler_module.js",
   "$resources_out_dir/quick_open/quick_open_module.js",
   "$resources_out_dir/resources/resources_module.js",
   "$resources_out_dir/sass/sass_module.js",
   "$resources_out_dir/security/security_module.js",
@@ skipping 247 matching lines @@
     ".eslintrc.js",
     "devtools-node-modules/",
     "front_end/",
     "scripts/buildbot/run_eslint.py",
     "scripts/local_node/",
     "//testing/scripts/common.py",
     "//testing/scripts/run_devtools_check.py",
     "//testing/xvfb.py",
   ]
 }