Bug 87939 - Syntax error in fast/filesystem/resources/file-writer-gc-blob.js
Summary: Syntax error in fast/filesystem/resources/file-writer-gc-blob.js
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Leo Yang
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-05-31 01:16 PDT by Leo Yang
Modified: 2012-06-05 06:18 PDT (History)
5 users (show)

See Also:

Attachments
Patch (1.81 KB, patch)
2012-05-31 01:20 PDT, Leo Yang 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Leo Yang 2012-05-31 01:16:18 PDT 
var blob = tenXBlob(new Blob["lorem ipsum"]); <-- parentheses are missed around [].
Comment 1 Leo Yang 2012-05-31 01:20:34 PDT 
Created attachment 145016 [details]
Patch
Comment 2 Alexey Proskuryakov 2012-05-31 10:17:47 PDT 
What effect does this have on test results?
Comment 3 Leo Yang 2012-05-31 18:44:26 PDT 
(In reply to comment #2)
> What effect does this have on test results?

JSC generates something like: Type Error: "undefined" is not a constructor. Apparently Blob["lorem ipsum"] is undefined.

You might ask why this case passes. I would say all portings except for chromium that uses v8 don't support filesystem API so they don't run this case. The BlackBerry porting is implementing filesystem API with JSC. This case will fail using JSC.
Comment 4 Alexey Proskuryakov 2012-05-31 22:41:59 PDT 
Makes sense. What about chromium - why doesn't the test fail there? It looks like a v8 bindings bug if Blob["lorem ipsum"] === Blob.
Comment 5 Leo Yang 2012-05-31 23:00:59 PDT 
(In reply to comment #4)
> Makes sense. What about chromium - why doesn't the test fail there? It looks like a v8 bindings bug if Blob["lorem ipsum"] === Blob.

I have no idea why there is no error for v8. The fix is harmless for v8 since the cr-linux ews bot is green. IIRC, cr-linux ews bot will run layout tests.
Comment 6 Kentaro Hara 2012-06-04 20:55:16 PDT 
(In reply to comment #5)
> (In reply to comment #4)
> I have no idea why there is no error for v8. The fix is harmless for v8 since the cr-linux ews bot is green. IIRC, cr-linux ews bot will run layout tests.

That's because file-writer-gc-blob.html is skipped in LayoutTests/platform/chromium/TestExpectations due to bug 85174.
Comment 7 Alexey Proskuryakov 2012-06-04 21:14:32 PDT 
Comment on attachment 145016 [details]
Patch

Cool.
Comment 8 WebKit Review Bot 2012-06-04 21:29:49 PDT 
Comment on attachment 145016 [details]
Patch

Clearing flags on attachment: 145016

Committed r119454: <http://trac.webkit.org/changeset/119454>
Comment 9 WebKit Review Bot 2012-06-04 21:29:54 PDT 
All reviewed patches have been landed.  Closing bug.
Comment 10 Kinuko Yasuda 2012-06-05 06:18:00 PDT 
Ah I just noticed this fix.  Thanks for fixing!