Closed
Bug 841466
Opened 11 years ago
Closed 11 years ago
Add GetParentObject() for WebIDL objects with a form owner
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla21
People
(Reporter: Ms2ger, Assigned: Ms2ger)
References
(Blocks 2 open bugs)
Details
(Keywords: regression)
Attachments
(2 files)
2.90 KB,
patch
|
bzbarsky
:
review+
|
Details | Diff | Splinter Review |
4.07 KB,
patch
|
bajaj
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
We use the parent chain to implement the scope for event handlers, and we broke that.
Updated•11 years ago
|
tracking-firefox21:
--- → ?
Assignee | ||
Updated•11 years ago
|
Assignee | ||
Comment 1•11 years ago
|
||
https://tbpl.mozilla.org/?tree=Try&rev=00e4bf7fa7e4
Attachment #714038 -
Flags: review?(bzbarsky)
Comment 2•11 years ago
|
||
Comment on attachment 714038 [details] [diff] [review] Patch v1 r=me. We need this on 20, right?
Attachment #714038 -
Flags: review?(bzbarsky) → review+
Assignee | ||
Comment 3•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/ca388aedb410
Flags: in-testsuite+
Assignee | ||
Comment 4•11 years ago
|
||
[Approval Request Comment] Bug caused by (feature/regressing bug #): Bug 824007 (legend) User impact if declined: possibly broken sites Testing completed (on m-c, etc.): on inbound; passes try Risk to taking this patch (and alternatives if risky): pretty low, fix is well understood and tested. String or UUID changes made by this patch: None
Attachment #714563 -
Flags: approval-mozilla-aurora?
Comment 5•11 years ago
|
||
Comment on attachment 714563 [details] [diff] [review] Patch for aurora Approving for aurora uplift even though this is only on inbound so far with the caveat that this pass testing and get merge to m-c before uplift to Aurora. With the long weekend in the US it would be great if this can land before Tues 6am Pacific when we do the merge to beta.
Attachment #714563 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Updated•11 years ago
|
status-firefox20:
--- → affected
status-firefox21:
--- → affected
Comment 6•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/ca388aedb410
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla21
Comment 8•11 years ago
|
||
Oh, it does matter what kind of object is returned from GetParentObject. Based on the documentation all that matters is that from the returned object one should eventually get to the global.
Comment 9•11 years ago
|
||
> Based on the documentation all that matters is that from the returned object one should
> eventually get to the global.
That's true for all objects that can't be on a scope chain of some script. As in, everything except nodes.
You need to log in
before you can comment on or make changes to this bug.
Description
•