DevTools: Hover over redirected request's initiator does not show anything |
||||||||
Issue descriptionIf you go to a site like theverge.com with cache disabled that has a redirected request and open devtools, then hover over the initiator column it throws an error and doesn't show anything.
,
Feb 7 2017
einbinder@ can you please request for M57 merge, So that it get picked in next M57 release. I am tagging all Desktop platforms.
,
Feb 7 2017
,
Feb 7 2017
Your change meets the bar and is auto-approved for M57. Please go ahead and merge the CL to branch 2987 manually. Please contact milestone owner if you have questions. Owners: amineer@(clank), cmasso@(bling), ketakid@(cros), govind@(desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Feb 8 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/0cab60e67973d3ebde4ca325fa40a711dd7d3cc6 commit 0cab60e67973d3ebde4ca325fa40a711dd7d3cc6 Author: einbinder <einbinder@chromium.org> Date: Tue Feb 07 23:59:31 2017 DevTools: Missing parenthesis on url in NetworkLog BUG= 689132 Review-Url: https://codereview.chromium.org/2667513002 Cr-Commit-Position: refs/heads/master@{#448396} (cherry picked from commit 6004f9b3a66dce2ad170a521529d3414e26fb538) NOTRY=true NOPRESUBMIT=true Review-Url: https://codereview.chromium.org/2680743005 Cr-Commit-Position: refs/branch-heads/2987@{#372} Cr-Branched-From: ad51088c0e8776e8dcd963dbe752c4035ba6dab6-refs/heads/master@{#444943} [modify] https://crrev.com/0cab60e67973d3ebde4ca325fa40a711dd7d3cc6/third_party/WebKit/Source/devtools/front_end/sdk/NetworkLog.js
,
Feb 8 2017
einbinder@ in order to verify this issue could you please help us with the steps to reproduce the scenario and the expected result, so that we reproduce the scenario from TE-End. Thank You...
,
Feb 8 2017
A friendly reminder that M57 Stable is launch is coming soon! Your bug is labelled as Stable ReleaseBlock, pls make sure to land the fix and get it merged into the release branch ASAP so it gets enough baking time in Beta (before Stable promotion). Thank you!
,
Feb 8 2017
if you go theverge.com, and hover over the initiator for the file analytics.js with status code 200, it should show the tooltip.
,
Feb 8 2017
Had an offline chat with einbinder@ and based on that this is a really small edge case. Depends on like the angle we move your mouse over the element and how long you leave it there.It's an obvious bug in the code but its hard to get it to manifest itself(Copied from Chat) Note : For now we are moving ahead with today's M57 release.
,
Feb 9 2017
Verified this issue on windows 10, Ubuntu 14.04 and Mac OS 10.12.3 using chrome latest M57-57.0.2987.37 by following steps mentioned in the comment #8. Observed the tool tip on the analytics.js initiator column. Hence adding TE-Verified label. Attaching the screencast for reference. Thank You...
,
Feb 16 2017
einbinder@ : As per the above comment # 10 its already verified, could you please mark the status to Fixed.
,
Feb 16 2017
A friendly reminder that M57 Stable is launch is coming VERY soon! Your bug is labelled as Stable ReleaseBlock, pls make sure to land the fix and get it merged into the release branch (2987) ASAP so it gets enough baking time in Beta (before Stable promotion). Thank you!
,
Feb 22 2017
URGENT - PTAL ASAP. We're getting VERY close to M57 Stable promotion. And this issue is marked as M57 stable release blocker. Pls make sure to land the fix and get it merged into the release branch ASAP so it gets enough baking time in Beta (before Stable promotion). Know that this issue shouldn't block the release? Remove the ReleaseBlock-Stable label or move to M58. Thank you.
,
Feb 23 2017
|
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by bugdroid1@chromium.org
, Feb 6 2017