Closed Bug 879404 Opened 8 years ago Closed 1 year ago
Stamp instead of PRTime in ns JSContext
Apparently PRTime doesn't guarantee monotonicity (ie that time always moves forward), so this can end up feeding negative durations for CCs to the profiler, which results in junky output. I think TimeStamp doesn't have that problem, so we should convert to it, at least for the CC logging stuff.
try run looked good https://tbpl.mozilla.org/?tree=Try&rev=b3bb655d7107
Andrew: are these TimeStamp patches still relevant for CC logging?
I've converted some of the uses of PRTime in another patch, but I guess we could still convert the rest.
One thing to watch out for here is the generation of the JSON for the GC takes a PRTime::Now() argument thing. That probably needs to get converted to taking a time delta computed from TimeStamp in nsJSEnvironment.
Component: DOM → DOM: Core & HTML
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → DUPLICATE
Duplicate of bug: 1580883
You need to log in before you can comment on or make changes to this bug.