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

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

 // 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.
 
 // For WinDDK ATL compatibility, these ATL headers must come first.
 #include "build/build_config.h"
 #if defined(OS_WIN)
 #include <atlbase.h>  // NOLINT
 #include <atlwin.h>  // NOLINT
 #endif
@@ skipping 85 matching lines @@
   int right_;
 
   DISALLOW_COPY_AND_ASSIGN(MirroringContext);
 };
 
 AutocompleteResultView::AutocompleteResultView(
     AutocompleteResultViewModel* model,
     int model_index,
     const gfx::Font& font,
     const gfx::Font& bold_font)
-    : model_(model),
+    : edge_item_padding_(LocationBarView::GetItemPadding()),
+      item_padding_(LocationBarView::GetItemPadding()),
+      minimum_text_vertical_padding_(kMinimumTextVerticalPadding),
+      model_(model),
       model_index_(model_index),
       normal_font_(font),
       bold_font_(bold_font),
       ellipsis_width_(font.GetStringWidth(string16(kEllipsis))),
       mirroring_context_(new MirroringContext()),
       keyword_icon_(new views::ImageView()),
       ALLOW_THIS_IN_INITIALIZER_LIST(
           animation_(new ui::SlideAnimation(this))) {
   CHECK_GE(model_index, 0);
   if (default_icon_size_ == 0) {
@@ skipping 46 matching lines @@
       colors[i][TEXT] =
           color_utils::AlphaBlend(SK_ColorBLACK, colors[i][BACKGROUND], 0xdd);
       colors[i][DIMMED_TEXT] =
           color_utils::AlphaBlend(SK_ColorBLACK, colors[i][BACKGROUND], 0xbb);
 #else
       colors[i][DIMMED_TEXT] =
           color_utils::AlphaBlend(colors[i][TEXT], colors[i][BACKGROUND], 128);
       colors[i][URL] = color_utils::GetReadableColor(SkColorSetRGB(0, 128, 0),
                                                      colors[i][BACKGROUND]);
 #endif
+
+      // When using non-default color schemes, the divider color may clash with
+      // the bubble border, which is currently "hard coded" in images as black
+      // with an 0x40 alpha channel.  See http://crbug.com/127861
+      colors[i][DIVIDER] = color_utils::AlphaBlend(colors[i][TEXT],

[Peter Kasting, 2012/05/11 23:01:41]

Nit: How about just:

      // TODO(joi): Programmatically draw the dropdown border using this color
      // as well.  (Right now it's drawn as black with 25% alpha.)
      colors[i][DIVIDER] =
          color_utils::AlphaBlend(colors[i][TEXT], colors[i][BACKGROUND], 0x34);

[Jói, 2012/05/14 18:29:45]

Done.

+                                                   colors[i][BACKGROUND],
+                                                   0x34);  // 20%
     }
     initialized = true;
   }
 
   return colors[state][kind];
 }
 
 void AutocompleteResultView::SetMatch(const AutocompleteMatch& match) {
   match_ = match;
   animation_->Reset();
@@ skipping 18 matching lines @@
 }
 
 void AutocompleteResultView::Invalidate() {
   keyword_icon_->SetImage(GetKeywordIcon());
   SchedulePaint();
 }
 
 gfx::Size AutocompleteResultView::GetPreferredSize() {
   return gfx::Size(0, std::max(
       default_icon_size_ + (kMinimumIconVerticalPadding * 2),
-      GetTextHeight() + (kMinimumTextVerticalPadding * 2)));
+      GetTextHeight() + (minimum_text_vertical_padding_ * 2)));
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // AutocompleteResultView, protected:
 
+AutocompleteResultView::ResultViewState
+    AutocompleteResultView::GetState() const {
+  if (model_->IsSelectedIndex(model_index_))
+    return SELECTED;
+  return model_->IsHoveredIndex(model_index_) ? HOVERED : NORMAL;
+}
+
 void AutocompleteResultView::PaintMatch(gfx::Canvas* canvas,
                                         const AutocompleteMatch& match,
                                         int x) {
   x = DrawString(canvas, match.contents, match.contents_class, false, x,
                  text_bounds_.y());
 
   // Paint the description.
   // TODO(pkasting): Because we paint in multiple separate pieces, we can wind
   // up with no space even for an ellipsis for one or both of these pieces.
   // Instead, we should paint the entire match as a single long string.  This
@@ skipping 25 matching lines @@
 
 // static
 bool AutocompleteResultView::SortRunsVisually(const RunData& lhs,
                                               const RunData& rhs) {
   return lhs.visual_order < rhs.visual_order;
 }
 
 // static
 int AutocompleteResultView::default_icon_size_ = 0;
 
-AutocompleteResultView::ResultViewState
-    AutocompleteResultView::GetState() const {
-  if (model_->IsSelectedIndex(model_index_))
-    return SELECTED;
-  return model_->IsHoveredIndex(model_index_) ? HOVERED : NORMAL;
-}
-
 const SkBitmap* AutocompleteResultView::GetIcon() const {
   const SkBitmap* bitmap = model_->GetIconIfExtensionMatch(model_index_);
   if (bitmap)
     return bitmap;
 
   int icon = match_.starred ?
       IDR_OMNIBOX_STAR : AutocompleteMatch::TypeToIcon(match_.type);
   if (GetState() == SELECTED) {
     switch (icon) {
       case IDR_OMNIBOX_EXTENSION_APP:
@@ skipping 252 matching lines @@
     }
   }
 
   // We couldn't draw anything.
   runs->clear();
 }
 
 void AutocompleteResultView::Layout() {
   const SkBitmap* icon = GetIcon();
 
-  icon_bounds_.SetRect(LocationBarView::kEdgeItemPadding +
-      ((icon->width() == default_icon_size_) ?
-          0 : LocationBarView::kIconInternalPadding),
-      (height() - icon->height()) / 2, icon->width(), icon->height());
+  int icon_padding = (icon->width() == default_icon_size_) ?

[Peter Kasting, 2012/05/11 23:01:41]

Nit: Frankly I was OK with how it was before, just with |edge_item_padding_|

+                      0 : LocationBarView::kIconInternalPadding;
+  icon_bounds_.SetRect(edge_item_padding_ + icon_padding,
+                       (height() - icon->height()) / 2,
+                       icon->width(),
+                       icon->height());
 
-  int text_x = LocationBarView::kEdgeItemPadding + default_icon_size_ +
-      LocationBarView::kItemPadding;
+  int text_x = edge_item_padding_ + default_icon_size_ + item_padding_;
   int text_height = GetTextHeight();
   int text_width;
 
   if (match_.associated_keyword.get()) {
     const int kw_collapsed_size = keyword_icon_->width() +

[Peter Kasting, 2012/05/11 23:01:41]

Nit: Wrap after '=' instead of '+'

[Jói, 2012/05/14 18:29:45]

Done.

-        LocationBarView::kEdgeItemPadding;
+        edge_item_padding_;
     const int max_kw_x = width() - kw_collapsed_size;
     const int kw_x = animation_->CurrentValueBetween(max_kw_x,

[Peter Kasting, 2012/05/11 23:01:41]

Nit: Wrap after '=' instead of ','

[Jói, 2012/05/14 18:29:45]

Done.

-        LocationBarView::kEdgeItemPadding);
-    const int kw_text_x = kw_x + keyword_icon_->width() +
-        LocationBarView::kItemPadding;
+        edge_item_padding_);
+    const int kw_text_x = kw_x + keyword_icon_->width() + item_padding_;
 
-    text_width = kw_x - text_x - LocationBarView::kItemPadding;
+    text_width = kw_x - text_x - item_padding_;
     keyword_text_bounds_.SetRect(kw_text_x, 0, std::max(

[Peter Kasting, 2012/05/11 23:01:41]

Nit: Wrap before "std::max(" instead of after

[Jói, 2012/05/14 18:29:45]

Done.

-        width() - kw_text_x - LocationBarView::kEdgeItemPadding, 0),
+        width() - kw_text_x - edge_item_padding_, 0),
         text_height);
     keyword_icon_->SetPosition(gfx::Point(kw_x,
         (height() - keyword_icon_->height()) / 2));
   } else {
-    text_width = width() - text_x - LocationBarView::kEdgeItemPadding;
+    text_width = width() - text_x - edge_item_padding_;
   }
 
   text_bounds_.SetRect(text_x, std::max(0, (height() - text_height) / 2),
       std::max(text_width, 0), text_height);
 }
 
 void AutocompleteResultView::OnBoundsChanged(
     const gfx::Rect& previous_bounds) {
   animation_->SetSlideDuration(width() / 4);
 }
 
 void AutocompleteResultView::OnPaint(gfx::Canvas* canvas) {
   const ResultViewState state = GetState();
   if (state != NORMAL)
     canvas->DrawColor(GetColor(state, BACKGROUND));
 
   if (!match_.associated_keyword.get() ||
       keyword_icon_->x() > icon_bounds_.right()) {
     // Paint the icon.
     canvas->DrawBitmapInt(*GetIcon(), GetMirroredXForRect(icon_bounds_),
-        icon_bounds_.y());
+                          icon_bounds_.y());
 
     // Paint the text.
     int x = GetMirroredXForRect(text_bounds_);
     mirroring_context_->Initialize(x, text_bounds_.width());
     PaintMatch(canvas, match_, x);
   }
 
   if (match_.associated_keyword.get()) {
     // Paint the keyword text.
     int x = GetMirroredXForRect(keyword_text_bounds_);
     mirroring_context_->Initialize(x, keyword_text_bounds_.width());
     PaintMatch(canvas, *match_.associated_keyword.get(), x);
   }
 }
 
 void AutocompleteResultView::AnimationProgressed(
     const ui::Animation* animation) {
   Layout();
   SchedulePaint();
 }