Fix a crash in the translate info bar.

chrome/browser/translate/translate_infobar_delegate.cc

Index: chrome/browser/translate/translate_infobar_delegate.cc
===================================================================
--- chrome/browser/translate/translate_infobar_delegate.cc	(revision 95783)
+++ chrome/browser/translate/translate_infobar_delegate.cc	(working copy)
@@ -35,10 +35,10 @@
   // context menu.
   DCHECK(type == TRANSLATING ||
       original_language != chrome::kUnknownLanguageCode);
-  if ((original_language != chrome::kUnknownLanguageCode &&
-          !TranslateManager::IsSupportedLanguage(original_language)) ||
-      !TranslateManager::IsSupportedLanguage(target_language))
-    return NULL;
+  // This must be validated by our callers.

[Peter Kasting, 2011/08/08 22:09:49]

Nit: How about this for comments and checks:

In header:

// 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.
... (declaration) ...;

In .cc file:

... (declaration) ... {
  DCHECK_NE(TRANSLATION_ERROR, type);
  DCHECK(TranslateManager::IsSupportedLanguage(original_language) ||
      ((type == TRANSLATING) &&
       (original_language == chrome::kUnknownLanguageCode)));
  DCHECK(TranslateManager::IsSupportedLanguage(target_language));
  ... (function body) ...

[MAD, 2011/08/09 14:19:41]

Done.

+  CHECK((original_language == chrome::kUnknownLanguageCode ||
+         TranslateManager::IsSupportedLanguage(original_language)) &&
+        TranslateManager::IsSupportedLanguage(target_language));
   TranslateInfoBarDelegate* delegate =
       new TranslateInfoBarDelegate(type, TranslateErrors::NONE, tab_contents,
                                    original_language, target_language);