[DevTools] Do not inherit ObjectPopoverHelper from PopoverHelper.

third_party/WebKit/Source/devtools/front_end/sources/JavaScriptSourceFrame.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 26 matching lines @@
    */
   constructor(uiSourceCode) {
     super(uiSourceCode);
     this._debuggerSourceCode = uiSourceCode;
 
     this._scriptsPanel = Sources.SourcesPanel.instance();
     this._breakpointManager = Bindings.breakpointManager;
     if (uiSourceCode.project().type() === Workspace.projectTypes.Debugger)
       this.element.classList.add('source-frame-debugger-script');
 
-    this._popoverHelper = new ObjectUI.ObjectPopoverHelper(
-        this._scriptsPanel.element, this._getPopoverAnchor.bind(this), this._resolveObjectForPopover.bind(this),
-        this._onHidePopover.bind(this), true);
+    this._popoverHelper = new UI.PopoverHelper(this._scriptsPanel.element, true);
+    this._popoverHelper.initializeCallbacks(
+        this._getPopoverAnchor.bind(this), this._showObjectPopover.bind(this), this._onHidePopover.bind(this));
     this._popoverHelper.setTimeout(250, 250);
+    this._popoverHelper.setHasPadding(true);
+    this._objectPopoverHelper = new ObjectUI.ObjectPopoverHelper();
+    this._scriptsPanel.element.addEventListener(
+        'scroll', this._popoverHelper.hidePopover.bind(this._popoverHelper), true);
 
     this.textEditor.element.addEventListener('keydown', this._onKeyDown.bind(this), true);
 
     this.textEditor.addEventListener(
         SourceFrame.SourcesTextEditor.Events.GutterClick, this._handleGutterClick.bind(this), this);
 
     this._breakpointManager.addEventListener(
         Bindings.BreakpointManager.Events.BreakpointAdded, this._breakpointAdded, this);
     this._breakpointManager.addEventListener(
         Bindings.BreakpointManager.Events.BreakpointRemoved, this._breakpointRemoved, this);
@@ skipping 354 matching lines @@
 
     var leftCorner = this.textEditor.cursorPositionToCoordinates(lineNumber, token.startColumn);
     var rightCorner = this.textEditor.cursorPositionToCoordinates(lineNumber, token.endColumn - 1);
     var anchorBox = new AnchorBox(leftCorner.x, leftCorner.y, rightCorner.x - leftCorner.x, leftCorner.height);
 
     anchorBox.highlight = {lineNumber: lineNumber, startColumn: token.startColumn, endColumn: token.endColumn - 1};
 
     return anchorBox;
   }
 
-  _resolveObjectForPopover(anchorBox, showCallback, objectGroupName) {
+  /**
+   * @param {!AnchorBox} anchorBox
+   * @return {!Promise<?SDK.RemoteObject>}
+   */
+  _resolveObjectForPopover(anchorBox) {
     var selectedCallFrame = UI.context.flavor(SDK.DebuggerModel.CallFrame);
-    if (!selectedCallFrame) {
-      this._popoverHelper.hidePopover();
-      return;
-    }
+    if (!selectedCallFrame)
+      return Promise.resolve(/** @type {?SDK.RemoteObject} */ (null));
     var lineNumber = anchorBox.highlight.lineNumber;
     var startHighlight = anchorBox.highlight.startColumn;
     var endHighlight = anchorBox.highlight.endColumn;
     var line = this.textEditor.line(lineNumber);
     if (!anchorBox.forSelection) {
       while (startHighlight > 1 && line.charAt(startHighlight - 1) === '.') {
         var token = this.textEditor.tokenAtTextPosition(lineNumber, startHighlight - 2);
-        if (!token || !token.type) {
-          this._popoverHelper.hidePopover();
-          return;
-        }
+        if (!token || !token.type)
+          return Promise.resolve(/** @type {?SDK.RemoteObject} */ (null));
         startHighlight = token.startColumn;
       }
     }
     var evaluationText = line.substring(startHighlight, endHighlight + 1);
-    Sources.SourceMapNamesResolver
+    return Sources.SourceMapNamesResolver
         .resolveExpression(
             selectedCallFrame, evaluationText, this._debuggerSourceCode, lineNumber, startHighlight, endHighlight)
         .then(onResolve.bind(this));
 
     /**
      * @param {?string=} text
+     * @return {!Promise<?SDK.RemoteObject>}
      * @this {Sources.JavaScriptSourceFrame}
      */
     function onResolve(text) {
+      var fulfill;
+      var promise = new Promise(x => fulfill = x);
       selectedCallFrame.evaluate(
-          text || evaluationText, objectGroupName, false, true, false, false, showObjectPopover.bind(this));
+          text || evaluationText, 'popover', false, true, false, false, showObjectPopover.bind(this, fulfill));
+      return promise;
     }
 
     /**
+     * @param {function(?SDK.RemoteObject)} fulfill
      * @param {?Protocol.Runtime.RemoteObject} result
      * @param {!Protocol.Runtime.ExceptionDetails=} exceptionDetails
      * @this {Sources.JavaScriptSourceFrame}
      */
-    function showObjectPopover(result, exceptionDetails) {
+    function showObjectPopover(fulfill, result, exceptionDetails) {
       var target = UI.context.flavor(SDK.Target);
       var potentiallyUpdatedCallFrame = UI.context.flavor(SDK.DebuggerModel.CallFrame);
-      if (selectedCallFrame !== potentiallyUpdatedCallFrame || !result) {
-        this._popoverHelper.hidePopover();
+      if (selectedCallFrame !== potentiallyUpdatedCallFrame || !result || exceptionDetails) {
+        fulfill(null);
         return;
       }
       this._popoverAnchorBox = anchorBox;
-      showCallback(target.runtimeModel.createRemoteObject(result), !!exceptionDetails, this._popoverAnchorBox);
-      // Popover may have been removed by showCallback().
+      this._popoverTarget = target;
+      fulfill(target.runtimeModel.createRemoteObject(result));
+      // Popover may have been removed by callback().

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

since with your change the callback will run in a subsequent microtask - this
code never happens.

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

Done.

       if (this._popoverAnchorBox) {
         var highlightRange = new Common.TextRange(lineNumber, startHighlight, lineNumber, endHighlight);
         this._popoverAnchorBox._highlightDescriptor =
             this.textEditor.highlightRange(highlightRange, 'source-frame-eval-expression');
       }
     }
   }
 
+  /**
+   * @param {!AnchorBox|!Element} anchorBox
+   * @param {!UI.GlassPane} popover
+   * @return {!Promise<boolean>}
+   */
+  _showObjectPopover(anchorBox, popover) {
+    return this._resolveObjectForPopover(/** @type {!AnchorBox} */ (anchorBox)).then(object => {
+      if (!object)
+        return false;
+      return this._objectPopoverHelper.buildObjectPopover(object, popover);
+    });
+  }
+
   _onHidePopover() {
+    this._objectPopoverHelper.dispose();

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

the this._objectPopoverHelper is created in the JSSF constructoror - we either
don't want to dispose it on every popover hide,
or we want to re-create it on every showPopover

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

Done.

+    if (this._popoverTarget) {
+      this._popoverTarget.runtimeModel.releaseObjectGroup('popover');
+      delete this._popoverTarget;
+    }
     if (!this._popoverAnchorBox)
       return;
     if (this._popoverAnchorBox._highlightDescriptor)
       this.textEditor.removeHighlight(this._popoverAnchorBox._highlightDescriptor);
     delete this._popoverAnchorBox;
   }
 
   _onKeyDown(event) {
     if (event.key === 'Escape') {
       if (this._popoverHelper.isPopoverVisible()) {
@@ skipping 969 matching lines @@
       return;
     this.bookmark.clear();
     this.bookmark = null;
   }
 };
 
 Sources.JavaScriptSourceFrame.BreakpointDecoration.bookmarkSymbol = Symbol('bookmark');
 Sources.JavaScriptSourceFrame.BreakpointDecoration._elementSymbolForTest = Symbol('element');
 
 Sources.JavaScriptSourceFrame.continueToLocationDecorationSymbol = Symbol('bookmark');