Make views::Label and views::Link auto-color themselves to be readable over their background colo...

chrome/browser/search_engines/template_url_fetcher_unittest.cc

Index: chrome/browser/search_engines/template_url_fetcher_unittest.cc
===================================================================
--- chrome/browser/search_engines/template_url_fetcher_unittest.cc	(revision 104769)
+++ chrome/browser/search_engines/template_url_fetcher_unittest.cc	(working copy)
@@ -29,12 +29,10 @@
   virtual ~TemplateURLFetcherTestCallbacks();
 
   // TemplateURLFetcherCallbacks implementation.
-  virtual void ConfirmSetDefaultSearchProvider(
-      TemplateURL* template_url,
-      TemplateURLService* template_url_service);
-  virtual void ConfirmAddSearchProvider(
-      TemplateURL* template_url,
-      Profile* profile);
+  virtual void ConfirmSetDefaultSearchProvider(TemplateURL* template_url,
+                                               Profile* profile);

[sky, 2011/10/11 14:54:10]

Since you're touching this code, how about adding OVERRIDE

+  virtual void ConfirmAddSearchProvider(TemplateURL* template_url,
+                                        Profile* profile);
 
  private:
   TemplateURLFetcherTest* test_;
@@ -68,12 +66,10 @@
 
   // TemplateURLFetcherCallbacks implementation.  (Although not derived from
   // this class, these methods handle those calls for the test.)
-  virtual void ConfirmSetDefaultSearchProvider(
-      TemplateURL* template_url,
-      TemplateURLService* template_url_service);
-  virtual void ConfirmAddSearchProvider(
-      TemplateURL* template_url,
-      Profile* profile);
+  virtual void ConfirmSetDefaultSearchProvider(TemplateURL* template_url,
+                                               Profile* profile);
+  virtual void ConfirmAddSearchProvider(TemplateURL* template_url,
+                                        Profile* profile);
 
  protected:
   // Schedules the download of the url.
@@ -114,8 +110,8 @@
 
 void TemplateURLFetcherTestCallbacks::ConfirmSetDefaultSearchProvider(
     TemplateURL* template_url,
-    TemplateURLService* template_url_service) {
-  test_->ConfirmSetDefaultSearchProvider(template_url, template_url_service);
+    Profile* profile) {
+  test_->ConfirmSetDefaultSearchProvider(template_url, profile);
 }
 
 void TemplateURLFetcherTestCallbacks::ConfirmAddSearchProvider(
@@ -142,7 +138,7 @@
 
 void TemplateURLFetcherTest::ConfirmSetDefaultSearchProvider(
     TemplateURL* template_url,
-    TemplateURLService* template_url_service) {
+    Profile* profile) {
   last_callback_template_url_.reset(template_url);
   set_default_called_++;
 }