Prevent crash when trying to dismiss an already dismissed infobar.

Prevent crash when trying to dismiss an already dismissed infobar.

 The owner of an infobar delegate might be NULL if the infobar is
already in the process of being deleted. Checking for this before asking
it to dismiss in the auto-login infobar.

TEST=None
BUG=None


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=145896

[qsr, 2012-07-10 09:07:36]

Changing reviewer as sky is OOO

[Ben Goodger (Google), 2012-07-10 15:15:49]

lgtm

http://codereview.chromium.org/10701080/diff/1/chrome/browser/ui/auto_login_i...
File chrome/browser/ui/auto_login_info_bar_delegate.cc (right):

http://codereview.chromium.org/10701080/diff/1/chrome/browser/ui/auto_login_i...
chrome/browser/ui/auto_login_info_bar_delegate.cc:225: if (owner())
note here the circumstances when owner() can be NULL.

It is unfortunate that owner() can be NULL.

[qsr, 2012-07-10 15:23:56]

http://codereview.chromium.org/10701080/diff/1/chrome/browser/ui/auto_login_i...
File chrome/browser/ui/auto_login_info_bar_delegate.cc (right):

http://codereview.chromium.org/10701080/diff/1/chrome/browser/ui/auto_login_i...
chrome/browser/ui/auto_login_info_bar_delegate.cc:225: if (owner())
 Done, and yes I agree it is unfortunate, but this happens when the
InfobarTabHelper decides to dismiss us, because he forgets about us, but the
object needs to continue to live for the animations.

[commit-bot: I haz the power, 2012-07-10 15:24:05]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/qsr@chromium.org/10701080/6

[commit-bot: I haz the power, 2012-07-10 17:20:59]

Change committed as 145896