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

Issue 704022 link

Starred by 1 user
Status: Fixed
Owner:
dgozman@chromium.org
Closed: Mar 2017
Cc:
nyerramilli@chromium.org
█ msrchandra@chromium.org
█ ranjitkan@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: Captured screenshot of Performance tab appears to be misplaced after clicking on it

Reported by sans...@etouch.net, Mar 22 2017 
Chrome Version: 59.0.3048.0 (Official Build) 1af111915f36842841682091b8c6882cae410831-refs/heads/master@{#458590}(32/64 Bit).
OS:Windows (7,8,8.1,10),Linux (14.04 LTS),Mac OS X(10.11.6,10.12.1)

Steps:
1. Launch Chrome and open devtool on NTP or any webpage
2. Go to Performance tab, perform recording and click on recorded screenshot (mostly on last/second last screenshot)
3. Observe

Actual: Captured screenshot appears to be misplaced after clicking on it ( for mostly last/second last screenshot) i.e it is seen on L.H.S even if clicking on RHS captured screenshot

Expected: Captured screenshot should appear properly after clicking on it i.e it should be seen on R.H.S if clicked on RHS captured screenshot

This is regression issue, broken in ‘M 59’ and below is narrow bisect:

Good build:59.0.3038.0
Bad build:59.0.3039.0
Actual_video.mp4
1000 KB View Download
Expected_video.mp4
323 KB View Download

Comment 1 by msrchandra@chromium.org, Mar 22 2017

Labels: hasbisect-per-revision
Owner: dgozman@chromium.org
Status: Assigned (was: Unconfirmed)
Using the per-revision bisect providing the bisect results,
Good build: 59.0.3038.0 (Revision:456244).
Bad build:59.0.3039.0 (Revision:4562985).

You are probably looking for a change made after 456259 (known good), but no later than 456260 (first known bad).
CHANGELOG URL:
  https://chromium.googlesource.com/chromium/src/+log/67c8ffe8dc73fb95bb8fcfe63fd5e1d2c34ee891..fc4bb36920dc06b4c961e32dfab83ecc30fd2561

@dgozman: Could you please look into the issue, pardon me if it has nothing to do with your changes and if possible please assign it to concern owner.
Thank You.

Comment 2 by ranjitkan@chromium.org, Mar 22 2017

Labels: ReleaseBlock-Stable
Adding release block label, please undo if not the case.
Project Member

Comment 3 by bugdroid1@chromium.org, Mar 23 2017 

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

commit 5786cea1a61ebae48b4635bf89544038f9fa4455
Author: dgozman <dgozman@chromium.org>
Date: Thu Mar 23 01:41:36 2017

[DevTools] Fix Popover click handling.

We used to not hide old popover when clicking to show new one, now we do.

BUG= 704022 

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

[modify] https://crrev.com/5786cea1a61ebae48b4635bf89544038f9fa4455/third_party/WebKit/Source/devtools/front_end/ui/Popover.js

Comment 4 by dgozman@chromium.org, Mar 24 2017

Status: Fixed (was: Assigned)

Comment 5 by sureshkumari@chromium.org, Mar 28 2017

Labels: TE-Verified-59.0.3053.3 TE-Veriifed-M59
Tested the issue on Windows-10,Mac-10.12.3 and Linux Ubuntu-14.04 using chrome version# 59.0.3053.3 with the steps mentioned in comment#0.
Observed that the fix is working as expected. Hence adding TE-Verified labels.
Please find the attached screen cast for the same.

Thanks!!
704022.mov
8.1 MB Download
►

Sign in to add a comment