Disable new bookmark apps on Mac.

Disable new bookmark apps on Mac.

There is insufficient time for this feature to be enabled for M46.
This CL disables it again, with the intention of enabling it for M47.

BUG=517682
Committed: https://crrev.com/96cb8287949019a962b09b8c3ff15358646f0c8d
Cr-Commit-Position: refs/heads/master@{#343853}

[dominickn, 2015-08-18 06:22:13]

dominickn@chromium.org changed reviewers:
+ benwells@chromium.org

[dominickn, 2015-08-18 06:22:13]

PTAL, thanks!

[benwells, 2015-08-18 06:32:07]

https://codereview.chromium.org/1299823002/diff/1/extensions/browser/api/mana...
File extensions/browser/api/management/management_api.cc (left):

https://codereview.chromium.org/1299823002/diff/1/extensions/browser/api/mana...
extensions/browser/api/management/management_api.cc:88: if
(!delegate->IsNewBookmarkAppsEnabled()) {
Why is this code being deleted? It only affects non-chromeos when the flag is
disabled. I believe it should stay as it was.

[dominickn, 2015-08-18 06:50:58]

Patchset #2 (id:20001) has been deleted

[dominickn, 2015-08-18 06:53:00]

Thanks!

https://codereview.chromium.org/1299823002/diff/1/extensions/browser/api/mana...
File extensions/browser/api/management/management_api.cc (left):

https://codereview.chromium.org/1299823002/diff/1/extensions/browser/api/mana...
extensions/browser/api/management/management_api.cc:88: if
(!delegate->IsNewBookmarkAppsEnabled()) {
On 2015/08/18 06:32:07, benwells wrote:
> Why is this code being deleted? It only affects non-chromeos when the flag is
> disabled. I believe it should stay as it was.

Done, had the mistaken thought that pinned and fullscreen functionality had been
removed.

[benwells, 2015-08-18 06:56:29]

lgtm

[dominickn, 2015-08-18 07:51:36]

The CQ bit was checked by dominickn@chromium.org

[commit-bot: I haz the power, 2015-08-18 07:51:55]

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

[commit-bot: I haz the power, 2015-08-18 08:00:51]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2015-08-18 08:01:31]

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