DevTools: Make sure a horizontal scrollbar doesn't appear in SuggestBox

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

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * @template T
  * @interface
  */
 UI.ListDelegate = function() {};
 
@@ skipping 34 matching lines @@
 
 /**
  * @template T
  */
 UI.ListControl = class {
   /**
    * @param {!UI.ListDelegate<T>} delegate
    */
   constructor(delegate) {
     this.element = createElement('div');
-    this.element.style.overflow = 'auto';
+    this.element.style.overflowY = 'auto';
+    this.element.style.overflowX = 'hidden';

[dgozman, 2016/12/30 02:30:40]

I think this line belongs to clients. I can easily imagine viewport with a
horizontal scrollbar, e.g. navigator view.

[einbinder, 2017/01/03 20:06:53]

The ListControl doesn't viewport horizontally, so it shouldn't scroll
horizontally. It can easily be wrapped in a div that scrolls horizontally.

[einbinder, 2017/01/05 18:45:32]

That makes sense. Done.

     this._topElement = this.element.createChild('div');
     this._bottomElement = this.element.createChild('div');
     this._firstIndex = 0;
     this._lastIndex = 0;
     this._renderedHeight = 0;
     this._topHeight = 0;
     this._bottomHeight = 0;
     this._clearViewport();
 
     /** @type {!Array<T>} */
@@ skipping 482 matching lines @@
     this._firstIndex = firstIndex;
     this._lastIndex = lastIndex;
     this._topHeight = this._offsetAtIndex(firstIndex);
     this._topElement.style.height = this._topHeight + 'px';
     this._bottomHeight = totalHeight - this._offsetAtIndex(lastIndex);
     this._bottomElement.style.height = this._bottomHeight + 'px';
     this._renderedHeight = totalHeight;
     this.element.scrollTop = scrollTop;
   }
 };