[Translate] Return before the new delegate is created if needed

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 40 matching lines @@
   // 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.
+  // translate infobar already present and if the infobar needs to be displayed.

[Peter Kasting, 2013/07/17 18:06:19]

Nit: Don't make this change; it's somewhat inaccurate in that this clause really
applies more broadly than where you have it, and it's vague and confusing
anyway.  People who want this level of detail can just read the function.

   static void Create(InfoBarService* infobar_service,
                      bool replace_existing_infobar,
                      Type infobar_type,
                      TranslateErrors::Type error_type,
                      PrefService* prefs,
                      const ShortcutConfiguration& shortcut_config,
                      const std::string& original_language,
                      const std::string& target_language);
 
   // Returns the number of languages supported.
@@ skipping 165 matching lines @@
 
   // The translation related preferences.
   TranslatePrefs prefs_;
 
   // Translation shortcut configuration
   ShortcutConfiguration shortcut_config_;
   DISALLOW_COPY_AND_ASSIGN(TranslateInfoBarDelegate);
 };
 
 #endif  // CHROME_BROWSER_TRANSLATE_TRANSLATE_INFOBAR_DELEGATE_H_