Regression: Default focus is missing in Application section of Devtools |
|||||
Issue descriptionChrome Version:60.3101.0 OS: Ubuntu 14.04 What steps will reproduce the problem? (1)Launch Chrome and open devtools in any page (2)Go to the Application tab and observe the default focus Actual:Default focus is missing on Manifest option Expected:Default focus should be seen on Manifest option This is Regression issue broken M-60 as it is working fine in M-59 beta. Will provide other info soon
,
May 16 2017
Manual Bisect Info: ==================== Good Build:60.0.3100.0 Bad Build: 60.0.3101.0 CHANGELOG URL: https://chromium.googlesource.com/chromium/src/+log/5764fa8dcfc1d110a7feaf39e26be8be583c1b54..0712667572b158d0613b5f5a2ec8fa6f61cada34 Suspecting https://codereview.chromium.org/2873843003 from above changelog @eostroukhov: Please confirm the issue and help in re-assigning if it is not related to your change.
,
May 17 2017
Issue 723544 has been merged into this issue.
,
May 18 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/d0471f6d597b98fb255a0190751c9ccc0636d09e commit d0471f6d597b98fb255a0190751c9ccc0636d09e Author: eostroukhov <eostroukhov@chromium.org> Date: Thu May 18 18:14:10 2017 [DevTools] Fix Application Panel initial selection 1. Select manifest panel if there's no stored selection. 2. Fix selection restoration for the top-level tree items. 3. Parent element will now be selected if selection cannot be restored because the element have been gone from the tree. BUG= 722716 Review-Url: https://codereview.chromium.org/2887183002 Cr-Commit-Position: refs/heads/master@{#472874} [modify] https://crrev.com/d0471f6d597b98fb255a0190751c9ccc0636d09e/third_party/WebKit/Source/devtools/front_end/resources/ApplicationPanelSidebar.js
,
May 18 2017
,
May 23 2017
Verified this issue on Ubuntu 14.04, Windows-10 and Mac OS 10.12 using chrome latest dev #60.0.3107.4 by following steps mentioned in the original comment, Observed the default focus is highlighted on Manifest as expected. Hence adding TE-Verified label. |
|||||
►
Sign in to add a comment |
|||||
Comment 1 by brajkumar@chromium.org
, May 16 2017