RESOLVED FIXED 125595
Web Inspector: Add Inspector Code Generation to JavaScriptCore for Runtime Domain
https://bugs.webkit.org/show_bug.cgi?id=125595
Summary Web Inspector: Add Inspector Code Generation to JavaScriptCore for Runtime Do...
Joseph Pecoraro
Reported 2013-12-11 11:39:21 PST
Next step for JSContext Inspection: Inspector Interface generation in JSC Some of the domains will be in JavaScriptCore. For example, the Runtime domain. This step will include: 1. Move the Runtime.json domain into JavaScriptCore 2. Move code generation scripts and add derived sources steps JavaScriptCore 3. Update CodeGeneration scripts to output dynamic filenames, and deal with a split protocol (some domains to generate, other domains are dependencies generated elsewhere) 4. Update the WebCore derived sources to account for the protocol split 5. Now that the protocol is split, update everyone that used to include "InspectorBackendCommands.js" to include the appropriate set of files => currently everyone will include both InspectorJSBackendCommands.js and InspectorWebBackendCommands.js The Runtime domain will not end up being the only domain pushed into JSC, but it is a good starting point because the Runtime.json protocol definition does not depend on any other domains for types. It is standalone. Eventually getting this Agent up and running in just JavaScriptCore is still a ways away (we are missing InjectedScript, InjectedScriptManager, InjectedScriptHost, etc), but this is a hefty step forward already!
Attachments
[PATCH] Proposed Fix (187.32 KB, patch)
2013-12-11 18:13 PST, Joseph Pecoraro
joepeck: commit-queue-
Archive of layout-test-results from webkit-ews-16 for mac-mountainlion-wk2 (485.28 KB, application/zip)
2013-12-11 19:32 PST, Build Bot
no flags
Archive of layout-test-results from webkit-ews-14 for mac-mountainlion-wk2 (482.65 KB, application/zip)
2013-12-11 20:18 PST, Build Bot
no flags
[PATCH] Proposed Fix (194.39 KB, patch)
2013-12-12 10:55 PST, Joseph Pecoraro
eflews.bot: commit-queue-
[PATCH] Proposed Fix (191.14 KB, patch)
2013-12-12 11:20 PST, Joseph Pecoraro
eflews.bot: commit-queue-
[PATCH] Proposed Fix (191.18 KB, patch)
2013-12-12 11:49 PST, Joseph Pecoraro
no flags
[PATCH] Proposed Fix (191.49 KB, patch)
2013-12-12 12:00 PST, Joseph Pecoraro
timothy: review+
eflews.bot: commit-queue-
[PATCH] For Bots 1 (191.90 KB, patch)
2013-12-12 12:45 PST, Joseph Pecoraro
eflews.bot: commit-queue-
[PATCH] For Bots 2 (192.32 KB, patch)
2013-12-12 13:11 PST, Joseph Pecoraro
eflews.bot: commit-queue-
[PATCH] For Bots 3 (192.48 KB, patch)
2013-12-12 13:32 PST, Joseph Pecoraro
eflews.bot: commit-queue-
Archive of layout-test-results from webkit-ews-15 for mac-mountainlion-wk2 (472.29 KB, application/zip)
2013-12-12 13:56 PST, Build Bot
no flags
[PATCH] For Bots 4 (192.62 KB, patch)
2013-12-12 14:17 PST, Joseph Pecoraro
no flags
[Patch] includes windows fixes (454.96 KB, patch)
2013-12-13 11:40 PST, Roger Fong
no flags
[PATCH] to land (195.60 KB, patch)
2013-12-13 12:28 PST, Joseph Pecoraro
no flags
Joseph Pecoraro
Comment 1 2013-12-11 18:13:53 PST
Created attachment 219026 [details] [PATCH] Proposed Fix Sorry, I don't think I can easily break this up into pieces. High level comment on how files are changing: JavaScriptCore gains: JavaScriptCore/inspector/prototype/Runtime.json JavaScriptCore/scripts/CodeGeneratorInspector.py (exported to PrivateHeaders so WebCore can use it) JavaScriptCore/scripts/CodeGeneratorInspectorStrings.py (exported to PrivateHeaders so WebCore can use it) JavaScriptCore/scripts/generate-combined-inspector-json.py (exported to PrivateHeaders so WebCore can use it) DerivedSources/InspectorJS.json (exported to PrivateHeaders so WebCore can use it) DerivedSources/InspectorJSBackendCommands.js DerivedSources/InspectorJSBackendDispatchers.{h,cpp} (exported to PrivateHeaders so WebCore can use it) DerivedSources/InspectorJSFrontendDispatchers.{h,cpp} (exported to PrivateHeaders so WebCore can use it) DerivedSources/InspectorJSTypeBuilders.{h,cpp} (exported to PrivateHeaders so WebCore can use it) WebCore's Derived Sources change name: DerivedSources/InspectorWeb.json DerivedSources/InspectorWebBackendCommands.js DerivedSources/InspectorWebBackendDispatchers.{h,cpp} DerivedSources/InspectorWebFrontendDispatchers.{h,cpp} DerivedSources/InspectorWebTypeBuilders.{h,cpp} WebInspectorUI and LayoutTests: Now we have to worry about loading both a InspectorJSBackendCommands.js and InspectorWebBackendCommands.js where appropriate.
WebKit Commit Bot
Comment 2 2013-12-11 18:15:32 PST
Attachment 219026 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'ChangeLog', u'GNUmakefile.am', u'LayoutTests/ChangeLog', u'LayoutTests/http/tests/inspector-protocol/resources/InspectorTest.js', u'Source/JavaScriptCore/CMakeLists.txt', u'Source/JavaScriptCore/ChangeLog', u'Source/JavaScriptCore/DerivedSources.make', u'Source/JavaScriptCore/GNUmakefile.am', u'Source/JavaScriptCore/GNUmakefile.list.am', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCore.vcxproj', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCore.vcxproj.filters', u'Source/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj', u'Source/JavaScriptCore/inspector/protocol/Runtime.json', u'Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py', u'Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspectorStrings.py', u'Source/JavaScriptCore/inspector/scripts/generate-combined-inspector-json.py', u'Source/WebCore/CMakeLists.txt', u'Source/WebCore/ChangeLog', u'Source/WebCore/DerivedSources.make', u'Source/WebCore/ForwardingHeaders/inspector/InspectorJSBackendDispatchers.h', u'Source/WebCore/ForwardingHeaders/inspector/InspectorJSFrontendDispatchers.h', u'Source/WebCore/ForwardingHeaders/inspector/InspectorJSTypeBuilders.h', u'Source/WebCore/GNUmakefile.am', u'Source/WebCore/GNUmakefile.list.am', u'Source/WebCore/WebCore.vcxproj/WebCore.vcxproj', u'Source/WebCore/WebCore.vcxproj/WebCore.vcxproj.filters', u'Source/WebCore/WebCore.xcodeproj/project.pbxproj', u'Source/WebCore/inspector/CodeGeneratorInspector.py', u'Source/WebCore/inspector/CodeGeneratorInspectorStrings.py', u'Source/WebCore/inspector/ConsoleMessage.h', u'Source/WebCore/inspector/InjectedScriptHost.cpp', u'Source/WebCore/inspector/InspectorAgent.cpp', u'Source/WebCore/inspector/InspectorAgent.h', u'Source/WebCore/inspector/InspectorApplicationCacheAgent.cpp', u'Source/WebCore/inspector/InspectorApplicationCacheAgent.h', u'Source/WebCore/inspector/InspectorCSSAgent.h', u'Source/WebCore/inspector/InspectorCanvasAgent.cpp', u'Source/WebCore/inspector/InspectorCanvasAgent.h', u'Source/WebCore/inspector/InspectorConsoleAgent.cpp', u'Source/WebCore/inspector/InspectorConsoleAgent.h', u'Source/WebCore/inspector/InspectorController.cpp', u'Source/WebCore/inspector/InspectorDOMAgent.cpp', u'Source/WebCore/inspector/InspectorDOMAgent.h', u'Source/WebCore/inspector/InspectorDOMDebuggerAgent.cpp', u'Source/WebCore/inspector/InspectorDOMDebuggerAgent.h', u'Source/WebCore/inspector/InspectorDOMStorageAgent.cpp', u'Source/WebCore/inspector/InspectorDOMStorageAgent.h', u'Source/WebCore/inspector/InspectorDatabaseAgent.cpp', u'Source/WebCore/inspector/InspectorDatabaseAgent.h', u'Source/WebCore/inspector/InspectorDatabaseResource.cpp', u'Source/WebCore/inspector/InspectorDatabaseResource.h', u'Source/WebCore/inspector/InspectorDebuggerAgent.cpp', u'Source/WebCore/inspector/InspectorDebuggerAgent.h', u'Source/WebCore/inspector/InspectorFrontendClientLocal.cpp', u'Source/WebCore/inspector/InspectorHeapProfilerAgent.h', u'Source/WebCore/inspector/InspectorIndexedDBAgent.cpp', u'Source/WebCore/inspector/InspectorIndexedDBAgent.h', u'Source/WebCore/inspector/InspectorInputAgent.h', u'Source/WebCore/inspector/InspectorLayerTreeAgent.cpp', u'Source/WebCore/inspector/InspectorLayerTreeAgent.h', u'Source/WebCore/inspector/InspectorMemoryAgent.cpp', u'Source/WebCore/inspector/InspectorMemoryAgent.h', u'Source/WebCore/inspector/InspectorPageAgent.cpp', u'Source/WebCore/inspector/InspectorPageAgent.h', u'Source/WebCore/inspector/InspectorProfilerAgent.cpp', u'Source/WebCore/inspector/InspectorProfilerAgent.h', u'Source/WebCore/inspector/InspectorResourceAgent.cpp', u'Source/WebCore/inspector/InspectorResourceAgent.h', u'Source/WebCore/inspector/InspectorRuntimeAgent.h', u'Source/WebCore/inspector/InspectorTimelineAgent.cpp', u'Source/WebCore/inspector/InspectorTimelineAgent.h', u'Source/WebCore/inspector/InspectorWorkerAgent.cpp', u'Source/WebCore/inspector/InspectorWorkerAgent.h', u'Source/WebCore/inspector/PageRuntimeAgent.h', u'Source/WebCore/inspector/ScriptCallFrame.cpp', u'Source/WebCore/inspector/Scripts/generate-combined-inspector-json.py', u'Source/WebCore/inspector/WorkerInspectorController.cpp', u'Source/WebCore/inspector/WorkerRuntimeAgent.h', u'Source/WebCore/inspector/protocol/Runtime.json', u'Source/WebInspectorUI/ChangeLog', u'Source/WebInspectorUI/Scripts/copy-user-interface-resources.sh', u'Source/WebInspectorUI/Scripts/update-InspectorBackendCommands.rb', u'Source/WebInspectorUI/UserInterface/InspectorBackendCommands.js', u'Source/WebInspectorUI/UserInterface/InspectorJSBackendCommands.js', u'Source/WebInspectorUI/UserInterface/InspectorWebBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/6.0/InspectorBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/6.0/InspectorWebBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/7.0/InspectorBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/7.0/InspectorWebBackendCommands.js', u'Source/WebInspectorUI/UserInterface/LoadInspectorBackendCommands.js', u'Source/WebInspectorUI/WebInspectorUI.vcxproj/WebInspectorUI.vcxproj', u'Source/WebInspectorUI/WebInspectorUI.vcxproj/WebInspectorUI.vcxproj.filters', u'Source/WebInspectorUI/WebInspectorUI.xcodeproj/project.pbxproj', '--commit-queue']" exit_code: 1 ERROR: Source/WebCore/inspector/InspectorIndexedDBAgent.h:36: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorCanvasAgent.h:37: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorCSSAgent.h:30: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorController.cpp:44: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorController.cpp:53: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorRuntimeAgent.h:36: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/PageRuntimeAgent.h:36: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorDebuggerAgent.h:40: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorTimelineAgent.cpp:43: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorWorkerAgent.h:34: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorDOMAgent.cpp:72: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorLayerTreeAgent.h:35: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:92: at least two spaces before inline comment [pep8/E261] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:108: trailing whitespace [pep8/W291] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:876: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1274: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1355: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1708: expected 2 blank lines, found 1 [pep8/E302] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1794: too many blank lines (5) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1869: expected 2 blank lines, found 1 [pep8/E302] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1884: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:286: [RawTypes.BaseType.get_raw_validator_call_text] Class 'BaseType' has no 'get_validate_method_params' member [pylint/E1101] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:661: [TypeModel.Enum.get_optional] Method should have "self" as first argument [pylint/E0213] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1618: [TypeData.get_binding] Undefined variable 'Error' [pylint/E0602] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1751: [Templates.get_this_script_path_] Method should have "self" as first argument [pylint/E0213] [5] ERROR: Source/WebCore/inspector/InspectorInputAgent.h:36: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorIndexedDBAgent.cpp:62: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorMemoryAgent.h:36: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorDOMStorageAgent.h:32: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorDOMDebuggerAgent.h:36: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorAgent.h:33: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorDOMAgent.h:37: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorResourceAgent.h:35: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorDatabaseAgent.h:35: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorCanvasAgent.cpp:44: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorConsoleAgent.h:33: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/WorkerInspectorController.cpp:39: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/WorkerInspectorController.cpp:43: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorDebuggerAgent.cpp:39: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorPageAgent.cpp:66: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorFrontendClientLocal.cpp:44: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorHeapProfilerAgent.h:37: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorResourceAgent.cpp:49: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorPageAgent.h:39: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorApplicationCacheAgent.cpp:37: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorAgent.cpp:43: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorDOMDebuggerAgent.cpp:41: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorProfilerAgent.h:36: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorTimelineAgent.h:37: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorDOMStorageAgent.cpp:42: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorApplicationCacheAgent.h:32: Alphabetical sorting problem. [build/include_order] [4] Total errors found: 51 in 85 files If any of these errors are false positives, please file a bug against check-webkit-style.
EFL EWS Bot
Comment 3 2013-12-11 18:17:19 PST
Comment on attachment 219026 [details] [PATCH] Proposed Fix Attachment 219026 [details] did not pass efl-ews (efl): Output: http://webkit-queues.appspot.com/results/48108115
Joseph Pecoraro
Comment 4 2013-12-11 18:42:26 PST
> [ 15%] Generating ../../DerivedSources/WebCore/InspectorWebBackendDispatchers.cpp, ../../DerivedSources/WebCore/InspectorWebBackendDispatchers.h, ../../DerivedSources/WebCore/InspectorWebFrontendDispatchers.cpp, ../../DerivedSources/WebCore/InspectorWebFrontendDispatchers.h, ../../DerivedSources/WebCore/InspectorWebTypeBuilders.cpp, ../../DerivedSources/WebCore/InspectorWebTypeBuilders.h > Traceback (most recent call last): > File "/mnt/eflews/webkit/WebKit/Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py", line 1741, in <module> > dependency_input_file = open(dependency_json_filename, "r") > IOError: [Errno 2] No such file or directory: '/mnt/eflews/webkit/WebKit/WebKitBuild/Release/DerivedSources/JavaScriptCore/InspectorJS.json' > make[2]: *** [DerivedSources/WebCore/InspectorWebBackendDispatchers.cpp] Error 1 > make[1]: *** [Source/WebCore/CMakeFiles/WebCore.dir/all] Error 2 > make: *** [all] Error 2 Oookay. So it looks like when building WebCore on EFL it has cleared or done something to the JavaScriptCore DerivedSources? I'm going to have to figure out how to carry InspectorJS.json over.
Build Bot
Comment 5 2013-12-11 19:32:34 PST
Comment on attachment 219026 [details] [PATCH] Proposed Fix Attachment 219026 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.appspot.com/results/47008264 New failing tests: http/tests/xmlhttprequest/access-control-repeated-failed-preflight-crash.html
Build Bot
Comment 6 2013-12-11 19:32:38 PST
Created attachment 219033 [details] Archive of layout-test-results from webkit-ews-16 for mac-mountainlion-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: webkit-ews-16 Port: mac-mountainlion-wk2 Platform: Mac OS X 10.8.5
EFL EWS Bot
Comment 7 2013-12-11 20:15:47 PST
Comment on attachment 219026 [details] [PATCH] Proposed Fix Attachment 219026 [details] did not pass efl-wk2-ews (efl-wk2): Output: http://webkit-queues.appspot.com/results/47618155
Build Bot
Comment 8 2013-12-11 20:18:23 PST
Comment on attachment 219026 [details] [PATCH] Proposed Fix Attachment 219026 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.appspot.com/results/47618152 New failing tests: http/tests/xmlhttprequest/access-control-repeated-failed-preflight-crash.html
Build Bot
Comment 9 2013-12-11 20:18:27 PST
Created attachment 219036 [details] Archive of layout-test-results from webkit-ews-14 for mac-mountainlion-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: webkit-ews-14 Port: mac-mountainlion-wk2 Platform: Mac OS X 10.8.5
kov's GTK+ EWS bot
Comment 10 2013-12-12 09:24:09 PST
Comment on attachment 219026 [details] [PATCH] Proposed Fix Attachment 219026 [details] did not pass gtk-ews (gtk): Output: http://webkit-queues.appspot.com/results/44318031
Joseph Pecoraro
Comment 11 2013-12-12 10:54:25 PST
Actually, all of the bots output don't make sense to me... Trying another patch.
Joseph Pecoraro
Comment 12 2013-12-12 10:55:21 PST
Created attachment 219097 [details] [PATCH] Proposed Fix
WebKit Commit Bot
Comment 13 2013-12-12 10:57:57 PST
Attachment 219097 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'ChangeLog', u'GNUmakefile.am', u'LayoutTests/ChangeLog', u'LayoutTests/http/tests/inspector-protocol/resources/InspectorTest.js', u'Source/JavaScriptCore/CMakeLists.txt', u'Source/JavaScriptCore/ChangeLog', u'Source/JavaScriptCore/DerivedSources.make', u'Source/JavaScriptCore/GNUmakefile.am', u'Source/JavaScriptCore/GNUmakefile.list.am', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCore.vcxproj', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCore.vcxproj.filters', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/copy-files.cmd', u'Source/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj', u'Source/JavaScriptCore/inspector/protocol/Runtime.json', u'Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py', u'Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspectorStrings.py', u'Source/JavaScriptCore/inspector/scripts/generate-combined-inspector-json.py', u'Source/WebCore/CMakeLists.txt', u'Source/WebCore/ChangeLog', u'Source/WebCore/DerivedSources.make', u'Source/WebCore/ForwardingHeaders/inspector/InspectorJSBackendDispatchers.h', u'Source/WebCore/ForwardingHeaders/inspector/InspectorJSFrontendDispatchers.h', u'Source/WebCore/ForwardingHeaders/inspector/InspectorJSTypeBuilders.h', u'Source/WebCore/GNUmakefile.am', u'Source/WebCore/GNUmakefile.list.am', u'Source/WebCore/WebCore.vcxproj/WebCore.vcxproj', u'Source/WebCore/WebCore.vcxproj/WebCore.vcxproj.filters', u'Source/WebCore/WebCore.xcodeproj/project.pbxproj', u'Source/WebCore/inspector/CodeGeneratorInspector.py', u'Source/WebCore/inspector/CodeGeneratorInspectorStrings.py', u'Source/WebCore/inspector/ConsoleMessage.h', u'Source/WebCore/inspector/InjectedScriptHost.cpp', u'Source/WebCore/inspector/InspectorAgent.cpp', u'Source/WebCore/inspector/InspectorAgent.h', u'Source/WebCore/inspector/InspectorApplicationCacheAgent.cpp', u'Source/WebCore/inspector/InspectorApplicationCacheAgent.h', u'Source/WebCore/inspector/InspectorCSSAgent.h', u'Source/WebCore/inspector/InspectorCanvasAgent.cpp', u'Source/WebCore/inspector/InspectorCanvasAgent.h', u'Source/WebCore/inspector/InspectorConsoleAgent.cpp', u'Source/WebCore/inspector/InspectorConsoleAgent.h', u'Source/WebCore/inspector/InspectorController.cpp', u'Source/WebCore/inspector/InspectorDOMAgent.cpp', u'Source/WebCore/inspector/InspectorDOMAgent.h', u'Source/WebCore/inspector/InspectorDOMDebuggerAgent.cpp', u'Source/WebCore/inspector/InspectorDOMDebuggerAgent.h', u'Source/WebCore/inspector/InspectorDOMStorageAgent.cpp', u'Source/WebCore/inspector/InspectorDOMStorageAgent.h', u'Source/WebCore/inspector/InspectorDatabaseAgent.cpp', u'Source/WebCore/inspector/InspectorDatabaseAgent.h', u'Source/WebCore/inspector/InspectorDatabaseResource.cpp', u'Source/WebCore/inspector/InspectorDatabaseResource.h', u'Source/WebCore/inspector/InspectorDebuggerAgent.cpp', u'Source/WebCore/inspector/InspectorDebuggerAgent.h', u'Source/WebCore/inspector/InspectorFrontendClientLocal.cpp', u'Source/WebCore/inspector/InspectorHeapProfilerAgent.h', u'Source/WebCore/inspector/InspectorIndexedDBAgent.cpp', u'Source/WebCore/inspector/InspectorIndexedDBAgent.h', u'Source/WebCore/inspector/InspectorInputAgent.h', u'Source/WebCore/inspector/InspectorLayerTreeAgent.cpp', u'Source/WebCore/inspector/InspectorLayerTreeAgent.h', u'Source/WebCore/inspector/InspectorMemoryAgent.cpp', u'Source/WebCore/inspector/InspectorMemoryAgent.h', u'Source/WebCore/inspector/InspectorPageAgent.cpp', u'Source/WebCore/inspector/InspectorPageAgent.h', u'Source/WebCore/inspector/InspectorProfilerAgent.cpp', u'Source/WebCore/inspector/InspectorProfilerAgent.h', u'Source/WebCore/inspector/InspectorResourceAgent.cpp', u'Source/WebCore/inspector/InspectorResourceAgent.h', u'Source/WebCore/inspector/InspectorRuntimeAgent.h', u'Source/WebCore/inspector/InspectorTimelineAgent.cpp', u'Source/WebCore/inspector/InspectorTimelineAgent.h', u'Source/WebCore/inspector/InspectorWorkerAgent.cpp', u'Source/WebCore/inspector/InspectorWorkerAgent.h', u'Source/WebCore/inspector/PageRuntimeAgent.h', u'Source/WebCore/inspector/ScriptCallFrame.cpp', u'Source/WebCore/inspector/Scripts/generate-combined-inspector-json.py', u'Source/WebCore/inspector/WorkerInspectorController.cpp', u'Source/WebCore/inspector/WorkerRuntimeAgent.h', u'Source/WebCore/inspector/protocol/Runtime.json', u'Source/WebInspectorUI/ChangeLog', u'Source/WebInspectorUI/Scripts/copy-user-interface-resources.sh', u'Source/WebInspectorUI/Scripts/update-InspectorBackendCommands.rb', u'Source/WebInspectorUI/UserInterface/InspectorBackendCommands.js', u'Source/WebInspectorUI/UserInterface/InspectorJSBackendCommands.js', u'Source/WebInspectorUI/UserInterface/InspectorWebBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/6.0/InspectorBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/6.0/InspectorWebBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/7.0/InspectorBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/7.0/InspectorWebBackendCommands.js', u'Source/WebInspectorUI/UserInterface/LoadInspectorBackendCommands.js', u'Source/WebInspectorUI/WebInspectorUI.vcxproj/WebInspectorUI.vcxproj', u'Source/WebInspectorUI/WebInspectorUI.vcxproj/WebInspectorUI.vcxproj.filters', u'Source/WebInspectorUI/WebInspectorUI.xcodeproj/project.pbxproj', '--commit-queue']" exit_code: 1 ERROR: Source/WebCore/inspector/InspectorIndexedDBAgent.h:36: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorCanvasAgent.h:37: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorCSSAgent.h:30: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorController.cpp:44: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorController.cpp:53: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorRuntimeAgent.h:36: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/PageRuntimeAgent.h:36: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorDebuggerAgent.h:40: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorTimelineAgent.cpp:43: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorWorkerAgent.h:34: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorDOMAgent.cpp:72: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorLayerTreeAgent.h:35: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:92: at least two spaces before inline comment [pep8/E261] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:108: trailing whitespace [pep8/W291] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:876: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1274: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1355: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1708: expected 2 blank lines, found 1 [pep8/E302] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1794: too many blank lines (5) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1869: expected 2 blank lines, found 1 [pep8/E302] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1884: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:286: [RawTypes.BaseType.get_raw_validator_call_text] Class 'BaseType' has no 'get_validate_method_params' member [pylint/E1101] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:661: [TypeModel.Enum.get_optional] Method should have "self" as first argument [pylint/E0213] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1618: [TypeData.get_binding] Undefined variable 'Error' [pylint/E0602] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1751: [Templates.get_this_script_path_] Method should have "self" as first argument [pylint/E0213] [5] ERROR: Source/WebCore/inspector/InspectorInputAgent.h:36: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorIndexedDBAgent.cpp:62: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorMemoryAgent.h:36: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorDOMStorageAgent.h:32: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorDOMDebuggerAgent.h:36: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorAgent.h:33: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorDOMAgent.h:37: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorResourceAgent.h:35: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorDatabaseAgent.h:35: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorCanvasAgent.cpp:44: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorConsoleAgent.h:33: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/WorkerInspectorController.cpp:39: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/WorkerInspectorController.cpp:43: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorDebuggerAgent.cpp:39: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorPageAgent.cpp:66: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorFrontendClientLocal.cpp:44: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorHeapProfilerAgent.h:37: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorResourceAgent.cpp:49: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorPageAgent.h:39: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorApplicationCacheAgent.cpp:37: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorAgent.cpp:43: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorDOMDebuggerAgent.cpp:41: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorProfilerAgent.h:36: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorTimelineAgent.h:37: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorDOMStorageAgent.cpp:42: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/inspector/InspectorApplicationCacheAgent.h:32: Alphabetical sorting problem. [build/include_order] [4] Total errors found: 51 in 86 files If any of these errors are false positives, please file a bug against check-webkit-style.
EFL EWS Bot
Comment 14 2013-12-12 11:02:57 PST
Comment on attachment 219097 [details] [PATCH] Proposed Fix Attachment 219097 [details] did not pass efl-ews (efl): Output: http://webkit-queues.appspot.com/results/46348081
EFL EWS Bot
Comment 15 2013-12-12 11:11:05 PST
Comment on attachment 219097 [details] [PATCH] Proposed Fix Attachment 219097 [details] did not pass efl-wk2-ews (efl-wk2): Output: http://webkit-queues.appspot.com/results/44418038
Joseph Pecoraro
Comment 16 2013-12-12 11:20:39 PST
Created attachment 219100 [details] [PATCH] Proposed Fix Noticed at least one typo in GNUMakefile. Added extra dependencies hoping that helps.
WebKit Commit Bot
Comment 17 2013-12-12 11:23:46 PST
Attachment 219100 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'ChangeLog', u'GNUmakefile.am', u'LayoutTests/ChangeLog', u'LayoutTests/http/tests/inspector-protocol/resources/InspectorTest.js', u'Source/JavaScriptCore/CMakeLists.txt', u'Source/JavaScriptCore/ChangeLog', u'Source/JavaScriptCore/DerivedSources.make', u'Source/JavaScriptCore/GNUmakefile.am', u'Source/JavaScriptCore/GNUmakefile.list.am', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCore.vcxproj', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCore.vcxproj.filters', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/copy-files.cmd', u'Source/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj', u'Source/JavaScriptCore/inspector/protocol/Runtime.json', u'Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py', u'Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspectorStrings.py', u'Source/JavaScriptCore/inspector/scripts/generate-combined-inspector-json.py', u'Source/WebCore/CMakeLists.txt', u'Source/WebCore/ChangeLog', u'Source/WebCore/DerivedSources.make', u'Source/WebCore/ForwardingHeaders/inspector/InspectorJSBackendDispatchers.h', u'Source/WebCore/ForwardingHeaders/inspector/InspectorJSFrontendDispatchers.h', u'Source/WebCore/ForwardingHeaders/inspector/InspectorJSTypeBuilders.h', u'Source/WebCore/GNUmakefile.am', u'Source/WebCore/GNUmakefile.list.am', u'Source/WebCore/WebCore.vcxproj/WebCore.vcxproj', u'Source/WebCore/WebCore.vcxproj/WebCore.vcxproj.filters', u'Source/WebCore/WebCore.xcodeproj/project.pbxproj', u'Source/WebCore/inspector/CodeGeneratorInspector.py', u'Source/WebCore/inspector/CodeGeneratorInspectorStrings.py', u'Source/WebCore/inspector/ConsoleMessage.h', u'Source/WebCore/inspector/InjectedScriptHost.cpp', u'Source/WebCore/inspector/InspectorAgent.cpp', u'Source/WebCore/inspector/InspectorAgent.h', u'Source/WebCore/inspector/InspectorApplicationCacheAgent.cpp', u'Source/WebCore/inspector/InspectorApplicationCacheAgent.h', u'Source/WebCore/inspector/InspectorCSSAgent.h', u'Source/WebCore/inspector/InspectorCanvasAgent.cpp', u'Source/WebCore/inspector/InspectorCanvasAgent.h', u'Source/WebCore/inspector/InspectorConsoleAgent.cpp', u'Source/WebCore/inspector/InspectorConsoleAgent.h', u'Source/WebCore/inspector/InspectorController.cpp', u'Source/WebCore/inspector/InspectorDOMAgent.cpp', u'Source/WebCore/inspector/InspectorDOMAgent.h', u'Source/WebCore/inspector/InspectorDOMDebuggerAgent.cpp', u'Source/WebCore/inspector/InspectorDOMDebuggerAgent.h', u'Source/WebCore/inspector/InspectorDOMStorageAgent.cpp', u'Source/WebCore/inspector/InspectorDOMStorageAgent.h', u'Source/WebCore/inspector/InspectorDatabaseAgent.cpp', u'Source/WebCore/inspector/InspectorDatabaseAgent.h', u'Source/WebCore/inspector/InspectorDatabaseResource.cpp', u'Source/WebCore/inspector/InspectorDatabaseResource.h', u'Source/WebCore/inspector/InspectorDebuggerAgent.cpp', u'Source/WebCore/inspector/InspectorDebuggerAgent.h', u'Source/WebCore/inspector/InspectorFrontendClientLocal.cpp', u'Source/WebCore/inspector/InspectorHeapProfilerAgent.h', u'Source/WebCore/inspector/InspectorIndexedDBAgent.cpp', u'Source/WebCore/inspector/InspectorIndexedDBAgent.h', u'Source/WebCore/inspector/InspectorInputAgent.h', u'Source/WebCore/inspector/InspectorLayerTreeAgent.cpp', u'Source/WebCore/inspector/InspectorLayerTreeAgent.h', u'Source/WebCore/inspector/InspectorMemoryAgent.cpp', u'Source/WebCore/inspector/InspectorMemoryAgent.h', u'Source/WebCore/inspector/InspectorPageAgent.cpp', u'Source/WebCore/inspector/InspectorPageAgent.h', u'Source/WebCore/inspector/InspectorProfilerAgent.cpp', u'Source/WebCore/inspector/InspectorProfilerAgent.h', u'Source/WebCore/inspector/InspectorResourceAgent.cpp', u'Source/WebCore/inspector/InspectorResourceAgent.h', u'Source/WebCore/inspector/InspectorRuntimeAgent.h', u'Source/WebCore/inspector/InspectorTimelineAgent.cpp', u'Source/WebCore/inspector/InspectorTimelineAgent.h', u'Source/WebCore/inspector/InspectorWorkerAgent.cpp', u'Source/WebCore/inspector/InspectorWorkerAgent.h', u'Source/WebCore/inspector/PageRuntimeAgent.h', u'Source/WebCore/inspector/ScriptCallFrame.cpp', u'Source/WebCore/inspector/Scripts/generate-combined-inspector-json.py', u'Source/WebCore/inspector/WorkerInspectorController.cpp', u'Source/WebCore/inspector/WorkerRuntimeAgent.h', u'Source/WebCore/inspector/protocol/Runtime.json', u'Source/WebInspectorUI/ChangeLog', u'Source/WebInspectorUI/Scripts/copy-user-interface-resources.sh', u'Source/WebInspectorUI/Scripts/update-InspectorBackendCommands.rb', u'Source/WebInspectorUI/UserInterface/InspectorBackendCommands.js', u'Source/WebInspectorUI/UserInterface/InspectorJSBackendCommands.js', u'Source/WebInspectorUI/UserInterface/InspectorWebBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/6.0/InspectorBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/6.0/InspectorWebBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/7.0/InspectorBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/7.0/InspectorWebBackendCommands.js', u'Source/WebInspectorUI/UserInterface/LoadInspectorBackendCommands.js', u'Source/WebInspectorUI/WebInspectorUI.vcxproj/WebInspectorUI.vcxproj', u'Source/WebInspectorUI/WebInspectorUI.vcxproj/WebInspectorUI.vcxproj.filters', u'Source/WebInspectorUI/WebInspectorUI.xcodeproj/project.pbxproj', '--commit-queue']" exit_code: 1 ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:876: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1274: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1355: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1795: too many blank lines (5) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1886: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:286: [RawTypes.BaseType.get_raw_validator_call_text] Class 'BaseType' has no 'get_validate_method_params' member [pylint/E1101] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:661: [TypeModel.Enum.get_optional] Method should have "self" as first argument [pylint/E0213] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1752: [Templates.get_this_script_path_] Method should have "self" as first argument [pylint/E0213] [5] Total errors found: 8 in 86 files If any of these errors are false positives, please file a bug against check-webkit-style.
EFL EWS Bot
Comment 18 2013-12-12 11:27:09 PST
Comment on attachment 219100 [details] [PATCH] Proposed Fix Attachment 219100 [details] did not pass efl-ews (efl): Output: http://webkit-queues.appspot.com/results/47808044
Joseph Pecoraro
Comment 19 2013-12-12 11:31:28 PST
GTK build error: > GEN DerivedSources/JavaScriptCore/InspectorJS.json > ... > GEN DerivedSources/JavaScriptCore/InspectorJSBackendDispatchers.cpp > ... > GEN DerivedSources/WebCore/InspectorWebBackendDispatchers.cpp > Traceback (most recent call last): > File "../../Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py", line 1733, in <module> > input_file = open(input_json_filename, "r") > IOError: [Errno 2] No such file or directory: 'DerivedSources/JavaScriptCore/InspectorJS.json' Maybe I need to use the absolute path to this file.
Joseph Pecoraro
Comment 20 2013-12-12 11:34:44 PST
(In reply to comment #19) > GTK build error: > > > GEN DerivedSources/JavaScriptCore/InspectorJS.json > > ... > > GEN DerivedSources/JavaScriptCore/InspectorJSBackendDispatchers.cpp > > ... > > GEN DerivedSources/WebCore/InspectorWebBackendDispatchers.cpp > > Traceback (most recent call last): > > File "../../Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py", line 1733, in <module> > > input_file = open(input_json_filename, "r") > > IOError: [Errno 2] No such file or directory: 'DerivedSources/JavaScriptCore/InspectorJS.json' > > Maybe I need to use the absolute path to this file. Oops, found it. I was outputting InspectorJS to WebCore's derived sources. Copy paste fail.
EFL EWS Bot
Comment 21 2013-12-12 11:38:38 PST
Comment on attachment 219100 [details] [PATCH] Proposed Fix Attachment 219100 [details] did not pass efl-wk2-ews (efl-wk2): Output: http://webkit-queues.appspot.com/results/47878014
Joseph Pecoraro
Comment 22 2013-12-12 11:43:31 PST
EFL Build error: > make[2]: *** No rule to make target `DerivedSources/JavaScriptCore/InspectorJS.json', needed by `DerivedSources/WebCore/InspectorWebBackendDispatchers.cpp'. Stop. That doesn't make sense to me. Source/JavaScriptCore/CMakeLists.txt has: add_custom_command( OUTPUT ${DERIVED_SOURCES_JAVASCRIPTCORE_DIR}/InspectorJS.json MAIN_DEPENDENCY ${JavaScriptCore_INSPECTOR_SCRIPTS_DIR}/generate-combined-inspector-json.py DEPENDS ${JavaScriptCore_INSPECTOR_DOMAINS} COMMAND ${PYTHON_EXECUTABLE} ${JavaScriptCore_INSPECTOR_SCRIPTS_DIR}/generate-combined-inspector-json.py ${JAVASCRIPTCORE_DIR}/inspector/protocol > ${DERIVED_SOURCES_JAVASCRIPTCORE_DIR}/InspectorJS.json VERBATIM) Is this approach (referencing a JSC makefile command output from the WebCore makefile command) not allowed?
Joseph Pecoraro
Comment 23 2013-12-12 11:49:07 PST
Created attachment 219104 [details] [PATCH] Proposed Fix Hopefully this fixes GTK.
WebKit Commit Bot
Comment 24 2013-12-12 11:51:53 PST
Attachment 219104 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'ChangeLog', u'GNUmakefile.am', u'LayoutTests/ChangeLog', u'LayoutTests/http/tests/inspector-protocol/resources/InspectorTest.js', u'Source/JavaScriptCore/CMakeLists.txt', u'Source/JavaScriptCore/ChangeLog', u'Source/JavaScriptCore/DerivedSources.make', u'Source/JavaScriptCore/GNUmakefile.am', u'Source/JavaScriptCore/GNUmakefile.list.am', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCore.vcxproj', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCore.vcxproj.filters', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/copy-files.cmd', u'Source/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj', u'Source/JavaScriptCore/inspector/protocol/Runtime.json', u'Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py', u'Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspectorStrings.py', u'Source/JavaScriptCore/inspector/scripts/generate-combined-inspector-json.py', u'Source/WebCore/CMakeLists.txt', u'Source/WebCore/ChangeLog', u'Source/WebCore/DerivedSources.make', u'Source/WebCore/ForwardingHeaders/inspector/InspectorJSBackendDispatchers.h', u'Source/WebCore/ForwardingHeaders/inspector/InspectorJSFrontendDispatchers.h', u'Source/WebCore/ForwardingHeaders/inspector/InspectorJSTypeBuilders.h', u'Source/WebCore/GNUmakefile.am', u'Source/WebCore/GNUmakefile.list.am', u'Source/WebCore/WebCore.vcxproj/WebCore.vcxproj', u'Source/WebCore/WebCore.vcxproj/WebCore.vcxproj.filters', u'Source/WebCore/WebCore.xcodeproj/project.pbxproj', u'Source/WebCore/inspector/CodeGeneratorInspector.py', u'Source/WebCore/inspector/CodeGeneratorInspectorStrings.py', u'Source/WebCore/inspector/ConsoleMessage.h', u'Source/WebCore/inspector/InjectedScriptHost.cpp', u'Source/WebCore/inspector/InspectorAgent.cpp', u'Source/WebCore/inspector/InspectorAgent.h', u'Source/WebCore/inspector/InspectorApplicationCacheAgent.cpp', u'Source/WebCore/inspector/InspectorApplicationCacheAgent.h', u'Source/WebCore/inspector/InspectorCSSAgent.h', u'Source/WebCore/inspector/InspectorCanvasAgent.cpp', u'Source/WebCore/inspector/InspectorCanvasAgent.h', u'Source/WebCore/inspector/InspectorConsoleAgent.cpp', u'Source/WebCore/inspector/InspectorConsoleAgent.h', u'Source/WebCore/inspector/InspectorController.cpp', u'Source/WebCore/inspector/InspectorDOMAgent.cpp', u'Source/WebCore/inspector/InspectorDOMAgent.h', u'Source/WebCore/inspector/InspectorDOMDebuggerAgent.cpp', u'Source/WebCore/inspector/InspectorDOMDebuggerAgent.h', u'Source/WebCore/inspector/InspectorDOMStorageAgent.cpp', u'Source/WebCore/inspector/InspectorDOMStorageAgent.h', u'Source/WebCore/inspector/InspectorDatabaseAgent.cpp', u'Source/WebCore/inspector/InspectorDatabaseAgent.h', u'Source/WebCore/inspector/InspectorDatabaseResource.cpp', u'Source/WebCore/inspector/InspectorDatabaseResource.h', u'Source/WebCore/inspector/InspectorDebuggerAgent.cpp', u'Source/WebCore/inspector/InspectorDebuggerAgent.h', u'Source/WebCore/inspector/InspectorFrontendClientLocal.cpp', u'Source/WebCore/inspector/InspectorHeapProfilerAgent.h', u'Source/WebCore/inspector/InspectorIndexedDBAgent.cpp', u'Source/WebCore/inspector/InspectorIndexedDBAgent.h', u'Source/WebCore/inspector/InspectorInputAgent.h', u'Source/WebCore/inspector/InspectorLayerTreeAgent.cpp', u'Source/WebCore/inspector/InspectorLayerTreeAgent.h', u'Source/WebCore/inspector/InspectorMemoryAgent.cpp', u'Source/WebCore/inspector/InspectorMemoryAgent.h', u'Source/WebCore/inspector/InspectorPageAgent.cpp', u'Source/WebCore/inspector/InspectorPageAgent.h', u'Source/WebCore/inspector/InspectorProfilerAgent.cpp', u'Source/WebCore/inspector/InspectorProfilerAgent.h', u'Source/WebCore/inspector/InspectorResourceAgent.cpp', u'Source/WebCore/inspector/InspectorResourceAgent.h', u'Source/WebCore/inspector/InspectorRuntimeAgent.h', u'Source/WebCore/inspector/InspectorTimelineAgent.cpp', u'Source/WebCore/inspector/InspectorTimelineAgent.h', u'Source/WebCore/inspector/InspectorWorkerAgent.cpp', u'Source/WebCore/inspector/InspectorWorkerAgent.h', u'Source/WebCore/inspector/PageRuntimeAgent.h', u'Source/WebCore/inspector/ScriptCallFrame.cpp', u'Source/WebCore/inspector/Scripts/generate-combined-inspector-json.py', u'Source/WebCore/inspector/WorkerInspectorController.cpp', u'Source/WebCore/inspector/WorkerRuntimeAgent.h', u'Source/WebCore/inspector/protocol/Runtime.json', u'Source/WebInspectorUI/ChangeLog', u'Source/WebInspectorUI/Scripts/copy-user-interface-resources.sh', u'Source/WebInspectorUI/Scripts/update-InspectorBackendCommands.rb', u'Source/WebInspectorUI/UserInterface/InspectorBackendCommands.js', u'Source/WebInspectorUI/UserInterface/InspectorJSBackendCommands.js', u'Source/WebInspectorUI/UserInterface/InspectorWebBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/6.0/InspectorBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/6.0/InspectorWebBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/7.0/InspectorBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/7.0/InspectorWebBackendCommands.js', u'Source/WebInspectorUI/UserInterface/LoadInspectorBackendCommands.js', u'Source/WebInspectorUI/WebInspectorUI.vcxproj/WebInspectorUI.vcxproj', u'Source/WebInspectorUI/WebInspectorUI.vcxproj/WebInspectorUI.vcxproj.filters', u'Source/WebInspectorUI/WebInspectorUI.xcodeproj/project.pbxproj', '--commit-queue']" exit_code: 1 ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:876: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1274: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1355: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1795: too many blank lines (5) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1886: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:286: [RawTypes.BaseType.get_raw_validator_call_text] Class 'BaseType' has no 'get_validate_method_params' member [pylint/E1101] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:661: [TypeModel.Enum.get_optional] Method should have "self" as first argument [pylint/E0213] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1752: [Templates.get_this_script_path_] Method should have "self" as first argument [pylint/E0213] [5] Total errors found: 8 in 86 files If any of these errors are false positives, please file a bug against check-webkit-style.
Joseph Pecoraro
Comment 25 2013-12-12 12:00:05 PST
Created attachment 219105 [details] [PATCH] Proposed Fix More GTK fixes. Speculative EFL fixes.
WebKit Commit Bot
Comment 26 2013-12-12 12:01:37 PST
Attachment 219105 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'ChangeLog', u'GNUmakefile.am', u'LayoutTests/ChangeLog', u'LayoutTests/http/tests/inspector-protocol/resources/InspectorTest.js', u'Source/JavaScriptCore/CMakeLists.txt', u'Source/JavaScriptCore/ChangeLog', u'Source/JavaScriptCore/DerivedSources.make', u'Source/JavaScriptCore/GNUmakefile.am', u'Source/JavaScriptCore/GNUmakefile.list.am', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCore.vcxproj', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCore.vcxproj.filters', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/copy-files.cmd', u'Source/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj', u'Source/JavaScriptCore/inspector/protocol/Runtime.json', u'Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py', u'Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspectorStrings.py', u'Source/JavaScriptCore/inspector/scripts/generate-combined-inspector-json.py', u'Source/WebCore/CMakeLists.txt', u'Source/WebCore/ChangeLog', u'Source/WebCore/DerivedSources.make', u'Source/WebCore/ForwardingHeaders/inspector/InspectorJSBackendDispatchers.h', u'Source/WebCore/ForwardingHeaders/inspector/InspectorJSFrontendDispatchers.h', u'Source/WebCore/ForwardingHeaders/inspector/InspectorJSTypeBuilders.h', u'Source/WebCore/GNUmakefile.am', u'Source/WebCore/GNUmakefile.list.am', u'Source/WebCore/WebCore.vcxproj/WebCore.vcxproj', u'Source/WebCore/WebCore.vcxproj/WebCore.vcxproj.filters', u'Source/WebCore/WebCore.xcodeproj/project.pbxproj', u'Source/WebCore/inspector/CodeGeneratorInspector.py', u'Source/WebCore/inspector/CodeGeneratorInspectorStrings.py', u'Source/WebCore/inspector/ConsoleMessage.h', u'Source/WebCore/inspector/InjectedScriptHost.cpp', u'Source/WebCore/inspector/InspectorAgent.cpp', u'Source/WebCore/inspector/InspectorAgent.h', u'Source/WebCore/inspector/InspectorApplicationCacheAgent.cpp', u'Source/WebCore/inspector/InspectorApplicationCacheAgent.h', u'Source/WebCore/inspector/InspectorCSSAgent.h', u'Source/WebCore/inspector/InspectorCanvasAgent.cpp', u'Source/WebCore/inspector/InspectorCanvasAgent.h', u'Source/WebCore/inspector/InspectorConsoleAgent.cpp', u'Source/WebCore/inspector/InspectorConsoleAgent.h', u'Source/WebCore/inspector/InspectorController.cpp', u'Source/WebCore/inspector/InspectorDOMAgent.cpp', u'Source/WebCore/inspector/InspectorDOMAgent.h', u'Source/WebCore/inspector/InspectorDOMDebuggerAgent.cpp', u'Source/WebCore/inspector/InspectorDOMDebuggerAgent.h', u'Source/WebCore/inspector/InspectorDOMStorageAgent.cpp', u'Source/WebCore/inspector/InspectorDOMStorageAgent.h', u'Source/WebCore/inspector/InspectorDatabaseAgent.cpp', u'Source/WebCore/inspector/InspectorDatabaseAgent.h', u'Source/WebCore/inspector/InspectorDatabaseResource.cpp', u'Source/WebCore/inspector/InspectorDatabaseResource.h', u'Source/WebCore/inspector/InspectorDebuggerAgent.cpp', u'Source/WebCore/inspector/InspectorDebuggerAgent.h', u'Source/WebCore/inspector/InspectorFrontendClientLocal.cpp', u'Source/WebCore/inspector/InspectorHeapProfilerAgent.h', u'Source/WebCore/inspector/InspectorIndexedDBAgent.cpp', u'Source/WebCore/inspector/InspectorIndexedDBAgent.h', u'Source/WebCore/inspector/InspectorInputAgent.h', u'Source/WebCore/inspector/InspectorLayerTreeAgent.cpp', u'Source/WebCore/inspector/InspectorLayerTreeAgent.h', u'Source/WebCore/inspector/InspectorMemoryAgent.cpp', u'Source/WebCore/inspector/InspectorMemoryAgent.h', u'Source/WebCore/inspector/InspectorPageAgent.cpp', u'Source/WebCore/inspector/InspectorPageAgent.h', u'Source/WebCore/inspector/InspectorProfilerAgent.cpp', u'Source/WebCore/inspector/InspectorProfilerAgent.h', u'Source/WebCore/inspector/InspectorResourceAgent.cpp', u'Source/WebCore/inspector/InspectorResourceAgent.h', u'Source/WebCore/inspector/InspectorRuntimeAgent.h', u'Source/WebCore/inspector/InspectorTimelineAgent.cpp', u'Source/WebCore/inspector/InspectorTimelineAgent.h', u'Source/WebCore/inspector/InspectorWorkerAgent.cpp', u'Source/WebCore/inspector/InspectorWorkerAgent.h', u'Source/WebCore/inspector/PageRuntimeAgent.h', u'Source/WebCore/inspector/ScriptCallFrame.cpp', u'Source/WebCore/inspector/Scripts/generate-combined-inspector-json.py', u'Source/WebCore/inspector/WorkerInspectorController.cpp', u'Source/WebCore/inspector/WorkerRuntimeAgent.h', u'Source/WebCore/inspector/protocol/Runtime.json', u'Source/WebInspectorUI/ChangeLog', u'Source/WebInspectorUI/Scripts/copy-user-interface-resources.sh', u'Source/WebInspectorUI/Scripts/update-InspectorBackendCommands.rb', u'Source/WebInspectorUI/UserInterface/InspectorBackendCommands.js', u'Source/WebInspectorUI/UserInterface/InspectorJSBackendCommands.js', u'Source/WebInspectorUI/UserInterface/InspectorWebBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/6.0/InspectorBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/6.0/InspectorWebBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/7.0/InspectorBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/7.0/InspectorWebBackendCommands.js', u'Source/WebInspectorUI/UserInterface/LoadInspectorBackendCommands.js', u'Source/WebInspectorUI/WebInspectorUI.vcxproj/WebInspectorUI.vcxproj', u'Source/WebInspectorUI/WebInspectorUI.vcxproj/WebInspectorUI.vcxproj.filters', u'Source/WebInspectorUI/WebInspectorUI.xcodeproj/project.pbxproj', '--commit-queue']" exit_code: 1 ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:876: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1274: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1355: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1795: too many blank lines (5) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1886: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:286: [RawTypes.BaseType.get_raw_validator_call_text] Class 'BaseType' has no 'get_validate_method_params' member [pylint/E1101] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:661: [TypeModel.Enum.get_optional] Method should have "self" as first argument [pylint/E0213] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1752: [Templates.get_this_script_path_] Method should have "self" as first argument [pylint/E0213] [5] Total errors found: 8 in 86 files If any of these errors are false positives, please file a bug against check-webkit-style.
EFL EWS Bot
Comment 27 2013-12-12 12:07:09 PST
Comment on attachment 219105 [details] [PATCH] Proposed Fix Attachment 219105 [details] did not pass efl-ews (efl): Output: http://webkit-queues.appspot.com/results/45498007
Timothy Hatcher
Comment 28 2013-12-12 12:17:10 PST
Comment on attachment 219105 [details] [PATCH] Proposed Fix View in context: https://bugs.webkit.org/attachment.cgi?id=219105&action=review > Source/WebInspectorUI/UserInterface/LoadInspectorBackendCommands.js:35 > + // FIXME: When we can inspect just a JSContext, we should not load InspectorWebBackendCommands.js. Would backendURLs ever be null in the case of inspecting a JSContext?
Joseph Pecoraro
Comment 29 2013-12-12 12:26:36 PST
GTK Build error: > make[1]: *** [Source/WebCore/bindings/js/libWebCore_la-ScriptProfile.lo] Error 1 > In file included from ../../Source/WebCore/bindings/js/ScriptProfile.h:38:0, > from ../../Source/WebCore/page/Console.h:33, > from ../../Source/WebCore/bindings/js/JSDOMWindowBase.cpp:27: > ./DerivedSources/WebCore/InspectorWebTypeBuilders.h:14:47: fatal error: inspector/InspectorJSTypeBuilders.h: No such file or directory > #include <inspector/InspectorJSTypeBuilders.h> Why? Other <inspector/Foo.h> includes work. This is a new file added to WebCore/ForwardingHeaders/inspector/InspectorJSTypeBuilders.h that ultimately references the JavaScriptCore/DerivedSources/InspectorJSTypeBuilders.h. I could understand maybe the second include not working, but this error message is about the first include.
EFL EWS Bot
Comment 30 2013-12-12 12:26:39 PST
Comment on attachment 219105 [details] [PATCH] Proposed Fix Attachment 219105 [details] did not pass efl-wk2-ews (efl-wk2): Output: http://webkit-queues.appspot.com/results/48418009
Joseph Pecoraro
Comment 31 2013-12-12 12:27:54 PST
(In reply to comment #28) > (From update of attachment 219105 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=219105&action=review > > > Source/WebInspectorUI/UserInterface/LoadInspectorBackendCommands.js:35 > > + // FIXME: When we can inspect just a JSContext, we should not load InspectorWebBackendCommands.js. > > Would backendURLs ever be null in the case of inspecting a JSContext? No. We would always want to have at least one backend commands with the list of domains we can use.
Joseph Pecoraro
Comment 32 2013-12-12 12:30:37 PST
EFL Build Error: (same as GTK's right now) > In file included from /mnt/eflews/webkit/WebKit/Source/WebCore/inspector/ScriptCallStack.h:35:0, > from /mnt/eflews/webkit/WebKit/Source/WebCore/page/PageConsole.h:33, > from /mnt/eflews/webkit/WebKit/Source/WebCore/css/CSSParser.cpp:70: > /mnt/eflews/webkit/WebKit/WebKitBuild/Release/DerivedSources/WebCore/InspectorWebTypeBuilders.h:14:47: fatal error: inspector/InspectorJSTypeBuilders.h: No such file or directory Again, I don't understand this issue with seeing forwarding headers. -- At least EFL and GTK are on the same page though. The files are being generated, they just are not compiling.
Joseph Pecoraro
Comment 33 2013-12-12 12:45:02 PST
Created attachment 219109 [details] [PATCH] For Bots 1 Wild guesses.
WebKit Commit Bot
Comment 34 2013-12-12 12:47:19 PST
Attachment 219109 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'ChangeLog', u'GNUmakefile.am', u'LayoutTests/ChangeLog', u'LayoutTests/http/tests/inspector-protocol/resources/InspectorTest.js', u'Source/JavaScriptCore/CMakeLists.txt', u'Source/JavaScriptCore/ChangeLog', u'Source/JavaScriptCore/DerivedSources.make', u'Source/JavaScriptCore/GNUmakefile.am', u'Source/JavaScriptCore/GNUmakefile.list.am', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCore.vcxproj', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCore.vcxproj.filters', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/copy-files.cmd', u'Source/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj', u'Source/JavaScriptCore/inspector/protocol/Runtime.json', u'Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py', u'Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspectorStrings.py', u'Source/JavaScriptCore/inspector/scripts/generate-combined-inspector-json.py', u'Source/WebCore/CMakeLists.txt', u'Source/WebCore/ChangeLog', u'Source/WebCore/DerivedSources.make', u'Source/WebCore/ForwardingHeaders/inspector/InspectorJSBackendDispatchers.h', u'Source/WebCore/ForwardingHeaders/inspector/InspectorJSFrontendDispatchers.h', u'Source/WebCore/ForwardingHeaders/inspector/InspectorJSTypeBuilders.h', u'Source/WebCore/GNUmakefile.am', u'Source/WebCore/GNUmakefile.list.am', u'Source/WebCore/WebCore.vcxproj/WebCore.vcxproj', u'Source/WebCore/WebCore.vcxproj/WebCore.vcxproj.filters', u'Source/WebCore/WebCore.xcodeproj/project.pbxproj', u'Source/WebCore/inspector/CodeGeneratorInspector.py', u'Source/WebCore/inspector/CodeGeneratorInspectorStrings.py', u'Source/WebCore/inspector/ConsoleMessage.h', u'Source/WebCore/inspector/InjectedScriptHost.cpp', u'Source/WebCore/inspector/InspectorAgent.cpp', u'Source/WebCore/inspector/InspectorAgent.h', u'Source/WebCore/inspector/InspectorApplicationCacheAgent.cpp', u'Source/WebCore/inspector/InspectorApplicationCacheAgent.h', u'Source/WebCore/inspector/InspectorCSSAgent.h', u'Source/WebCore/inspector/InspectorCanvasAgent.cpp', u'Source/WebCore/inspector/InspectorCanvasAgent.h', u'Source/WebCore/inspector/InspectorConsoleAgent.cpp', u'Source/WebCore/inspector/InspectorConsoleAgent.h', u'Source/WebCore/inspector/InspectorController.cpp', u'Source/WebCore/inspector/InspectorDOMAgent.cpp', u'Source/WebCore/inspector/InspectorDOMAgent.h', u'Source/WebCore/inspector/InspectorDOMDebuggerAgent.cpp', u'Source/WebCore/inspector/InspectorDOMDebuggerAgent.h', u'Source/WebCore/inspector/InspectorDOMStorageAgent.cpp', u'Source/WebCore/inspector/InspectorDOMStorageAgent.h', u'Source/WebCore/inspector/InspectorDatabaseAgent.cpp', u'Source/WebCore/inspector/InspectorDatabaseAgent.h', u'Source/WebCore/inspector/InspectorDatabaseResource.cpp', u'Source/WebCore/inspector/InspectorDatabaseResource.h', u'Source/WebCore/inspector/InspectorDebuggerAgent.cpp', u'Source/WebCore/inspector/InspectorDebuggerAgent.h', u'Source/WebCore/inspector/InspectorFrontendClientLocal.cpp', u'Source/WebCore/inspector/InspectorHeapProfilerAgent.h', u'Source/WebCore/inspector/InspectorIndexedDBAgent.cpp', u'Source/WebCore/inspector/InspectorIndexedDBAgent.h', u'Source/WebCore/inspector/InspectorInputAgent.h', u'Source/WebCore/inspector/InspectorLayerTreeAgent.cpp', u'Source/WebCore/inspector/InspectorLayerTreeAgent.h', u'Source/WebCore/inspector/InspectorMemoryAgent.cpp', u'Source/WebCore/inspector/InspectorMemoryAgent.h', u'Source/WebCore/inspector/InspectorPageAgent.cpp', u'Source/WebCore/inspector/InspectorPageAgent.h', u'Source/WebCore/inspector/InspectorProfilerAgent.cpp', u'Source/WebCore/inspector/InspectorProfilerAgent.h', u'Source/WebCore/inspector/InspectorResourceAgent.cpp', u'Source/WebCore/inspector/InspectorResourceAgent.h', u'Source/WebCore/inspector/InspectorRuntimeAgent.h', u'Source/WebCore/inspector/InspectorTimelineAgent.cpp', u'Source/WebCore/inspector/InspectorTimelineAgent.h', u'Source/WebCore/inspector/InspectorWorkerAgent.cpp', u'Source/WebCore/inspector/InspectorWorkerAgent.h', u'Source/WebCore/inspector/PageRuntimeAgent.h', u'Source/WebCore/inspector/ScriptCallFrame.cpp', u'Source/WebCore/inspector/Scripts/generate-combined-inspector-json.py', u'Source/WebCore/inspector/WorkerInspectorController.cpp', u'Source/WebCore/inspector/WorkerRuntimeAgent.h', u'Source/WebCore/inspector/protocol/Runtime.json', u'Source/WebInspectorUI/ChangeLog', u'Source/WebInspectorUI/Scripts/copy-user-interface-resources.sh', u'Source/WebInspectorUI/Scripts/update-InspectorBackendCommands.rb', u'Source/WebInspectorUI/UserInterface/InspectorBackendCommands.js', u'Source/WebInspectorUI/UserInterface/InspectorJSBackendCommands.js', u'Source/WebInspectorUI/UserInterface/InspectorWebBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/6.0/InspectorBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/6.0/InspectorWebBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/7.0/InspectorBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/7.0/InspectorWebBackendCommands.js', u'Source/WebInspectorUI/UserInterface/LoadInspectorBackendCommands.js', u'Source/WebInspectorUI/WebInspectorUI.vcxproj/WebInspectorUI.vcxproj', u'Source/WebInspectorUI/WebInspectorUI.vcxproj/WebInspectorUI.vcxproj.filters', u'Source/WebInspectorUI/WebInspectorUI.xcodeproj/project.pbxproj', '--commit-queue']" exit_code: 1 ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:876: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1274: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1355: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1795: too many blank lines (5) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1886: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:286: [RawTypes.BaseType.get_raw_validator_call_text] Class 'BaseType' has no 'get_validate_method_params' member [pylint/E1101] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:661: [TypeModel.Enum.get_optional] Method should have "self" as first argument [pylint/E0213] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1752: [Templates.get_this_script_path_] Method should have "self" as first argument [pylint/E0213] [5] Total errors found: 8 in 86 files If any of these errors are false positives, please file a bug against check-webkit-style.
EFL EWS Bot
Comment 35 2013-12-12 12:55:19 PST
Comment on attachment 219109 [details] [PATCH] For Bots 1 Attachment 219109 [details] did not pass efl-ews (efl): Output: http://webkit-queues.appspot.com/results/45628023
Joseph Pecoraro
Comment 36 2013-12-12 13:11:02 PST
Created attachment 219112 [details] [PATCH] For Bots 2 More guessing, but I feel good about this guess.
WebKit Commit Bot
Comment 37 2013-12-12 13:13:37 PST
Attachment 219112 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'ChangeLog', u'GNUmakefile.am', u'LayoutTests/ChangeLog', u'LayoutTests/http/tests/inspector-protocol/resources/InspectorTest.js', u'Source/JavaScriptCore/CMakeLists.txt', u'Source/JavaScriptCore/ChangeLog', u'Source/JavaScriptCore/DerivedSources.make', u'Source/JavaScriptCore/GNUmakefile.am', u'Source/JavaScriptCore/GNUmakefile.list.am', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCore.vcxproj', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCore.vcxproj.filters', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/copy-files.cmd', u'Source/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj', u'Source/JavaScriptCore/inspector/protocol/Runtime.json', u'Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py', u'Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspectorStrings.py', u'Source/JavaScriptCore/inspector/scripts/generate-combined-inspector-json.py', u'Source/WebCore/CMakeLists.txt', u'Source/WebCore/ChangeLog', u'Source/WebCore/DerivedSources.make', u'Source/WebCore/ForwardingHeaders/inspector/InspectorJSBackendDispatchers.h', u'Source/WebCore/ForwardingHeaders/inspector/InspectorJSFrontendDispatchers.h', u'Source/WebCore/ForwardingHeaders/inspector/InspectorJSTypeBuilders.h', u'Source/WebCore/GNUmakefile.am', u'Source/WebCore/GNUmakefile.list.am', u'Source/WebCore/WebCore.vcxproj/WebCore.vcxproj', u'Source/WebCore/WebCore.vcxproj/WebCore.vcxproj.filters', u'Source/WebCore/WebCore.xcodeproj/project.pbxproj', u'Source/WebCore/inspector/CodeGeneratorInspector.py', u'Source/WebCore/inspector/CodeGeneratorInspectorStrings.py', u'Source/WebCore/inspector/ConsoleMessage.h', u'Source/WebCore/inspector/InjectedScriptHost.cpp', u'Source/WebCore/inspector/InspectorAgent.cpp', u'Source/WebCore/inspector/InspectorAgent.h', u'Source/WebCore/inspector/InspectorApplicationCacheAgent.cpp', u'Source/WebCore/inspector/InspectorApplicationCacheAgent.h', u'Source/WebCore/inspector/InspectorCSSAgent.h', u'Source/WebCore/inspector/InspectorCanvasAgent.cpp', u'Source/WebCore/inspector/InspectorCanvasAgent.h', u'Source/WebCore/inspector/InspectorConsoleAgent.cpp', u'Source/WebCore/inspector/InspectorConsoleAgent.h', u'Source/WebCore/inspector/InspectorController.cpp', u'Source/WebCore/inspector/InspectorDOMAgent.cpp', u'Source/WebCore/inspector/InspectorDOMAgent.h', u'Source/WebCore/inspector/InspectorDOMDebuggerAgent.cpp', u'Source/WebCore/inspector/InspectorDOMDebuggerAgent.h', u'Source/WebCore/inspector/InspectorDOMStorageAgent.cpp', u'Source/WebCore/inspector/InspectorDOMStorageAgent.h', u'Source/WebCore/inspector/InspectorDatabaseAgent.cpp', u'Source/WebCore/inspector/InspectorDatabaseAgent.h', u'Source/WebCore/inspector/InspectorDatabaseResource.cpp', u'Source/WebCore/inspector/InspectorDatabaseResource.h', u'Source/WebCore/inspector/InspectorDebuggerAgent.cpp', u'Source/WebCore/inspector/InspectorDebuggerAgent.h', u'Source/WebCore/inspector/InspectorFrontendClientLocal.cpp', u'Source/WebCore/inspector/InspectorHeapProfilerAgent.h', u'Source/WebCore/inspector/InspectorIndexedDBAgent.cpp', u'Source/WebCore/inspector/InspectorIndexedDBAgent.h', u'Source/WebCore/inspector/InspectorInputAgent.h', u'Source/WebCore/inspector/InspectorLayerTreeAgent.cpp', u'Source/WebCore/inspector/InspectorLayerTreeAgent.h', u'Source/WebCore/inspector/InspectorMemoryAgent.cpp', u'Source/WebCore/inspector/InspectorMemoryAgent.h', u'Source/WebCore/inspector/InspectorPageAgent.cpp', u'Source/WebCore/inspector/InspectorPageAgent.h', u'Source/WebCore/inspector/InspectorProfilerAgent.cpp', u'Source/WebCore/inspector/InspectorProfilerAgent.h', u'Source/WebCore/inspector/InspectorResourceAgent.cpp', u'Source/WebCore/inspector/InspectorResourceAgent.h', u'Source/WebCore/inspector/InspectorRuntimeAgent.h', u'Source/WebCore/inspector/InspectorTimelineAgent.cpp', u'Source/WebCore/inspector/InspectorTimelineAgent.h', u'Source/WebCore/inspector/InspectorWorkerAgent.cpp', u'Source/WebCore/inspector/InspectorWorkerAgent.h', u'Source/WebCore/inspector/PageRuntimeAgent.h', u'Source/WebCore/inspector/ScriptCallFrame.cpp', u'Source/WebCore/inspector/Scripts/generate-combined-inspector-json.py', u'Source/WebCore/inspector/WorkerInspectorController.cpp', u'Source/WebCore/inspector/WorkerRuntimeAgent.h', u'Source/WebCore/inspector/protocol/Runtime.json', u'Source/WebInspectorUI/ChangeLog', u'Source/WebInspectorUI/Scripts/copy-user-interface-resources.sh', u'Source/WebInspectorUI/Scripts/update-InspectorBackendCommands.rb', u'Source/WebInspectorUI/UserInterface/InspectorBackendCommands.js', u'Source/WebInspectorUI/UserInterface/InspectorJSBackendCommands.js', u'Source/WebInspectorUI/UserInterface/InspectorWebBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/6.0/InspectorBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/6.0/InspectorWebBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/7.0/InspectorBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/7.0/InspectorWebBackendCommands.js', u'Source/WebInspectorUI/UserInterface/LoadInspectorBackendCommands.js', u'Source/WebInspectorUI/WebInspectorUI.vcxproj/WebInspectorUI.vcxproj', u'Source/WebInspectorUI/WebInspectorUI.vcxproj/WebInspectorUI.vcxproj.filters', u'Source/WebInspectorUI/WebInspectorUI.xcodeproj/project.pbxproj', '--commit-queue']" exit_code: 1 ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:876: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1274: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1355: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1795: too many blank lines (5) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1886: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:286: [RawTypes.BaseType.get_raw_validator_call_text] Class 'BaseType' has no 'get_validate_method_params' member [pylint/E1101] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:661: [TypeModel.Enum.get_optional] Method should have "self" as first argument [pylint/E0213] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1752: [Templates.get_this_script_path_] Method should have "self" as first argument [pylint/E0213] [5] Total errors found: 8 in 86 files If any of these errors are false positives, please file a bug against check-webkit-style.
EFL EWS Bot
Comment 38 2013-12-12 13:15:38 PST
Comment on attachment 219112 [details] [PATCH] For Bots 2 Attachment 219112 [details] did not pass efl-ews (efl): Output: http://webkit-queues.appspot.com/results/48488005
Joseph Pecoraro
Comment 39 2013-12-12 13:32:55 PST
Created attachment 219114 [details] [PATCH] For Bots 3
WebKit Commit Bot
Comment 40 2013-12-12 13:51:56 PST
Attachment 219114 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'ChangeLog', u'GNUmakefile.am', u'LayoutTests/ChangeLog', u'LayoutTests/http/tests/inspector-protocol/resources/InspectorTest.js', u'Source/JavaScriptCore/CMakeLists.txt', u'Source/JavaScriptCore/ChangeLog', u'Source/JavaScriptCore/DerivedSources.make', u'Source/JavaScriptCore/GNUmakefile.am', u'Source/JavaScriptCore/GNUmakefile.list.am', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCore.vcxproj', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCore.vcxproj.filters', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/copy-files.cmd', u'Source/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj', u'Source/JavaScriptCore/inspector/protocol/Runtime.json', u'Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py', u'Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspectorStrings.py', u'Source/JavaScriptCore/inspector/scripts/generate-combined-inspector-json.py', u'Source/WebCore/CMakeLists.txt', u'Source/WebCore/ChangeLog', u'Source/WebCore/DerivedSources.make', u'Source/WebCore/ForwardingHeaders/inspector/InspectorJSBackendDispatchers.h', u'Source/WebCore/ForwardingHeaders/inspector/InspectorJSFrontendDispatchers.h', u'Source/WebCore/ForwardingHeaders/inspector/InspectorJSTypeBuilders.h', u'Source/WebCore/GNUmakefile.am', u'Source/WebCore/GNUmakefile.list.am', u'Source/WebCore/WebCore.vcxproj/WebCore.vcxproj', u'Source/WebCore/WebCore.vcxproj/WebCore.vcxproj.filters', u'Source/WebCore/WebCore.xcodeproj/project.pbxproj', u'Source/WebCore/inspector/CodeGeneratorInspector.py', u'Source/WebCore/inspector/CodeGeneratorInspectorStrings.py', u'Source/WebCore/inspector/ConsoleMessage.h', u'Source/WebCore/inspector/InjectedScriptHost.cpp', u'Source/WebCore/inspector/InspectorAgent.cpp', u'Source/WebCore/inspector/InspectorAgent.h', u'Source/WebCore/inspector/InspectorApplicationCacheAgent.cpp', u'Source/WebCore/inspector/InspectorApplicationCacheAgent.h', u'Source/WebCore/inspector/InspectorCSSAgent.h', u'Source/WebCore/inspector/InspectorCanvasAgent.cpp', u'Source/WebCore/inspector/InspectorCanvasAgent.h', u'Source/WebCore/inspector/InspectorConsoleAgent.cpp', u'Source/WebCore/inspector/InspectorConsoleAgent.h', u'Source/WebCore/inspector/InspectorController.cpp', u'Source/WebCore/inspector/InspectorDOMAgent.cpp', u'Source/WebCore/inspector/InspectorDOMAgent.h', u'Source/WebCore/inspector/InspectorDOMDebuggerAgent.cpp', u'Source/WebCore/inspector/InspectorDOMDebuggerAgent.h', u'Source/WebCore/inspector/InspectorDOMStorageAgent.cpp', u'Source/WebCore/inspector/InspectorDOMStorageAgent.h', u'Source/WebCore/inspector/InspectorDatabaseAgent.cpp', u'Source/WebCore/inspector/InspectorDatabaseAgent.h', u'Source/WebCore/inspector/InspectorDatabaseResource.cpp', u'Source/WebCore/inspector/InspectorDatabaseResource.h', u'Source/WebCore/inspector/InspectorDebuggerAgent.cpp', u'Source/WebCore/inspector/InspectorDebuggerAgent.h', u'Source/WebCore/inspector/InspectorFrontendClientLocal.cpp', u'Source/WebCore/inspector/InspectorHeapProfilerAgent.h', u'Source/WebCore/inspector/InspectorIndexedDBAgent.cpp', u'Source/WebCore/inspector/InspectorIndexedDBAgent.h', u'Source/WebCore/inspector/InspectorInputAgent.h', u'Source/WebCore/inspector/InspectorLayerTreeAgent.cpp', u'Source/WebCore/inspector/InspectorLayerTreeAgent.h', u'Source/WebCore/inspector/InspectorMemoryAgent.cpp', u'Source/WebCore/inspector/InspectorMemoryAgent.h', u'Source/WebCore/inspector/InspectorPageAgent.cpp', u'Source/WebCore/inspector/InspectorPageAgent.h', u'Source/WebCore/inspector/InspectorProfilerAgent.cpp', u'Source/WebCore/inspector/InspectorProfilerAgent.h', u'Source/WebCore/inspector/InspectorResourceAgent.cpp', u'Source/WebCore/inspector/InspectorResourceAgent.h', u'Source/WebCore/inspector/InspectorRuntimeAgent.h', u'Source/WebCore/inspector/InspectorTimelineAgent.cpp', u'Source/WebCore/inspector/InspectorTimelineAgent.h', u'Source/WebCore/inspector/InspectorWorkerAgent.cpp', u'Source/WebCore/inspector/InspectorWorkerAgent.h', u'Source/WebCore/inspector/PageRuntimeAgent.h', u'Source/WebCore/inspector/ScriptCallFrame.cpp', u'Source/WebCore/inspector/Scripts/generate-combined-inspector-json.py', u'Source/WebCore/inspector/WorkerInspectorController.cpp', u'Source/WebCore/inspector/WorkerRuntimeAgent.h', u'Source/WebCore/inspector/protocol/Runtime.json', u'Source/WebInspectorUI/ChangeLog', u'Source/WebInspectorUI/Scripts/copy-user-interface-resources.sh', u'Source/WebInspectorUI/Scripts/update-InspectorBackendCommands.rb', u'Source/WebInspectorUI/UserInterface/InspectorBackendCommands.js', u'Source/WebInspectorUI/UserInterface/InspectorJSBackendCommands.js', u'Source/WebInspectorUI/UserInterface/InspectorWebBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/6.0/InspectorBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/6.0/InspectorWebBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/7.0/InspectorBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/7.0/InspectorWebBackendCommands.js', u'Source/WebInspectorUI/UserInterface/LoadInspectorBackendCommands.js', u'Source/WebInspectorUI/WebInspectorUI.vcxproj/WebInspectorUI.vcxproj', u'Source/WebInspectorUI/WebInspectorUI.vcxproj/WebInspectorUI.vcxproj.filters', u'Source/WebInspectorUI/WebInspectorUI.xcodeproj/project.pbxproj', '--commit-queue']" exit_code: 1 ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:876: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1274: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1355: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1795: too many blank lines (5) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1886: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:286: [RawTypes.BaseType.get_raw_validator_call_text] Class 'BaseType' has no 'get_validate_method_params' member [pylint/E1101] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:661: [TypeModel.Enum.get_optional] Method should have "self" as first argument [pylint/E0213] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1752: [Templates.get_this_script_path_] Method should have "self" as first argument [pylint/E0213] [5] Total errors found: 8 in 86 files If any of these errors are false positives, please file a bug against check-webkit-style.
EFL EWS Bot
Comment 41 2013-12-12 13:54:50 PST
Comment on attachment 219114 [details] [PATCH] For Bots 3 Attachment 219114 [details] did not pass efl-ews (efl): Output: http://webkit-queues.appspot.com/results/48498018
Build Bot
Comment 42 2013-12-12 13:56:52 PST
Comment on attachment 219105 [details] [PATCH] Proposed Fix Attachment 219105 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.appspot.com/results/48098362 New failing tests: http/tests/xmlhttprequest/access-control-repeated-failed-preflight-crash.html
Build Bot
Comment 43 2013-12-12 13:56:57 PST
Created attachment 219117 [details] Archive of layout-test-results from webkit-ews-15 for mac-mountainlion-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: webkit-ews-15 Port: mac-mountainlion-wk2 Platform: Mac OS X 10.8.5
EFL EWS Bot
Comment 44 2013-12-12 13:59:24 PST
Comment on attachment 219114 [details] [PATCH] For Bots 3 Attachment 219114 [details] did not pass efl-wk2-ews (efl-wk2): Output: http://webkit-queues.appspot.com/results/48558009
Joseph Pecoraro
Comment 45 2013-12-12 14:17:45 PST
Created attachment 219118 [details] [PATCH] For Bots 4
WebKit Commit Bot
Comment 46 2013-12-12 14:21:23 PST
Attachment 219118 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'ChangeLog', u'GNUmakefile.am', u'LayoutTests/ChangeLog', u'LayoutTests/http/tests/inspector-protocol/resources/InspectorTest.js', u'Source/JavaScriptCore/CMakeLists.txt', u'Source/JavaScriptCore/ChangeLog', u'Source/JavaScriptCore/DerivedSources.make', u'Source/JavaScriptCore/GNUmakefile.am', u'Source/JavaScriptCore/GNUmakefile.list.am', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCore.vcxproj', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCore.vcxproj.filters', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/copy-files.cmd', u'Source/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj', u'Source/JavaScriptCore/inspector/protocol/Runtime.json', u'Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py', u'Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspectorStrings.py', u'Source/JavaScriptCore/inspector/scripts/generate-combined-inspector-json.py', u'Source/WebCore/CMakeLists.txt', u'Source/WebCore/ChangeLog', u'Source/WebCore/DerivedSources.make', u'Source/WebCore/ForwardingHeaders/inspector/InspectorJSBackendDispatchers.h', u'Source/WebCore/ForwardingHeaders/inspector/InspectorJSFrontendDispatchers.h', u'Source/WebCore/ForwardingHeaders/inspector/InspectorJSTypeBuilders.h', u'Source/WebCore/GNUmakefile.am', u'Source/WebCore/GNUmakefile.list.am', u'Source/WebCore/WebCore.vcxproj/WebCore.vcxproj', u'Source/WebCore/WebCore.vcxproj/WebCore.vcxproj.filters', u'Source/WebCore/WebCore.xcodeproj/project.pbxproj', u'Source/WebCore/inspector/CodeGeneratorInspector.py', u'Source/WebCore/inspector/CodeGeneratorInspectorStrings.py', u'Source/WebCore/inspector/ConsoleMessage.h', u'Source/WebCore/inspector/InjectedScriptHost.cpp', u'Source/WebCore/inspector/InspectorAgent.cpp', u'Source/WebCore/inspector/InspectorAgent.h', u'Source/WebCore/inspector/InspectorApplicationCacheAgent.cpp', u'Source/WebCore/inspector/InspectorApplicationCacheAgent.h', u'Source/WebCore/inspector/InspectorCSSAgent.h', u'Source/WebCore/inspector/InspectorCanvasAgent.cpp', u'Source/WebCore/inspector/InspectorCanvasAgent.h', u'Source/WebCore/inspector/InspectorConsoleAgent.cpp', u'Source/WebCore/inspector/InspectorConsoleAgent.h', u'Source/WebCore/inspector/InspectorController.cpp', u'Source/WebCore/inspector/InspectorDOMAgent.cpp', u'Source/WebCore/inspector/InspectorDOMAgent.h', u'Source/WebCore/inspector/InspectorDOMDebuggerAgent.cpp', u'Source/WebCore/inspector/InspectorDOMDebuggerAgent.h', u'Source/WebCore/inspector/InspectorDOMStorageAgent.cpp', u'Source/WebCore/inspector/InspectorDOMStorageAgent.h', u'Source/WebCore/inspector/InspectorDatabaseAgent.cpp', u'Source/WebCore/inspector/InspectorDatabaseAgent.h', u'Source/WebCore/inspector/InspectorDatabaseResource.cpp', u'Source/WebCore/inspector/InspectorDatabaseResource.h', u'Source/WebCore/inspector/InspectorDebuggerAgent.cpp', u'Source/WebCore/inspector/InspectorDebuggerAgent.h', u'Source/WebCore/inspector/InspectorFrontendClientLocal.cpp', u'Source/WebCore/inspector/InspectorHeapProfilerAgent.h', u'Source/WebCore/inspector/InspectorIndexedDBAgent.cpp', u'Source/WebCore/inspector/InspectorIndexedDBAgent.h', u'Source/WebCore/inspector/InspectorInputAgent.h', u'Source/WebCore/inspector/InspectorLayerTreeAgent.cpp', u'Source/WebCore/inspector/InspectorLayerTreeAgent.h', u'Source/WebCore/inspector/InspectorMemoryAgent.cpp', u'Source/WebCore/inspector/InspectorMemoryAgent.h', u'Source/WebCore/inspector/InspectorPageAgent.cpp', u'Source/WebCore/inspector/InspectorPageAgent.h', u'Source/WebCore/inspector/InspectorProfilerAgent.cpp', u'Source/WebCore/inspector/InspectorProfilerAgent.h', u'Source/WebCore/inspector/InspectorResourceAgent.cpp', u'Source/WebCore/inspector/InspectorResourceAgent.h', u'Source/WebCore/inspector/InspectorRuntimeAgent.h', u'Source/WebCore/inspector/InspectorTimelineAgent.cpp', u'Source/WebCore/inspector/InspectorTimelineAgent.h', u'Source/WebCore/inspector/InspectorWorkerAgent.cpp', u'Source/WebCore/inspector/InspectorWorkerAgent.h', u'Source/WebCore/inspector/PageRuntimeAgent.h', u'Source/WebCore/inspector/ScriptCallFrame.cpp', u'Source/WebCore/inspector/Scripts/generate-combined-inspector-json.py', u'Source/WebCore/inspector/WorkerInspectorController.cpp', u'Source/WebCore/inspector/WorkerRuntimeAgent.h', u'Source/WebCore/inspector/protocol/Runtime.json', u'Source/WebInspectorUI/ChangeLog', u'Source/WebInspectorUI/Scripts/copy-user-interface-resources.sh', u'Source/WebInspectorUI/Scripts/update-InspectorBackendCommands.rb', u'Source/WebInspectorUI/UserInterface/InspectorBackendCommands.js', u'Source/WebInspectorUI/UserInterface/InspectorJSBackendCommands.js', u'Source/WebInspectorUI/UserInterface/InspectorWebBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/6.0/InspectorBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/6.0/InspectorWebBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/7.0/InspectorBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/7.0/InspectorWebBackendCommands.js', u'Source/WebInspectorUI/UserInterface/LoadInspectorBackendCommands.js', u'Source/WebInspectorUI/WebInspectorUI.vcxproj/WebInspectorUI.vcxproj', u'Source/WebInspectorUI/WebInspectorUI.vcxproj/WebInspectorUI.vcxproj.filters', u'Source/WebInspectorUI/WebInspectorUI.xcodeproj/project.pbxproj', '--commit-queue']" exit_code: 1 ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:876: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1274: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1355: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1795: too many blank lines (5) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1886: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:286: [RawTypes.BaseType.get_raw_validator_call_text] Class 'BaseType' has no 'get_validate_method_params' member [pylint/E1101] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:661: [TypeModel.Enum.get_optional] Method should have "self" as first argument [pylint/E0213] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1752: [Templates.get_this_script_path_] Method should have "self" as first argument [pylint/E0213] [5] Total errors found: 8 in 86 files If any of these errors are false positives, please file a bug against check-webkit-style.
kov's GTK+ EWS bot
Comment 47 2013-12-12 15:10:24 PST
Comment on attachment 219105 [details] [PATCH] Proposed Fix Attachment 219105 [details] did not pass gtk-ews (gtk): Output: http://webkit-queues.appspot.com/results/48508041
Joseph Pecoraro
Comment 48 2013-12-13 10:27:29 PST
Yay, EFL and GTK have built! Roger Fong is helping me with Windows. I have a fix for Mac as well since my changes to the WebInspectorUI.framework build phase actually uncovered something accidentally working there.
Roger Fong
Comment 49 2013-12-13 11:40:13 PST
Created attachment 219177 [details] [Patch] includes windows fixes
Joseph Pecoraro
Comment 50 2013-12-13 12:28:33 PST
Created attachment 219179 [details] [PATCH] to land Here we go!
WebKit Commit Bot
Comment 51 2013-12-13 12:30:58 PST
Attachment 219179 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'ChangeLog', u'GNUmakefile.am', u'LayoutTests/ChangeLog', u'LayoutTests/http/tests/inspector-protocol/resources/InspectorTest.js', u'Source/JavaScriptCore/CMakeLists.txt', u'Source/JavaScriptCore/ChangeLog', u'Source/JavaScriptCore/DerivedSources.make', u'Source/JavaScriptCore/GNUmakefile.am', u'Source/JavaScriptCore/GNUmakefile.list.am', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCore.vcxproj', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCore.vcxproj.filters', u'Source/JavaScriptCore/JavaScriptCore.vcxproj/copy-files.cmd', u'Source/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj', u'Source/JavaScriptCore/inspector/protocol/Runtime.json', u'Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py', u'Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspectorStrings.py', u'Source/JavaScriptCore/inspector/scripts/generate-combined-inspector-json.py', u'Source/WebCore/CMakeLists.txt', u'Source/WebCore/ChangeLog', u'Source/WebCore/DerivedSources.make', u'Source/WebCore/ForwardingHeaders/inspector/InspectorJSBackendDispatchers.h', u'Source/WebCore/ForwardingHeaders/inspector/InspectorJSFrontendDispatchers.h', u'Source/WebCore/ForwardingHeaders/inspector/InspectorJSTypeBuilders.h', u'Source/WebCore/GNUmakefile.am', u'Source/WebCore/GNUmakefile.list.am', u'Source/WebCore/WebCore.vcxproj/WebCore.vcxproj', u'Source/WebCore/WebCore.vcxproj/WebCore.vcxproj.filters', u'Source/WebCore/WebCore.vcxproj/build-generated-files.sh', u'Source/WebCore/WebCore.xcodeproj/project.pbxproj', u'Source/WebCore/inspector/CodeGeneratorInspector.py', u'Source/WebCore/inspector/CodeGeneratorInspectorStrings.py', u'Source/WebCore/inspector/ConsoleMessage.h', u'Source/WebCore/inspector/InjectedScriptHost.cpp', u'Source/WebCore/inspector/InspectorAgent.cpp', u'Source/WebCore/inspector/InspectorAgent.h', u'Source/WebCore/inspector/InspectorApplicationCacheAgent.cpp', u'Source/WebCore/inspector/InspectorApplicationCacheAgent.h', u'Source/WebCore/inspector/InspectorCSSAgent.h', u'Source/WebCore/inspector/InspectorCanvasAgent.cpp', u'Source/WebCore/inspector/InspectorCanvasAgent.h', u'Source/WebCore/inspector/InspectorConsoleAgent.cpp', u'Source/WebCore/inspector/InspectorConsoleAgent.h', u'Source/WebCore/inspector/InspectorController.cpp', u'Source/WebCore/inspector/InspectorDOMAgent.cpp', u'Source/WebCore/inspector/InspectorDOMAgent.h', u'Source/WebCore/inspector/InspectorDOMDebuggerAgent.cpp', u'Source/WebCore/inspector/InspectorDOMDebuggerAgent.h', u'Source/WebCore/inspector/InspectorDOMStorageAgent.cpp', u'Source/WebCore/inspector/InspectorDOMStorageAgent.h', u'Source/WebCore/inspector/InspectorDatabaseAgent.cpp', u'Source/WebCore/inspector/InspectorDatabaseAgent.h', u'Source/WebCore/inspector/InspectorDatabaseResource.cpp', u'Source/WebCore/inspector/InspectorDatabaseResource.h', u'Source/WebCore/inspector/InspectorDebuggerAgent.cpp', u'Source/WebCore/inspector/InspectorDebuggerAgent.h', u'Source/WebCore/inspector/InspectorFrontendClientLocal.cpp', u'Source/WebCore/inspector/InspectorHeapProfilerAgent.h', u'Source/WebCore/inspector/InspectorIndexedDBAgent.cpp', u'Source/WebCore/inspector/InspectorIndexedDBAgent.h', u'Source/WebCore/inspector/InspectorInputAgent.h', u'Source/WebCore/inspector/InspectorLayerTreeAgent.cpp', u'Source/WebCore/inspector/InspectorLayerTreeAgent.h', u'Source/WebCore/inspector/InspectorMemoryAgent.cpp', u'Source/WebCore/inspector/InspectorMemoryAgent.h', u'Source/WebCore/inspector/InspectorPageAgent.cpp', u'Source/WebCore/inspector/InspectorPageAgent.h', u'Source/WebCore/inspector/InspectorProfilerAgent.cpp', u'Source/WebCore/inspector/InspectorProfilerAgent.h', u'Source/WebCore/inspector/InspectorResourceAgent.cpp', u'Source/WebCore/inspector/InspectorResourceAgent.h', u'Source/WebCore/inspector/InspectorRuntimeAgent.h', u'Source/WebCore/inspector/InspectorTimelineAgent.cpp', u'Source/WebCore/inspector/InspectorTimelineAgent.h', u'Source/WebCore/inspector/InspectorWorkerAgent.cpp', u'Source/WebCore/inspector/InspectorWorkerAgent.h', u'Source/WebCore/inspector/PageRuntimeAgent.h', u'Source/WebCore/inspector/ScriptCallFrame.cpp', u'Source/WebCore/inspector/Scripts/generate-combined-inspector-json.py', u'Source/WebCore/inspector/WorkerInspectorController.cpp', u'Source/WebCore/inspector/WorkerRuntimeAgent.h', u'Source/WebCore/inspector/protocol/Runtime.json', u'Source/WebCore/make-generated-sources.sh', u'Source/WebInspectorUI/ChangeLog', u'Source/WebInspectorUI/Scripts/copy-user-interface-resources.sh', u'Source/WebInspectorUI/Scripts/update-InspectorBackendCommands.rb', u'Source/WebInspectorUI/UserInterface/InspectorBackendCommands.js', u'Source/WebInspectorUI/UserInterface/InspectorJSBackendCommands.js', u'Source/WebInspectorUI/UserInterface/InspectorWebBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/6.0/InspectorBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/6.0/InspectorWebBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/7.0/InspectorBackendCommands.js', u'Source/WebInspectorUI/UserInterface/Legacy/7.0/InspectorWebBackendCommands.js', u'Source/WebInspectorUI/UserInterface/LoadInspectorBackendCommands.js', u'Source/WebInspectorUI/WebInspectorUI.vcxproj/WebInspectorUI.vcxproj', u'Source/WebInspectorUI/WebInspectorUI.vcxproj/WebInspectorUI.vcxproj.filters', u'Source/WebInspectorUI/WebInspectorUI.xcodeproj/project.pbxproj', '--commit-queue']" exit_code: 1 ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:876: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1274: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1355: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1795: too many blank lines (5) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1886: too many blank lines (2) [pep8/E303] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:286: [RawTypes.BaseType.get_raw_validator_call_text] Class 'BaseType' has no 'get_validate_method_params' member [pylint/E1101] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:661: [TypeModel.Enum.get_optional] Method should have "self" as first argument [pylint/E0213] [5] ERROR: Source/JavaScriptCore/inspector/scripts/CodeGeneratorInspector.py:1752: [Templates.get_this_script_path_] Method should have "self" as first argument [pylint/E0213] [5] Total errors found: 8 in 88 files If any of these errors are false positives, please file a bug against check-webkit-style.
Blaze Burg
Comment 52 2014-01-11 18:31:47 PST
This got landed, right?
Radar WebKit Bug Importer
Comment 53 2014-01-11 18:32:08 PST
Joseph Pecoraro
Comment 54 2014-01-13 10:39:28 PST
Sorry, yes this landed a while back: <http://trac.webkit.org/changeset/160557>
Note You need to log in before you can comment on or make changes to this bug.