Clean up NativeTheme (particularly CommonTheme).

chrome/browser/ui/views/profiles/profile_chooser_view.cc

 // Copyright 2014 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/ui/views/profiles/profile_chooser_view.h"
 
 #include "base/macros.h"
 #include "base/metrics/user_metrics.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/browser_process.h"
@@ skipping 202 matching lines @@
     SetImage(STATE_NORMAL, icon);
   }
 
   ~BackgroundColorHoverButton() override {}
 
  private:
   // views::LabelButton:
   void OnPaint(gfx::Canvas* canvas) override {
     if ((state() == STATE_PRESSED) ||
         (state() == STATE_HOVERED)) {
-      canvas->DrawColor(GetNativeTheme()->GetSystemColor(
-          ui::NativeTheme::kColorId_ButtonHoverBackgroundColor));
+      canvas->DrawColor(SkColorSetRGB(0xEA, 0xEA, 0xEA));

[sky, 2016/10/03 16:12:05]

Why is a hard coded color good here?

[Evan Stade, 2016/10/03 19:46:44]

a) we don't need it to vary depending on the native theme
b) I suspect it should be replaced by an ink drop highlight.

p.s. did you see my comments on ps1? I attempted to pre-explain a few things
there.

     }
     LabelButton::OnPaint(canvas);
   }
 
   DISALLOW_COPY_AND_ASSIGN(BackgroundColorHoverButton);
 };
 
 // SizedContainer -------------------------------------------------
 
 // A simple container view that takes an explicit preferred size.
@@ skipping 2070 matching lines @@
       IncognitoModePrefs::GetAvailability(browser_->profile()->GetPrefs()) !=
           IncognitoModePrefs::DISABLED;
   return incognito_available && !browser_->profile()->IsGuestSession();
 }
 
 void ProfileChooserView::PostActionPerformed(
     ProfileMetrics::ProfileDesktopMenu action_performed) {
   ProfileMetrics::LogProfileDesktopMenu(action_performed, gaia_service_type_);
   gaia_service_type_ = signin::GAIA_SERVICE_TYPE_NONE;
 }