GN: Fix common_audio linking on iOS

GN: Fix common_audio linking on iOS

The invalid condition made the test be included for iOS, which
fails linking.

BUG=webrtc:5949, webrtc:5544
NOTRY=True

Committed: https://crrev.com/34db033cd4489f6bc6a60549a73e555aa57a5bc3
Cr-Commit-Position: refs/heads/master@{#13970}

[kjellander_webrtc, 2016-08-30 09:41:08]

kjellander@webrtc.org changed reviewers:
+ ehmaldonado@webrtc.org

[ehmaldonado_webrtc, 2016-08-30 09:44:38]

On 2016/08/30 09:41:08, kjellander_webrtc wrote:

Only for arm?

[kjellander_webrtc, 2016-08-30 09:51:06]

On 2016/08/30 09:44:38, ehmaldonado_webrtc wrote:
> On 2016/08/30 09:41:08, kjellander_webrtc wrote:
> 
> Only for arm?

Yeah, as stated in the bug
(https://bugs.chromium.org/p/webrtc/issues/detail?id=5544) it actually builds
for ARM64, so I didn't see the point of not having the test for that platform. 

I should have mentioned that...

[kjellander_webrtc, 2016-08-30 09:51:18]

Description was changed from

==========
GN: Fix common_audio linking on iOS

The invalid condition made the test be included for iOS, which
fails linking.

BUG=webrtc:5949
NOTRY=True
==========

to

==========
GN: Fix common_audio linking on iOS

The invalid condition made the test be included for iOS, which
fails linking.

BUG=webrtc:5949, webrtc:5544
NOTRY=True
==========

[ehmaldonado_webrtc, 2016-08-30 09:55:39]

On 2016/08/30 09:51:06, kjellander_webrtc wrote:
> On 2016/08/30 09:44:38, ehmaldonado_webrtc wrote:
> > On 2016/08/30 09:41:08, kjellander_webrtc wrote:
> > 
> > Only for arm?
> 
> Yeah, as stated in the bug
> (https://bugs.chromium.org/p/webrtc/issues/detail?id=5544) it actually builds
> for ARM64, so I didn't see the point of not having the test for that platform.

> 
> I should have mentioned that...

LGTM

[kjellander_webrtc, 2016-08-30 11:14:22]

On 2016/08/30 09:55:39, ehmaldonado_webrtc wrote:
> On 2016/08/30 09:51:06, kjellander_webrtc wrote:
> > On 2016/08/30 09:44:38, ehmaldonado_webrtc wrote:
> > > On 2016/08/30 09:41:08, kjellander_webrtc wrote:
> > > 
> > > Only for arm?
> > 
> > Yeah, as stated in the bug
> > (https://bugs.chromium.org/p/webrtc/issues/detail?id=5544) it actually
builds
> > for ARM64, so I didn't see the point of not having the test for that
platform.
> 
> > 
> > I should have mentioned that...
> 
> LGTM

https://codereview.webrtc.org/2291443002/#ps250001 being green demonstrates this
is correct (even post-GN flip).

[kjellander_webrtc, 2016-08-30 11:14:24]

The CQ bit was checked by kjellander@webrtc.org

[commit-bot: I haz the power, 2016-08-30 11:14:33]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2016-08-30 11:17:15]

Description was changed from

==========
GN: Fix common_audio linking on iOS

The invalid condition made the test be included for iOS, which
fails linking.

BUG=webrtc:5949, webrtc:5544
NOTRY=True
==========

to

==========
GN: Fix common_audio linking on iOS

The invalid condition made the test be included for iOS, which
fails linking.

BUG=webrtc:5949, webrtc:5544
NOTRY=True
==========

[commit-bot: I haz the power, 2016-08-30 11:17:16]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-30 11:17:23]

Description was changed from

==========
GN: Fix common_audio linking on iOS

The invalid condition made the test be included for iOS, which
fails linking.

BUG=webrtc:5949, webrtc:5544
NOTRY=True
==========

to

==========
GN: Fix common_audio linking on iOS

The invalid condition made the test be included for iOS, which
fails linking.

BUG=webrtc:5949, webrtc:5544
NOTRY=True

Committed: https://crrev.com/34db033cd4489f6bc6a60549a73e555aa57a5bc3
Cr-Commit-Position: refs/heads/master@{#13970}
==========

[commit-bot: I haz the power, 2016-08-30 11:17:24]

Patchset 1 (id:??) landed as
https://crrev.com/34db033cd4489f6bc6a60549a73e555aa57a5bc3
Cr-Commit-Position: refs/heads/master@{#13970}