Closed
Bug 824517
Opened 12 years ago
Closed 12 years ago
Quickstub the .performance property on Window objects
Categories
(Core :: DOM: Core & HTML, defect)
Tracking
()
RESOLVED
FIXED
mozilla20
People
(Reporter: bzbarsky, Assigned: bzbarsky)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
|
(deleted),
patch
|
peterv
:
review+
|
Details | Diff | Splinter Review |
This came up in bug 824495. Getting .performance is slow because it ends up doing xpconnectery.
The attached patch speeds up the window.performance.now testcase in bug 824495 by 2x or so, on Mac.
|
Assignee | |
Comment 1•12 years ago
|
||
Attachment #695532 -
Flags: review?(peterv)
|
||
Comment 2•12 years ago
|
||
Comment on attachment 695532 [details] [diff] [review]
Quickstub window.performance.
Review of attachment 695532 [details] [diff] [review]:
-----------------------------------------------------------------
::: dom/base/nsGlobalWindow.cpp
@@ +10770,5 @@
> }
>
> // static
> bool
> +nsPIDOMWindow::HasPerformanceSupport()
Trailing whitespace.
Attachment #695532 -
Flags: review?(peterv) → review+
|
|
Assignee | |
Comment 3•12 years ago
|
||
http://hg.mozilla.org/integration/mozilla-inbound/rev/1d51451f6674 with that fixed.
Marking in-testsuite- like all other performance bugs. :(
Flags: in-testsuite-
Whiteboard: [need review]
Target Milestone: --- → mozilla20
|
||
Comment 4•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
|
||
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
•