Move browser_protocol.pdl to third_party/blink/public/devtools_protocol

Bug: 1018108

This still retains a copy of protocol.json for compatibility with
devtools front_end.
Also, remove a presubmit for front-end compilation upon a change to PDL
since the front-end is in another repository now.

Change-Id: I2faea62feca80776d2fc0a0b4ea00a90439ae005
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1898538
Commit-Queue: Andrey Kosyakov <caseq@chromium.org>
Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#718903}

chrome/browser/devtools/BUILD.gn

@@ -45,11 +45,11 @@ if (!is_android) {
     script = _inspector_protocol + "/concatenate_protocols.py"
 
     deps = [
-      "//third_party/blink/renderer/core/inspector:protocol_version",
+      "//third_party/blink/public/devtools_protocol:protocol_version",
     ]
 
     _blink_protocol_path =
-        "$root_gen_dir/third_party/blink/renderer/core/inspector/protocol.json"
+        "$root_gen_dir/third_party/blink/public/devtools_protocol/protocol.json"
     inputs = [
       _blink_protocol_path,
     ]

components/autofill_assistant/browser/devtools/BUILD.gn

@@ -54,10 +54,10 @@ foreach(domain, devtools_domains) {
 action("gen_devtools_client_api") {
   script = "devtools_api/client_api_generator.py"
   deps = [
-    "//third_party/blink/renderer/core/inspector:protocol_version",
+    "//third_party/blink/public/devtools_protocol:protocol_version",
   ]
   inputs = [
-    "$root_gen_dir/third_party/blink/renderer/core/inspector/protocol.json",
+    "$root_gen_dir/third_party/blink/public/devtools_protocol/protocol.json",
   ]
 
   outputs = generated_devtools_api

content/browser/devtools/BUILD.gn

@@ -29,7 +29,7 @@ grit("devtools_resources") {
   grit_flags = [
     "-E",
     "protocol_file=" + rebase_path(
-            "$root_gen_dir/third_party/blink/renderer/core/inspector/protocol.json",
+            "$root_gen_dir/third_party/blink/public/devtools_protocol/protocol.json",
             root_build_dir),
   ]
 
@@ -37,7 +37,7 @@ grit("devtools_resources") {
       [ "SHARED_INTERMEDIATE_DIR=" + rebase_path(root_gen_dir, root_build_dir) ]
 
   deps = [
-    "//third_party/blink/renderer/core/inspector:protocol_version",
+    "//third_party/blink/public/devtools_protocol:protocol_version",
 
     # This is the action that generates out .grd input file.
     "//third_party/blink/public:blink_generate_devtools_grd",
@@ -47,10 +47,10 @@ grit("devtools_resources") {
 inspector_protocol_generate("protocol_sources") {
   visibility = [ "//content/browser" ]
   deps = [
-    "//third_party/blink/renderer/core/inspector:protocol_version",
+    "//third_party/blink/public/devtools_protocol:protocol_version",
   ]
   _blink_protocol_path = rebase_path(
-          "$root_gen_dir/third_party/blink/renderer/core/inspector/protocol.json",
+          "$root_gen_dir/third_party/blink/public/devtools_protocol/protocol.json",
           root_build_dir)
 
   inspector_protocol_dir = "//third_party/inspector_protocol"
@@ -59,7 +59,7 @@ inspector_protocol_generate("protocol_sources") {
   config_values = [ "protocol.path=$_blink_protocol_path" ]
 
   inputs = [
-    "$root_gen_dir/third_party/blink/renderer/core/inspector/protocol.json",
+    "$root_gen_dir/third_party/blink/public/devtools_protocol/protocol.json",
     config_file,
   ]
 

headless/BUILD.gn

@@ -170,10 +170,10 @@ foreach(domain, devtools_domains) {
 action("gen_devtools_client_api") {
   script = "lib/browser/devtools_api/client_api_generator.py"
   deps = [
-    "//third_party/blink/renderer/core/inspector:protocol_version",
+    "//third_party/blink/public/devtools_protocol:protocol_version",
   ]
   inputs = [
-    "$root_gen_dir/third_party/blink/renderer/core/inspector/protocol.json",
+    "$root_gen_dir/third_party/blink/public/devtools_protocol/protocol.json",
   ]
 
   outputs = generated_devtools_api
@@ -213,10 +213,10 @@ if (headless_fontconfig_utils && !is_fuchsia) {
 inspector_protocol_generate("protocol_sources") {
   visibility = [ "//headless:headless_shared_sources" ]
   deps = [
-    "//third_party/blink/renderer/core/inspector:protocol_version",
+    "//third_party/blink/public/devtools_protocol:protocol_version",
   ]
   _blink_protocol_path = rebase_path(
-          "$root_gen_dir/third_party/blink/renderer/core/inspector/protocol.json",
+          "$root_gen_dir/third_party/blink/public/devtools_protocol/protocol.json",
           root_build_dir)
 
   inspector_protocol_dir = "//third_party/inspector_protocol"
@@ -225,7 +225,7 @@ inspector_protocol_generate("protocol_sources") {
   config_values = [ "protocol.path=$_blink_protocol_path" ]
 
   inputs = [
-    "$root_gen_dir/third_party/blink/renderer/core/inspector/protocol.json",
+    "$root_gen_dir/third_party/blink/public/devtools_protocol/protocol.json",
     config_file,
   ]
 

third_party/blink/public/devtools_protocol/BUILD.gn

+import("//third_party/inspector_protocol/inspector_protocol.gni")
+import("//v8/gni/v8.gni")
+
+_inspector_protocol_dir = "//third_party/inspector_protocol"
+_protocol_out_dir = "$root_gen_dir/third_party/blink/public/devtools_protocol"
+
+action("protocol_compatibility_check") {
+  script = _inspector_protocol_dir + "/check_protocol_compatibility.py"
+
+  inputs = [
+    "browser_protocol.pdl",
+    "browser_protocol-1.3.json",
+    v8_inspector_js_protocol,
+  ]
+  _stamp = "$_protocol_out_dir/browser_protocol.stamp"
+  outputs = [
+    _stamp,
+  ]
+
+  args = [
+    "--stamp",
+    rebase_path(_stamp, root_build_dir),
+    rebase_path("browser_protocol.pdl", root_build_dir),
+    rebase_path(v8_inspector_js_protocol, root_build_dir),
+  ]
+}
+
+action("protocol_version") {
+  deps = [
+    ":protocol_compatibility_check",
+  ]
+  script = _inspector_protocol_dir + "/concatenate_protocols.py"
+
+  inputs = [
+    "browser_protocol.pdl",
+    v8_inspector_js_protocol,
+  ]
+  output_file = "$_protocol_out_dir/protocol.json"
+  outputs = [
+    output_file,
+  ]
+
+  args = [
+    rebase_path("browser_protocol.pdl", root_build_dir),
+    rebase_path(v8_inspector_js_protocol, root_build_dir),
+    rebase_path(output_file, root_build_dir),
+  ]
+}

third_party/blink/public/devtools_protocol/OWNERS

+alph@chromium.org
+caseq@chromium.org
+dgozman@chromium.org
+pfeldman@chromium.org
+
+# COMPONENT: Platform>DevTools

third_party/blink/renderer/core/inspector/BUILD.gn

@@ -114,6 +114,8 @@ blink_core_sources("inspector") {
 
 inspector_protocol_generate("protocol_sources") {
   inspector_protocol_dir = _inspector_protocol_dir
+  browser_protocol =
+      "//third_party/blink/public/devtools_protocol/browser_protocol.pdl"
   out_dir = blink_core_output_dir
 
   config_file = "inspector_protocol_config.json"
@@ -121,8 +123,8 @@ inspector_protocol_generate("protocol_sources") {
   config_values = [ "imported.path=$_imported" ]
 
   inputs = [
-    "browser_protocol.pdl",
     v8_inspector_js_protocol,
+    browser_protocol,
     "inspector_protocol_config.json",
   ]
 
@@ -217,45 +219,17 @@ jumbo_source_set("generated") {
   ]
 }
 
-action("protocol_compatibility_check") {
-  script = _inspector_protocol_dir + "/check_protocol_compatibility.py"
-
-  inputs = [
-    "browser_protocol.pdl",
-    "browser_protocol-1.3.json",
-    v8_inspector_js_protocol,
-  ]
-  _stamp = "$blink_core_output_dir/inspector/browser_protocol.stamp"
-  outputs = [
-    _stamp,
-  ]
-
-  args = [
-    "--stamp",
-    rebase_path(_stamp, root_build_dir),
-    rebase_path("browser_protocol.pdl", root_build_dir),
-    rebase_path(v8_inspector_js_protocol, root_build_dir),
-  ]
-}
-
-action("protocol_version") {
-  deps = [
-    ":protocol_compatibility_check",
-  ]
-  script = _inspector_protocol_dir + "/concatenate_protocols.py"
-
-  inputs = [
-    "browser_protocol.pdl",
-    v8_inspector_js_protocol,
+# TODO(caseq): remove once devtools-frontend is updated to use the
+# correct location.
+copy("protocol_version") {
+  sources = [
+    "$root_gen_dir/third_party/blink/public/devtools_protocol/protocol.json",
   ]
   output_file = "$blink_core_output_dir/inspector/protocol.json"
   outputs = [
     output_file,
   ]
-
-  args = [
-    rebase_path("browser_protocol.pdl", root_build_dir),
-    rebase_path(v8_inspector_js_protocol, root_build_dir),
-    rebase_path(output_file, root_build_dir),
+  deps = [
+    "//third_party/blink/public/devtools_protocol:protocol_version",
   ]
 }

third_party/blink/renderer/core/inspector/OWNERS

@@ -4,12 +4,4 @@ dgozman@chromium.org
 pfeldman@chromium.org
 yangguo@chromium.org
 
-# Changes to remote debugging protocol require devtools review to
-# ensure backwards compatibility and committment to maintain.
-per-file browser_protocol.pdl=set noparent
-per-file browser_protocol.pdl=alph@chromium.org
-per-file browser_protocol.pdl=dgozman@chromium.org
-per-file browser_protocol.pdl=caseq@chromium.org
-per-file browser_protocol.pdl=pfeldman@chromium.org
-
 # COMPONENT: Platform>DevTools

third_party/blink/renderer/core/inspector/PRESUBMIT.py

-# Copyright 2017 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.
-
-
-def _CompileDevtoolsFrontend(input_api, output_api):
-    # Need to get all affected files from change (not just within this subtree)
-    local_paths = [f.AbsoluteLocalPath() for f in input_api.change.AffectedFiles()]
-    devtools = input_api.os_path.realpath(
-        input_api.os_path.join(input_api.PresubmitLocalPath(), "..", "..", "..", "..", "devtools-frontend", "src"))
-
-    # If a devtools file is changed, the PRESUBMIT hook in Source/devtools
-    # will run closure compiler
-    if (any("browser_protocol.pdl" in path for path in local_paths) and
-            all(devtools not in path for path in local_paths)):
-        compile_path = input_api.os_path.join(
-            input_api.PresubmitLocalPath(), "..", "..", "..", "..", "devtools-frontend", "src", "scripts", "test", "run_type_check.py")
-        out, _ = input_api.subprocess.Popen(
-            [input_api.python_executable, compile_path], stdout=input_api.subprocess.PIPE,
-            stderr=input_api.subprocess.STDOUT).communicate()
-        if "ERROR" in out or "WARNING" in out:
-            return [output_api.PresubmitError(out)]
-        if "NOTE" in out:
-            return [output_api.PresubmitPromptWarning(out)]
-    return []
-
-
-def CheckChangeOnUpload(input_api, output_api):
-    results = []
-    results.extend(_CompileDevtoolsFrontend(input_api, output_api))
-    return results
-
-
-def CheckChangeOnCommit(input_api, output_api):
-    return []

third_party/blink/renderer/core/inspector/inspector_protocol_config.json

 {
     "protocol": {
-        "path": "browser_protocol.pdl",
+        "path": "../../../public/devtools_protocol/browser_protocol.pdl",
         "package": "third_party/blink/renderer/core/inspector/protocol",
         "output": "inspector/protocol",
         "namespace": ["blink", "protocol"],