Bug 160635

Summary: Web Inspector: clean up prefixing of Automation protocol generated files
Product: WebKit Reporter: BJ Burg <bburg>
Component: Web InspectorAssignee: BJ Burg <bburg>
Severity: Normal CC: bburg, commit-queue, joepeck, keith_miller, mark.lam, mattbaker, msaboff, nvasilyev, saam, timothy, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: All   
OS: All   
Description Flags
Proposed Fix timothy: review+, bburg: commit-queue-

Description BJ Burg 2016-08-06 14:55:17 PDT
Comment 1 Radar WebKit Bug Importer 2016-08-06 14:55:30 PDT
Comment 2 BJ Burg 2016-08-06 21:37:01 PDT
Created attachment 285520 [details]
Proposed Fix
Comment 3 BJ Burg 2016-08-06 21:37:25 PDT
Comment on attachment 285520 [details]
Proposed Fix

Needs to be landed with internal changes, do not cq+.
Comment 4 WebKit Commit Bot 2016-08-06 21:38:16 PDT
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 5 WebKit Commit Bot 2016-08-06 21:38:23 PDT
Attachment 285520 [details] did not pass style-queue:

ERROR: Source/JavaScriptCore/inspector/scripts/codegen/objc_generator.py:97:  [ObjCGenerator.protocol_name] Instance of 'ObjCGenerator' has no 'model' member  [pylint/E1101] [5]
ERROR: Source/JavaScriptCore/inspector/scripts/codegen/cpp_generator.py:51:  [CppGenerator.protocol_name] Instance of 'CppGenerator' has no 'model' member  [pylint/E1101] [5]
Total errors found: 2 in 18 files

If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 6 BJ Burg 2016-08-08 09:45:09 PDT
Committed r204255: <http://trac.webkit.org/changeset/204255>