Change infobar creation to use a public static Create() method on the infobar delegate classes.

chrome/browser/translate/translate_infobar_delegate.h

 // Copyright (c) 2011 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.
 
 #ifndef CHROME_BROWSER_TRANSLATE_TRANSLATE_INFOBAR_DELEGATE_H_
 #define CHROME_BROWSER_TRANSLATE_TRANSLATE_INFOBAR_DELEGATE_H_
 
 #include <string>
 #include <utility>
 #include <vector>
@@ skipping 19 matching lines @@
 
   // The types of background color animations.
   enum BackgroundAnimationType {
     NONE,
     NORMAL_TO_ERROR,
     ERROR_TO_NORMAL
   };
 
   static const size_t kNoIndex;
 
-  // Factory method to create a non-error translate infobar. |original_language|
-  // and |target_language| must be ASCII language codes (e.g. "en", "fr", etc.)
-  // for languages the TranslateManager supports translating. The lone exception
-  // is when the user initiates translation from the context menu, in which case
-  // it's legal to call this with |type| == TRANSLATING and
-  // |originalLanguage| == kUnknownLanguageCode.
-  static TranslateInfoBarDelegate* CreateDelegate(
-      Type infobar_type,
-      InfoBarService* infobar_service,
-      PrefService* prefs,
-      const std::string& original_language,
-      const std::string& target_language);
+  virtual ~TranslateInfoBarDelegate();
 
-  // Factory method to create an error translate infobar.
-  static TranslateInfoBarDelegate* CreateErrorDelegate(
-      TranslateErrors::Type error_type,
-      InfoBarService* infobar_service,
-      PrefService* prefs,
-      const std::string& original_language,
-      const std::string& target_language);
-
-  virtual ~TranslateInfoBarDelegate();
+  // Factory method to create a translate infobar.  |error_type| must be
+  // specified iff |infobar_type| == TRANSLATION_ERROR.  For other infobar
+  // types, |original_language| and |target_language| must be ASCII language
+  // codes (e.g. "en", "fr", etc.) for languages the TranslateManager supports
+  // translating.  The lone exception is when the user initiates translation
+  // from the context menu, in which case it's legal to call this with
+  // |infobar_type| == TRANSLATING and
+  // |original_language| == kUnknownLanguageCode.
+  //
+  // If |replace_existing_infobar| is true, the infobar is created and added to
+  // |infobar_service|, replacing any other translate infobar already present
+  // there.  Otherwise, the infobar will only be added if there is no other
+  // translate infobar already present.
+  static void Create(InfoBarService* infobar_service,
+                     bool replace_existing_infobar,
+                     Type infobar_type,
+                     TranslateErrors::Type error_type,
+                     PrefService* prefs,
+                     const std::string& original_language,
+                     const std::string& target_language);
 
   // Returns the number of languages supported.
   size_t num_languages() const { return languages_.size(); }
 
   // Returns the ISO code for the language at |index|.
   std::string language_code_at(size_t index) const {
     DCHECK_LT(index, num_languages());
     return languages_[index].first;
   }
 
@@ skipping 144 matching lines @@
   // The error that occurred when trying to translate (NONE if no error).
   TranslateErrors::Type error_type_;
 
   // The translation related preferences.
   TranslatePrefs prefs_;
 
   DISALLOW_COPY_AND_ASSIGN(TranslateInfoBarDelegate);
 };
 
 #endif  // CHROME_BROWSER_TRANSLATE_TRANSLATE_INFOBAR_DELEGATE_H_