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

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

 // Copyright (c) 2012 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.
 
 'use strict';
 
 /**
  * This variable is checked in SelectFileDialogExtensionBrowserTest.
  * @type {number}
  */
@@ skipping 975 matching lines @@
     if (loadTimeData.getBoolean('ASH'))
       this.dialogDom_.setAttribute('ash', 'true');
 
     this.filePopup_ = null;
 
     this.searchBoxWrapper_ =
         this.dialogDom_.querySelector('.search-box-wrapper');
     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();
     autocompleteList.id = 'autocomplete-list';
     autocompleteList.autoExpands = true;
     autocompleteList.requestSuggestions =
         this.requestAutocompleteSuggestions_.bind(this);
     // function(item) {}.bind(this) does not work here, as it's a constructor.
     var self = this;
     autocompleteList.itemConstructor = function(item) {
@@ skipping 1331 matching lines @@
                            path.substring(rootPath.length);
   };
 
   /**
    * Updates search box value when directory gets changed.
    * @private
    */
   FileManager.prototype.updateSearchBoxOnDirChange_ = function() {
     if (!this.searchBox_.disabled) {
       this.searchBox_.value = '';
-      this.updateSearchBoxClass_();
+      this.updateSearchBoxStyles_();
     }
   };
 
   /**
    * Update the gear menu.
    * @private
    */
   FileManager.prototype.updateGearMenu_ = function() {
     var hideItemsForDrive = !this.isOnDrive();
     this.syncButton.hidden = hideItemsForDrive;
@@ skipping 1055 matching lines @@
 
   /**
    * Invoked when the search box is changed.
    *
    * @param {Event} event The 'changed' event.
    * @private
    */
   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
       // {@code DirectoryModel.search()}.
       if (this.directoryModel_.isSearching() &&
           this.lastSearchQuery_ != searchString) {
         this.doSearch('');
       }
 
       // On drive, incremental search is not invoked since we have an auto-
       // complete suggestion instead.
       return;
     }
 
     this.search_(searchString);
   };
 
   /**
    * Updates search box's CSS classes.
    * These classes are refered from CSS.
    *
    * @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;

[yoshiki, 2013/05/24 09:56:16]

Add the comment '// in px.'

[hirono, 2013/05/24 10:36:43]

Done.

+    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';
   };
 
   /**
    * Search files and update the list with the search result.
    *
    * @param {string} searchString String to be searched with.
    * @private
    */
   FileManager.prototype.search_ = function(searchString) {
     var noResultsDiv = this.document_.getElementById('no-search-results');
@@ skipping 366 matching lines @@
   /**
    * Set the flag expressing whether the ctrl key is pressed or not.
    * @param {boolean} flag New value of the flag
    * @private
    */
   FileManager.prototype.setCtrlKeyPressed_ = function(flag) {
     this.ctrlKeyPressed_ = flag;
     this.document_.querySelector('#drive-clear-local-cache').canExecuteChange();
     this.document_.querySelector('#drive-reload').canExecuteChange();
   };
+
+  /**
+   * TextMeasure constructor.
+   *
+   * TextMeasure is a measure for text that returns the width of text.

[yoshiki, 2013/05/24 09:56:16]

I think this approach (creating dummySpan and measure it) is somewhat hacky.
Please write the comment about the implementation.

[hirono, 2013/05/24 10:36:43]

Done.

+   *
+   * @param {HTMLElement} element Element that has styles of measured text. The
+   *     width of text is mesures like as it is rendered in this element.
+   */
+  var TextMeasure = function(element) {
+    this.dummySpan = document.createElement('span');

[yoshiki, 2013/05/24 09:56:16]

This is a private variable, so please add an underscode at the end of the name.

[hirono, 2013/05/24 10:36:43]

Done.

+    this.dummySpan = document.getElementsByTagName('body')[0]

[yoshiki, 2013/05/24 09:56:16]

Pass FileManager.dialogDom_ or (dialogDom_.ownerDocument) from the caller and
use it instead of direct use of 'document'.

[hirono, 2013/05/24 10:36:43]

How about using element.document?
|element| is already passed as an argument.

+                             .appendChild(this.dummySpan);
+    this.dummySpan.style.position = 'absolute';
+    this.dummySpan.style.visibility = 'hidden';
+    var styles = getComputedStyle(element, '');

[yoshiki, 2013/05/24 09:58:33]

window.getComputedStyle()

[hirono, 2013/05/24 10:36:43]

Done.

+    var copiedStyles = ['fontSize',

[yoshiki, 2013/05/24 09:56:16]

"attributesToBeCopied" (or something like it) is better than "copiesStyles"?

[hirono, 2013/05/24 10:36:43]

attributes looks like HTML attributes. So 'stylesToBeCopied' is OK?
 

+                        'fontStyle',
+                        'fontWeight',
+                        'fontFamily',
+                        'letterSpacing'];
+    for (var i = 0; i < copiedStyles.length; i++) {
+      console.log(copiedStyles[i] + ': ' + styles[copiedStyles[i]]);

[yoshiki, 2013/05/24 09:56:16]

remove debug log

[hirono, 2013/05/24 10:36:43]

Done.

+      this.dummySpan.style[copiedStyles[i]] = styles[copiedStyles[i]];
+    }
+  };
+
+  TextMeasure.prototype.getWidth = function(text) {
+    this.dummySpan.innerText = text;
+    return this.dummySpan.getBoundingClientRect().width;

[yoshiki, 2013/05/24 09:56:16]

getBoundingClientRect() returns null if the element is hidden. Please add
null-check.

[hirono, 2013/05/24 10:36:43]

Done.

+  };
+
 })();