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

Issue 797575 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Jan 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Load time on network tabs shows negative for https://developers.google.com/apis-explorer/#p/

Reported by joshung...@gmail.com, Dec 25 2017

Issue description

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

Steps to reproduce the problem:
1. Open devtools and the network tab. 
2. Visit https://developers.google.com/apis-explorer/#p/
3. Wait a couple of seconds and notice the Load: time.

What is the expected behavior?
Probably something bigger than 0ms?

What went wrong?
Negative load time displays such as -783 ms

Did this work before? N/A 

Chrome version: 65.0.3302.0  Channel: n/a
OS Version: OS X 10.12.6
Flash Version:
 
Google_APIs_Explorer.png
43.9 KB View Download
This this might be the best way to improve my load times please share the tricks ;)
Labels: Needs-Triage-M65
Cc: sc00335...@techmahindra.com
Labels: -Type-Bug -Pri-2 ReleaseBlock-Stable Triaged-ET M-65 hasbisect OS-Linux OS-Windows Pri-1 Type-Bug-Regression
Owner: dgozman@chromium.org
Status: Assigned (was: Unconfirmed)
Able to reproduce this issue on reported version 65.0.3302.0 and on latest canary 65.0.3305.0 using Windows 10, Mac 10.13.1 and Ubuntu 14.04 with URL given in comment#0

Good Build: 65.0.3293.0
Bad Build: 65.0.3294.0

Unable to do per-revision bisect as builds got crashed on invoking. Hence performed chromium bisect.

You are probably looking for a change made after 523829 (known good), but no later than 523838 (first known bad).
CHANGELOG URL:
 https://chromium.googlesource.com/chromium/src/+log/7d4f463847c0dd17130b102b2694ddffa71e29f1..6938342187ebab13ceb617f2b5cf80bf84f8a2d2

Reviewed-on: https://chromium-review.googlesource.com/818076

Suspecting above change from changelog.

@dgozman:Please confirm the issue and help in re-assigning if it is not related to your change.

Adding RB-Stable as this is a recent regression. Please change if not the case.

Thanks!
dgozman@ Gentle Ping! This issue is marked as RB-Stable for M65, could you please take a look in to this issue.

Thanks!
Still we are able to reproduce the issue on Windows,Mac & Linux as per C#0. 
dgozman@,Could you please take a look as it is marked as stable blocker.
Thanks..!
Friendly ping to get an update on this issue as it is marked as stable blocker & still reproducible on latest canary-65.0.3322.3.
Thanks..!
Project Member

Comment 7 by bugdroid1@chromium.org, Jan 17 2018

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

commit bf84368c819b46098f7c1743e52a7ca81ceae81d
Author: Dmitry Gozman <dgozman@chromium.org>
Date: Wed Jan 17 04:37:15 2018

[DevTools] Only use main target's start time as base time in Network panel

Bug:  797575 
Change-Id: If7e432d8ed428666395fd02fad2efa56d9f5bbc7
Reviewed-on: https://chromium-review.googlesource.com/868847
Reviewed-by: Andrey Lushnikov <lushnikov@chromium.org>
Commit-Queue: Dmitry Gozman <dgozman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#529608}
[modify] https://crrev.com/bf84368c819b46098f7c1743e52a7ca81ceae81d/third_party/WebKit/Source/devtools/front_end/network/NetworkLogView.js

Labels: TE-Verified-M65 TE-Verified-65.0.3324.0
Verified the fix on Mac 10.13.1, Windows-10 and Ubuntu 14.04 using Chrome version #65.0.3324.0 as per the comment #0.
Attaching screenshot for reference.
Observed that the Load time is seen positive[say 1.6 s]
Hence, the fix is working as expected.
Adding the verified labels.

Thanks...!!
797575.PNG
398 KB View Download
Status: Fixed (was: Assigned)
Labels: Merge-TBD
[Auto-generated comment by a script] We noticed that this issue is targeted for M-65; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-65 label, otherwise remove Merge-TBD label. Thanks.
Labels: -Merge-TBD
Looks like this did it to M65.

Sign in to add a comment