Update NEON compiler flags for Skia to use the SK_ prefix.

Update NEON compiler flags for Skia to use the SK_ prefix.

The old defines will be left in place until the next Skia DEPS
roll. After that roll Skia will be switched to using the SK_
prefixed defines and the old reserved (i.e. __ARM*) defines can be
removed.

BUG=skia:2785
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=286901

[djsollen, 2014-07-31 16:05:54]

https://codereview.chromium.org/430253002/diff/1/skia/skia_library_opts.gyp
File skia/skia_library_opts.gyp (left):

https://codereview.chromium.org/430253002/diff/1/skia/skia_library_opts.gyp#o...
skia/skia_library_opts.gyp:68: [ 'arm_version >= 7 and arm_neon == 1', {
this is not needed as it is pulled in from the skia_common.gypi

https://codereview.chromium.org/430253002/diff/1/skia/skia_library_opts.gyp#o...
skia/skia_library_opts.gyp:335: 'conditions': [
same here

[mtklein, 2014-07-31 16:08:34]

lgtm

[djsollen, 2014-07-31 16:15:54]

The CQ bit was checked by djsollen@google.com

[commit-bot: I haz the power, 2014-07-31 16:17:16]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/djsollen@google.com/430253002/1

[commit-bot: I haz the power, 2014-07-31 16:17:17]

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

[commit-bot: I haz the power, 2014-07-31 16:17:18]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[mtklein_C, 2014-07-31 16:17:51]

lgtm

[djsollen, 2014-07-31 16:18:01]

The CQ bit was checked by djsollen@google.com

[commit-bot: I haz the power, 2014-07-31 16:20:19]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/djsollen@google.com/430253002/1

[commit-bot: I haz the power, 2014-07-31 16:20:21]

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

[commit-bot: I haz the power, 2014-07-31 16:20:22]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[bungeman-chromium, 2014-07-31 16:24:24]

lgtm

[djsollen, 2014-07-31 16:24:34]

The CQ bit was checked by djsollen@google.com

[commit-bot: I haz the power, 2014-07-31 16:26:01]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/djsollen@google.com/430253002/1

[commit-bot: I haz the power, 2014-07-31 20:17:50]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_chromium_rel on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel/...)

[commit-bot: I haz the power, 2014-07-31 22:05:43]

Change committed as 286901