Exclude Android-only cpu-features.h from non-Android builds

Exclude Android-only cpu-features.h from non-Android builds

This is needed when building Chromium for ARM.

Patch by Joel Stanley <joel@chromium.org>

[agl, 2009-11-17 16:47:22]

NACK for now. I need to chat to some people about this first.

[agl1, 2009-11-17 23:39:25]

LGTM. Mike Reed should probably give the final ok.

http://codereview.appspot.com/154182/diff/8/1003
File core/SkBitmapProcState_filter.h (right):

http://codereview.appspot.com/154182/diff/8/1003#newcode18
core/SkBitmapProcState_filter.h:18: #include <machine/cpu-features.h>
Skia style != google3 style so one shouldn't reformat the include line.

http://codereview.appspot.com/154182/diff/8/1001
File opts/SkBitmapProcState_opts_arm.cpp (right):

http://codereview.appspot.com/154182/diff/8/1001#newcode18
opts/SkBitmapProcState_opts_arm.cpp:18: #include <machine/cpu-features.h>
Probably best to match the indent here too.

http://codereview.appspot.com/154182/diff/8/1002
File opts/SkBlitRow_opts_arm.cpp (right):

http://codereview.appspot.com/154182/diff/8/1002#newcode19
opts/SkBlitRow_opts_arm.cpp:19: #include <machine/cpu-features.h>
Probably best to match the indent here too.

[Joel Stanley, 2009-11-17 23:56:45]

http://codereview.appspot.com/154182/diff/8/1003
File core/SkBitmapProcState_filter.h (right):

http://codereview.appspot.com/154182/diff/8/1003#newcode18
core/SkBitmapProcState_filter.h:18: #include <machine/cpu-features.h>
On 2009/11/17 23:39:25, agl1 wrote:
> Skia style != google3 style so one shouldn't reformat the include line.

Done.

http://codereview.appspot.com/154182/diff/8/1001
File opts/SkBitmapProcState_opts_arm.cpp (right):

http://codereview.appspot.com/154182/diff/8/1001#newcode18
opts/SkBitmapProcState_opts_arm.cpp:18: #include <machine/cpu-features.h>
On 2009/11/17 23:39:25, agl1 wrote:
> Probably best to match the indent here too.

Done.

http://codereview.appspot.com/154182/diff/8/1002
File opts/SkBlitRow_opts_arm.cpp (right):

http://codereview.appspot.com/154182/diff/8/1002#newcode19
opts/SkBlitRow_opts_arm.cpp:19: #include <machine/cpu-features.h>
On 2009/11/17 23:39:25, agl1 wrote:
> Probably best to match the indent here too.

Done.

[agl, 2009-11-19 01:06:35]

If this is messing up your build, then it's probably ok to land it now. If Mike
objects, we can always fix it later.

[Joel Stanley, 2009-11-19 01:15:24]

On 2009/11/19 01:06:35, agl wrote:
> If this is messing up your build, then it's probably ok to land it now. If
Mike
> objects, we can always fix it later.

Thanks, that would be helpful.  Are you able to commit for me?

How do I go about getting it pulled into chrome's tree?