Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(132)

Unified Diff: src/v8-inspector/inspector.gyp

Issue 2184733002: [inspector] add inspector protocol parser test build target. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: add missing file Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: src/v8-inspector/inspector.gyp
diff --git a/src/v8-inspector/inspector.gyp b/src/v8-inspector/inspector.gyp
new file mode 100644
index 0000000000000000000000000000000000000000..0ecfd91eedb37c87b9b36c1dc506a96a95a12a91
--- /dev/null
+++ b/src/v8-inspector/inspector.gyp
@@ -0,0 +1,142 @@
+# Copyright 2016 the V8 project authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
Michael Achenbach 2016/07/26 14:27:45 Reviewed gyp consistency. Somebody with more knowl
+{ 'variables': {
+ 'protocol_path': '../../platform/inspector_protocol',
+ 'inspector_path': '../../src/v8-inspector',
+ 'protocol_sources': [
+ '<(SHARED_INTERMEDIATE_DIR)/inspector/Console.cpp',
+ '<(SHARED_INTERMEDIATE_DIR)/inspector/Console.h',
+ '<(SHARED_INTERMEDIATE_DIR)/inspector/Debugger.cpp',
+ '<(SHARED_INTERMEDIATE_DIR)/inspector/Debugger.h',
+ '<(SHARED_INTERMEDIATE_DIR)/inspector/HeapProfiler.cpp',
+ '<(SHARED_INTERMEDIATE_DIR)/inspector/HeapProfiler.h',
+ '<(SHARED_INTERMEDIATE_DIR)/inspector/Profiler.cpp',
+ '<(SHARED_INTERMEDIATE_DIR)/inspector/Profiler.h',
+ '<(SHARED_INTERMEDIATE_DIR)/inspector/Runtime.cpp',
+ '<(SHARED_INTERMEDIATE_DIR)/inspector/Runtime.h',
+ ]
+ },
+ 'targets': [
+ { 'target_name': 'inspector_protocol_version',
+ 'type': 'none',
+ 'actions': [
+ {
+ 'action_name': 'generate_inspector_protocol_version',
+ 'inputs': [
+ '<(protocol_path)/generate-inspector-protocol-version',
+ '<(inspector_path)/js_protocol.json',
+ ],
+ 'outputs': [
+ '<(SHARED_INTERMEDIATE_DIR)/inspector/protocol.json',
+ ],
+ 'action': [
+ 'python',
+ '<(protocol_path)/generate-inspector-protocol-version',
+ '--o',
+ '<@(_outputs)',
+ '<(inspector_path)/js_protocol.json'
+ ],
+ 'message': 'Validate inspector protocol for backwards compatibility and generate version file',
+ },
+ ]
+ },
+ { 'target_name': 'inspector_protocol_sources',
+ 'type': 'none',
+ 'dependencies': ['inspector_protocol_version'],
+ 'variables': {
+ 'jinja_module_files': [
+ # jinja2/__init__.py contains version string, so sufficient for package
+ '../third_party/jinja2/__init__.py',
+ '../third_party/markupsafe/__init__.py', # jinja2 dep
+ ]
+ },
+ 'actions': [
+ {
+ 'action_name': 'generate_inspector_protocol_sources',
+ 'inputs': [
+ # Source generator script.
+ '<(protocol_path)/CodeGenerator.py',
+ # Source code templates.
+ '<(protocol_path)/Exported_h.template',
+ '<(protocol_path)/Imported_h.template',
+ '<(protocol_path)/TypeBuilder_h.template',
+ '<(protocol_path)/TypeBuilder_cpp.template',
+ # Protocol definition.
+ '<(inspector_path)/js_protocol.json',
+ ],
+ 'outputs': [
+ '<@(protocol_sources)',
+ ],
+ 'action': [
+ 'python',
+ '<(protocol_path)/CodeGenerator.py',
+ '--protocol', '<(inspector_path)/js_protocol.json',
+ '--string_type', 'String',
+ '--export_macro', 'PLATFORM_EXPORT',
+ '--output_dir', '<(SHARED_INTERMEDIATE_DIR)/inspector',
+ '--output_package', 'inspector',
+ ],
+ 'message': 'Generating Inspector protocol backend sources from json definitions',
+ },
+ ]
+ },
+ { 'target_name': 'inspector_protocol',
+ 'type': 'static_library',
+ 'dependencies': [
+ 'inspector_protocol_sources',
+ '../../testing/gmock.gyp:gmock',
Michael Achenbach 2016/07/26 14:27:45 Why is this dependent on gmock and gtest and not t
Yang 2016/07/27 06:42:25 You are right. Fixed.
+ '../../testing/gtest.gyp:gtest',
+ ],
+ 'include_dirs+': [
+ '../..',
+ '<(SHARED_INTERMEDIATE_DIR)',
+ ],
+ 'defines': [
+ 'V8_INSPECTOR_USE_STL',
+ ],
+ 'sources': [
+ '<@(protocol_sources)',
+ '<(protocol_path)/Allocator.h',
+ '<(protocol_path)/Array.h',
+ '<(protocol_path)/BackendCallback.h',
+ '<(protocol_path)/CodeGenerator.py',
+ '<(protocol_path)/Collections.h',
+ '<(protocol_path)/DispatcherBase.cpp',
+ '<(protocol_path)/DispatcherBase.h',
+ '<(protocol_path)/ErrorSupport.cpp',
+ '<(protocol_path)/ErrorSupport.h',
+ '<(protocol_path)/FrontendChannel.h',
+ '<(protocol_path)/Maybe.h',
+ '<(protocol_path)/Object.cpp',
+ '<(protocol_path)/Object.h',
+ '<(protocol_path)/Parser.cpp',
+ '<(protocol_path)/Parser.h',
+ '<(protocol_path)/Platform.h',
+ '<(protocol_path)/PlatformSTL.h',
+ '<(protocol_path)/String16.h',
+ '<(protocol_path)/String16STL.cpp',
+ '<(protocol_path)/String16STL.h',
+ '<(protocol_path)/ValueConversions.h',
+ '<(protocol_path)/Values.cpp',
+ '<(protocol_path)/Values.h',
+ ]
+ },
+ { 'target_name': 'inspector_protocol_parser_test',
+ 'type': 'executable',
+ 'dependencies': ['inspector_protocol'],
+ 'include_dirs+': [
+ '../..',
+ '../../testing/gtest/include',
+ ],
+ 'defines': [
+ 'V8_INSPECTOR_USE_STL',
+ ],
+ 'sources': [
+ '<(protocol_path)/ParserTest.cpp',
+ '../../test/inspector_protocol/RunTests.cpp',
+ ]
+ },
+ ],
+}

Powered by Google App Engine
This is Rietveld 408576698