DialogDelegate: allow platforms to prevent Cancel being default

DialogDelegate: allow platforms to prevent Cancel being default

On Mac, DIALOG_BUTTON_CANCEL should never have default button styling, so
DialogDelegate::UpdateButton() should avoid setting cancel buttons as default.

BUG=605652
Committed: https://crrev.com/9139823c60ba620c8d344e9ef70ea5f2eea680e0
Cr-Commit-Position: refs/heads/master@{#406276}

[Elly Fong-Jones, 2016-07-14 17:37:11]

Description was changed from

==========
DialogDelegate: allow platforms to prevent Cancel being default

On Mac, DIALOG_BUTTON_CANCEL should never have default button styling, so
DialogDelegate::UpdateButton() should avoid setting cancel buttons as default.

BUG=605652
==========

to

==========
DialogDelegate: allow platforms to prevent Cancel being default

On Mac, DIALOG_BUTTON_CANCEL should never have default button styling, so
DialogDelegate::UpdateButton() should avoid setting cancel buttons as default.

BUG=605652
==========

[Elly Fong-Jones, 2016-07-14 17:37:11]

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

[Elly Fong-Jones, 2016-07-14 17:37:18]

sky: ptal? :)

[sky, 2016-07-18 15:50:42]

LGTM

[Elly Fong-Jones, 2016-07-18 18:27:17]

The CQ bit was checked by ellyjones@chromium.org

[commit-bot: I haz the power, 2016-07-18 18:27:51]

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-07-18 19:35:59]

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

[commit-bot: I haz the power, 2016-07-18 19:36:00]

Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Elly Fong-Jones, 2016-07-19 14:30:23]

The CQ bit was checked by ellyjones@chromium.org

[Elly Fong-Jones, 2016-07-19 14:30:23]

The patchset sent to the CQ was uploaded after l-g-t-m from sky@chromium.org
Link to the patchset: https://codereview.chromium.org/2151933002/#ps20001
(title: "fix unit test for mac")

[commit-bot: I haz the power, 2016-07-19 14:30:51]

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-07-19 15:15:46]

Description was changed from

==========
DialogDelegate: allow platforms to prevent Cancel being default

On Mac, DIALOG_BUTTON_CANCEL should never have default button styling, so
DialogDelegate::UpdateButton() should avoid setting cancel buttons as default.

BUG=605652
==========

to

==========
DialogDelegate: allow platforms to prevent Cancel being default

On Mac, DIALOG_BUTTON_CANCEL should never have default button styling, so
DialogDelegate::UpdateButton() should avoid setting cancel buttons as default.

BUG=605652
==========

[commit-bot: I haz the power, 2016-07-19 15:15:48]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-07-19 15:15:51]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-19 15:18:32]

Description was changed from

==========
DialogDelegate: allow platforms to prevent Cancel being default

On Mac, DIALOG_BUTTON_CANCEL should never have default button styling, so
DialogDelegate::UpdateButton() should avoid setting cancel buttons as default.

BUG=605652
==========

to

==========
DialogDelegate: allow platforms to prevent Cancel being default

On Mac, DIALOG_BUTTON_CANCEL should never have default button styling, so
DialogDelegate::UpdateButton() should avoid setting cancel buttons as default.

BUG=605652

Committed: https://crrev.com/9139823c60ba620c8d344e9ef70ea5f2eea680e0
Cr-Commit-Position: refs/heads/master@{#406276}
==========

[commit-bot: I haz the power, 2016-07-19 15:18:33]

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