Closed Bug 773326 Opened 12 years ago Closed 12 years ago

Inverted thread check in getJSToNativeConversionTemplate::onFailure

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla17

People

(Reporter: Ms2ger, Assigned: Ms2ger)

References

(Blocks 1 open bug)

Details

Attachments

(1 file, 1 obsolete file)

Attached patch Patch v1 (obsolete) (deleted) — Splinter Review
      No description provided.
Attachment #641510 - Flags: review?(peterv)
Attached patch Patch v1.1 (deleted) — Splinter Review
Now with a test.
Assignee: nobody → Ms2ger
Attachment #641510 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #641510 - Flags: review?(peterv)
Attachment #641545 - Flags: review?(peterv)
Comment on attachment 641545 [details] [diff] [review]
Patch v1.1

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

Should probably try to get this on branches.
Attachment #641545 - Flags: review?(peterv) → review+
Code was fixed in bug 776685, landed the test as:

https://hg.mozilla.org/mozilla-central/rev/67e0ec3563f6
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
Backed out with the mass tree revert to get rid of the OS X M5 orange:
https://hg.mozilla.org/mozilla-central/rev/c801b99d726f
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
https://hg.mozilla.org/mozilla-central/rev/393b9e4e7404
Status: REOPENED → RESOLVED
Closed: 12 years ago12 years ago
Resolution: --- → FIXED
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: