Remove Metro mode avatar/profile switcher assets

Remove Metro mode avatar/profile switcher assets

BUG=558054
Committed: https://crrev.com/327f4816de86c9408c8638e9c91add65424affd3
Cr-Commit-Position: refs/heads/master@{#370906}

[tapted, 2016-01-20 23:43:04]

tapted@chromium.org changed reviewers:
+ msw@chromium.org, oshima@chromium.org

[tapted, 2016-01-20 23:43:04]

msw: please review new_avatar_button.cc
oshima: for chrome/app/theme

(context: these are in the way of some refactoring/cleanups I'm trying to do for
the toolkit-views browser frame on mac)

[oshima, 2016-01-20 23:44:30]

lgtm

[msw, 2016-01-21 18:18:01]

lgtm with a nit

https://codereview.chromium.org/1605243002/diff/1/chrome/browser/ui/views/pro...
File chrome/browser/ui/views/profiles/new_avatar_button.cc (right):

https://codereview.chromium.org/1605243002/diff/1/chrome/browser/ui/views/pro...
chrome/browser/ui/views/profiles/new_avatar_button.cc:9: #include
"base/win/windows_version.h"
nit: remove

[tapted, 2016-01-22 04:49:33]

https://codereview.chromium.org/1605243002/diff/1/chrome/browser/ui/views/pro...
File chrome/browser/ui/views/profiles/new_avatar_button.cc (right):

https://codereview.chromium.org/1605243002/diff/1/chrome/browser/ui/views/pro...
chrome/browser/ui/views/profiles/new_avatar_button.cc:9: #include
"base/win/windows_version.h"
On 2016/01/21 18:18:01, msw wrote:
> nit: remove

Done.

[tapted, 2016-01-22 04:49:43]

The CQ bit was checked by tapted@chromium.org

[tapted, 2016-01-22 04:49:43]

The patchset sent to the CQ was uploaded after l-g-t-m from msw@chromium.org,
oshima@chromium.org
Link to the patchset: https://codereview.chromium.org/1605243002/#ps20001
(title: "remove unused header")

[commit-bot: I haz the power, 2016-01-22 04:50:00]

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

[commit-bot: I haz the power, 2016-01-22 04:55:11]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-01-22 04:56:24]

Description was changed from

==========
Remove Metro mode avatar/profile switcher assets

BUG=558054
==========

to

==========
Remove Metro mode avatar/profile switcher assets

BUG=558054

Committed: https://crrev.com/327f4816de86c9408c8638e9c91add65424affd3
Cr-Commit-Position: refs/heads/master@{#370906}
==========

[commit-bot: I haz the power, 2016-01-22 04:56:24]

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

[tapted, 2016-01-25 21:23:25]

https://codereview.chromium.org/1605243002/diff/20001/chrome/browser/ui/views...
File chrome/browser/ui/views/profiles/new_avatar_button.cc (left):

https://codereview.chromium.org/1605243002/diff/20001/chrome/browser/ui/views...
chrome/browser/ui/views/profiles/new_avatar_button.cc:84: } else if
(base::win::GetVersion() >= base::win::VERSION_WIN8 ||
Argh. This is an `||` not an `&&`. What a trap. I want to rename these IDR
constants anyway for Mac. But for now this needs to get reverted #shamecube.

[tapted, 2016-01-25 21:23:55]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/1631973002/ by tapted@chromium.org.

The reason for reverting is: Broke Win10

BUG=581097.