Move TranslateInfoBarDelegate and OptionsMenuModel to the Translate component.

components/translate/core/browser/translate_client.h

Index: components/translate/core/browser/translate_client.h
diff --git a/components/translate/core/browser/translate_client.h b/components/translate/core/browser/translate_client.h
index 6a2c3775ea5523aed040ca3c32f6f6c5344269e2..57a91606a5ec0a8d0b6a5ea29c216790fbb3c79c 100644
--- a/components/translate/core/browser/translate_client.h
+++ b/components/translate/core/browser/translate_client.h
@@ -8,6 +8,7 @@
 #include <string>
 
 #include "base/memory/ref_counted.h"
+#include "base/memory/scoped_ptr.h"
 #include "components/translate/core/browser/translate_prefs.h"
 #include "components/translate/core/browser/translate_step.h"
 #include "components/translate/core/common/translate_errors.h"
@@ -16,6 +17,11 @@ class GURL;
 class PrefService;
 class TranslateAcceptLanguages;
 class TranslateDriver;
+class TranslateInfoBarDelegate;
+
+namespace infobars {
+class InfoBar;
+}
 
 // A client interface that needs to be supplied to TranslateManager by the
 // embedder.
@@ -36,6 +42,13 @@ class TranslateClient {
   // Returns the associated TranslateAcceptLanguages.
   virtual TranslateAcceptLanguages* GetTranslateAcceptLanguages() = 0;
 
+  // Returns the resource ID of the icon to be shown for the Translate infobars.
+  virtual int GetInfobarIconID() const = 0;
+
+  // Returns a translate infobar that owns |delegate|.
+  virtual scoped_ptr<infobars::InfoBar> CreateInfoBar(
+      scoped_ptr<TranslateInfoBarDelegate> delegate) = 0;

[Peter Kasting, 2014/05/28 19:25:55]

Does this really need to be virtual?  It seems like maybe it could be static, if
we're allowed to declare a static function here that we define in chrome/ (maybe
we can't).

If this can't be static, it seems like at least it could be const?

[droger, 2014/06/02 11:30:15]

I made it const.

The component should be standalone, and cannot rely on chrome/ for its
implementation.
It would not link.
Maybe we could work around these issues to make it link -- I'm not sure -- , but
I don't think it is something we want.

+
   // Called when the embedder should present UI to the user corresponding to the
   // user's current |step|.
   virtual void ShowTranslateUI(translate::TranslateStep step,