fix armv7 non-neon webrtc builds

webrtc/common_audio/common_audio.gyp

Index: webrtc/common_audio/common_audio.gyp
diff --git a/webrtc/common_audio/common_audio.gyp b/webrtc/common_audio/common_audio.gyp
index 36e6a65ebd086d5cfca542df3344d72e3add683b..9f7be2d8e192b925ca73ac9a1381cd371bf4c707 100644
--- a/webrtc/common_audio/common_audio.gyp
+++ b/webrtc/common_audio/common_audio.gyp
@@ -146,7 +146,7 @@
             'signal_processing/spl_sqrt_floor.c',
           ],
           'conditions': [
-            ['arm_version>=7', {
+            ['(arm_version>=7 and (arm_neon==1 or arm_neon_optional==1)) or target_arch=="arm64"', {

[Andrew MacDonald, 2015/06/17 03:29:23]

This is inside a target_arch=="arm" block so is certainly not right.

[Andrew MacDonald, 2015/06/17 05:36:32]

I think this should just be 'arm_neon==1 or arm_neon_optional == 1', but we
should update the other places we have neon targets as well.

We should probably add a build_arm_neon flag in common.gypi to consolidate this.

[Mostyn Bramley-Moore, 2015/06/17 07:42:26]

I tried this earlier, but got lost in the nested gyp scopes when trying to set
arm_neon to 1 whenever target_arch=="arm64".
Sounds good to me- I will experiment some more.

[Mostyn Bramley-Moore, 2015/06/17 07:42:26]

I don't think it's incorrect, but this is definitely not a good/final solution.

[zhongwei, 2015/06/17 11:44:50]

target_arch=="arm64" is not needed here.  arm64 is handled at line 160.

[jridges, 2015/06/17 15:56:49]

There is the same problem here with ARMv7 being confused with NEON. The
"filter_ar_fast_q12" module has nothing to do with NEON.

               'dependencies': ['common_audio_neon',],
               'sources': [
                 'signal_processing/filter_ar_fast_q12_armv7.S',
@@ -212,7 +212,7 @@
         },
       ],  # targets
     }],
-    ['target_arch=="arm" and arm_version>=7 or target_arch=="arm64"', {
+    ['(target_arch=="arm" and arm_version>=7 and (arm_neon==1 or arm_neon_optional==1)) or target_arch=="arm64"', {

[jridges, 2015/06/17 15:56:49]

Is there a definitive answer as to when the arm_neon and arm_neon_optional
variables get set? This seems like overkill to have to include the architecture
here as well.

       'targets': [
         {
           'target_name': 'common_audio_neon',