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

Unified Diff: third_party/WebKit/Source/devtools/front_end/ui/Popover.js

Issue 2527503002: Revert of DevTools: all swatches should have a default focused element (Closed)
Patch Set: Created 4 years, 1 month 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/ui/Popover.js
diff --git a/third_party/WebKit/Source/devtools/front_end/ui/Popover.js b/third_party/WebKit/Source/devtools/front_end/ui/Popover.js
index 55b6dcaff49a1e59eaa75cead14e657a86d6a7fe..7d27e2a0479f642d69ae58ead704421972410b3e 100644
--- a/third_party/WebKit/Source/devtools/front_end/ui/Popover.js
+++ b/third_party/WebKit/Source/devtools/front_end/ui/Popover.js
@@ -83,11 +83,8 @@
this._contentElement = contentElement;
// This should not happen, but we hide previous popup to be on the safe side.
- var restoreFocus;
- if (UI.Popover._popover) {
- restoreFocus = UI.Popover._popover.hasFocus();
+ if (UI.Popover._popover)
UI.Popover._popover.hide();
- }
UI.Popover._popover = this;
var document = anchor instanceof Element ? anchor.ownerDocument : contentElement.ownerDocument;
@@ -102,15 +99,10 @@
document.body.appendChild(this._containerElement);
super.show(this._containerElement);
- if (view) {
+ if (view)
view.show(this._contentDiv);
- if (restoreFocus)
- view.focus();
- } else {
+ else
this._contentDiv.appendChild(this._contentElement);
- if (restoreFocus)
- this._contentElement.focus();
- }
this.positionElement(anchor, this._preferredWidth, this._preferredHeight, arrowDirection);

Powered by Google App Engine
This is Rietveld 408576698