Issue 671727 link

Starred by 1 user

Issue metadata

Status:	Fixed
Owner:	panicker@chromium.org
Closed:	Dec 2016
Cc:	panicker@chromium.org yoavweiss@chromium.org npm@chromium.org
Components:	Blink>PerformanceAPIs
EstimatedDays:	----
NextAction:	----
OS:	----
Pri:	3
Type:	Bug

Blocking:
issue 635596

Long Tasks API: clamp timestamp to ms granularity for cross-origin

Project Member Reported by panicker@chromium.org, Dec 6 2016

Issue description

When reporting to cross-origin frames, clamp the timestamp (duration and also start-time) to millisecond granularity.

Project Member

Comment 1 by bugdroid1@chromium.org, Dec 21 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/0a41be818b1ae20f9ff14f828184efc2ddf29448

commit 0a41be818b1ae20f9ff14f828184efc2ddf29448
Author: panicker <panicker@chromium.org>
Date: Wed Dec 21 03:02:44 2016

- Remove incorrect wrapper on DomHighResTimestamp as it's already in milliseconds
- Clamp resolution of timing to 1 millisecond

BUG= 671727 

Review-Url: https://codereview.chromium.org/2593853002
Cr-Commit-Position: refs/heads/master@{#439989}

[modify] https://crrev.com/0a41be818b1ae20f9ff14f828184efc2ddf29448/third_party/WebKit/LayoutTests/fast/performance/longtasktiming.html
[modify] https://crrev.com/0a41be818b1ae20f9ff14f828184efc2ddf29448/third_party/WebKit/Source/core/timing/PerformanceBase.cpp
[modify] https://crrev.com/0a41be818b1ae20f9ff14f828184efc2ddf29448/third_party/WebKit/Source/core/timing/PerformanceBase.h
[modify] https://crrev.com/0a41be818b1ae20f9ff14f828184efc2ddf29448/third_party/WebKit/Source/core/timing/PerformanceLongTaskTiming.cpp

Comment 2 by panicker@chromium.org, Dec 21 2016

Status: Fixed (was: Assigned)

►

Issue 671727 link

Issue metadata

Long Tasks API: clamp timestamp to ms granularity for cross-origin

Issue description

Comment 1 by bugdroid1@chromium.org, Dec 21 2016

Comment 1 Deleted

Comment 2 by panicker@chromium.org, Dec 21 2016

Comment 2 Deleted

Sign in to add a comment