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

Issue 668638 link

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



Sign in to add a comment

Regression : Unable to scroll summary data using Up/Down arrow keys in ’Summary’ tab under devtools timeline.

Reported by yfulgaon...@etouch.net, Nov 25 2016 
Chrome Version : 57.0.2931.0 5059e49d0a21c1c311653c1613de16b27aefc963-refs/heads/master@{#434310} 32/64 bit
OS : Mac(10.11.6, 10.12.1, 10.12), Windows(7,8,8.1,10), Linux (14.04 LTS)

What steps will reproduce the problem?
1. Launch chrome, open devtools, navigate to ’Timeline’ tab and hit ‘Cmd + R’ to record the reload.
2. Once recording is finished, click anywhere inside ‘Summery’ tab and press Up/Down arrow keys.
3. Observe the scroll bar in ‘Summary’ tab.

Actual : Unable to scroll summary data using Up/Down arrow keys in ’Summary’ tab.
Expected : User should be able to scroll using Up/Down arrow keys in ’Summary’ tab.

This is a regression issue broken in ‘M-56’, below is the Manual Regression range and will soon update other info.
Good build : 56.0.2888.0
Bad build : 56.0.2890.0
Expected_Summary_Scroll.mov
7.1 MB Download
Actual_Summary_Scrolling.mov
7.3 MB Download

Comment 1 by msrchandra@chromium.org, Nov 25 2016

Labels: hasbisect-per-revision ReleaseBlock-Stable
Owner: pfeldman@chromium.org
Status: Assigned (was: Unconfirmed)
Using the per-revision bisect providing the bisect results,
Good build: 56.0.2888.0 (Revision: 424625).
Bad build: 56.0.2890.0 (Revision: 425218).

You are probably looking for a change made after 424894 (known good), but no later than 424895 (first known bad).
CHANGELOG URL:
  https://chromium.googlesource.com/chromium/src/+log/9d82356b6496bb090bd743be75774a9c4b9918e3..2cbfc042012e00bc5863caa8c55de9d25f71d5e3

@pfeldman -- 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.
Adding RB Label as this is a recent Regression. Please remove if not required.

Thank You.

Comment 2 by tkonch...@chromium.org, Nov 30 2016 

Still able to reproduce the issue on win10 chrome version 57.0.2936.0

pfeldman@, Could you please take a look

Comment 3 by hdodda@chromium.org, Dec 7 2016 

Issue is still seen in mac os 10.11.6  using chrome version 57.0.2943.0 

@pfeldman -- Could you please look into this.'

Thanks!

Comment 4 by pfeldman@chromium.org, Dec 8 2016

Labels: -ReleaseBlock-Stable
Owner: alph@chromium.org

Comment 5 by tkonch...@chromium.org, Dec 19 2016 

Still able to reproduce the issue on mac 10.12.2 chrome version 57.0.2956.0	

alph@, Any update on this please

Comment 6 by krajshree@chromium.org, Jan 4 2017 

Just to update, still able to reproduce this issue on mac 10.12.2 and Win-10 using latest chrome version #57.0.2970.0.

In the latest canary #57.0.2970.0, after opening devtools the "timeline" tab has been replaced by "performance" tab.

alph@ - Could you please have a look into this issue.

Thanks...!!

Comment 7 by alph@chromium.org, Jan 4 2017

Labels: -Pri-1 Hotlist-Polish Pri-2
Project Member

Comment 8 by bugdroid1@chromium.org, Jan 4 2017 

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

commit fa2dbcca4d97bc42123ac5ee8c40b15f8dc85b97
Author: alph <alph@chromium.org>
Date: Wed Jan 04 23:40:23 2017

DevTools: Allow timeline details scrolling with keyboard.

BUG= 668638 

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

[modify] https://crrev.com/fa2dbcca4d97bc42123ac5ee8c40b15f8dc85b97/third_party/WebKit/Source/devtools/front_end/timeline/TimelinePanel.js

Comment 9 by alph@chromium.org, Jan 4 2017

Status: Fixed (was: Assigned)

Comment 10 by pfeldman@chromium.org, Jan 5 2017

Labels: -Hotlist-Polish
►

Sign in to add a comment