Closed
Bug 481336
Opened 16 years ago
Closed 16 years ago
Consider quickstubbing some of the nsIDOMNSHTMLAnchorElement properties
Categories
(Core :: DOM: Core & HTML, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: bzbarsky, Assigned: benjamin)
References
Details
(Keywords: fixed1.9.1, perf)
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
benjamin
:
review+
beltzner
:
approval1.9.1+
|
Details | Diff | Splinter Review |
See bug 481131. Quickstubbing .hash seems to have a pretty major impact... Do we want to do that, and if so what else off this interface do we want to quickstub?
Assignee | ||
Comment 2•16 years ago
|
||
Oh, we already do href/rel/target. I'll add name and hash
Assignee | ||
Comment 3•16 years ago
|
||
Attachment #365425 -
Flags: review?(bzbarsky)
Reporter | ||
Comment 4•16 years ago
|
||
Comment on attachment 365425 [details] [diff] [review]
Quickstub nsIDOMNSHTMLAnchorElement stuff, rev. 1
Looks good.
Attachment #365425 -
Flags: review?(bzbarsky) → review+
Reporter | ||
Comment 5•16 years ago
|
||
Though actually, want to move the NS stuff next to nsIDOMNSHTMLAnchorElement.text?
Assignee | ||
Comment 6•16 years ago
|
||
Attachment #365425 -
Attachment is obsolete: true
Attachment #366639 -
Flags: review+
Assignee | ||
Comment 7•16 years ago
|
||
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•16 years ago
|
Attachment #366639 -
Flags: approval1.9.1?
Assignee | ||
Comment 8•16 years ago
|
||
Comment on attachment 366639 [details] [diff] [review]
Quickstub nsIDOMNSHTMLAnchorElement stuff, rev. 1.1
very low-risk, improves performance on a real-world site that might be representative
Updated•16 years ago
|
Flags: wanted1.9.1?
Comment 9•16 years ago
|
||
Comment on attachment 366639 [details] [diff] [review]
Quickstub nsIDOMNSHTMLAnchorElement stuff, rev. 1.1
a191=beltzner
Attachment #366639 -
Flags: approval1.9.1? → approval1.9.1+
Assignee | ||
Comment 10•16 years ago
|
||
Flags: wanted1.9.1?
Keywords: fixed1.9.1
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•