Don't clear the request's transaction in IDBChild::Recv[Success|Error] since it may be too early

RESOLVED FIXED in Firefox 16

Status

()

Core
DOM: IndexedDB
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: khuey, Assigned: khuey)

Tracking

unspecified
mozilla17
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox16 fixed)

Details

Attachments

(1 attachment)

Created attachment 648078 [details] [diff] [review]
Patch

Got r=bent irl.
Attachment #648078 - Flags: review+
http://hg.mozilla.org/integration/mozilla-inbound/rev/0f2965ada1cd
https://hg.mozilla.org/mozilla-central/rev/0f2965ada1cd
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
Blocks: 778180
Blocks: 766642, 769624

Updated

5 years ago
status-firefox16: --- → affected
Comment on attachment 648078 [details] [diff] [review]
Patch

[Approval Request Comment]
Bug caused by (feature/regressing bug #): Implementing IndexedDB for e10s
User impact if declined: Annoyance to the sheriffs.
Testing completed (on m-c, etc.): Spent a week on m-c, kills orange dead.
Risk to taking this patch (and alternatives if risky): Very low risk.
String or UUID changes made by this patch: None.

Lets take this on Aurora to avoid oranges.  This code isn't actually used (it's for B2G) in any shipping product, but it is tested.
Attachment #648078 - Flags: approval-mozilla-aurora?
Comment on attachment 648078 [details] [diff] [review]
Patch

much love for things that kill orange dead.  land away!
Attachment #648078 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
https://hg.mozilla.org/releases/mozilla-aurora/rev/eaa94120ccbb
status-firefox16: affected → fixed
You need to log in before you can comment on or make changes to this bug.