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 38 matching lines @@
       weak_java_auto_login_delegate_.get(env);
   DCHECK(delegate.obj());
   Java_AutoLoginDelegate_cancelLogIn(env, delegate.obj(),
                                      reinterpret_cast<intptr_t>(this));
   return true;
 }
 
 void AutoLoginInfoBarDelegateAndroid::LoginSuccess(JNIEnv* env,
                                                    jobject obj,
                                                    jstring result) {
-  if (!owner())
+  if (!infobar()->owner())
      return;  // We're closing; don't call anything, it might access the owner.
 
   // TODO(miguelg): Test whether the Stop() and RemoveInfoBar() calls here are
   // necessary, or whether OpenURL() will do this for us.
-  content::WebContents* web_contents = owner()->web_contents();
-  web_contents->Stop();
-  owner()->RemoveInfoBar(this);
+  content::WebContents* contents = web_contents();
+  contents->Stop();
+  infobar()->RemoveSelf();
   // WARNING: |this| may be deleted at this point!  Do not access any members!
-  web_contents->OpenURL(content::OpenURLParams(
+  contents->OpenURL(content::OpenURLParams(
       GURL(base::android::ConvertJavaStringToUTF8(env, result)),
       content::Referrer(), CURRENT_TAB, content::PAGE_TRANSITION_AUTO_BOOKMARK,
       false));
 }
 
 void AutoLoginInfoBarDelegateAndroid::LoginFailed(JNIEnv* env, jobject obj) {
-  if (!owner())
+  if (!infobar()->owner())
      return;  // We're closing; don't call anything, it might access the owner.
 
   // TODO(miguelg): Using SimpleAlertInfoBarDelegate::Create() animates in a new
   // infobar while we animate the current one closed.  It would be better to use
   // ReplaceInfoBar().
   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);
 }