Implement Blob.text()/Blob.arrayBuffer()/Blob.stream()
Categories
(Core :: DOM: File, enhancement)
Tracking
()
People
(Reporter: baku, Assigned: baku)
References
Details
(Keywords: dev-doc-complete)
Attachments
(7 files)
|
47 bytes,
text/x-phabricator-request
|
Details | Review | |
|
47 bytes,
text/x-phabricator-request
|
Details | Review | |
|
47 bytes,
text/x-phabricator-request
|
Details | Review | |
|
47 bytes,
text/x-phabricator-request
|
Details | Review | |
|
47 bytes,
text/x-phabricator-request
|
Details | Review | |
|
47 bytes,
text/x-phabricator-request
|
Details | Review | |
|
47 bytes,
text/x-phabricator-request
|
Details | Review |
These 3 methods are new in the spec. Chrome is going to ship them (they already pass WPT tests). Would be nice to ship them too.
|
Assignee | |
Comment 1•5 years ago
|
||
|
|
Assignee | |
Comment 2•5 years ago
|
||
Depends on D33826
|
|
Assignee | |
Comment 3•5 years ago
|
||
Depends on D33827
|
|
Assignee | |
Comment 4•5 years ago
|
||
Depends on D33828
|
|
Assignee | |
Comment 5•5 years ago
|
||
Depends on D33829
|
|
Assignee | |
Comment 6•5 years ago
|
||
Depends on D33830
|
|
Assignee | |
Comment 7•5 years ago
|
||
|
||
Comment 8•5 years ago
|
||
Bugbug thinks this bug is a enhancement, but please change it back in case of error.
|
||
Comment 10•5 years ago
|
||
Pushed by amarchesini@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/b83fb79138d7 Implement Blob.text()/Blob.arrayBuffer()/Blob.stream() - part 1 - Rename FetchStream to BodyStream, r=smaug https://hg.mozilla.org/integration/autoland/rev/c96028ee8aa4 Implement Blob.text()/Blob.arrayBuffer()/Blob.stream() - part 2 - FetchBodyConsumer doesn't need to be a template class, r=smaug https://hg.mozilla.org/integration/autoland/rev/0a49beaf7f21 Implement Blob.text()/Blob.arrayBuffer()/Blob.stream() - part 3 - Rename FetchBodyConsumer to BodyConsumer, r=smaug https://hg.mozilla.org/integration/autoland/rev/28eb82bc47f7 Implement Blob.text()/Blob.arrayBuffer()/Blob.stream() - part 4 - blob.text()/blob.arrayBuffer(), r=smaug https://hg.mozilla.org/integration/autoland/rev/318564bd3614 Implement Blob.text()/Blob.arrayBuffer()/Blob.stream() - part 5 - blob.stream(), r=smaug https://hg.mozilla.org/integration/autoland/rev/b7387f0cad9f Implement Blob.text()/Blob.arrayBuffer()/Blob.stream() - part 6 - tests, r=smaug https://hg.mozilla.org/integration/autoland/rev/4f5df7ee7541 Implement Blob.text()/Blob.arrayBuffer()/Blob.stream() - part 7 - WPT, r=smaug
|
||
Updated•5 years ago
|
|
||
Comment 11•5 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/b83fb79138d7
https://hg.mozilla.org/mozilla-central/rev/c96028ee8aa4
https://hg.mozilla.org/mozilla-central/rev/0a49beaf7f21
https://hg.mozilla.org/mozilla-central/rev/28eb82bc47f7
https://hg.mozilla.org/mozilla-central/rev/318564bd3614
https://hg.mozilla.org/mozilla-central/rev/b7387f0cad9f
https://hg.mozilla.org/mozilla-central/rev/4f5df7ee7541
|
||
Comment 12•5 years ago
|
||
:baku,
Do you think this is notable in the release note? If so, care to nominate is? Thank you.
|
|
Assignee | |
Comment 13•5 years ago
|
||
Release Note Request (optional, but appreciated)
[Why is this notable]: It's an important change in the FileAPI and in how to read data from Blob objects.
[Affects Firefox for Android]: no
[Suggested wording]: Blob.text(), Blob.arrayBuffer(), Blob.stream() are 3 new methods to read data from a Blob object.
[Links (documentation, blog post, etc)]: https://w3c.github.io/FileAPI/#dom-blob-stream https://w3c.github.io/FileAPI/#dom-blob-text https://w3c.github.io/FileAPI/#dom-blob-arraybuffer
|
||
Comment 14•5 years ago
|
||
That change seems of interest to web developers not the general firefox userbase, so I guess more suitable for MDN than the main release notes?
|
||
Comment 15•5 years ago
|
||
Andrea, can you clarify if you want this mentioned in both our developer's notes (MDN page) and our general notes on www.mozilla.org or only on our developer's notes? Thanks
|
|
Assignee | |
Comment 16•5 years ago
|
||
(In reply to Pascal Chevrel:pascalc from comment #15)
Andrea, can you clarify if you want this mentioned in both our developer's notes (MDN page) and our general notes on www.mozilla.org or only on our developer's notes? Thanks
This is for web developers only. Thank you.
|
|
||
Comment 18•5 years ago
|
||
Documentation has been updated:
- Updated Blob.stream()
- Updated Blob.text()
- Updated Blob.arrayBuffer()
- Cleaned up Blob.size
- Cleaned up Blob.type
- Cleaned up Blob; converted a sample to live
- Cleaned up the Blob() constructor
- Submitted PR 4816 to add new items to BCD
Description
•