skia/ext: do not assume Skia's headers include windows.h

skia/ext: do not assume Skia's headers include windows.h

Committed: https://crrev.com/b765188e433cb3fea3ce675553269d8b660b8053
Cr-Commit-Position: refs/heads/master@{#398129}

[hal.canary, 2016-06-06 19:54:15]

The CQ bit was checked by halcanary@google.com to run a CQ dry run

[hal.canary, 2016-06-06 19:54:33]

halcanary@google.com changed reviewers:
+ bungeman@chromium.org, bungeman@google.com

[hal.canary, 2016-06-06 19:54:34]

PTAL

[commit-bot: I haz the power, 2016-06-06 19:54:50]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2041113002/1

[bungeman-skia, 2016-06-06 20:04:50]

https://codereview.chromium.org/2041113002/diff/1/skia/ext/platform_surface.h
File skia/ext/platform_surface.h (right):

https://codereview.chromium.org/2041113002/diff/1/skia/ext/platform_surface.h...
skia/ext/platform_surface.h:11: #include <windows.h>
Does this work if moved below the Skia includes to the platform ifdef block
below with USE_CAIRO and OS_MACOSX. It should probably come first there so it
looks like the one inside the 'skia' namespace.

[hal.canary, 2016-06-06 20:07:05]

https://codereview.chromium.org/2041113002/diff/1/skia/ext/platform_surface.h
File skia/ext/platform_surface.h (right):

https://codereview.chromium.org/2041113002/diff/1/skia/ext/platform_surface.h...
skia/ext/platform_surface.h:11: #include <windows.h>
On 2016/06/06 20:04:50, bungeman-skia wrote:
> Does this work if moved below the Skia includes to the platform ifdef block
> below with USE_CAIRO and OS_MACOSX. It should probably come first there so it
> looks like the one inside the 'skia' namespace.

Done.

[bungeman-skia, 2016-06-06 20:08:21]

Description was changed from

==========
skia/ext: do not assume Skia's headers include windows.h

.
==========

to

==========
skia/ext: do not assume Skia's headers include windows.h
==========

[bungeman-skia, 2016-06-06 20:08:38]

lgtm

[bungeman-chromium, 2016-06-06 20:08:53]

The CQ bit was checked by bungeman@chromium.org

[bungeman-chromium, 2016-06-06 20:08:53]

lgtm

[commit-bot: I haz the power, 2016-06-06 20:09:35]

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

[commit-bot: I haz the power, 2016-06-06 21:22:55]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-06 21:27:04]

Description was changed from

==========
skia/ext: do not assume Skia's headers include windows.h
==========

to

==========
skia/ext: do not assume Skia's headers include windows.h

Committed: https://crrev.com/b765188e433cb3fea3ce675553269d8b660b8053
Cr-Commit-Position: refs/heads/master@{#398129}
==========

[commit-bot: I haz the power, 2016-06-06 21:27:05]

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