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

chrome/browser/ui/views/autocomplete/touch_autocomplete_popup_contents_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_TOUCH_AUTOCOMPLETE_POPUP_CONTENTS_VIEW_H_
 #define CHROME_BROWSER_UI_VIEWS_AUTOCOMPLETE_TOUCH_AUTOCOMPLETE_POPUP_CONTENTS_VIEW_H_
 #pragma once
 
 #include "base/compiler_specific.h"
 #include "chrome/browser/ui/views/autocomplete/autocomplete_popup_contents_view.h"
@@ skipping 10 matching lines @@
 class View;
 }
 
 class TouchAutocompleteResultView : public AutocompleteResultView {
  public:
   TouchAutocompleteResultView(AutocompleteResultViewModel* model,
                               int model_index,
                               const gfx::Font& font,
                               const gfx::Font& bold_font);
 
+  using AutocompleteResultView::GetState;

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

Nit: I tend to prefer avoiding using statements, and I think they may be banned
entirely in headers anyway.  (Plus it doesn't look like you actually need this
in the .cc file?)

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

This one is just to make the method public in this subclass instead of protected
as it is in the base.

There are two other alternatives, I felt this one was neatest.  The others are:

a) Add a [ friend class TouchAutocompletePopupContentsView ] declaration.  This
exposes more of the object than necessary;

b) Spell out what the using clause does, i.e. add a method like

AutocompleteResultView::ResultViewState GetState() {
  return AutocompleteResultView::GetState();
}

which I'm pretty sure is semantically identical to the using statement, just
more statements of code and more work to maintain (when the function signature
changes).

We've been using this type of using statement in several places in the code
where subclasses (especially but I think not exclusively testing subclasses)
need to change the access to some members.

+
  private:
   virtual ~TouchAutocompleteResultView();
 
   // AutocompleteResultView:
   virtual void PaintMatch(gfx::Canvas* canvas,
                           const AutocompleteMatch& match,
                           int x) OVERRIDE;
   virtual int GetTextHeight() const OVERRIDE;
+  virtual void AdjustIconBounds(const AutocompleteMatch& match,
+                                gfx::Rect* bounds) OVERRIDE;
 
   DISALLOW_COPY_AND_ASSIGN(TouchAutocompleteResultView);
 };
 
 class TouchAutocompletePopupContentsView
     : public AutocompletePopupContentsView {
  public:
   TouchAutocompletePopupContentsView(const gfx::Font& font,
                                      OmniboxView* omnibox_view,
                                      AutocompleteEditModel* edit_model,
                                      views::View* location_bar);
   virtual ~TouchAutocompletePopupContentsView();
 
   // AutocompletePopupContentsView:
   virtual void UpdatePopupAppearance() OVERRIDE;
-  virtual void LayoutChildren() OVERRIDE;
 
  protected:
   // AutocompletePopupContentsView:
   virtual void PaintResultViews(gfx::Canvas* canvas) OVERRIDE;
-  virtual int CalculatePopupHeight() OVERRIDE;
   virtual AutocompleteResultView* CreateResultView(
       AutocompleteResultViewModel* model,
       int model_index,
       const gfx::Font& font,
       const gfx::Font& bold_font) OVERRIDE;
 
  private:
   std::vector<View*> GetVisibleChildren();
 
   DISALLOW_COPY_AND_ASSIGN(TouchAutocompletePopupContentsView);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_AUTOCOMPLETE_TOUCH_AUTOCOMPLETE_POPUP_CONTENTS_VIEW_H_