Print Preview Print Destination Search Widget

chrome/browser/resources/print_preview/search/search_box.html

Index: chrome/browser/resources/print_preview/search/search_box.html
diff --git a/chrome/browser/resources/print_preview/search/search_box.html b/chrome/browser/resources/print_preview/search/search_box.html
new file mode 100644
index 0000000000000000000000000000000000000000..3ac652ae5d75a9630e0ac01167eceec54a94cb88
--- /dev/null
+++ b/chrome/browser/resources/print_preview/search/search_box.html
@@ -0,0 +1,6 @@
+<div id="search-box" class="search-box">

[Dan Beam, 2012/05/29 21:25:50]

any reason to have both a class and ID? do you have multiple search-box's on the
page or plan to have more?

[Robert Toscano, 2012/05/29 22:17:36]

The ID is there because I actually reference the element in JS by ID, and the
class is there so that I can style it. I know that I can style based on the ID,
but really the style is based on the class. Right now, this is a unique element,
but I wanted to leave room for it being used in multiple places.

+  <img src="../images/search.png" class="search-box-icon"/>
+  <input type="text"

[Dan Beam, 2012/05/29 21:25:50]

seems like this wrapping is taking up too many lines, think the preferred is

  <input type="text" class="search-box-input"
      i18n-values="placeholder:searchBoxPlaceholder"/>

also, you don't really need the / in self-closing tags in HTML (just XHTML), but
that's another religious debate...

[Robert Toscano, 2012/05/29 22:17:36]

Done.
Yea I know I don't need "/" for HTML, but if we ever wanted to migrate to XHTML,
it would be one less step. I also would just always end with "/" just so I don't
ever leave it out when I shouldn't.

+         class="search-box-input"
+         i18n-values="placeholder:searchBoxPlaceholder"/>
+</div>