MacViews: Fix BrowserViewTest.RepeatedAccelerators.

MacViews: Fix BrowserViewTest.RepeatedAccelerators.

crbug.com/620315 unified Cocoa and Views accelerators, removing almost all
Ctrl-accelerators from MacViews in the process. We must use Cmd+L in order to
focus Omnibox on Mac.

BUG=620315
Committed: https://crrev.com/23052ba5652db0942cb1b014314be33a5d372171
Cr-Commit-Position: refs/heads/master@{#434574}

[themblsha, 2016-11-25 16:51:22]

mblsha@yandex-team.ru changed reviewers:
+ pkasting@chromium.org

[themblsha, 2016-11-25 17:08:16]

Description was changed from

==========
MacViews: Fix BrowserViewTest.RepeatedAccelerators.

crbug.com/620315 unified Cocoa and Views accelerators, removing almost all
Ctrl-accelerators from MacViews in the process. We must use Cmd+L in order to
focus Omnibox on Mac.
==========

to

==========
MacViews: Fix BrowserViewTest.RepeatedAccelerators.

crbug.com/620315 unified Cocoa and Views accelerators, removing almost all
Ctrl-accelerators from MacViews in the process. We must use Cmd+L in order to
focus Omnibox on Mac.

BUG=620315
==========

[Peter Kasting, 2016-11-26 06:05:45]

LGTM, but it seems like the "if Mac COMMAND, else CONTROL" is a theme.  Consider
finding a good place to centrally define this and using it in all applicable
files.

[themblsha, 2016-11-26 14:43:15]

On 2016/11/26 06:05:45, Peter Kasting wrote:
> LGTM, but it seems like the "if Mac COMMAND, else CONTROL" is a theme. 
Consider
> finding a good place to centrally define this and using it in all applicable
> files.

Ok, I'll add it to my future pile of CLs :-)

[themblsha, 2016-11-26 14:43:18]

The CQ bit was checked by mblsha@yandex-team.ru

[commit-bot: I haz the power, 2016-11-26 14:43:27]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-26 17:05:01]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1480171398878460, "parent_rev":
"6f86086af11c888f0514cfb773ab3ee81adff0ec", "commit_rev":
"ad954f3618350878742f81cbe53bf10f91c94f10"}

[commit-bot: I haz the power, 2016-11-26 17:05:31]

Description was changed from

==========
MacViews: Fix BrowserViewTest.RepeatedAccelerators.

crbug.com/620315 unified Cocoa and Views accelerators, removing almost all
Ctrl-accelerators from MacViews in the process. We must use Cmd+L in order to
focus Omnibox on Mac.

BUG=620315
==========

to

==========
MacViews: Fix BrowserViewTest.RepeatedAccelerators.

crbug.com/620315 unified Cocoa and Views accelerators, removing almost all
Ctrl-accelerators from MacViews in the process. We must use Cmd+L in order to
focus Omnibox on Mac.

BUG=620315
==========

[commit-bot: I haz the power, 2016-11-26 17:05:32]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-26 17:07:17]

Description was changed from

==========
MacViews: Fix BrowserViewTest.RepeatedAccelerators.

crbug.com/620315 unified Cocoa and Views accelerators, removing almost all
Ctrl-accelerators from MacViews in the process. We must use Cmd+L in order to
focus Omnibox on Mac.

BUG=620315
==========

to

==========
MacViews: Fix BrowserViewTest.RepeatedAccelerators.

crbug.com/620315 unified Cocoa and Views accelerators, removing almost all
Ctrl-accelerators from MacViews in the process. We must use Cmd+L in order to
focus Omnibox on Mac.

BUG=620315

Committed: https://crrev.com/23052ba5652db0942cb1b014314be33a5d372171
Cr-Commit-Position: refs/heads/master@{#434574}
==========

[commit-bot: I haz the power, 2016-11-26 17:07:18]

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