New issue
Advanced search Search tips

Issue 673472 link

Starred by 1 user
Status: Fixed
Owner:
einbinder@chromium.org
Closed: Jan 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

DevTools: Inspect Element opens the last used panel before switching to Elements

Project Member Reported by einbinder@chromium.org, Dec 12 2016 
When doing Right-click Inspect Element, the last used panel is opened. Then it quickly switches to the elements panel. This wastes time loading the wrong panel, and flashes the wrong panel.
Project Member

Comment 1 by bugdroid1@chromium.org, Dec 16 2016 

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

commit dfa567bd9bfd0625abd29223e3ab038c508e5ae2
Author: einbinder <einbinder@chromium.org>
Date: Fri Dec 16 01:15:52 2016

DevTools: Open Elements panel sooner on Inspect Element

This patch adds a query parameter "panel" which sets the default panel
when devtools loads.

BUG= 673472 
TBR=sky

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

[modify] https://crrev.com/dfa567bd9bfd0625abd29223e3ab038c508e5ae2/chrome/browser/chromeos/extensions/file_manager/private_api_misc.cc
[modify] https://crrev.com/dfa567bd9bfd0625abd29223e3ab038c508e5ae2/chrome/browser/devtools/devtools_toggle_action.cc
[modify] https://crrev.com/dfa567bd9bfd0625abd29223e3ab038c508e5ae2/chrome/browser/devtools/devtools_toggle_action.h
[modify] https://crrev.com/dfa567bd9bfd0625abd29223e3ab038c508e5ae2/chrome/browser/devtools/devtools_window.cc
[modify] https://crrev.com/dfa567bd9bfd0625abd29223e3ab038c508e5ae2/chrome/browser/devtools/devtools_window.h
[modify] https://crrev.com/dfa567bd9bfd0625abd29223e3ab038c508e5ae2/chrome/browser/ui/browser_command_controller.cc
[modify] https://crrev.com/dfa567bd9bfd0625abd29223e3ab038c508e5ae2/chrome/browser/ui/browser_commands.cc
[modify] https://crrev.com/dfa567bd9bfd0625abd29223e3ab038c508e5ae2/chrome/browser/ui/views/extensions/extension_popup.cc
[modify] https://crrev.com/dfa567bd9bfd0625abd29223e3ab038c508e5ae2/chrome/browser/ui/webui/devtools_ui.cc
[modify] https://crrev.com/dfa567bd9bfd0625abd29223e3ab038c508e5ae2/third_party/WebKit/Source/devtools/front_end/devtools_compatibility.js
[modify] https://crrev.com/dfa567bd9bfd0625abd29223e3ab038c508e5ae2/third_party/WebKit/Source/devtools/front_end/ui/InspectorView.js
[modify] https://crrev.com/dfa567bd9bfd0625abd29223e3ab038c508e5ae2/third_party/WebKit/Source/devtools/front_end/ui/TabbedPane.js
[modify] https://crrev.com/dfa567bd9bfd0625abd29223e3ab038c508e5ae2/third_party/WebKit/Source/devtools/front_end/ui/View.js
Project Member

Comment 2 by bugdroid1@chromium.org, Dec 22 2016 

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

commit 2de169544dd69ffc7e31e87be92add166397ed12
Author: einbinder <einbinder@chromium.org>
Date: Thu Dec 22 00:18:42 2016

DevTools: Add missing case to DoAction switch

BUG= 673472 

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

[modify] https://crrev.com/2de169544dd69ffc7e31e87be92add166397ed12/chrome/browser/devtools/devtools_window.cc

Comment 3 by einbinder@chromium.org, Jan 5 2017

Status: Fixed (was: Assigned)
►

Sign in to add a comment