Bug 160192 – Binding generator should expose the visible interface name in error messages

• Status: RESOLVED
• Resolution: FIXED
• Component: WebCore Misc.

[youenn fablet]

Currently the binding generator is generating error messages with the interface name, which may be different from the name exposed to scripts.

[youenn fablet]

Created attachment 284572 [details]
Patch

[youenn fablet]

Created attachment 284576 [details]
Patch

[Build Bot]

Comment on attachment 284576 [details]
Patch

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

New failing tests:
media/encrypted-media/encrypted-media-v2-syntax.html
fast/dom/Window/getComputedStyle-missing-parameter.html
js/getOwnPropertyDescriptor-window-attributes.html
js/dom/toString-and-valueOf-override.html
webaudio/mediaelementaudiosourcenode.html
editing/selection/extend.html
webaudio/decode-audio-data-basic.html
webaudio/createMediaStreamSource-null.html
js/getOwnPropertyDescriptor-unforgeable-attributes.html

[Build Bot]

Created attachment 284579 [details]
Archive of layout-test-results from ews102 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews102  Port: mac-yosemite  Platform: Mac OS X 10.10.5

[Build Bot]

Comment on attachment 284576 [details]
Patch

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

New failing tests:
media/encrypted-media/encrypted-media-v2-syntax.html
fast/dom/Window/getComputedStyle-missing-parameter.html
js/getOwnPropertyDescriptor-window-attributes.html
js/dom/toString-and-valueOf-override.html
webaudio/decode-audio-data-basic.html
editing/selection/extend.html
webaudio/mediaelementaudiosourcenode.html
webaudio/createMediaStreamSource-null.html
js/getOwnPropertyDescriptor-unforgeable-attributes.html

[Build Bot]

Created attachment 284580 [details]
Archive of layout-test-results from ews105 for mac-yosemite-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews105  Port: mac-yosemite-wk2  Platform: Mac OS X 10.10.5

[Build Bot]

Comment on attachment 284576 [details]
Patch

Attachment 284576 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: http://webkit-queues.webkit.org/results/1755945

New failing tests:
editing/selection/extend.html
js/getOwnPropertyDescriptor-window-attributes.html
fast/dom/Window/getComputedStyle-missing-parameter.html
js/getOwnPropertyDescriptor-unforgeable-attributes.html
js/dom/toString-and-valueOf-override.html

[Build Bot]

Created attachment 284581 [details]
Archive of layout-test-results from ews121 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews121  Port: ios-simulator-wk2  Platform: Mac OS X 10.11.5

[Build Bot]

Comment on attachment 284576 [details]
Patch

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

New failing tests:
media/encrypted-media/encrypted-media-v2-syntax.html
fast/dom/Window/getComputedStyle-missing-parameter.html
js/getOwnPropertyDescriptor-window-attributes.html
js/dom/toString-and-valueOf-override.html
webaudio/decode-audio-data-basic.html
editing/selection/extend.html
webaudio/mediaelementaudiosourcenode.html
webaudio/createMediaStreamSource-null.html
js/getOwnPropertyDescriptor-unforgeable-attributes.html

[Build Bot]

Created attachment 284582 [details]
Archive of layout-test-results from ews117 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews117  Port: mac-yosemite  Platform: Mac OS X 10.10.5

[youenn fablet]

Created attachment 284590 [details]
Rebasing tests

[Alex Christensen]

Comment on attachment 284590 [details]
Rebasing tests

View in context: https://bugs.webkit.org/attachment.cgi?id=284590&action=review

What's the motivation for this change?

> LayoutTests/media/encrypted-media/encrypted-media-v2-syntax-expected.txt:25
> -TEST(mediaKeySession.update(null)) THROWS('TypeError: Argument 1 (\'key\') to MediaKeySession.update must be an instance of Uint8Array') OK
> +TEST(mediaKeySession.update(null)) THROWS('TypeError: Argument 1 (\'key\') to webkitMediaKeySession.update must be an instance of Uint8Array') FAIL

This OK to FAIL doesn't look good.

[youenn fablet]

(In reply to comment #12)
> Comment on attachment 284590 [details]
> Rebasing tests
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=284590&action=review
> 
> What's the motivation for this change?

Improve error message quality.
Using FetchRequest or FetchResponse is misleading for web developers.
We should use Request or Response.

> 
> > LayoutTests/media/encrypted-media/encrypted-media-v2-syntax-expected.txt:25
> > -TEST(mediaKeySession.update(null)) THROWS('TypeError: Argument 1 (\'key\') to MediaKeySession.update must be an instance of Uint8Array') OK
> > +TEST(mediaKeySession.update(null)) THROWS('TypeError: Argument 1 (\'key\') to webkitMediaKeySession.update must be an instance of Uint8Array') FAIL
> 
> This OK to FAIL doesn't look good.

Right, the first letter should be capital W.

[Alex Christensen]

(In reply to comment #13)
> > What's the motivation for this change?
> 
> Improve error message quality.
> Using FetchRequest or FetchResponse is misleading for web developers.
> We should use Request or Response.
Can this be shown in a layout test?

[youenn fablet]

(In reply to comment #14)
> (In reply to comment #13)
> > > What's the motivation for this change?
> > 
> > Improve error message quality.
> > Using FetchRequest or FetchResponse is misleading for web developers.
> > We should use Request or Response.
> Can this be shown in a layout test?

There is one test in fetch/fetch-error-messages.hrml.
JS built-ins fetch API correctly use "Request"/"Response".
But binding generated code does not.

If that helps, I can add another test in that file to exercice this for binding generated methods.
That said, this is somehow redundant with all rebased tests in terms of non-regression.

[youenn fablet]

Created attachment 284763 [details]
Adding FetchRequest test

[WebKit Commit Bot]

Comment on attachment 284763 [details]
Adding FetchRequest test

Clearing flags on attachment: 284763

Committed r203858: <http://trac.webkit.org/changeset/203858>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.