cocoa browser: remove non-material toolbar support

cocoa browser: remove non-material toolbar support

This is part 1 of removing non-material top chrome support.

BUG=649368
Committed: https://crrev.com/9cceb8fa96365b8f0678384d9385b555f81e1d9b
Cr-Commit-Position: refs/heads/master@{#420593}

[Elly Fong-Jones, 2016-09-22 18:36:29]

Description was changed from

==========
cocoa browser: remove non-material toolbar support

This is part 1 of removing non-material top chrome support.

BUG=649368
==========

to

==========
cocoa browser: remove non-material toolbar support

This is part 1 of removing non-material top chrome support.

BUG=649368
==========

[Elly Fong-Jones, 2016-09-22 18:36:29]

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

[Elly Fong-Jones, 2016-09-22 18:36:36]

avi: ptal? :)

[Avi (use Gerrit), 2016-09-22 19:11:18]

stampity stamp lgtm

[Elly Fong-Jones, 2016-09-23 10:49:49]

The CQ bit was checked by ellyjones@chromium.org

[commit-bot: I haz the power, 2016-09-23 10:49:59]

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-09-23 11:30:35]

Description was changed from

==========
cocoa browser: remove non-material toolbar support

This is part 1 of removing non-material top chrome support.

BUG=649368
==========

to

==========
cocoa browser: remove non-material toolbar support

This is part 1 of removing non-material top chrome support.

BUG=649368
==========

[commit-bot: I haz the power, 2016-09-23 11:30:36]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-23 11:33:18]

Description was changed from

==========
cocoa browser: remove non-material toolbar support

This is part 1 of removing non-material top chrome support.

BUG=649368
==========

to

==========
cocoa browser: remove non-material toolbar support

This is part 1 of removing non-material top chrome support.

BUG=649368

Committed: https://crrev.com/9cceb8fa96365b8f0678384d9385b555f81e1d9b
Cr-Commit-Position: refs/heads/master@{#420593}
==========

[commit-bot: I haz the power, 2016-09-23 11:33:19]

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

[Evan Stade, 2016-09-27 19:34:50]

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

[Evan Stade, 2016-09-27 19:34:52]

https://codereview.chromium.org/2361153002/diff/1/chrome/browser/ui/cocoa/too...
File chrome/browser/ui/cocoa/toolbar/toolbar_controller.mm (right):

https://codereview.chromium.org/2361153002/diff/1/chrome/browser/ui/cocoa/too...
chrome/browser/ui/cocoa/toolbar/toolbar_controller.mm:83: const CGFloat
kMaterialDesignLocationBarHeight = 28;
nit: remove references to MD

https://codereview.chromium.org/2361153002/diff/1/chrome/browser/ui/cocoa/too...
chrome/browser/ui/cocoa/toolbar/toolbar_controller.mm:335: // Replace the app
button from the nib with an AppToolbarButton instance for
can you remove AppToolbarButtonCell now too?

https://codereview.chromium.org/2361153002/diff/1/chrome/browser/ui/cocoa/too...
chrome/browser/ui/cocoa/toolbar/toolbar_controller.mm:336: // Material Design.
nit: remove references to material design (since it's the only design)