Infobar system refactor.

chrome/browser/ui/android/infobars/auto_login_infobar_delegate_android.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/auto_login_infobar_delegate_android.h"
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_helper.h"
 #include "base/android/jni_string.h"
 #include "base/strings/utf_string_conversions.h"
+#include "chrome/browser/infobars/infobar.h"
 #include "chrome/browser/infobars/simple_alert_infobar_delegate.h"
 #include "chrome/browser/ui/auto_login_infobar_delegate.h"
 #include "content/public/browser/web_contents.h"
 #include "grit/generated_resources.h"
 #include "grit/theme_resources.h"
 #include "jni/AutoLoginDelegate_jni.h"
 #include "ui/base/l10n/l10n_util.h"
 
 using base::android::ConvertUTF8ToJavaString;
 using base::android::ScopedJavaLocalRef;
 
+
 AutoLoginInfoBarDelegateAndroid::AutoLoginInfoBarDelegateAndroid(
-    InfoBarService* owner,
     const Params& params)
-    : AutoLoginInfoBarDelegate(owner, params),
+    : AutoLoginInfoBarDelegate(params),
       params_(params) {
 }
 
 AutoLoginInfoBarDelegateAndroid::~AutoLoginInfoBarDelegateAndroid() {
 }
 
 bool AutoLoginInfoBarDelegateAndroid::AttachAccount(
     JavaObjectWeakGlobalRef weak_java_auto_login_delegate) {
   weak_java_auto_login_delegate_ = weak_java_auto_login_delegate;
   JNIEnv* env = base::android::AttachCurrentThread();
@@ skipping 18 matching lines @@
 string16 AutoLoginInfoBarDelegateAndroid::GetMessageText() const {
   return l10n_util::GetStringFUTF16(IDS_AUTOLOGIN_INFOBAR_MESSAGE,
                                     UTF8ToUTF16(user_));
 }
 
 bool AutoLoginInfoBarDelegateAndroid::Accept() {
   JNIEnv* env = base::android::AttachCurrentThread();
   ScopedJavaLocalRef<jobject> delegate =
       weak_java_auto_login_delegate_.get(env);
   DCHECK(delegate.obj());
-
   Java_AutoLoginDelegate_logIn(env, delegate.obj(),
                                reinterpret_cast<jint>(this));
-
   // Do not close the infobar on accept, it will be closed as part
   // of the log in callback.
   return false;
 }
 
 bool AutoLoginInfoBarDelegateAndroid::Cancel() {
   JNIEnv* env = base::android::AttachCurrentThread();
   ScopedJavaLocalRef<jobject> delegate =
       weak_java_auto_login_delegate_.get(env);
   DCHECK(delegate.obj());
   Java_AutoLoginDelegate_cancelLogIn(env, delegate.obj(),
                                      reinterpret_cast<jint>(this));
   return true;
 }
 
 void AutoLoginInfoBarDelegateAndroid::LoginSuccess(JNIEnv* env,
                                                    jobject obj,
                                                    jstring result) {
-  if (owner()) {
-    content::WebContents* web_contents = owner()->web_contents();
-    if (web_contents) {
-      web_contents->Stop();
-      web_contents->OpenURL(content::OpenURLParams(
-          GURL(base::android::ConvertJavaStringToUTF8(env, result)),
-          content::Referrer(), CURRENT_TAB,
-          content::PAGE_TRANSITION_AUTO_BOOKMARK, false));
-    }
-    owner()->RemoveInfoBar(this);
+  // TODO(pkasting): Can this and LoginFailed() really be called when the
+  // infobar is unowned?
+  if (infobar()->owner()) {
+    content::WebContents* contents = web_contents();
+    contents->Stop();
+    // It's not obvious whether calling OpenURL() will auto-close us or not.  If
+    // it did, we wouldn't want to try to RemoveSelf() afterwards.  So for
+    // safety, we always call RemoveSelf() directly, and then navigate
+    // afterwards.
+    infobar()->RemoveSelf();
+    // WARNING: |this| may be deleted at this point!  Do not access any members!
+    contents->OpenURL(content::OpenURLParams(
+        GURL(base::android::ConvertJavaStringToUTF8(env, result)),
+        content::Referrer(), CURRENT_TAB,
+        content::PAGE_TRANSITION_LINK, false));
   }
 }
 
 void AutoLoginInfoBarDelegateAndroid::LoginFailed(JNIEnv* env, jobject obj) {
   ScopedJavaLocalRef<jobject> delegate =
       weak_java_auto_login_delegate_.get(env);
   DCHECK(delegate.obj());
-  if (owner()) {
+  if (infobar()->owner()) {
     SimpleAlertInfoBarDelegate::Create(
-        owner(), IDR_INFOBAR_WARNING,
+        infobar()->owner(), IDR_INFOBAR_WARNING,
         l10n_util::GetStringUTF16(IDS_AUTO_LOGIN_FAILED), false);
-    owner()->RemoveInfoBar(this);
+    infobar()->RemoveSelf();
   }
 }
 
 void AutoLoginInfoBarDelegateAndroid::LoginDismiss(JNIEnv* env, jobject obj) {
-  if (owner())
-    owner()->RemoveInfoBar(this);
+  infobar()->RemoveSelf();
 }
 
 bool AutoLoginInfoBarDelegateAndroid::Register(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }