Don't try to use ManagePasswordsBubbleModel if the password bubble was closed.

Don't try to use ManagePasswordsBubbleModel if the password bubble was closed.

The bubble may be closed by some navigation in the content area. However, if the user pressed a button before closing and released after then the button action is still sent. Moreover, all the objects like NSWindow, NSWindowController, NSViewController are still alive. The UI code should not ping ManagePasswordsBubbleModel which is actually destroyed.

BUG=579726
Committed: https://crrev.com/a3aaa89baf57b6cbbf4905ccbaccc4aedb08b4e7
Cr-Commit-Position: refs/heads/master@{#371513}

[vasilii, 2016-01-26 14:05:42]

Description was changed from

==========
Don't try to use ManagePasswordsBubbleModel if the password bubble was closed.

BUG=579726
==========

to

==========
Don't try to use ManagePasswordsBubbleModel if the password bubble was closed.

The bubble may be closed by some navigation in the content area. However, if the
user pressed a button before closing and released after then the button action
is still sent. Moreover, all the objects like NSWindow, NSWindowController,
NSViewController are still alive. The UI code should not ping
ManagePasswordsBubbleModel which is actually destroyed.

BUG=579726
==========

[vasilii, 2016-01-26 14:05:42]

vasilii@chromium.org changed reviewers:
+ vabr@chromium.org

[vasilii, 2016-01-26 14:06:39]

Hi Vaclav,

please review the CL.

[vabr (Chromium), 2016-01-26 14:35:26]

lgtm

[vasilii, 2016-01-26 15:01:54]

The CQ bit was checked by vasilii@chromium.org

[commit-bot: I haz the power, 2016-01-26 15:02:25]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1637043002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1637043002/1

[commit-bot: I haz the power, 2016-01-26 15:07:09]

Description was changed from

==========
Don't try to use ManagePasswordsBubbleModel if the password bubble was closed.

The bubble may be closed by some navigation in the content area. However, if the
user pressed a button before closing and released after then the button action
is still sent. Moreover, all the objects like NSWindow, NSWindowController,
NSViewController are still alive. The UI code should not ping
ManagePasswordsBubbleModel which is actually destroyed.

BUG=579726
==========

to

==========
Don't try to use ManagePasswordsBubbleModel if the password bubble was closed.

The bubble may be closed by some navigation in the content area. However, if the
user pressed a button before closing and released after then the button action
is still sent. Moreover, all the objects like NSWindow, NSWindowController,
NSViewController are still alive. The UI code should not ping
ManagePasswordsBubbleModel which is actually destroyed.

BUG=579726
==========

[commit-bot: I haz the power, 2016-01-26 15:07:10]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-26 15:08:16]

Description was changed from

==========
Don't try to use ManagePasswordsBubbleModel if the password bubble was closed.

The bubble may be closed by some navigation in the content area. However, if the
user pressed a button before closing and released after then the button action
is still sent. Moreover, all the objects like NSWindow, NSWindowController,
NSViewController are still alive. The UI code should not ping
ManagePasswordsBubbleModel which is actually destroyed.

BUG=579726
==========

to

==========
Don't try to use ManagePasswordsBubbleModel if the password bubble was closed.

The bubble may be closed by some navigation in the content area. However, if the
user pressed a button before closing and released after then the button action
is still sent. Moreover, all the objects like NSWindow, NSWindowController,
NSViewController are still alive. The UI code should not ping
ManagePasswordsBubbleModel which is actually destroyed.

BUG=579726

Committed: https://crrev.com/a3aaa89baf57b6cbbf4905ccbaccc4aedb08b4e7
Cr-Commit-Position: refs/heads/master@{#371513}
==========

[commit-bot: I haz the power, 2016-01-26 15:08:16]

Patchset 1 (id:??) landed as
https://crrev.com/a3aaa89baf57b6cbbf4905ccbaccc4aedb08b4e7
Cr-Commit-Position: refs/heads/master@{#371513}