[DevTools] Make UI.GlassPane position contentElement for different overlay controls.

third_party/WebKit/Source/devtools/front_end/ui/Dialog.js

 /*
  * Copyright (C) 2012 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 23 matching lines @@
 UI.Dialog = class extends UI.Widget {
   constructor() {
     super(true);
     this.markAsRoot();
     this.registerRequiredCSS('ui/dialog.css');
 
     this.contentElement.createChild('content');
     this.contentElement.tabIndex = 0;
     this.contentElement.addEventListener('focus', this._onFocus.bind(this), false);
     this._keyDownBound = this._onKeyDown.bind(this);
+    this._onClickBound = this._onGlassPaneClick.bind(this);
+    this._dimmed = false;
+    this._wrapsContent = false;
+    this._maxSize = null;
+    /** @type {?number} */
+    this._positionX = null;
+    /** @type {?number} */
+    this._positionY = null;
 
-    this._wrapsContent = false;
-    this._dimmed = false;
     /** @type {!Map<!HTMLElement, number>} */
     this._tabIndexMap = new Map();
   }
 
   /**
    * @return {boolean}
    */
   static hasInstance() {
     return !!UI.Dialog._instance;
   }
 
   /**
-   * @param {!UI.Widget} view
-   */
-  static setModalHostView(view) {
-    UI.Dialog._modalHostView = view;
-  }
-
-  /**
-   * FIXME: make utility method in Dialog, so clients use it instead of this getter.
-   * Method should be like Dialog.showModalElement(position params, reposition callback).
-   * @return {?UI.Widget}
-   */
-  static modalHostView() {
-    return UI.Dialog._modalHostView;
-  }
-
-  static modalHostRepositioned() {
-    if (UI.Dialog._instance)
-      UI.Dialog._instance._position();
-  }
-
-  /**
    * @override
+   * @suppressGlobalPropertiesCheck
+   * TODO(dgozman): pass document in constructor.
    */
   show() {
     if (UI.Dialog._instance)
       UI.Dialog._instance.detach();
     UI.Dialog._instance = this;
 
-    var document = /** @type {!Document} */ (UI.Dialog._modalHostView.element.ownerDocument);
     this._disableTabIndexOnElements(document);
 
-    this._glassPane = new UI.GlassPane(document, this._dimmed);
-    this._glassPane.element.addEventListener('click', this._onGlassPaneClick.bind(this), false);
-    this.element.ownerDocument.body.addEventListener('keydown', this._keyDownBound, false);
+    this._glassPane = new UI.GlassPane(document, this._dimmed, true /* blockPointerEvents*/);
+    var container = createElement('div');
+    this._glassPane.setContentElement(container);
+    this._glassPane.show();
+    super.show(container);
+    this._glassPane.setMaxContentSize(this._effectiveMaxSize());
+    this._glassPane.setContentPosition(this._positionX, this._positionY);

[pfeldman, 2017/02/01 00:07:18]

setContentPosition/setContentAnchorBox would be mutually exclusive calls that
are positioning content.

[dgozman, 2017/02/01 16:19:44]

Done.

+    this._glassPane.positionContent();
 
-    super.show(this._glassPane.element);
+    this._glassPane.element.addEventListener('click', this._onClickBound, false);

[pfeldman, 2017/02/01 00:07:18]

this._glassPane.on('exit', () => {})

[dgozman, 2017/02/01 16:19:44]

Reworked.

+    this.contentElement.addEventListener('keydown', this._keyDownBound, false);
 
-    this._position();
     this._focusRestorer = new UI.WidgetFocusRestorer(this);
   }
 
   /**
    * @override
    */
   detach() {
     this._focusRestorer.restore();
 
-    this.element.ownerDocument.body.removeEventListener('keydown', this._keyDownBound, false);
+    this.contentElement.removeEventListener('keydown', this._keyDownBound, false);
+    this._glassPane.element.removeEventListener('click', this._onClickBound, false);
+
     super.detach();
-
-    this._glassPane.dispose();
+    this._glassPane.hide();
     delete this._glassPane;
 
     this._restoreTabIndexOnElements();
 
     delete UI.Dialog._instance;
   }
 
   addCloseButton() {
     var closeButton = this.contentElement.createChild('div', 'dialog-close-button', 'dt-close-button');
     closeButton.gray = true;
     closeButton.addEventListener('click', () => this.detach(), false);
   }
 
   /**
-   * @param {number=} positionX
-   * @param {number=} positionY
+   * @param {?number} positionX
+   * @param {?number} positionY
    */
   setPosition(positionX, positionY) {
-    this._defaultPositionX = positionX;
-    this._defaultPositionY = positionY;
+    this._positionX = positionX;
+    this._positionY = positionY;
   }
 
   /**
    * @param {!UI.Size} size
    */
   setMaxSize(size) {
     this._maxSize = size;
   }
 
   /**
+   * @return {?UI.Size}
+   */
+  _effectiveMaxSize() {
+    if (!this._wrapsContent)
+      return this._maxSize;
+    return new UI.Size(this.contentElement.offsetWidth, this.contentElement.offsetHeight).clipTo(this._maxSize);
+  }
+
+  /**
    * @param {boolean} wraps
    */
   setWrapsContent(wraps) {
+    this._wrapsContent = wraps;
     this.element.classList.toggle('wraps-content', wraps);
-    this._wrapsContent = wraps;
   }
 
   /**
    * @param {boolean} dimmed
    */
   setDimmed(dimmed) {
     this._dimmed = dimmed;
   }
 
   contentResized() {
-    if (this._wrapsContent)
-      this._position();
+    if (!this._wrapsContent || !this._glassPane)
+      return;
+    this._glassPane.setMaxContentSize(this._effectiveMaxSize());
   }
 
   /**
    * @param {!Document} document
    */
   _disableTabIndexOnElements(document) {
     this._tabIndexMap.clear();
     for (var node = document; node; node = node.traverseNextNode(document)) {
       if (node instanceof HTMLElement) {
         var element = /** @type {!HTMLElement} */ (node);
@@ skipping 20 matching lines @@
   }
 
   /**
    * @param {!Event} event
    */
   _onGlassPaneClick(event) {
     if (!this.element.isSelfOrAncestor(/** @type {?Node} */ (event.target)))
       this.detach();
   }
 
-  _position() {
-    var container = UI.Dialog._modalHostView.element;
-
-    var width = container.offsetWidth - 10;
-    var height = container.offsetHeight - 10;
-
-    if (this._wrapsContent) {
-      width = Math.min(width, this.contentElement.offsetWidth);
-      height = Math.min(height, this.contentElement.offsetHeight);
-    }
-
-    if (this._maxSize) {
-      width = Math.min(width, this._maxSize.width);
-      height = Math.min(height, this._maxSize.height);
-    }
-
-    var positionX;
-    if (typeof this._defaultPositionX === 'number') {
-      positionX = this._defaultPositionX;
-    } else {
-      positionX = (container.offsetWidth - width) / 2;
-      positionX = Number.constrain(positionX, 0, container.offsetWidth - width);
-    }
-
-    var positionY;
-    if (typeof this._defaultPositionY === 'number') {
-      positionY = this._defaultPositionY;
-    } else {
-      positionY = (container.offsetHeight - height) / 2;
-      positionY = Number.constrain(positionY, 0, container.offsetHeight - height);
-    }
-
-    this.element.style.width = width + 'px';
-    this.element.style.height = height + 'px';
-    this.element.positionAt(positionX, positionY, container);
-  }
-
   /**
    * @param {!Event} event
    */
   _onKeyDown(event) {
     if (event.keyCode === UI.KeyboardShortcut.Keys.Esc.code) {
       event.consume(true);
       this.detach();
     }
   }
 };
-
-
-/** @type {?Element} */
-UI.Dialog._previousFocusedElement = null;
-
-/** @type {?UI.Widget} */
-UI.Dialog._modalHostView = null;