[Mac] Fix a crash in handling keyboard shortcuts for empty bookmark folder menus.

[Mac] Fix a crash in handling keyboard shortcuts for empty bookmark folder menus.

Empty bookmark bar folder menus have an "(empty)" button that is not associated
with an actual model node. This leads to a DCHECK in the BookmarkBarController,
but crashes in release mode.

BUG=690424
Review-Url: https://codereview.chromium.org/2752433002
Cr-Commit-Position: refs/heads/master@{#456460}
Committed: https://chromium.googlesource.com/chromium/src/+/8da61374a11432c3b6c3acf43d50d562439dba02

[Robert Sesek, 2017-03-13 18:16:09]

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

[commit-bot: I haz the power, 2017-03-13 18:17:03]

Dry run: 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, 2017-03-13 18:46:30]

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

[commit-bot: I haz the power, 2017-03-13 18:46:31]

Dry run: This issue passed the CQ dry run.

[Robert Sesek, 2017-03-13 18:53:44]

rsesek@chromium.org changed reviewers:
+ lgrey@chromium.org

[lgrey, 2017-03-13 18:56:52]

LGTM

[Robert Sesek, 2017-03-13 19:24:51]

The CQ bit was checked by rsesek@chromium.org

[commit-bot: I haz the power, 2017-03-13 19:25:24]

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, 2017-03-13 19:25:25]

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

[commit-bot: I haz the power, 2017-03-13 19:25:26]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
Committers are members of the group "project-chromium-committers".
Note that this has nothing to do with OWNERS files.

[Robert Sesek, 2017-03-13 19:28:06]

rsesek@chromium.org changed reviewers:
+ ellyjones@chromium.org

[Elly Fong-Jones, 2017-03-13 20:14:08]

lgtm

[Robert Sesek, 2017-03-13 20:14:29]

The CQ bit was checked by rsesek@chromium.org

[commit-bot: I haz the power, 2017-03-13 20:15:37]

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, 2017-03-13 20:22:15]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1489436069381880, "parent_rev":
"9434e88d23f40824a07de46780aa7199c409a888", "commit_rev":
"8da61374a11432c3b6c3acf43d50d562439dba02"}

[commit-bot: I haz the power, 2017-03-13 20:22:58]

Description was changed from

==========
[Mac] Fix a crash in handling keyboard shortcuts for empty bookmark folder
menus.

Empty bookmark bar folder menus have an "(empty)" button that is not associated
with an actual model node. This leads to a DCHECK in the BookmarkBarController,
but crashes in release mode.

BUG=690424
==========

to

==========
[Mac] Fix a crash in handling keyboard shortcuts for empty bookmark folder
menus.

Empty bookmark bar folder menus have an "(empty)" button that is not associated
with an actual model node. This leads to a DCHECK in the BookmarkBarController,
but crashes in release mode.

BUG=690424

Review-Url: https://codereview.chromium.org/2752433002
Cr-Commit-Position: refs/heads/master@{#456460}
Committed:
https://chromium.googlesource.com/chromium/src/+/8da61374a11432c3b6c3acf43d50...
==========

[commit-bot: I haz the power, 2017-03-13 20:22:59]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/8da61374a11432c3b6c3acf43d50...