Infobar system refactor.

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

Index: chrome/browser/ui/android/infobars/infobar_android.cc
===================================================================
--- chrome/browser/ui/android/infobars/infobar_android.cc	(revision 226624)
+++ chrome/browser/ui/android/infobars/infobar_android.cc	(working copy)
@@ -28,11 +28,8 @@
 
 // InfoBarAndroid -------------------------------------------------------------
 
-InfoBarAndroid::InfoBarAndroid(InfoBarService* owner, InfoBarDelegate* delegate)
-    : InfoBar(owner, delegate),
-      delegate_(delegate) {
-  DCHECK(delegate_);
-  DCHECK(delegate_->owner());
+InfoBarAndroid::InfoBarAndroid(scoped_ptr<InfoBarDelegate> delegate)
+    : InfoBar(delegate.Pass()) {
 }
 
 InfoBarAndroid::~InfoBarAndroid() {
@@ -60,19 +57,18 @@
                                      jobject obj,
                                      jint action,
                                      jstring action_value) {
-  DCHECK(delegate_);
   std::string value = base::android::ConvertJavaStringToUTF8(env, action_value);
   ProcessButton(action, value);
 }
 
 void InfoBarAndroid::OnCloseButtonClicked(JNIEnv* env, jobject obj) {
-  delegate_->InfoBarDismissed();
+  delegate()->InfoBarDismissed();
+  CloseInfoBar();  // TODO(pkasting): Is this correct?

[Miguel Garcia, 2013/10/07 15:19:00]

This method is being removed in https://codereview.chromium.org/26176007/

Right now, closing the java infobar here is not needed. When the user clicks the
X button in java we first close the java bar, then notify C++ via this method.
This was a mistake really and but it was done like that in an attempt to share
the java implementation by java only infobars and C++ infobars. We are trying to
get rid of the former and have all infobars have a C++ equivalent.

As part of the cleanup we are making it very explicit what is the logic for java
only infobars and C++ infobars (so we can transition the former in a second
step)  https://codereview.chromium.org/26263002/ is another step in that
direction. There I add a RemoveSelf after this so the infobar can be dismissed
properly. 

[Peter Kasting, 2013/10/07 20:19:24]

Sounds like I should just wait for that to go in and then take another look
here.

 }
 
 void InfoBarAndroid::OnInfoBarClosed(JNIEnv* env, jobject obj) {
   java_info_bar_.Reset();  // So we don't notify Java.
-  if (owner())
-    RemoveSelf();
+  RemoveSelf();
 }
 
 void InfoBarAndroid::CloseJavaInfoBar() {
@@ -83,14 +79,12 @@
 }
 
 int InfoBarAndroid::GetEnumeratedIconId() {
-  DCHECK(delegate_);
-  return ResourceMapper::MapFromChromiumId(delegate_->GetIconID());
+  return ResourceMapper::MapFromChromiumId(delegate()->GetIconID());
 }
 
 void InfoBarAndroid::CloseInfoBar() {
   CloseJavaInfoBar();
-  if (owner())
-    RemoveSelf();
+  RemoveSelf();
 }