Remove unnecessary wrapper on DomHighResTimestamp, and clamp resolution to 1 millisecond

third_party/WebKit/Source/core/timing/PerformanceLongTaskTiming.cpp

Index: third_party/WebKit/Source/core/timing/PerformanceLongTaskTiming.cpp
diff --git a/third_party/WebKit/Source/core/timing/PerformanceLongTaskTiming.cpp b/third_party/WebKit/Source/core/timing/PerformanceLongTaskTiming.cpp
index f8266a5224eed9a044d86b3fb3149888d5db85ad..325fd639ed9d9878729314761e9586c84ba52830 100644
--- a/third_party/WebKit/Source/core/timing/PerformanceLongTaskTiming.cpp
+++ b/third_party/WebKit/Source/core/timing/PerformanceLongTaskTiming.cpp
@@ -8,13 +8,24 @@
 
 namespace blink {
 
+namespace {
+
+double clampToMillisecond(double timeInMillis) {
+  // Long task times are clamped to 1 millisecond for security.
+  return floor(timeInMillis);
+}
+}

[skobes, 2016/12/20 23:48:38]

Add a blank line above this one, and a comment like this:

}  // namespace

[panicker, 2016/12/21 00:10:08]

Done.

+
 PerformanceLongTaskTiming::PerformanceLongTaskTiming(double startTime,
                                                      double endTime,
                                                      String name,
                                                      String culpritFrameSrc,
                                                      String culpritFrameId,
                                                      String culpritFrameName)
-    : PerformanceEntry(name, "longtask", startTime, endTime),
+    : PerformanceEntry(name,
+                       "longtask",
+                       clampToMillisecond(startTime),
+                       clampToMillisecond(endTime)),
       m_culpritFrameSrc(culpritFrameSrc),
       m_culpritFrameId(culpritFrameId),
       m_culpritFrameName(culpritFrameName) {}