Closed Bug 827214 Opened 7 years ago Closed 7 years ago

Actually convert HTML lists to WebIDL

Categories

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

x86
macOS
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla21
Tracking Status
firefox20 --- fixed
firefox21 --- fixed

People

(Reporter: bzbarsky, Assigned: bzbarsky)

References

Details

Attachments

(1 file)

Bug 824823 didn't quite manage to convert HTML lists to WebIDL.  We added the classes and so forth... but we didn't start creating instances of those objects!

Fixing that makes the firstChild test on Dromaeo faster, but lastChild is still being slow for some reason.
(In reply to Boris Zbarsky (:bz) from comment #0)
> Fixing that makes the firstChild test on Dromaeo faster, but lastChild is
> still being slow for some reason.

Possibly related: GetLastChild calls GetChildArray, which is virtual.
http://mxr.mozilla.org/mozilla-central/source/content/base/public/nsINode.h#1034
Comment on attachment 698548 [details] [diff] [review]
Fix HTML lists to really be webidl.

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

Hah.
Attachment #698548 - Flags: review?(peterv) → review+
> Possibly related: GetLastChild calls GetChildArray, which is virtual.

Oh!  I'd totally forgotten that.  We really need to finish that conversion.  :(  That might certainly explain part of what's going on there...  But also, we're going through genericGetter for some reason, which can't possibly help.  I still need to sort that out.

Try is green, so https://hg.mozilla.org/integration/mozilla-inbound/rev/557e0a6e3ca7
Flags: in-testsuite?
Target Milestone: --- → mozilla20
Target Milestone: mozilla20 → ---
Improvement: Mozilla-Inbound-Non-PGO - Dromaeo (DOM) - Win7 - 8.93% increase
Duplicate of this bug: 827516
Blocks: 824823
No longer depends on: 824823
Comment on attachment 698548 [details] [diff] [review]
Fix HTML lists to really be webidl.

Given bug 827516 we need this on Aurora 20, I think.

[Approval Request Comment]
Bug caused by (feature/regressing bug #): Bug 824823
User impact if declined: Assertion failures and possible security issues.
Testing completed (on m-c, etc.): Passes tests.
Risk to taking this patch (and alternatives if risky): Very low risk, I think. 
   The other option is to fully turn off the WebIDL bindings for lists, which
   is doable if needed.
String or UUID changes made by this patch: None.
Attachment #698548 - Flags: approval-mozilla-aurora?
https://hg.mozilla.org/mozilla-central/rev/557e0a6e3ca7
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla21
Attachment #698548 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
I have verified that bug 827516 is also fixed on Aurora.
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.