Actually convert HTML lists to WebIDL

RESOLVED FIXED in Firefox 20

Status

()

Core
DOM
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: bz, Assigned: bz)

Tracking

unspecified
mozilla21
x86
Mac OS X
Points:
---
Bug Flags:
in-testsuite ?

Firefox Tracking Flags

(firefox20 fixed, firefox21 fixed)

Details

Attachments

(1 attachment)

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.
Created attachment 698548 [details] [diff] [review]
Fix HTML lists to really be webidl.
Attachment #698548 - Flags: review?(peterv)
(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?

Comment 8

5 years ago
https://hg.mozilla.org/mozilla-central/rev/557e0a6e3ca7
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla21

Updated

5 years ago
Attachment #698548 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
https://hg.mozilla.org/releases/mozilla-aurora/rev/10e6c5983d6e
status-firefox20: --- → fixed
status-firefox21: --- → fixed
I have verified that bug 827516 is also fixed on Aurora.
You need to log in before you can comment on or make changes to this bug.