Bug 69692 – [File API] URL methods required arguments

• Component: New Bugs
• Status: RESOLVED INVALID

[Mark Pilgrim (Google)]

According to http://dev.w3.org/2006/webapi/FileAPI/#creating-revoking , the URL object has two methods, each of which as a required parameter:

createObjectURL(blob)
revokeObjectURL(url)

This test calls those methods without the required parameter, and logs the results.

WebKit passes this test.

[Mark Pilgrim (Google)]

Created attachment 110252 [details]
Patch

[WebKit Review Bot]

Comment on attachment 110252 [details]
Patch

Clearing flags on attachment: 110252

Committed r97544: <http://trac.webkit.org/changeset/97544>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.

[Csaba Osztrogonác]

This test fails on Qt and on SL bot:

--- /Volumes/Big/slave/snowleopard-intel-debug-tests/build/layout-test-results/fast/files/url-required-arguments-expected.txt 
+++ /Volumes/Big/slave/snowleopard-intel-debug-tests/build/layout-test-results/fast/files/url-required-arguments-actual.txt 
@@ -2,7 +2,7 @@
 
 On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
 
-PASS webkitURL.createObjectURL() threw exception TypeError: Type error.
+PASS webkitURL.createObjectURL() threw exception TypeError: Not enough arguments.
 PASS webkitURL.revokeObjectURL() threw exception TypeError: Not enough arguments.
 PASS successfullyParsed is true

Is it a JSC vs. V8 difference problem? As far as I remember in this case JSC result should be the platform independent result and the V8 result should platform specific. Reopen to fix expected results.

[Adam Barth]

-PASS webkitURL.createObjectURL() threw exception TypeError: Type error.
+PASS webkitURL.createObjectURL() threw exception TypeError: Not enough arguments.

Not enough arguments is the correct text of the exception.  We should change V8 to match JSC.

@Mark: Would you like to take care of this, or should I?

[Csaba Osztrogonác]

I added a Qt specific expected file to make the buildbot green. 
Please remove it with the proper fix. Thanks in advance.

[Ryosuke Niwa]

Moved JSC results to cross-platform dir. and then moved V8 specific results to platform/chromium.
http://trac.webkit.org/changeset/97632

[Anne van Kesteren]

Seems this was only applicable to V8.