[DevTools] Fix UI.EmptyWidget scrollbars

third_party/WebKit/Source/devtools/front_end/ui/EmptyWidget.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 20 matching lines @@
 /**
  * @unrestricted
  */
 UI.EmptyWidget = class extends UI.VBox {
   /**
    * @param {string} text
    */
   constructor(text) {
     super();
     this.registerRequiredCSS('ui/emptyWidget.css');
-    this.element.classList.add('empty-view');
-    this.textElement = this.element.createChild('h2');
+    this.element.classList.add('empty-view-scroller');
+    this._panel = this.element.createChild('div', 'empty-view');

[caseq, 2017/05/20 00:49:05]

nit: panel is a bit overloaded. _contentElement or _innerElement perhaps?

[eostroukhov, 2017/05/22 22:21:16]

Done.

+    this.textElement = this._panel.createChild('h2');

[caseq, 2017/05/20 00:49:05]

Can we make it private? I don't think it's used outside.

[eostroukhov, 2017/05/22 22:21:16]

Done.

     this.textElement.textContent = text;
   }
 
   /**
    * @return {!Element}
    */
   appendParagraph() {
-    return this.element.createChild('p');
+    return this._panel.createChild('p');

[caseq, 2017/05/20 00:49:05]

Shouldn't this be under textElement?

[eostroukhov, 2017/05/22 22:21:16]

I don't think so. textElement is a main text, that is wrapped in a header
element. Paragraphs appended here serve more as a description text.

   }
 
   /**
    * @param {string} text
    */
   set text(text) {
     this.textElement.textContent = text;
   }
 };