Bug 81558 – Web Inspector: CodeGeneratorInspector.py: switch Runtime, Network and DOM domains to typed API

• Component: Web Inspector (Deprecated)
• Status: RESOLVED FIXED

[Peter Rybin]

Do not allow InspectorObject and other raw interfaces from Runtime, Network and DOM domains anymore.

[Peter Rybin]

Created attachment 132670 [details]
Patch

[Peter Rybin]

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

[WebKit Review Bot]

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.

[Build Bot]

Comment on attachment 132670 [details]
Patch

Attachment 132670 [details] did not pass win-ews (win):
Output: http://queues.webkit.org/results/11985701

[Build Bot]

Comment on attachment 132670 [details]
Patch

Attachment 132670 [details] did not pass mac-ews (mac):
Output: http://queues.webkit.org/results/11990056

[Build Bot]

Comment on attachment 132670 [details]
Patch

Attachment 132670 [details] did not pass mac-ews (mac):
Output: http://queues.webkit.org/results/11991060

[WebKit Review Bot]

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

[Pavel Feldman]

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.

[Peter Rybin]

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

[Peter Rybin]

Created attachment 133104 [details]
Patch

[Timothy Hatcher]

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

[Peter Rybin]

Created attachment 133110 [details]
Patch

[Timothy Hatcher]

Comment on attachment 133110 [details]
Patch

My previous request still stands. Please revise the ChangeLog.

[WebKit Review Bot]

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.

[Peter Rybin]

(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

[Peter Rybin]

Created attachment 133114 [details]
Do not commit. Mac still has a problem

[WebKit Review Bot]

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.

[Build Bot]

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

[Build Bot]

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

[WebKit Review Bot]

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

[WebKit Review Bot]

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

[Timothy Hatcher]

(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.

[Peter Rybin]

Created attachment 133355 [details]
Do not commit. Mac still has a problem

[WebKit Review Bot]

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.

[Build Bot]

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

[WebKit Review Bot]

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

[Build Bot]

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

[Peter Rybin]

Created attachment 133511 [details]
Do not commit. Mac still has a problem

[Peter Rybin]

Created attachment 133533 [details]
Do not commit. Mac still has a problem

[WebKit Review Bot]

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.

[WebKit Review Bot]

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

[WebKit Review Bot]

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

[Peter Rybin]

Created attachment 133574 [details]
Do not commit. Tests still have a problem

[WebKit Review Bot]

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.

[WebKit Review Bot]

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

[Peter Rybin]

Created attachment 133911 [details]
Patch

[WebKit Review Bot]

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.

[WebKit Review Bot]

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

[WebKit Review Bot]

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

[Peter Rybin]

Created attachment 134196 [details]
Patch

[WebKit Review Bot]

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.

[WebKit Review Bot]

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

[WebKit Review Bot]

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

[WebKit Review Bot]

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

[WebKit Review Bot]

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

[Peter Rybin]

Created attachment 134326 [details]
Patch

[WebKit Review Bot]

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.

[Vsevolod Vlasov]

Committed r112428: <http://trac.webkit.org/changeset/112428>

[Vsevolod Vlasov]

Comment on attachment 134326 [details]
Patch

Clearing r?