Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(766)

Unified Diff: third_party/WebKit/Source/devtools/front_end/profiler/ProfilesPanel.js

Issue 2366613003: [DevTools] Focus sidebar trees in certain panels by default. (Closed)
Patch Set: Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/devtools/front_end/profiler/ProfilesPanel.js
diff --git a/third_party/WebKit/Source/devtools/front_end/profiler/ProfilesPanel.js b/third_party/WebKit/Source/devtools/front_end/profiler/ProfilesPanel.js
index 9d785c8806c51b358a86c3461ee6818d0c11268c..2f42580662fc43f20aa308c7741b1b1a2c6a573e 100644
--- a/third_party/WebKit/Source/devtools/front_end/profiler/ProfilesPanel.js
+++ b/third_party/WebKit/Source/devtools/front_end/profiler/ProfilesPanel.js
@@ -448,7 +448,6 @@ WebInspector.ProfilesPanel = function()
this._sidebarTree = new TreeOutlineInShadow();
this._sidebarTree.registerRequiredCSS("profiler/profilesSidebarTree.css");
this.panelSidebarElement().appendChild(this._sidebarTree.element);
- this.setDefaultFocusedElement(this._sidebarTree.element);
this._sidebarTree.appendChild(this.profilesItemTreeElement);
@@ -915,6 +914,14 @@ WebInspector.ProfilesPanel.prototype = {
WebInspector.context.setFlavor(WebInspector.ProfilesPanel, this);
},
+ /**
+ * @override
+ */
+ focus: function()
+ {
+ this._sidebarTree.focus();
+ },
+
willHide: function()
{
WebInspector.context.setFlavor(WebInspector.ProfilesPanel, null);

Powered by Google App Engine
This is Rietveld 408576698