Simplify the translate infobar code some.

chrome/browser/ui/cocoa/infobars/translate_infobar_unittest.mm

Index: chrome/browser/ui/cocoa/infobars/translate_infobar_unittest.mm
===================================================================
--- chrome/browser/ui/cocoa/infobars/translate_infobar_unittest.mm	(revision 157463)
+++ chrome/browser/ui/cocoa/infobars/translate_infobar_unittest.mm	(working copy)
@@ -54,27 +54,16 @@
 
   }
 
-  virtual string16 GetDisplayNameForLocale(const std::string& language_code) {
-    return ASCIIToUTF16("Foo");
-  }
-
-  virtual bool IsLanguageBlacklisted() {
-    return false;
-  }
-
-  virtual bool IsSiteBlacklisted() {
-    return false;
-  }
-
-  virtual bool ShouldAlwaysTranslate() {
-    return false;
-  }
-
   MOCK_METHOD0(Translate, void());
   MOCK_METHOD0(RevertTranslation, void());
+
   MOCK_METHOD0(TranslationDeclined, void());
+
+  virtual bool IsLanguageBlacklisted() { return false; }

[sail, 2012/09/19 04:44:06]

should these have OVERRIDE on them?

[Peter Kasting, 2012/09/19 04:53:32]

Yes.  Good catch.

   MOCK_METHOD0(ToggleLanguageBlacklist, void());
+  virtual bool IsSiteBlacklisted() { return false; }
   MOCK_METHOD0(ToggleSiteBlacklist, void());
+  virtual bool ShouldAlwaysTranslate() { return false; }
   MOCK_METHOD0(ToggleAlwaysTranslate, void());
 };