Regression : [Devtools] inspect button does not work when the focus is on profiles tab. |
|||||||
Issue descriptionChrome Version : 52.0.2718.0 (Official Build) OS Version : ubuntu 14.04 What steps will reproduce the problem? (1)Launch chrome and open Devtools on any page. (2)Open profiles tab and collect a javascript cpu profile. (3)click on inspect button and select an element on the page to inspect it and observe. What is the expected result? The focus should navigate to elements tab and the corresponding code for the element selected should be highlighted. What happens instead? inspect button does not work on step-3 This is a regression issue broken in M-52
,
Apr 27 2016
Able to reproduce the issue on Mac 10.11.4 using chrome version 52.0.2718.0
,
May 3 2016
Tested this on Windows 7 for Google Chrome Dev Version - 52.0.2723.0 and the issue is still reproducible. Thank you.
,
May 4 2016
Issue still persists on 52.0.2724.0.
,
May 10 2016
Able to reproduce the issue on Mac 10.11.4 using chrome version 52.0.2729.4 dgozman@, Could you please take a look at this.
,
May 11 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/67722794ed6f4ce1cdcc4cbc02ca7ce0fc454c40 commit 67722794ed6f4ce1cdcc4cbc02ca7ce0fc454c40 Author: dgozman <dgozman@chromium.org> Date: Wed May 11 02:31:44 2016 [DevTools] Fix InspectorOverlay breakage after suspend. Split up clearInternal and clear, with latter resetting agents references. BUG= 607070 Review-Url: https://codereview.chromium.org/1957303003 Cr-Commit-Position: refs/heads/master@{#392832} [add] https://crrev.com/67722794ed6f4ce1cdcc4cbc02ca7ce0fc454c40/third_party/WebKit/LayoutTests/inspector/elements/inspect-mode-after-profiling-expected.txt [add] https://crrev.com/67722794ed6f4ce1cdcc4cbc02ca7ce0fc454c40/third_party/WebKit/LayoutTests/inspector/elements/inspect-mode-after-profiling.html [modify] https://crrev.com/67722794ed6f4ce1cdcc4cbc02ca7ce0fc454c40/third_party/WebKit/Source/web/InspectorOverlay.cpp [modify] https://crrev.com/67722794ed6f4ce1cdcc4cbc02ca7ce0fc454c40/third_party/WebKit/Source/web/InspectorOverlay.h
,
May 11 2016
,
Sep 28 2016
[Auto-generated comment by a script] We noticed that this issue is targeted for M-52; 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-52 label, otherwise remove Merge-TBD label. Thanks.
,
Sep 28 2016
[Bulk edit] Our blockerbot script was offline; it was recently brought back online, and thus labeled many old issues (including this one) erroneously. Removing Merge-TBD label since all milestones for this issue are already completed; no further work should be done. |
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by radhakri...@techmahindra.com
, Apr 27 2016Owner: dgozman@chromium.org
Status: Assigned (was: Unconfirmed)