Files.app: Fit the size of search box as its contents size.

chrome/browser/resources/file_manager/js/file_manager.js

Index: chrome/browser/resources/file_manager/js/file_manager.js
diff --git a/chrome/browser/resources/file_manager/js/file_manager.js b/chrome/browser/resources/file_manager/js/file_manager.js
index bd6972be36f43fbd503ca28b0dc465154b3803df..6f0c2931805ea76a73ee206c9e54ab911772faf4 100644
--- a/chrome/browser/resources/file_manager/js/file_manager.js
+++ b/chrome/browser/resources/file_manager/js/file_manager.js
@@ -993,6 +993,21 @@ DialogType.isModal = function(type) {
     this.searchBox_ = this.dialogDom_.querySelector('#search-box');
     this.searchBox_.addEventListener(
         'input', this.onSearchBoxUpdate_.bind(this));
+    this.searchTextMeasure_ = new TextMeasure(this.searchBox_);
+    if (util.platform.newUI()) {
+      this.searchIcon_ = this.dialogDom_.querySelector('#search-icon');
+      this.searchIcon_.addEventListener(
+          'click',
+          function() { this.searchBox_.focus(); }.bind(this));
+      this.searchClearButton_ =
+          this.dialogDom_.querySelector('#search-clear-button');
+      this.searchClearButton_.addEventListener(
+          'click',
+          function() {
+            this.searchBox_.value = '';
+            this.onSearchBoxUpdate_();
+          }.bind(this));
+    }
     this.lastSearchQuery_ = '';
 
     var autocompleteList = new cr.ui.AutocompleteList();
@@ -2344,7 +2359,7 @@ DialogType.isModal = function(type) {
   FileManager.prototype.updateSearchBoxOnDirChange_ = function() {
     if (!this.searchBox_.disabled) {
       this.searchBox_.value = '';
-      this.updateSearchBoxClass_();
+      this.updateSearchBoxStyles_();
     }
   };
 
@@ -3420,7 +3435,7 @@ DialogType.isModal = function(type) {
   FileManager.prototype.onSearchBoxUpdate_ = function(event) {
     var searchString = this.searchBox_.value;
 
-    this.updateSearchBoxClass_();
+    this.updateSearchBoxStyles_();
     if (this.isOnDrive()) {
       // When the search text is changed, finishes the search and showes back
       // the last directory by passing an empty string to
@@ -3444,8 +3459,15 @@ DialogType.isModal = function(type) {
    *
    * @private
    */
-  FileManager.prototype.updateSearchBoxClass_ = function() {
-    this.searchBox_.classList.toggle('has-text', !!this.searchBox_.value);
+  FileManager.prototype.updateSearchBoxStyles_ = function() {
+    if (!util.platform.newUI())
+      return;
+    var TEXT_BOX_PADDING = 16; // in px.
+    this.searchBoxWrapper_.classList.toggle('has-text',
+                                            !!this.searchBox_.value);
+    var width = this.searchTextMeasure_.getWidth(this.searchBox_.value) +
+                TEXT_BOX_PADDING;
+    this.searchBox_.style.width = width + 'px';
   };
 
   /**
@@ -3832,4 +3854,45 @@ DialogType.isModal = function(type) {
     this.document_.querySelector('#drive-clear-local-cache').canExecuteChange();
     this.document_.querySelector('#drive-reload').canExecuteChange();
   };
+
+  /**

[yoshiki, 2013/05/24 14:49:54]

nit: Please move this class to the first part of this file. (maybe around L17 or
L78 is good place).  Utility class/function should be placed at the first of the
file.

[hirono, 2013/05/27 01:58:28]

Done.

+   * TextMeasure constructor.
+   *
+   * TextMeasure is a measure for text that returns the width of text.  This
+   * class has a dummy span element. When measuring the width of text, it sets
+   * the text to the element and obtains the element's size by
+   * getBoundingClientRect.
+   *
+   * @param {HTMLElement} element Element that has styles of measured text. The
+   *     width of text is mesures like as it is rendered in this element.

[yoshiki, 2013/05/24 14:49:54]

nit: add '@constructor'

[hirono, 2013/05/27 01:58:28]

Done.

+   */
+  var TextMeasure = function(element) {
+    var doc = element.ownerDocument;
+    this.dummySpan_ = doc.createElement('span');
+    this.dummySpan_ = doc.getElementsByTagName('body')[0]
+                         .appendChild(this.dummySpan_);

[yoshiki, 2013/05/24 14:49:54]

nit: a dot should be at the last of the previous line.

[hirono, 2013/05/27 01:58:28]

Done.

+    this.dummySpan_.style.position = 'absolute';
+    this.dummySpan_.style.visibility = 'hidden';
+    var styles = window.getComputedStyle(element, '');
+    var stylesToBeCopied = ['fontSize',
+                            'fontStyle',
+                            'fontWeight',
+                            'fontFamily',
+                            'letterSpacing'];
+    for (var i = 0; i < stylesToBeCopied.length; i++) {
+      this.dummySpan_.style[stylesToBeCopied[i]] = styles[stylesToBeCopied[i]];
+    }
+  };
+
+  /**
+   * Measures the widht of text.
+   *
+   * @param {string} text Text that is measured the width.
+   */
+  TextMeasure.prototype.getWidth = function(text) {
+    this.dummySpan_.innerText = text;
+    var rect = this.dummySpan_.getBoundingClientRect();
+    return rect ? rect.width : 0;
+  };
+
 })();