ObjectWrapper.jsm needs to provide |type| property for the new blob

RESOLVED FIXED in Firefox 19

Status

()

Core
DOM
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: Gene Lian (I already quit Mozilla), Assigned: Gene Lian (I already quit Mozilla))

Tracking

Trunk
mozilla20
Points:
---

Firefox Tracking Flags

(firefox19 fixed, firefox20 fixed, b2g18 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Comment 1

6 years ago
Created attachment 692568 [details] [diff] [review]
Patch
Attachment #692568 - Flags: review?(jonas)
(Assignee)

Comment 2

6 years ago
Let's see try result first before checking: https://tbpl.mozilla.org/?tree=Try&rev=2d036c027a46, although it's unlikely to cause any harm.
(Assignee)

Comment 3

6 years ago
Hi Jonas, do we need to consider this as bb+?
blocking-basecamp: --- → ?
(Assignee)

Comment 4

6 years ago
(In reply to Gene Lian [:gene] from comment #2)
> Let's see try result first before checking:
> https://tbpl.mozilla.org/?tree=Try&rev=2d036c027a46, although it's unlikely
> to cause any harm.

All green!
Keywords: checkin-needed
Comment on attachment 692568 [details] [diff] [review]
Patch

Review of attachment 692568 [details] [diff] [review]:
-----------------------------------------------------------------

I don't know if it's blocking, but it's definitely safe enough to uplift.

It might very well have bad effects for end users since we use the type of the blob when a blob is shared over bluetooth.
Attachment #692568 - Flags: approval-mozilla-b2g18+
Attachment #692568 - Flags: approval-mozilla-aurora+
Removing nomination for now. Feel free to renominate if you think it's important.
blocking-basecamp: ? → ---

Comment 8

6 years ago
https://hg.mozilla.org/mozilla-central/rev/86e0432487a7
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla20
https://hg.mozilla.org/releases/mozilla-aurora/rev/cb4021304583
https://hg.mozilla.org/releases/mozilla-b2g18/rev/db4a9ec36559
status-b2g18: --- → fixed
status-firefox19: --- → fixed
status-firefox20: --- → fixed
You need to log in before you can comment on or make changes to this bug.