Bug 154508 - Web Inspector: it should be possible to omit generated code guarded by INSPECTOR_ALTERNATE_DISPATCHERS
Summary: Web Inspector: it should be possible to omit generated code guarded by INSPEC...
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (show other bugs)
Version: WebKit Nightly Build
Hardware: All All
: P2 Normal
Assignee: BJ Burg
Keywords: InRadar
Depends on:
Reported: 2016-02-20 22:37 PST by BJ Burg
Modified: 2016-02-21 16:08 PST (History)
12 users (show)

See Also:

Proposed Fix (21.04 KB, patch)
2016-02-20 22:51 PST, BJ Burg
timothy: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description BJ Burg 2016-02-20 22:37:03 PST
patch coming.
Comment 1 Radar WebKit Bug Importer 2016-02-20 22:37:19 PST
Comment 2 BJ Burg 2016-02-20 22:51:23 PST
Created attachment 271876 [details]
Proposed Fix
Comment 3 WebKit Commit Bot 2016-02-20 22:52:47 PST
This patch modifies the inspector protocol generator. Please ensure that you have rebaselined any generator test results (i.e., by running `Tools/Scripts/run-inspector-generator-tests --reset-results`)
Comment 4 WebKit Commit Bot 2016-02-20 22:52:55 PST
Attachment 271876 [details] did not pass style-queue:

ERROR: Source/JavaScriptCore/inspector/scripts/codegen/generate_cpp_backend_dispatcher_header.py:70:  [CppBackendDispatcherHeaderGenerator.generate_output] Instance of 'CppBackendDispatcherHeaderGenerator' has no 'model' member  [pylint/E1101] [5]
ERROR: Source/JavaScriptCore/inspector/scripts/codegen/generate_cpp_backend_dispatcher_header.py:205:  [CppBackendDispatcherHeaderGenerator._generate_dispatcher_declarations_for_domain] Instance of 'CppBackendDispatcherHeaderGenerator' has no 'model' member  [pylint/E1101] [5]
ERROR: Source/JavaScriptCore/inspector/scripts/codegen/generate_cpp_backend_dispatcher_implementation.py:59:  [CppBackendDispatcherImplementationGenerator.generate_output] Instance of 'CppBackendDispatcherImplementationGenerator' has no 'model' member  [pylint/E1101] [5]
ERROR: Source/JavaScriptCore/inspector/scripts/codegen/generate_cpp_backend_dispatcher_implementation.py:281:  [CppBackendDispatcherImplementationGenerator._generate_dispatcher_implementation_for_command] Instance of 'CppBackendDispatcherImplementationGenerator' has no 'model' member  [pylint/E1101] [5]
Total errors found: 4 in 10 files

If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 5 BJ Burg 2016-02-21 16:08:35 PST
Committed r196881: <http://trac.webkit.org/changeset/196881>