Closed
Bug 1005089
Opened 10 years ago
Closed 10 years ago
crash in mozilla::dom::WrapNewBindingObjectHelper<nsRefPtr<mozilla::dom::EventTarget>, true>::Wrap(JSContext*, nsRefPtr<mozilla::dom::EventTarget> const&, JS::MutableHandle<JS::Value>)
Categories
(Core :: DOM: Core & HTML, defect)
Tracking
()
RESOLVED
FIXED
mozilla32
People
(Reporter: aaronmt, Assigned: smaug)
Details
(Keywords: crash)
Crash Data
Attachments
(2 files)
2.95 KB,
patch
|
bzbarsky
:
review+
|
Details | Diff | Splinter Review |
5.01 KB,
patch
|
Details | Diff | Splinter Review |
This bug was filed from the Socorro interface and is report bp-401a6439-5547-4749-b520-15b202140501. =============================================================
Comment 1•10 years ago
|
||
Olli, Touch.webidl lists "target" as not nullable. This crash shows it seems to be null sometimes? Is that expected?
Component: JavaScript Engine → DOM
Flags: needinfo?(bugs)
Assignee | ||
Comment 3•10 years ago
|
||
But I can see a case... if there is some native anonymous content
Assignee | ||
Comment 4•10 years ago
|
||
Thinking about other approaches for trunk, but this should be fine for branches.
Assignee: nobody → bugs
Attachment #8416707 -
Flags: review?(bzbarsky)
Comment 5•10 years ago
|
||
Comment on attachment 8416707 [details] [diff] [review] branch safe patch r=me
Attachment #8416707 -
Flags: review?(bzbarsky) → review+
Assignee | ||
Comment 6•10 years ago
|
||
Commit message "Bug 1005089, let touch.target be null, r=bz"
Keywords: checkin-needed
Comment 7•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/79686836f1f7
Keywords: checkin-needed
Comment 8•10 years ago
|
||
sorry had to backout this change for bustage like https://tbpl.mozilla.org/php/getParsedLog.php?id=39037844&tree=Mozilla-Inbound
Assignee | ||
Comment 9•10 years ago
|
||
Apparently I had managed to not re-compile everything. https://hg.mozilla.org/integration/mozilla-inbound/rev/74d7fb43bb44
Comment 10•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/74d7fb43bb44
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla32
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
•