[Extensions UI] Fix a crash when clicking on a bubble's learn more

[Extensions UI] Fix a crash when clicking on a bubble's learn more

Clicking on a bubble's learn more link causes a tab to open, which in
turn causes the bubble to be cleaned up (as part of losing focus to the
new tab). Fix the order so that we clean up before opening the new tab
and add a regression test. Also add end-to-end tests for clicking on the
action and dismiss buttons.

BUG=622117
TBR=avi@chromium.org (small changes to cocoa browsertest)

Committed: https://crrev.com/bb826f1bd289d9147c4e0b630831b0ab6ae81afd
Cr-Commit-Position: refs/heads/master@{#401412}

[Devlin, 2016-06-22 01:37:46]

Patchset #2 (id:20001) has been deleted

[Devlin, 2016-06-22 01:37:51]

Patchset #1 (id:1) has been deleted

[Devlin, 2016-06-22 01:38:17]

rdevlin.cronin@chromium.org changed reviewers:
+ finnur@chromium.org

[Devlin, 2016-06-22 01:38:17]

Finnur, mind taking a look?  One of those awesome 1-line fix, 150+ lines of
tests :)

[Finnur, 2016-06-22 10:36:47]

LGTM

[Devlin, 2016-06-22 20:08:42]

Description was changed from

==========
[Extensions UI] Fix a crash when clicking on a bubble's learn more

Clicking on a bubble's learn more link causes a tab to open, which in
turn causes the bubble to be cleaned up (as part of losing focus to the
new tab). Fix the order so that we clean up before opening the new tab
and add a regression test. Also add end-to-end tests for clicking on the
action and dismiss buttons.

BUG=622117
==========

to

==========
[Extensions UI] Fix a crash when clicking on a bubble's learn more

Clicking on a bubble's learn more link causes a tab to open, which in
turn causes the bubble to be cleaned up (as part of losing focus to the
new tab). Fix the order so that we clean up before opening the new tab
and add a regression test. Also add end-to-end tests for clicking on the
action and dismiss buttons.

BUG=622117
TBR=avi@chromium.org (small changes to cocoa browsertest)
==========

[Devlin, 2016-06-22 20:08:46]

The CQ bit was checked by rdevlin.cronin@chromium.org

[Devlin, 2016-06-22 20:08:46]

The patchset sent to the CQ was uploaded after l-g-t-m from finnur@chromium.org
Link to the patchset: https://codereview.chromium.org/2086193002/#ps60001
(title: "Fix Views clicking")

[commit-bot: I haz the power, 2016-06-22 20:09:20]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2086193002/60001

[commit-bot: I haz the power, 2016-06-22 21:30:25]

Description was changed from

==========
[Extensions UI] Fix a crash when clicking on a bubble's learn more

Clicking on a bubble's learn more link causes a tab to open, which in
turn causes the bubble to be cleaned up (as part of losing focus to the
new tab). Fix the order so that we clean up before opening the new tab
and add a regression test. Also add end-to-end tests for clicking on the
action and dismiss buttons.

BUG=622117
TBR=avi@chromium.org (small changes to cocoa browsertest)
==========

to

==========
[Extensions UI] Fix a crash when clicking on a bubble's learn more

Clicking on a bubble's learn more link causes a tab to open, which in
turn causes the bubble to be cleaned up (as part of losing focus to the
new tab). Fix the order so that we clean up before opening the new tab
and add a regression test. Also add end-to-end tests for clicking on the
action and dismiss buttons.

BUG=622117
TBR=avi@chromium.org (small changes to cocoa browsertest)
==========

[commit-bot: I haz the power, 2016-06-22 21:30:28]

Committed patchset #2 (id:60001)

[commit-bot: I haz the power, 2016-06-22 21:34:03]

Description was changed from

==========
[Extensions UI] Fix a crash when clicking on a bubble's learn more

Clicking on a bubble's learn more link causes a tab to open, which in
turn causes the bubble to be cleaned up (as part of losing focus to the
new tab). Fix the order so that we clean up before opening the new tab
and add a regression test. Also add end-to-end tests for clicking on the
action and dismiss buttons.

BUG=622117
TBR=avi@chromium.org (small changes to cocoa browsertest)
==========

to

==========
[Extensions UI] Fix a crash when clicking on a bubble's learn more

Clicking on a bubble's learn more link causes a tab to open, which in
turn causes the bubble to be cleaned up (as part of losing focus to the
new tab). Fix the order so that we clean up before opening the new tab
and add a regression test. Also add end-to-end tests for clicking on the
action and dismiss buttons.

BUG=622117
TBR=avi@chromium.org (small changes to cocoa browsertest)

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

[commit-bot: I haz the power, 2016-06-22 21:34:04]

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