New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 704340 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Apr 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 2
Type: Bug



Sign in to add a comment

Server-Timing header interpreted with wrong units

Reported by dvpdin...@gmail.com, Mar 22 2017

Issue description

UserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_3) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/59.0.3047.0 Safari/537.36

Steps to reproduce the problem:
A Server-Timing header of `view=230.5, db=32.9` should be interpreted in milliseconds

What is the expected behavior?
The DevTools should report 32.9ms in db and 230.5ms in view.

What went wrong?
The DevTools incorrectly interprets the values as seconds, leading to a result of 32.9s in db and 3.8min in view.

Did this work before? No 

Chrome version: 59.0.3047.0  Channel: dev
OS Version: OS X 10.12.3
Flash Version: 

See https://github.com/WICG/server-timing/issues/11
 
Labels: Needs-Triage-M59
Cc: sureshkumari@chromium.org
Labels: Needs-Feedback
dvpdiner2@ could you please provide a sample test file to triage the issue from TE end.

Thanks..

Comment 3 by dvpdin...@gmail.com, Mar 29 2017

This is a simple NodeJS server that can run without dependencies and serves a single text page with a Server-Timing header.

It looks like this is working correctly in the current dev channel (59) but it is showing the wrong values in Chrome 56 and 57. Looks like this might have been fixed in the last commit of 594800.

I'll try to check Chrome 58 to figure out where I need to return seconds and where I need to return milliseconds.
server-timing-test.js
441 bytes View Download
Project Member

Comment 4 by sheriffbot@chromium.org, Mar 29 2017

Labels: -Needs-Feedback
Thank you for providing more feedback. Adding requester "sureshkumari@chromium.org" to the cc list and removing "Needs-Feedback" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: TE-NeedsTriageHelp
Cc: paulir...@chromium.org
Components: -Platform>DevTools Platform>DevTools>Network
Labels: -TE-NeedsTriageHelp -Needs-Triage-M59 Needs-Feedback
Owner: allada@chromium.org
Status: Assigned (was: Unconfirmed)
Hi dvpdiner2, This should have been fixed here: https://codereview.chromium.org/2689833002/

It appears this bug was reported against Chrome 59, which is strange because it should have been fixed in this version.

Can you try again and make sure it is still broken?
screenshot.png
55.7 KB View Download
It looks to be working as intended in Chrome 59. Thanks.
Status: Fixed (was: Assigned)
Thanks!

Sign in to add a comment