First stab at touch optimized omnibox auto-complete per sgabriel's mocks.

chrome/browser/ui/views/autocomplete/autocomplete_result_view.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_VIEWS_AUTOCOMPLETE_AUTOCOMPLETE_RESULT_VIEW_H_
 #define CHROME_BROWSER_UI_VIEWS_AUTOCOMPLETE_AUTOCOMPLETE_RESULT_VIEW_H_
 #pragma once
 
 #include "chrome/browser/autocomplete/autocomplete_match.h"
 #include "third_party/skia/include/core/SkColor.h"
@@ skipping 42 matching lines @@
   void SetMatch(const AutocompleteMatch& match);
 
   void ShowKeyword(bool show_keyword);
 
   void Invalidate();
 
   // views::View:
   virtual gfx::Size GetPreferredSize() OVERRIDE;
 
  protected:
+  ResultViewState GetState() const;
+
   virtual void PaintMatch(gfx::Canvas* canvas,
                           const AutocompleteMatch& match,
                           int x);
 
+  // The base class never adjusts the bounds for the icon.  Subclasses
+  // may adjust them e.g. if they are using a multi-line or horizontal
+  // layout.
+  virtual void AdjustIconBounds(const AutocompleteMatch& match,
+                                gfx::Rect* bounds);
+
   // Returns the height of the text portion of the result view. In the base
   // class, this is the height of one line of text.
   virtual int GetTextHeight() const;
 
   // Draws the specified |text| into the canvas, using highlighting provided by
   // |classifications|. If |force_dim| is true, ACMatchClassification::DIM is
   // added to all of the classifications. Returns the x position to the right
   // of the string.
   int DrawString(gfx::Canvas* canvas,
                  const string16& text,
                  const ACMatchClassifications& classifications,
                  bool force_dim,
                  int x,
                  int y);
 
   const gfx::Rect& text_bounds() const { return text_bounds_; }
 
+  // Default values cached here, may be overridden by subclasses.

[Peter Kasting, 2012/05/04 23:33:31]

Nit: The Google style guide says to use overrideable accessors or else setters
rather than exposing data members to subclasses directly.

[Jói, 2012/05/04 23:52:19]

Wanted to avoid the virtual methods since these get accessed a lot.  I guess
I'll make these private and add protected setters to comply with that bit of the
style guide.

[Jói, 2012/05/07 16:26:18]

Done.

+  int edge_item_padding_;
+  int item_padding_;
+  int minimum_text_vertical_padding_;
+
  private:
   struct ClassificationData;
   typedef std::vector<ClassificationData> Classifications;
 
   struct RunData;
   typedef std::vector<RunData> Runs;
 
   // Predicate functions for use when sorting the runs.
   static bool SortRunsLogically(const RunData& lhs, const RunData& rhs);
   static bool SortRunsVisually(const RunData& lhs, const RunData& rhs);
 
-  ResultViewState GetState() const;
   const SkBitmap* GetIcon() const;
   const SkBitmap* GetKeywordIcon() const;
 
   // Elides |runs| to fit in |remaining_width|.  The runs in |runs| should be in
   // logical order.
   //
   // When we need to elide a run, the ellipsis will be placed at the end of that
   // run.  This means that if we elide a run whose visual direction is opposite
   // that of the drawing context, the ellipsis will not be at the "end" of the
   // drawn string.  For example, if in an LTR context we have the LTR run
@@ skipping 37 matching lines @@
 
   gfx::Rect keyword_text_bounds_;
   scoped_ptr<views::ImageView> keyword_icon_;
 
   scoped_ptr<ui::SlideAnimation> animation_;
 
   DISALLOW_COPY_AND_ASSIGN(AutocompleteResultView);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_AUTOCOMPLETE_AUTOCOMPLETE_RESULT_VIEW_H_