Closed
Bug 736687
Opened 12 years ago
Closed 12 years ago
Deprecate MozBlobBuilder
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla14
People
(Reporter: emk, Assigned: emk)
References
Details
(Keywords: addon-compat, dev-doc-complete)
Attachments
(1 file)
15.03 KB,
patch
|
sicking
:
review+
|
Details | Diff | Splinter Review |
Blob constructor was proposed as a replacement of BlobBuilder.
Yeah. I think for now we should just add a warning whenever BlobBuilder is used. Eventually we should be able to remove it.
Assignee | ||
Comment 2•12 years ago
|
||
Attachment #606941 -
Flags: review?(jonas) → review+
Assignee | ||
Updated•12 years ago
|
Keywords: checkin-needed
Comment 3•12 years ago
|
||
http://hg.mozilla.org/integration/mozilla-inbound/rev/5b2e4538f43e
Keywords: checkin-needed
Target Milestone: --- → mozilla14
Comment 4•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/5b2e4538f43e
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment 5•12 years ago
|
||
I saw this deprecation warning so I tried to use the Blob constructor, but I get this error: new Blob([new ArrayBuffer(10)]) [10:37:43.420] [Exception... "An attempt was made to use an object that is not, or is no longer, usable" code: "11" nsresult: "0x8053000b (InvalidStateError)" location: "Web Console Line: 1"] Should I file a bug, or is the Blob constructor not ready for use yet?
Assignee | ||
Comment 6•12 years ago
|
||
Please file a bug and block this bug and CC me.
Comment 7•12 years ago
|
||
RE dev-doc-needed: https://developer.mozilla.org/en/Document_Object_Model_%28DOM%29/BlobBuilder still says "As MozBlobBuilder" for Firefox I imagine we need to add a line there with "14.0 (14.0) As BlobBuilder" or something?
Assignee | ||
Comment 8•12 years ago
|
||
No, MozBlobBuilder is not (and will never be) unprefixed. Instead it will be removed.
Comment 9•12 years ago
|
||
Comment on attachment 606941 [details] [diff] [review] Warn use of MozBlobBuilder >diff --git a/dom/indexedDB/test/test_file_cross_database_copying.html b/dom/indexedDB/test/test_file_cross_database_copying.html >--- a/dom/indexedDB/test/test_file_cross_database_copying.html >+++ b/dom/indexedDB/test/test_file_cross_database_copying.html >@@ -56,19 +56,16 @@ > let result = event.target.result; > verifyBlob(result, fileData.file, 1); > yield; > > if (!refResult) { > refResult = result; > continue; > } >- >- netscape.security.PrivilegeManager.enablePrivilege("UniversalXPConnect"); >- isnot(result.mozFullPath, refResult.mozFullPath, "Different os files"); why did you remove that ?
Assignee | ||
Comment 10•12 years ago
|
||
Because Blob constructor doesn't support getFile() equivalent function.
Comment 11•12 years ago
|
||
(In reply to Masatoshi Kimura [:emk] from comment #10) > Because Blob constructor doesn't support getFile() equivalent function. The automatic test is now almost useless (w/o that check) I see that getFile() in file.js now returns only blobs, so files (nsIDOMFile) are not tested at all, I mean they are not tested in entire test suite. There must be a way to create a File, no ?
Assignee | ||
Comment 12•12 years ago
|
||
Filed bug 736687.
Assignee | ||
Comment 13•12 years ago
|
||
Sorry, bug 742941.
Comment 14•12 years ago
|
||
thanks!
Comment 15•12 years ago
|
||
Documentation updated: https://developer.mozilla.org/en/DOM/BlobBuilder Listed on Firefox 14 for developers.
Keywords: dev-doc-needed → dev-doc-complete
Updated•5 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•