Closed
Bug 690372
Opened 13 years ago
Closed 13 years ago
Give an IID to dom::Element
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla11
People
(Reporter: kaze, Assigned: kaze)
References
Details
Attachments
(1 file, 1 obsolete file)
2.57 KB,
patch
|
ehsan.akhgari
:
review+
mounir
:
checkin+
|
Details | Diff | Splinter Review |
We want to use nsCOMPtr<dom::Element> instead of nsCOMPtr<nsIDOMElement> in a few cases, e.g. for bug 684187.
Assignee | ||
Updated•13 years ago
|
Assignee: nobody → kaze
Assignee | ||
Comment 1•13 years ago
|
||
Assignee | ||
Updated•13 years ago
|
Status: NEW → ASSIGNED
Comment 2•13 years ago
|
||
Comment on attachment 563431 [details] [diff] [review] patch proposal Review of attachment 563431 [details] [diff] [review]: ----------------------------------------------------------------- ::: content/base/public/Element.h @@ +204,5 @@ > > nsEventStates mState; > }; > > +NS_DEFINE_STATIC_IID_ACCESSOR(Element, NS_ELEMENT_IID) I _think_ this should go into a cpp file. nsGenericElement.cpp sounds like a good choice.
Comment 3•13 years ago
|
||
No, that typically goes in the header (e.g. see nsINode.h).
Comment 4•13 years ago
|
||
Comment on attachment 563431 [details] [diff] [review] patch proposal Right! Sorry :-)
Attachment #563431 -
Flags: review+
Comment 5•13 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/5336c87067f7
Target Milestone: --- → mozilla10
Comment 6•13 years ago
|
||
Sorry, I backed this out on inbound: https://hg.mozilla.org/integration/mozilla-inbound/rev/d1b5c48c22c7 because of crashes in test_fullscreen-api.html: https://tbpl.mozilla.org/php/getParsedLog.php?id=6632003&tree=Mozilla-Inbound
Target Milestone: mozilla10 → ---
Comment 7•13 years ago
|
||
http://mxr.mozilla.org/mozilla-central/source/content/base/src/nsDocument.cpp#8633 (Which I just happen to have a fix for in bug 688547)
Assignee | ||
Comment 8•13 years ago
|
||
(adding NS_INTERFACE_MAP_ENTRY) This passes test_fullscreen-api.html here (Ubuntu 11.10).
Attachment #563431 -
Attachment is obsolete: true
Attachment #568041 -
Flags: review?(ehsan)
Comment 9•13 years ago
|
||
Comment on attachment 568041 [details] [diff] [review] patch proposal Ah, good catch! Have this gone through try?
Attachment #568041 -
Flags: review?(ehsan) → review+
Updated•13 years ago
|
Component: General → DOM
QA Contact: general → general
Assignee | ||
Comment 10•13 years ago
|
||
https://tbpl.mozilla.org/?tree=Try&rev=5230666d62db The tests on Windows are not significant on this page (“infrastructure exception”). Anyway, there’s a fresh tbpl for bug 684187 which includes this patch: https://tbpl.mozilla.org/?tree=Try&rev=907654f23f5e
Component: DOM → General
Updated•13 years ago
|
Component: General → DOM
Assignee | ||
Updated•13 years ago
|
Keywords: checkin-needed
Updated•13 years ago
|
Updated•13 years ago
|
Attachment #568041 -
Flags: checkin+
Comment 11•13 years ago
|
||
Has this landed? It seems like this patch is not in yet. In that case, what are we waiting on?
Comment 12•13 years ago
|
||
(In reply to Ehsan Akhgari [:ehsan] from comment #11) > Has this landed? It seems like this patch is not in yet. In that case, > what are we waiting on? It's in inbound.
Comment 13•13 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/35b60f8565a7
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
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
•