macviews: reinstate close button

macviews: reinstate close button

Blanket removal of close buttons turns out to have been hasty, since many of
our dialogs are designed around having them, not just visually but
semantically. For now, return the behavior to that of other platforms while
we reconsider this UI design.

BUG=637178
Committed: https://crrev.com/4aad4a8d18daf041d47bc0d9bb489f05bd5f5188
Cr-Commit-Position: refs/heads/master@{#413156}

[Elly Fong-Jones, 2016-08-18 15:46:29]

Description was changed from

==========
macviews: reinstate close button

Blanket removal of close buttons turns out to have been hasty, since many of
our dialogs are designed around having them, not just visually but
semantically. For now, return the behavior to that of other platforms while
we reconsider this UI design.

BUG=637178
==========

to

==========
macviews: reinstate close button

Blanket removal of close buttons turns out to have been hasty, since many of
our dialogs are designed around having them, not just visually but
semantically. For now, return the behavior to that of other platforms while
we reconsider this UI design.

BUG=637178
==========

[Elly Fong-Jones, 2016-08-18 15:46:29]

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

[Elly Fong-Jones, 2016-08-18 15:46:36]

msw: ptal? :)

[msw, 2016-08-18 16:44:13]

lgtm

[Elly Fong-Jones, 2016-08-18 17:35:21]

The CQ bit was checked by ellyjones@chromium.org

[commit-bot: I haz the power, 2016-08-18 17:35:53]

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-08-18 19:33:09]

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

[commit-bot: I haz the power, 2016-08-18 19:33:10]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[Elly Fong-Jones, 2016-08-19 14:23:10]

The CQ bit was checked by ellyjones@chromium.org

[commit-bot: I haz the power, 2016-08-19 14:23:32]

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-08-19 16:08:27]

Description was changed from

==========
macviews: reinstate close button

Blanket removal of close buttons turns out to have been hasty, since many of
our dialogs are designed around having them, not just visually but
semantically. For now, return the behavior to that of other platforms while
we reconsider this UI design.

BUG=637178
==========

to

==========
macviews: reinstate close button

Blanket removal of close buttons turns out to have been hasty, since many of
our dialogs are designed around having them, not just visually but
semantically. For now, return the behavior to that of other platforms while
we reconsider this UI design.

BUG=637178
==========

[commit-bot: I haz the power, 2016-08-19 16:08:28]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-19 16:11:06]

Description was changed from

==========
macviews: reinstate close button

Blanket removal of close buttons turns out to have been hasty, since many of
our dialogs are designed around having them, not just visually but
semantically. For now, return the behavior to that of other platforms while
we reconsider this UI design.

BUG=637178
==========

to

==========
macviews: reinstate close button

Blanket removal of close buttons turns out to have been hasty, since many of
our dialogs are designed around having them, not just visually but
semantically. For now, return the behavior to that of other platforms while
we reconsider this UI design.

BUG=637178

Committed: https://crrev.com/4aad4a8d18daf041d47bc0d9bb489f05bd5f5188
Cr-Commit-Position: refs/heads/master@{#413156}
==========

[commit-bot: I haz the power, 2016-08-19 16:11:07]

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