Closed
Bug 757670
Opened 12 years ago
Closed 12 years ago
make nsIPresShell::GetLinkLocation faster
Categories
(Core :: Disability Access APIs, defect)
Core
Disability Access APIs
Tracking
()
RESOLVED
FIXED
mozilla15
People
(Reporter: surkov, Assigned: surkov)
References
(Blocks 1 open bug)
Details
(Keywords: access)
Attachments
(1 file, 1 obsolete file)
|
(deleted),
patch
|
bzbarsky
:
review+
tbsaunde
:
review+
|
Details | Diff | Splinter Review |
6% we spend for it (for bug 732872 case), most of time is spent for URI computation for nsHTMLAnchorElement.
the nsHTMLAnchorElement fix isn't nice but probably ok (if all this code is going to be reworked one day).
Attachment #626258 -
Flags: review?(bzbarsky)
|
||
Comment 1•12 years ago
|
||
Comment on attachment 626258 [details] [diff] [review]
patch
Is there a reason this isn't an nsContentUtils method? And if it really always takes an element, the argument should be Element.
Past that, I guess this is OK...
|
Assignee | |
Comment 2•12 years ago
|
||
Attachment #626258 -
Attachment is obsolete: true
Attachment #626258 -
Flags: review?(bzbarsky)
Attachment #626318 -
Flags: review?(bzbarsky)
|
|
Assignee | |
Updated•12 years ago
|
Attachment #626318 -
Flags: review?(trev.saunders)
|
||
Updated•12 years ago
|
Attachment #626318 -
Flags: review?(trev.saunders) → review+
|
|
||
Comment 3•12 years ago
|
||
Comment on attachment 626318 [details] [diff] [review]
patch2
Er, also please nuke the DEBUG_dr chunk and the no-op StripChars business.
r=me with that.
Attachment #626318 -
Flags: review?(bzbarsky) → review+
|
|
Assignee | |
Comment 4•12 years ago
|
||
Target Milestone: --- → mozilla15
|
||
Comment 5•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•