Converts some of the omnibox related classes to support multiple

chrome/browser/ui/views/omnibox/omnibox_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 10 matching lines @@
 #include "grit/generated_resources.h"
 #include "grit/theme_resources.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/native_theme/native_theme.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/base/text/text_elider.h"
 #include "ui/gfx/canvas.h"
 #include "ui/gfx/color_utils.h"
 #include "ui/gfx/render_text.h"
 
+#if defined(OS_WIN)
+#include "ui/base/native_theme/native_theme_win.h"
+#endif
+
+#if defined(USE_AURA)
+#include "ui/base/native_theme/native_theme_aura.h"
+#endif
+
 namespace {
 
 const char16 kEllipsis[] = { 0x2026, 0x0 };
 
 // The minimum distance between the top and bottom of the {icon|text} and the
 // top or bottom of the row.
 const int kMinimumIconVerticalPadding = 2;
 const int kMinimumTextVerticalPadding = 3;
 
 }  // namespace
@@ skipping 87 matching lines @@
   keyword_icon_->set_owned_by_client();
   keyword_icon_->EnableCanvasFlippingForRTLUI(true);
   keyword_icon_->SetImage(GetKeywordIcon());
   keyword_icon_->SizeToPreferredSize();
 }
 
 OmniboxResultView::~OmniboxResultView() {
 }
 
 // static
-SkColor OmniboxResultView::GetColor(ResultViewState state, ColorKind kind) {
-  static bool initialized = false;
-  static SkColor colors[NUM_STATES][NUM_KINDS];
-  if (!initialized) {
+SkColor OmniboxResultView::GetColor(
+    const ui::NativeTheme* theme,
+    ResultViewState state,
+    ColorKind kind) {
 #if defined(OS_WIN)
-    colors[NORMAL][BACKGROUND] = color_utils::GetSysSkColor(COLOR_WINDOW);
-    colors[SELECTED][BACKGROUND] = color_utils::GetSysSkColor(COLOR_HIGHLIGHT);
-    colors[NORMAL][TEXT] = color_utils::GetSysSkColor(COLOR_WINDOWTEXT);
-    colors[SELECTED][TEXT] = color_utils::GetSysSkColor(COLOR_HIGHLIGHTTEXT);
-#elif defined(USE_AURA)
-    const ui::NativeTheme* theme = ui::NativeTheme::instance();
-    colors[SELECTED][BACKGROUND] = theme->GetSystemColor(
-        ui::NativeTheme::kColorId_TextfieldSelectionBackgroundFocused);
-    colors[NORMAL][BACKGROUND] = theme->GetSystemColor(
-        ui::NativeTheme::kColorId_TextfieldDefaultBackground);
-    colors[NORMAL][URL] = SkColorSetARGB(0xff, 0x00, 0x99, 0x33);
-    colors[SELECTED][URL] = SkColorSetARGB(0xff, 0x00, 0x66, 0x22);
-    colors[HOVERED][URL] = SkColorSetARGB(0xff, 0x00, 0x66, 0x22);
-#else
-    // TODO(beng): source from theme provider.
-    colors[NORMAL][BACKGROUND] = SK_ColorWHITE;
-    colors[SELECTED][BACKGROUND] = SK_ColorBLUE;
-    colors[NORMAL][TEXT] = SK_ColorBLACK;
-    colors[SELECTED][TEXT] = SK_ColorWHITE;
+  if (theme == ui::NativeThemeWin::instance()) {
+    static bool initialized = false;
+    static SkColor colors[NUM_STATES][NUM_KINDS];
+    if (!initialized) {
+      colors[NORMAL][BACKGROUND] = color_utils::GetSysSkColor(COLOR_WINDOW);
+      colors[SELECTED][BACKGROUND] =
+          color_utils::GetSysSkColor(COLOR_HIGHLIGHT);
+      colors[NORMAL][TEXT] = color_utils::GetSysSkColor(COLOR_WINDOWTEXT);
+      colors[SELECTED][TEXT] = color_utils::GetSysSkColor(COLOR_HIGHLIGHTTEXT);
+      CommonInitColors(theme, colors);
+      initialized = true;
+    }
+    return colors[state][kind];
+  }
 #endif
-    colors[HOVERED][BACKGROUND] =
-        color_utils::AlphaBlend(colors[SELECTED][BACKGROUND],
-                                colors[NORMAL][BACKGROUND], 64);
-    colors[HOVERED][TEXT] = colors[NORMAL][TEXT];
-    for (int i = 0; i < NUM_STATES; ++i) {
-#if defined(USE_AURA)
-      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
-
-      // 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);
+  {

[sky, 2012/11/08 02:04:24]

I put this in a block so that there is no conflict with initialized and colors.

[Peter Kasting, 2012/11/08 18:55:47]

Might be slightly clearer to add a qualifier to the Windows-specific variables
in the above block, e.g. "colors" -> "win_colors" or something.

+    static bool initialized = false;
+    static SkColor colors[NUM_STATES][NUM_KINDS];
+    if (!initialized) {
+      colors[SELECTED][BACKGROUND] = theme->GetSystemColor(
+          ui::NativeTheme::kColorId_TextfieldSelectionBackgroundFocused);
+      colors[NORMAL][BACKGROUND] = theme->GetSystemColor(
+          ui::NativeTheme::kColorId_TextfieldDefaultBackground);
+      colors[NORMAL][URL] = SkColorSetARGB(0xff, 0x00, 0x99, 0x33);
+      colors[SELECTED][URL] = SkColorSetARGB(0xff, 0x00, 0x66, 0x22);
+      colors[HOVERED][URL] = SkColorSetARGB(0xff, 0x00, 0x66, 0x22);
+      CommonInitColors(theme, colors);
+      initialized = true;
     }
-    initialized = true;
+    return colors[state][kind];
   }
-
-  return colors[state][kind];
 }
 
 void OmniboxResultView::SetMatch(const AutocompleteMatch& match) {
   match_ = match;
   animation_->Reset();
 
   if (match.associated_keyword.get()) {
     keyword_icon_->SetImage(GetKeywordIcon());
 
     if (!keyword_icon_->parent())
@@ skipping 56 matching lines @@
     DrawString(canvas, match.description, match.description_class, true, x,
                text_bounds_.y());
   }
 }
 
 int OmniboxResultView::GetTextHeight() const {
   return std::max(normal_font_.GetHeight(), bold_font_.GetHeight());
 }
 
 // static
+void OmniboxResultView::CommonInitColors(const ui::NativeTheme* theme,
+                                         SkColor colors[][NUM_KINDS]) {
+  colors[HOVERED][BACKGROUND] =
+      color_utils::AlphaBlend(colors[SELECTED][BACKGROUND],
+                              colors[NORMAL][BACKGROUND], 64);
+  colors[HOVERED][TEXT] = colors[NORMAL][TEXT];
+#if defined(USE_AURA)
+  const bool is_aura = theme == ui::NativeThemeAura::instance();
+#else
+  const bool is_aura = false;
+#endif
+  for (int i = 0; i < NUM_STATES; ++i) {
+    if (is_aura) {
+      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]);
+    }
+
+    // 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);
+  }
+}
+
+// static
 bool OmniboxResultView::SortRunsLogically(const RunData& lhs,
                                           const RunData& rhs) {
   return lhs.run_start < rhs.run_start;
 }
 
 // static
 bool OmniboxResultView::SortRunsVisually(const RunData& lhs,
                                          const RunData& rhs) {
   return lhs.visual_order < rhs.visual_order;
 }
@@ skipping 61 matching lines @@
   // Split the text into visual runs.  We do this first so that we don't need to
   // worry about whether our eliding might change the visual display in
   // unintended ways, e.g. by removing directional markings or by adding an
   // ellipsis that's not enclosed in appropriate markings.
   base::i18n::BiDiLineIterator bidi_line;
   if (!bidi_line.Open(text, base::i18n::IsRTL(), is_url))
     return x;
   const int num_runs = bidi_line.CountRuns();
   ScopedVector<gfx::RenderText> render_texts;
   Runs runs;
+  const ui::NativeTheme* theme = GetNativeTheme();
   for (int run = 0; run < num_runs; ++run) {
     int run_start_int = 0, run_length_int = 0;
     // The index we pass to GetVisualRun corresponds to the position of the run
     // in the displayed text. For example, the string "Google in HEBREW" (where
     // HEBREW is text in the Hebrew language) has two runs: "Google in " which
     // is an LTR run, and "HEBREW" which is an RTL run. In an LTR context, the
     // run "Google in " has the index 0 (since it is the leftmost run
     // displayed). In an RTL context, the same run has the index 1 because it
     // is the rightmost run. This is why the order in which we traverse the
     // runs is different depending on the locale direction.
@@ skipping 23 matching lines @@
       current_run->classifications.push_back(ClassificationData());
       ClassificationData* current_data =
           &current_run->classifications.back();
       current_data->text = text.substr(text_start, text_end - text_start);
 
       // Calculate style-related data.
       const int style = classifications[i].style;
       const bool use_bold_font = !!(style & ACMatchClassification::MATCH);
       current_data->font = &(use_bold_font ? bold_font_ : normal_font_);
       const ResultViewState state = GetState();
-      if (style & ACMatchClassification::URL)
-        current_data->color = GetColor(state, URL);
-      else if (style & ACMatchClassification::DIM)
-        current_data->color = GetColor(state, DIMMED_TEXT);
-      else
-        current_data->color = GetColor(state, force_dim ? DIMMED_TEXT : TEXT);
+      if (style & ACMatchClassification::URL) {
+        current_data->color = GetColor(theme, state, URL);
+      } else if (style & ACMatchClassification::DIM) {
+        current_data->color = GetColor(theme, state, DIMMED_TEXT);
+      } else {
+        current_data->color = GetColor(theme, state,
+                                       force_dim ? DIMMED_TEXT : TEXT);
+      }
 
       render_texts.push_back(gfx::RenderText::CreateInstance());
       current_data->render_text = render_texts.back();
       current_data->render_text->SetFont(*current_data->font);
       current_data->render_text->SetText(current_data->text);
 
       gfx::StyleRange style_range;
       style_range.foreground = current_data->color;
       style_range.font_style = current_data->font->GetStyle();
       current_data->render_text->set_default_style(style_range);
@@ skipping 186 matching lines @@
       std::max(text_width, 0), text_height);
 }
 
 void OmniboxResultView::OnBoundsChanged(const gfx::Rect& previous_bounds) {
   animation_->SetSlideDuration(width() / 4);
 }
 
 void OmniboxResultView::OnPaint(gfx::Canvas* canvas) {
   const ResultViewState state = GetState();
   if (state != NORMAL)
-    canvas->DrawColor(GetColor(state, BACKGROUND));
+    canvas->DrawColor(GetColor(GetNativeTheme(), state, BACKGROUND));
 
   if (!match_.associated_keyword.get() ||
       keyword_icon_->x() > icon_bounds_.right()) {
     // Paint the icon.
     canvas->DrawImageInt(GetIcon(), GetMirroredXForRect(icon_bounds_),
                          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 OmniboxResultView::AnimationProgressed(const ui::Animation* animation) {
   Layout();
   SchedulePaint();
 }