Don't set a default action for ToolbarActionsBarBubbles.

Don't set a default action for ToolbarActionsBarBubbles.

This isn't totally correct; some of these bubbles should have defaults
and some should not. But this was the behavior previous to dc0d234f7c87
so the more conservative approach is to revert to the old behavior and
later change/improve individual bubbles that should have defaults by
adding them.

This patch is designed to be easy/safe to merge to M52.

BUG=621122
Committed: https://crrev.com/0013e8398ca2cb80185a08bcf14358bc4310b27a
Cr-Commit-Position: refs/heads/master@{#400503}

[Evan Stade, 2016-06-17 17:11:27]

estade@chromium.org changed reviewers:
+ pkasting@chromium.org

[Evan Stade, 2016-06-17 17:11:35]

The CQ bit was checked by estade@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-06-17 17:11:52]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2076993002/1

[commit-bot: I haz the power, 2016-06-17 17:50:15]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-06-17 17:50:16]

Dry run: This issue passed the CQ dry run.

[Peter Kasting, 2016-06-17 19:14:41]

LGTM

https://codereview.chromium.org/2076993002/diff/1/chrome/browser/ui/views/too...
File chrome/browser/ui/views/toolbar/toolbar_actions_bar_bubble_views.cc
(right):

https://codereview.chromium.org/2076993002/diff/1/chrome/browser/ui/views/too...
chrome/browser/ui/views/toolbar/toolbar_actions_bar_bubble_views.cc:106: return
ui::DIALOG_BUTTON_NONE;
Nit: May want a relevant TODO here

[Evan Stade, 2016-06-17 20:22:12]

The CQ bit was checked by estade@chromium.org

[Evan Stade, 2016-06-17 20:22:12]

The patchset sent to the CQ was uploaded after l-g-t-m from
pkasting@chromium.org
Link to the patchset: https://codereview.chromium.org/2076993002/#ps20001
(title: "add todo")

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

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

[commit-bot: I haz the power, 2016-06-17 21:07:16]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-17 21:09:35]

Description was changed from

==========
Don't set a default action for ToolbarActionsBarBubbles.

This isn't totally correct; some of these bubbles should have defaults
and some should not. But this was the behavior previous to dc0d234f7c87
so the more conservative approach is to revert to the old behavior and
later change/improve individual bubbles that should have defaults by
adding them.

This patch is designed to be easy/safe to merge to M52.

BUG=621122
==========

to

==========
Don't set a default action for ToolbarActionsBarBubbles.

This isn't totally correct; some of these bubbles should have defaults
and some should not. But this was the behavior previous to dc0d234f7c87
so the more conservative approach is to revert to the old behavior and
later change/improve individual bubbles that should have defaults by
adding them.

This patch is designed to be easy/safe to merge to M52.

BUG=621122

Committed: https://crrev.com/0013e8398ca2cb80185a08bcf14358bc4310b27a
Cr-Commit-Position: refs/heads/master@{#400503}
==========

[commit-bot: I haz the power, 2016-06-17 21:09:37]

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