Closed Bug 824517 Opened 7 years ago Closed 7 years ago

Quickstub the .performance property on Window objects

Categories

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

x86
macOS
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla20

People

(Reporter: bzbarsky, Assigned: bzbarsky)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

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.
Blocks: 824495
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+
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
https://hg.mozilla.org/mozilla-central/rev/1d51451f6674
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.