Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(657)

Unified Diff: third_party/WebKit/Source/core/timing/PerformanceBase.cpp

Issue 2593853002: Remove unnecessary wrapper on DomHighResTimestamp, and clamp resolution to 1 millisecond (Closed)
Patch Set: address format nit Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/timing/PerformanceBase.cpp
diff --git a/third_party/WebKit/Source/core/timing/PerformanceBase.cpp b/third_party/WebKit/Source/core/timing/PerformanceBase.cpp
index 2b8ea03f51329215022d97cac1c29240c8a2927a..e77a457e81ea5ac75b3d9a7640a116a0dd11d4d3 100644
--- a/third_party/WebKit/Source/core/timing/PerformanceBase.cpp
+++ b/third_party/WebKit/Source/core/timing/PerformanceBase.cpp
@@ -433,9 +433,9 @@ void PerformanceBase::addLongTaskTiming(double startTime,
if (!hasObserverFor(PerformanceEntry::LongTask))
return;
PerformanceEntry* entry = PerformanceLongTaskTiming::create(
- monotonicTimeToDOMHighResTimeStampInMillis(startTime),
- monotonicTimeToDOMHighResTimeStampInMillis(endTime), name,
- culpritFrameSrc, culpritFrameId, culpritFrameName);
+ monotonicTimeToDOMHighResTimeStamp(startTime),
+ monotonicTimeToDOMHighResTimeStamp(endTime), name, culpritFrameSrc,
+ culpritFrameId, culpritFrameName);
notifyObserversOfEntry(*entry);
}
@@ -562,11 +562,6 @@ DOMHighResTimeStamp PerformanceBase::monotonicTimeToDOMHighResTimeStamp(
clampTimeResolution(timeInSeconds));
}
-double PerformanceBase::monotonicTimeToDOMHighResTimeStampInMillis(
- DOMHighResTimeStamp monotonicTime) const {
- return monotonicTimeToDOMHighResTimeStamp(monotonicTime) * 1000;
-}
-
DOMHighResTimeStamp PerformanceBase::now() const {
return monotonicTimeToDOMHighResTimeStamp(monotonicallyIncreasingTime());
}

Powered by Google App Engine
This is Rietveld 408576698