[InfoBar] Always close native infobars from C++. Part II

chrome/browser/ui/android/infobars/translate_infobar.cc

 // Copyright 2013 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.
 
 #include "chrome/browser/ui/android/infobars/translate_infobar.h"
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_array.h"
 #include "base/android/jni_helper.h"
 #include "chrome/browser/translate/translate_infobar_delegate.h"
@@ skipping 41 matching lines @@
       ShouldDisplayNeverTranslateInfoBarOnCancel(),
       java_languages.obj());
 }
 
 void TranslateInfoBar::ProcessButton(
     int action, const std::string& action_value) {
   if (!owner())
      return; // We're closing; don't call anything, it might access the owner.
 
   if (action == InfoBarAndroid::ACTION_TRANSLATE) {
-    // Do not close the infobar upon translate since it will be replaced by a
-    // different one which will close this current infobar.
     delegate_->Translate();
     return;
   }
 
   if (action == InfoBarAndroid::ACTION_CANCEL)
     delegate_->TranslationDeclined();
   else if (action == InfoBarAndroid::ACTION_TRANSLATE_SHOW_ORIGINAL)
     delegate_->RevertTranslation();
   else
     DCHECK_EQ(InfoBarAndroid::ACTION_NONE, action);
@@ skipping 54 matching lines @@
           TranslateInfoBarDelegate::BEFORE_TRANSLATE) &&
       (delegate_->ShouldShowNeverTranslateShortcut());
 }
 
 
 // Native JNI methods ---------------------------------------------------------
 
 bool RegisterTranslateInfoBarDelegate(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }