Switch devtools frontend to external DEPS

Bug: 1011259
Change-Id: If6a80fa169c5286ed536688ab39f67c37a27faad
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1855938Reviewed-by: Michael Achenbach <machenbach@chromium.org>
Reviewed-by: Rick Byers <rbyers@chromium.org>
Commit-Queue: Yang Guo <yangguo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#707733}

testing/buildbot/gn_isolate_map.pyl

@@ -1020,17 +1020,17 @@
   },
   "devtools_closure_compile": {
     "args": [
-      "../../third_party/blink/renderer/devtools/scripts/compile_frontend.py",
+      "../../third_party/devtools-frontend/src/scripts/compile_frontend.py",
     ],
-    "label": "//third_party/blink/renderer/devtools:devtools_closure_compile",
+    "label": "//third_party/devtools-frontend/src:devtools_closure_compile",
     "script": "//testing/scripts/run_devtools_check.py",
     "type": "script",
   },
   "devtools_eslint": {
     "args": [
-      "../../third_party/blink/renderer/devtools/scripts/lint_javascript.py",
+      "../../third_party/devtools-frontend/src/scripts/lint_javascript.py",
     ],
-    "label": "//third_party/blink/renderer/devtools:devtools_eslint",
+    "label": "//third_party/devtools-frontend/src:devtools_eslint",
     "script": "//testing/scripts/run_devtools_check.py",
     "type": "script",
   },

third_party/blink/public/public_features.gni

@@ -13,7 +13,7 @@ declare_args() {
 
   # If external_devtools_frontend is set to true, use DevTools frontend that is
   # pulled as external dependency.
-  external_devtools_frontend = false
+  external_devtools_frontend = true
 }
 
 # Unhandled Tap enable means Contextual Search aka Tap to Search.

third_party/blink/web_tests/inspector-protocol/heap-profiler/resources/heap-snapshot-common.js

@@ -7,17 +7,17 @@
   // This script is supposed to be evaluated in inspector-protocol/heap-profiler tests
   // and the relative paths below are relative to that location.
   await testRunner.loadScriptModule(
-      '../../../../blink/renderer/devtools/front_end/platform/utilities.js');
+      '../../../../devtools-frontend/src/front_end/platform/utilities.js');
   await testRunner.loadScriptModule(
-      '../../../../blink/renderer/devtools/front_end/common/UIString.js');
+      '../../../../devtools-frontend/src/front_end/common/UIString.js');
   await testRunner.loadScriptModule(
-      '../../../../blink/renderer/devtools/front_end/heap_snapshot_model/HeapSnapshotModel.js');
+      '../../../../devtools-frontend/src/front_end/heap_snapshot_model/HeapSnapshotModel.js');
   await testRunner.loadScriptModule(
-      '../../../../blink/renderer/devtools/front_end/heap_snapshot_worker/HeapSnapshot.js');
+      '../../../../devtools-frontend/src/front_end/heap_snapshot_worker/HeapSnapshot.js');
   await testRunner.loadScriptModule(
-      '../../../../blink/renderer/devtools/front_end/text_utils/TextUtils.js');
+      '../../../../devtools-frontend/src/front_end/text_utils/TextUtils.js');
   await testRunner.loadScriptModule(
-      '../../../../blink/renderer/devtools/front_end/heap_snapshot_worker/HeapSnapshotLoader.js');
+      '../../../../devtools-frontend/src/front_end/heap_snapshot_worker/HeapSnapshotLoader.js');
 
   // Expose the (de)serialize code from Common because the worker expects it on self.
   // TODO(https://crbug.com/680046) Remove the dupe code below.