Do not allow InspectorObject and other raw interfaces from Runtime, Network and DOM domains anymore.
Created attachment 132670 [details] Patch
List of new FIXME to review (line numbers are in new file versions). Source/WebCore/inspector/InjectedScript.cpp:177 Source/WebCore/inspector/InspectorDOMAgent.cpp:414 Source/WebCore/inspector/InspectorDOMAgent.cpp:1557
Attachment 132670 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCor..." exit_code: 1 Source/WebCore/inspector/InjectedScript.cpp:66: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.cpp:76: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.cpp:112: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.cpp:263: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:66: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:73: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:85: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:108: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Total errors found: 8 in 17 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 132670 [details] Patch Attachment 132670 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/11985701
Comment on attachment 132670 [details] Patch Attachment 132670 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/11990056
Comment on attachment 132670 [details] Patch Attachment 132670 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/11991060
Comment on attachment 132670 [details] Patch Attachment 132670 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/12000027 New failing tests: inspector/elements/dom-agent-query-selector.html inspector/styles/undo-add-new-rule.html inspector/audits/audits-panel-functional.html inspector/styles/styles-add-new-rule.html inspector/styles/add-new-rule-inline-style-csp.html inspector/styles/commit-selector.html inspector/styles/undo-set-selector-text.html inspector/audits/audits-panel-noimages-functional.html
Comment on attachment 132670 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=132670&action=review So why do the tests fail? > Source/WebCore/inspector/CodeGeneratorInspector.py:2261 > + nit: 3 blank lines? > Source/WebCore/inspector/InjectedScript.cpp:176 > + RefPtr<TypeBuilder::Runtime::RemoteObject> result = TypeBuilder::Runtime::RemoteObject::create() Wrap should never through. We should return ErrorString in this case instead. > Source/WebCore/inspector/InspectorDOMAgent.cpp:414 > + // FIXME: initialize root or error string param. I am not sure I get the comment right. What you should do here is return ErrorString claiming that document is not available. > Source/WebCore/inspector/InspectorDOMAgent.cpp:1557 > + // FIXME: make a proper default value or return error message. return error.
Comment on attachment 132670 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=132670&action=review >> Source/WebCore/inspector/CodeGeneratorInspector.py:2261 >> + > > nit: 3 blank lines? Done >> Source/WebCore/inspector/InjectedScript.cpp:176 >> + RefPtr<TypeBuilder::Runtime::RemoteObject> result = TypeBuilder::Runtime::RemoteObject::create() > > Wrap should never through. We should return ErrorString in this case instead. Done >> Source/WebCore/inspector/InspectorDOMAgent.cpp:414 >> + // FIXME: initialize root or error string param. > > I am not sure I get the comment right. What you should do here is return ErrorString claiming that document is not available. Done
Created attachment 133104 [details] Patch
Comment on attachment 133104 [details] Patch Please revise the ChangeLog and provide information on what changed and why (overall summary and per function). More info with an example: http://www.webkit.org/coding/contributing.html#changelogs
Created attachment 133110 [details] Patch
Comment on attachment 133110 [details] Patch My previous request still stands. Please revise the ChangeLog.
Attachment 133110 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCor..." exit_code: 1 Source/WebCore/inspector/InjectedScript.cpp:66: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.cpp:76: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.cpp:112: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.cpp:259: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:66: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:73: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:85: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:108: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Total errors found: 8 in 18 files If any of these errors are false positives, please file a bug against check-webkit-style.
(In reply to comment #11) > (From update of attachment 133104 [details]) > Please revise the ChangeLog and provide information on what changed and why (overall summary and per function). > > More info with an example: http://www.webkit.org/coding/contributing.html#changelogs Hi Timothy The change is pretty much mechanical. In all functions of 3 listed domains InspectorObject and InspectorArray are completely replaced with corresponding auto-generated types from TypeBuilder according to Inspector.json protocol definition. Are you sure you want this description per function? This is a mechanical work, and all I can write in functions is probably a repeating statement or retell the raw diff. Peter
Created attachment 133114 [details] Do not commit. Mac still has a problem
Attachment 133114 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCor..." exit_code: 1 Source/WebCore/inspector/InjectedScript.cpp:66: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.cpp:76: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.cpp:112: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.cpp:259: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:66: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:73: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:85: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:108: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Total errors found: 8 in 18 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 133114 [details] Do not commit. Mac still has a problem Attachment 133114 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/12070687
Comment on attachment 133114 [details] Do not commit. Mac still has a problem Attachment 133114 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/12102003
Comment on attachment 133114 [details] Do not commit. Mac still has a problem Attachment 133114 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/12090637 New failing tests: inspector/elements/dom-agent-query-selector.html inspector/styles/undo-add-new-rule.html inspector/audits/audits-panel-functional.html inspector/styles/styles-add-new-rule.html inspector/styles/add-new-rule-inline-style-csp.html inspector/styles/commit-selector.html inspector/styles/undo-set-selector-text.html inspector/audits/audits-panel-noimages-functional.html
Comment on attachment 133114 [details] Do not commit. Mac still has a problem Attachment 133114 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/12071686 New failing tests: inspector/elements/dom-agent-query-selector.html inspector/styles/undo-add-new-rule.html inspector/audits/audits-panel-functional.html inspector/styles/styles-add-new-rule.html inspector/styles/add-new-rule-inline-style-csp.html inspector/styles/commit-selector.html inspector/styles/undo-set-selector-text.html inspector/audits/audits-panel-noimages-functional.html
(In reply to comment #15) > (In reply to comment #11) > > (From update of attachment 133104 [details] [details]) > > Please revise the ChangeLog and provide information on what changed and why (overall summary and per function). > > > > More info with an example: http://www.webkit.org/coding/contributing.html#changelogs > > Hi Timothy > The change is pretty much mechanical. In all functions of 3 listed domains InspectorObject and InspectorArray are completely replaced with corresponding auto-generated types from TypeBuilder according to Inspector.json protocol definition. > Are you sure you want this description per function? This is a mechanical work, and all I can write in functions is probably a repeating statement or retell the raw diff. > Peter The new comment at the top of the ChangeLog is good. Thanks for adding that. Per function comments in this case are not needed.
Created attachment 133355 [details] Do not commit. Mac still has a problem
Attachment 133355 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCor..." exit_code: 1 Source/WebCore/inspector/InjectedScript.cpp:66: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.cpp:76: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.cpp:112: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.cpp:259: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:66: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:73: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:85: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:108: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Total errors found: 8 in 19 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 133355 [details] Do not commit. Mac still has a problem Attachment 133355 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/12121198
Comment on attachment 133355 [details] Do not commit. Mac still has a problem Attachment 133355 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/12103020 New failing tests: inspector/elements/dom-agent-query-selector.html inspector/styles/undo-add-new-rule.html inspector/audits/audits-panel-functional.html inspector/styles/styles-add-new-rule.html inspector/styles/add-new-rule-inline-style-csp.html inspector/styles/commit-selector.html inspector/styles/undo-set-selector-text.html inspector/audits/audits-panel-noimages-functional.html
Comment on attachment 133355 [details] Do not commit. Mac still has a problem Attachment 133355 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/12122329
Created attachment 133511 [details] Do not commit. Mac still has a problem
Created attachment 133533 [details] Do not commit. Mac still has a problem
Attachment 133533 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCor..." exit_code: 1 Source/WebCore/inspector/InjectedScript.cpp:66: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.cpp:76: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.cpp:112: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.cpp:259: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:66: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:73: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:85: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:108: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Total errors found: 8 in 19 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 133533 [details] Do not commit. Mac still has a problem Attachment 133533 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/12117782 New failing tests: inspector/elements/dom-agent-query-selector.html inspector/styles/undo-add-new-rule.html inspector/audits/audits-panel-functional.html inspector/styles/styles-add-new-rule.html inspector/styles/add-new-rule-inline-style-csp.html inspector/styles/commit-selector.html inspector/styles/undo-set-selector-text.html inspector/audits/audits-panel-noimages-functional.html
Created attachment 133548 [details] Archive of layout-test-results from ec2-cr-linux-04 The attached test failures were seen while running run-webkit-tests on the chromium-ews. Bot: ec2-cr-linux-04 Port: <class 'webkitpy.common.config.ports.ChromiumXVFBPort'> Platform: Linux-2.6.35-28-virtual-x86_64-with-Ubuntu-10.10-maverick
Created attachment 133574 [details] Do not commit. Tests still have a problem
Attachment 133574 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCor..." exit_code: 1 Source/WebCore/inspector/InjectedScript.cpp:66: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.cpp:76: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.cpp:112: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.cpp:259: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:66: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:73: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:85: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:108: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Total errors found: 8 in 19 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 133574 [details] Do not commit. Tests still have a problem Rejecting attachment 133574 [details] from commit-queue. Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 2 Last 500 characters of output: ce/WebCore/inspector/InspectorRuntimeAgent.h patching file Source/WebCore/inspector/InspectorValues.h patching file Source/WebCore/inspector/ScriptCallFrame.cpp patching file Source/WebCore/inspector/ScriptCallFrame.h patching file Source/WebCore/inspector/ScriptCallStack.cpp patching file Source/WebCore/inspector/ScriptCallStack.h Failed to run "[u'/mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply', u'--force', u'--reviewer', u'Pavel Feld..." exit_code: 1 cwd: /mnt/git/webkit-commit-queue/ Full output: http://queues.webkit.org/results/12132816
Created attachment 133911 [details] Patch
Attachment 133911 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCor..." exit_code: 1 Source/WebCore/inspector/InjectedScript.cpp:66: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.cpp:76: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.cpp:112: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.cpp:259: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:66: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:73: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:85: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:108: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Total errors found: 8 in 18 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 133911 [details] Patch Attachment 133911 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/12141228 New failing tests: inspector/elements/dom-agent-query-selector.html inspector/styles/undo-add-new-rule.html inspector/audits/audits-panel-functional.html inspector/styles/styles-add-new-rule.html inspector/styles/add-new-rule-inline-style-csp.html inspector/styles/commit-selector.html http/tests/inspector/modify-cross-domain-rule.html inspector/styles/undo-set-selector-text.html inspector/audits/audits-panel-noimages-functional.html
Created attachment 133945 [details] Archive of layout-test-results from ec2-cr-linux-02 The attached test failures were seen while running run-webkit-tests on the chromium-ews. Bot: ec2-cr-linux-02 Port: <class 'webkitpy.common.config.ports.ChromiumXVFBPort'> Platform: Linux-2.6.35-28-virtual-x86_64-with-Ubuntu-10.10-maverick
Created attachment 134196 [details] Patch
Attachment 134196 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCor..." exit_code: 1 Source/WebCore/inspector/InjectedScript.cpp:66: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.cpp:76: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.cpp:112: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.cpp:259: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:66: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:73: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:85: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:108: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Total errors found: 8 in 18 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 134196 [details] Patch Attachment 134196 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/12145921 New failing tests: inspector/audits/audits-panel-functional.html inspector/audits/audits-panel-noimages-functional.html
Created attachment 134209 [details] Archive of layout-test-results from ec2-cr-linux-02 The attached test failures were seen while running run-webkit-tests on the chromium-ews. Bot: ec2-cr-linux-02 Port: <class 'webkitpy.common.config.ports.ChromiumXVFBPort'> Platform: Linux-2.6.35-28-virtual-x86_64-with-Ubuntu-10.10-maverick
Comment on attachment 134196 [details] Patch Attachment 134196 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/12142935 New failing tests: inspector/audits/audits-panel-functional.html inspector/audits/audits-panel-noimages-functional.html
Created attachment 134215 [details] Archive of layout-test-results from ec2-cr-linux-01 The attached test failures were seen while running run-webkit-tests on the chromium-ews. Bot: ec2-cr-linux-01 Port: <class 'webkitpy.common.config.ports.ChromiumXVFBPort'> Platform: Linux-2.6.35-28-virtual-x86_64-with-Ubuntu-10.10-maverick
Created attachment 134326 [details] Patch
Attachment 134326 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCor..." exit_code: 1 Source/WebCore/inspector/InjectedScript.cpp:66: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.cpp:76: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.cpp:112: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.cpp:259: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:66: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:73: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:85: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Source/WebCore/inspector/InjectedScript.h:108: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] Total errors found: 8 in 18 files If any of these errors are false positives, please file a bug against check-webkit-style.
Committed r112428: <http://trac.webkit.org/changeset/112428>
Comment on attachment 134326 [details] Patch Clearing r?