deps: switch to upstream v8_inspector

This change picks up v8_inspector directly from the upstream chromium
repository [1]; dropping the intermediate repository. The upstream
code has been refactored substantially to make it easy to share code
without adaptation with Node.js.

The deps/v8_inspector directory is now simply a gathering of
dependencies:

* platform/v8_inspector: vendored from [2].
* platform/inspector_protocol: vendored from [3].
* deps/jinja2: vendored from [4].
* deps/markupsafe: vendored from [5].

[1]: https://chromium.googlesource.com/chromium/src/third_party/WebKit/Source/platform
[2]: https://chromium.googlesource.com/chromium/src/third_party/WebKit/Source/platform/v8_inspector
[3]: https://chromium.googlesource.com/chromium/src/third_party/WebKit/Source/platform/inspector_protocol
[4]: https://github.com/mitsuhiko/jinja2
[5]: https://github.com/mitsuhiko/markupsafe

PR-URL: #7302
Reviewed-By: indutny - Fedor Indutny <[email protected]>
Reviewed-By: bnoordhuis - Ben Noordhuis <[email protected]>

Author: ofrobots

configure

@@ -816,6 +816,7 @@ def configure_node(o):
 
   o['variables']['asan'] = int(options.enable_asan or 0)
   o['variables']['v8_inspector'] = b(not options.without_inspector)
+  o['variables']['debug_devtools'] = 'node'
 
   if options.use_xcode and options.use_ninja:
     raise Exception('--xcode and --ninja cannot be used together.')

deps/v8_inspector/README.md

@@ -1,2 +1,10 @@
-# v8_inspector
-# v8_inspector
+V8 Inspector support for Node.js
+================================
+
+This directory is a gathering of dependencies for Node.js support for the
+[Chrome Debug Protocol][https://developer.chrome.com/devtools/docs/debugger-protocol].
+
+* platform/v8_inspector: vendored from https://chromium.googlesource.com/chromium/src/third_party/WebKit/Source/platform/v8_inspector
+* platform/inspector_protocol: vendored from https://chromium.googlesource.com/chromium/src/third_party/WebKit/Source/platform/inspector_protocol
+* deps/jinja2: vendored from https://github.com/mitsuhiko/jinja2
+* deps/markupsafe: vendored from https://github.com/mitsuhiko/markupsafe

deps/v8_inspector/devtools/Inspector-1.1.json

@@ -5,9 +5,9 @@
 #ifndef Array_h
 #define Array_h
 
-#include "platform/PlatformExport.h"
 #include "platform/inspector_protocol/Collections.h"
 #include "platform/inspector_protocol/ErrorSupport.h"
+#include "platform/inspector_protocol/Platform.h"
 #include "platform/inspector_protocol/String16.h"
 #include "platform/inspector_protocol/ValueConversions.h"
 #include "platform/inspector_protocol/Values.h"
@@ -91,7 +91,7 @@ class Array {
             errors->addError("array expected");
             return nullptr;
         }
-        std::unique_ptr<Array<T>> result = wrapUnique(new Array<T>());
+        std::unique_ptr<Array<T>> result(new Array<T>());
         errors->push();
         for (size_t i = 0; i < array->size(); ++i) {
             errors->setName(String16::number(i));

deps/v8_inspector/devtools/protocol.json

@@ -0,0 +1,23 @@
+// Copyright (c) 2016 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.
+
+#ifndef BackendCallback_h
+#define BackendCallback_h
+
+#include "platform/inspector_protocol/ErrorSupport.h"
+#include "platform/inspector_protocol/Platform.h"
+
+namespace blink {
+namespace protocol {
+
+class PLATFORM_EXPORT BackendCallback {
+public:
+    virtual ~BackendCallback() { }
+    virtual void sendFailure(const ErrorString&) = 0;
+};
+
+} // namespace platform
+} // namespace blink
+
+#endif // !defined(BackendCallback_h)