Create a new autocomplete popup for touch. This CL depends on

chrome/browser/autocomplete/autocomplete_edit_view_views.cc

 // Copyright (c) 2011 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.
 
 #include "chrome/browser/autocomplete/autocomplete_edit_view_views.h"
 
 #include "base/logging.h"
 #include "base/string_util.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/autocomplete/autocomplete_edit.h"
 #include "chrome/browser/autocomplete/autocomplete_match.h"
 #include "chrome/browser/autocomplete/autocomplete_popup_model.h"
 #include "chrome/browser/command_updater.h"
 #include "chrome/browser/tab_contents/tab_contents.h"
 #include "chrome/browser/ui/views/autocomplete/autocomplete_popup_contents_view.h"
+#include "chrome/browser/ui/views/autocomplete/touch_autocomplete_popup_contents_view.h"
 #include "chrome/browser/ui/views/location_bar/location_bar_view.h"
 #include "chrome/common/notification_service.h"
 #include "googleurl/src/gurl.h"
 #include "grit/generated_resources.h"
 #include "net/base/escape.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/gfx/font.h"
 #include "views/border.h"
 #include "views/layout/fill_layout.h"
 
@@ skipping 76 matching lines @@
 }  // namespace
 
 AutocompleteEditViewViews::AutocompleteEditViewViews(
     AutocompleteEditController* controller,
     ToolbarModel* toolbar_model,
     Profile* profile,
     CommandUpdater* command_updater,
     bool popup_window_mode,
     const views::View* location_bar)
     : model_(new AutocompleteEditModel(this, controller, profile)),
-      popup_view_(new AutocompletePopupContentsView(
-          gfx::Font(), this, model_.get(), profile, location_bar)),
+      popup_view_(CreatePopupView(profile, location_bar)),
       controller_(controller),
       toolbar_model_(toolbar_model),
       command_updater_(command_updater),
       popup_window_mode_(popup_window_mode),
       security_level_(ToolbarModel::NONE),
       delete_was_pressed_(false),
       delete_at_end_pressed_(false) {
   set_border(views::Border::CreateEmptyBorder(kAutocompleteVerticalMargin, 0,
                                               kAutocompleteVerticalMargin, 0));
 }
@@ skipping 509 matching lines @@
 
 string16 AutocompleteEditViewViews::GetSelectedText() const {
   // TODO(oshima): Support instant, IME.
   return textfield_->GetSelectedText();
 }
 
 void AutocompleteEditViewViews::SelectRange(size_t caret, size_t end) {
   const views::TextRange range(caret, end);
   textfield_->SelectRange(range);
 }
+
+AutocompletePopupView* AutocompleteEditViewViews::CreatePopupView(
+    Profile* profile, const views::View* location_bar) {

[Peter Kasting, 2011/02/17 23:08:57]

Nit: One arg per line

[varunjain, 2011/02/18 00:16:34]

Done.

+#if defined(TOUCH_UI)
+  return new TouchAutocompletePopupContentsView(
+      gfx::Font(), this, model_.get(), profile, location_bar);

[Peter Kasting, 2011/02/17 23:08:57]

Nit: Why not bring this second line out of the #if?

[varunjain, 2011/02/18 00:16:34]

It looked better to me that way... but anyways. Done.

+#else
+  return new AutocompletePopupContentsView(
+      gfx::Font(), this, model_.get(), profile, location_bar);
+#endif
+}