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

Unified Diff: third_party/WebKit/Source/devtools/front_end/elements/ElementsPanel.js

Issue 2740803002: [DevTools] Simplify Popover API (Closed)
Patch Set: addressed comments Created 3 years, 9 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/elements/ElementsPanel.js
diff --git a/third_party/WebKit/Source/devtools/front_end/elements/ElementsPanel.js b/third_party/WebKit/Source/devtools/front_end/elements/ElementsPanel.js
index 13148835b365eaae24b18147865908d0477622fd..5df8f45a6f2d563d9c295b727228ed0a0f38bdfb 100644
--- a/third_party/WebKit/Source/devtools/front_end/elements/ElementsPanel.js
+++ b/third_party/WebKit/Source/devtools/front_end/elements/ElementsPanel.js
@@ -581,22 +581,27 @@ Elements.ElementsPanel = class extends UI.Panel {
/**
* @param {!Element} link
- * @param {!UI.Popover} popover
+ * @param {!UI.GlassPane} popover
+ * @return {!Promise<boolean>}
*/
_showPopover(link, popover) {
var node = this.selectedDOMNode();
- if (node) {
- Components.DOMPresentationUtils.buildImagePreviewContents(
- node.target(), link[Elements.ElementsTreeElement.HrefSymbol], true, showPopover);
- }
+ if (!node)
+ return Promise.resolve(false);
+
+ var fulfill;
+ var promise = new Promise(x => fulfill = x);
+ Components.DOMPresentationUtils.buildImagePreviewContents(
+ node.target(), link[Elements.ElementsTreeElement.HrefSymbol], true, showPopover);
+ return promise;
/**
* @param {!Element=} contents
*/
function showPopover(contents) {
- if (!contents)
- return;
- popover.showForAnchor(contents, link);
+ if (contents)
+ popover.contentElement.appendChild(contents);
+ fulfill(!!contents);
}
}
@@ -903,6 +908,7 @@ Elements.ElementsPanel = class extends UI.Panel {
if (this._popoverHelper)
this._popoverHelper.hidePopover();
this._popoverHelper = new UI.PopoverHelper(tabbedPane.element);
+ this._popoverHelper.setHasPadding(true);
this._popoverHelper.initializeCallbacks(this._getPopoverAnchor.bind(this), this._showPopover.bind(this));
this._popoverHelper.setTimeout(0);

Powered by Google App Engine
This is Rietveld 408576698