[DevTools] Do not inherit ObjectPopoverHelper from PopoverHelper.

third_party/WebKit/Source/devtools/front_end/object_ui/ObjectPopoverHelper.js

 /*
  * Copyright (C) 2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 13 matching lines @@
  * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 /**
  * @unrestricted
  */
-ObjectUI.ObjectPopoverHelper = class extends UI.PopoverHelper {
-  /**
-   * @param {!Element} panelElement
-   * @param {function(!Element, !Event):(!Element|!AnchorBox|undefined)} getAnchor
-   * @param {function(!Element, function(!SDK.RemoteObject, boolean, !Element=):undefined, string):undefined} queryObject
-   * @param {function()=} onHide
-   * @param {boolean=} disableOnClick
-   */
-  constructor(panelElement, getAnchor, queryObject, onHide, disableOnClick) {
-    super(panelElement, disableOnClick);
-    this.initializeCallbacks(getAnchor, this._showObjectPopover.bind(this), this._onHideObjectPopover.bind(this));
-    this.setHasPadding(true);
-    this._queryObject = queryObject;
-    this._onHideCallback = onHide;
-    this._popoverObjectGroup = 'popover';
-    panelElement.addEventListener('scroll', this.hidePopover.bind(this), true);
+ObjectUI.ObjectPopoverHelper = class {
+  constructor() {

[lushnikov, 2017/03/13 17:55:02]

let's create an instnace with popover and remote object

[dgozman, 2017/03/13 19:57:24]

Done.

+    this._disposed = false;
   }
 
   /**
-   * @param {!Element} element
+   * @param {!SDK.RemoteObject} result
    * @param {!UI.GlassPane} popover
    * @return {!Promise<boolean>}
    */
-  _showObjectPopover(element, popover) {
+  buildObjectPopover(result, popover) {

[lushnikov, 2017/03/13 17:55:02]

maybe you can even make this a static factory which returns
Promise<ObjectPopoverHelper>

[dgozman, 2017/03/13 19:57:24]

Done.

+    this._disposed = false;
+
     var fulfill;
     var promise = new Promise(x => fulfill = x);
 
     /**
      * @param {!SDK.RemoteObject} funcObject
      * @param {!Element} popoverContentElement
      * @param {!Element} popoverValueElement
      * @param {?Array.<!SDK.RemoteObjectProperty>} properties
      * @param {?Array.<!SDK.RemoteObjectProperty>} internalProperties
      * @this {ObjectUI.ObjectPopoverHelper}
@@ skipping 33 matching lines @@
       var rawLocation = response.location;
       var linkContainer = title.createChild('div', 'function-title-link-container');
       var sourceURL = rawLocation && rawLocation.script() ? rawLocation.script().sourceURL : null;
       if (rawLocation && sourceURL)
         linkContainer.appendChild(this._lazyLinkifier().linkifyRawLocation(rawLocation, sourceURL));
       container.appendChild(popoverContentElement);
       popover.contentElement.appendChild(container);
       fulfill(true);
     }
 
-    /**
-     * @param {!SDK.RemoteObject} result
-     * @param {boolean} wasThrown
-     * @param {!Element|!AnchorBox=} anchorOverride
-     * @this {ObjectUI.ObjectPopoverHelper}
-     */
-    function didQueryObject(result, wasThrown, anchorOverride) {
-      if (this._disposed) {
-        fulfill(false);
-        return;
+    var description = result.description.trimEnd(ObjectUI.ObjectPopoverHelper.MaxPopoverTextLength);
+    var popoverContentElement = null;
+    if (result.type !== 'object') {

[lushnikov, 2017/03/13 17:55:02]

brief examination didn't reveal any code changes here - is it just a move? 

[dgozman, 2017/03/13 19:57:24]

Exactly.

+      popoverContentElement = createElement('span');
+      UI.appendStyle(popoverContentElement, 'object_ui/objectValue.css');
+      UI.appendStyle(popoverContentElement, 'object_ui/objectPopover.css');
+      var valueElement = popoverContentElement.createChild('span', 'monospace object-value-' + result.type);
+      valueElement.style.whiteSpace = 'pre';
+
+      if (result.type === 'string')
+        valueElement.createTextChildren('"', description, '"');
+      else if (result.type !== 'function')
+        valueElement.textContent = description;
+
+      if (result.type === 'function') {
+        result.getOwnProperties(
+            false /* generatePreview */,
+            didGetFunctionProperties.bind(this, result, popoverContentElement, valueElement));
+        return promise;
       }
-      if (wasThrown) {
-        this.hidePopover();
-        fulfill(false);
-        return;
+      popover.contentElement.appendChild(popoverContentElement);
+      fulfill(true);
+    } else {
+      if (result.subtype === 'node') {
+        SDK.DOMModel.highlightObjectAsDOMNode(result);
+        this._resultHighlightedAsDOM = true;
       }
-      this._objectTarget = result.target();
-      var anchor = anchorOverride || element;
-      popover.setContentAnchorBox(anchor instanceof AnchorBox ? anchor : anchor.boxInWindow());
-      var description = result.description.trimEnd(ObjectUI.ObjectPopoverHelper.MaxPopoverTextLength);
-      var popoverContentElement = null;
-      if (result.type !== 'object') {
-        popoverContentElement = createElement('span');
-        UI.appendStyle(popoverContentElement, 'object_ui/objectValue.css');
+
+      if (result.customPreview()) {
+        var customPreviewComponent = new ObjectUI.CustomPreviewComponent(result);
+        customPreviewComponent.expandIfPossible();
+        popoverContentElement = customPreviewComponent.element;
+      } else {
+        popoverContentElement = createElementWithClass('div', 'object-popover-content');
         UI.appendStyle(popoverContentElement, 'object_ui/objectPopover.css');
-        var valueElement = popoverContentElement.createChild('span', 'monospace object-value-' + result.type);
-        valueElement.style.whiteSpace = 'pre';
-
-        if (result.type === 'string')
-          valueElement.createTextChildren('"', description, '"');
-        else if (result.type !== 'function')
-          valueElement.textContent = description;
-
-        if (result.type === 'function') {
-          result.getOwnProperties(
-              false /* generatePreview */,
-              didGetFunctionProperties.bind(this, result, popoverContentElement, valueElement));
-          return;
-        }
-        popover.contentElement.appendChild(popoverContentElement);
-        fulfill(true);
-      } else {
-        if (result.subtype === 'node') {
-          SDK.DOMModel.highlightObjectAsDOMNode(result);
-          this._resultHighlightedAsDOM = true;
-        }
-
-        if (result.customPreview()) {
-          var customPreviewComponent = new ObjectUI.CustomPreviewComponent(result);
-          customPreviewComponent.expandIfPossible();
-          popoverContentElement = customPreviewComponent.element;
-        } else {
-          popoverContentElement = createElementWithClass('div', 'object-popover-content');
-          UI.appendStyle(popoverContentElement, 'object_ui/objectPopover.css');
-          var titleElement = popoverContentElement.createChild('div', 'monospace object-popover-title');
-          titleElement.createChild('span').textContent = description;
-          var section = new ObjectUI.ObjectPropertiesSection(result, '', this._lazyLinkifier());
-          section.element.classList.add('object-popover-tree');
-          section.titleLessMode();
-          popoverContentElement.appendChild(section.element);
-        }
-        popover.setMaxContentSize(new UI.Size(300, 250));
-        popover.setSizeBehavior(UI.GlassPane.SizeBehavior.SetExactSize);
-        popover.contentElement.appendChild(popoverContentElement);
-        fulfill(true);
+        var titleElement = popoverContentElement.createChild('div', 'monospace object-popover-title');
+        titleElement.createChild('span').textContent = description;
+        var section = new ObjectUI.ObjectPropertiesSection(result, '', this._lazyLinkifier());
+        section.element.classList.add('object-popover-tree');
+        section.titleLessMode();
+        popoverContentElement.appendChild(section.element);
       }
+      popover.setMaxContentSize(new UI.Size(300, 250));
+      popover.setSizeBehavior(UI.GlassPane.SizeBehavior.SetExactSize);
+      popover.contentElement.appendChild(popoverContentElement);
+      fulfill(true);
     }
-    this._disposed = false;
-    this._queryObject(element, didQueryObject.bind(this), this._popoverObjectGroup);
     return promise;
   }
 
-  _onHideObjectPopover() {
+  dispose() {
     this._disposed = true;
     if (this._resultHighlightedAsDOM) {
       SDK.DOMModel.hideDOMNodeHighlight();
       delete this._resultHighlightedAsDOM;
     }
     if (this._linkifier) {
       this._linkifier.dispose();
       delete this._linkifier;
     }
-    if (this._onHideCallback)
-      this._onHideCallback();
-    if (this._objectTarget) {
-      this._objectTarget.runtimeModel.releaseObjectGroup(this._popoverObjectGroup);
-      delete this._objectTarget;
-    }
   }
 
   /**
    * @return {!Components.Linkifier}
    */
   _lazyLinkifier() {
     if (!this._linkifier)
       this._linkifier = new Components.Linkifier();
     return this._linkifier;
   }
 };
 
 ObjectUI.ObjectPopoverHelper.MaxPopoverTextLength = 10000;