Use TimeStamp::NowLoRes() in nsAppShell

RESOLVED FIXED in mozilla21

Status

()

Core
Widget: Win32
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: mayhemer, Assigned: mayhemer)

Tracking

Trunk
mozilla21
x86
Windows 7
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
Created attachment 708724 [details] [diff] [review]
v1

nsAppShell is checking for 1000ms time difference but is using hi-res TimeStamp::Now().  This is an ideal first candidate for using the NowLoRes() much faster function.

My checks to call of TimeStamp::Now() shows these are one of the most frequent consumers.
Attachment #708724 - Flags: review?(roc)
Attachment #708724 - Flags: review?(roc) → review+
(Assignee)

Comment 1

5 years ago
Comment on attachment 708724 [details] [diff] [review]
v1

https://hg.mozilla.org/integration/mozilla-inbound/rev/cc3b4a2c1968
Attachment #708724 - Flags: checkin+
https://hg.mozilla.org/mozilla-central/rev/cc3b4a2c1968
Assignee: nobody → honzab.moz
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla21
You need to log in before you can comment on or make changes to this bug.