Ignores SecRandomCopyBytes() warning for GYP build.

r14326 added '-Wno-unused-result' to 'WARNING_CFLAGS!' which removes the
flag. This change moves it to WARNING_CFLAGS, and makes it work for both
iOS and mac.

BUG=webrtc:6396
NOTRY=True
NOPRESUBMIT=True
NOTREECHECKS=True

Committed: https://crrev.com/20684110fdbde0818579eea472ee376b31249fde
Cr-Commit-Position: refs/heads/master@{#14378}

[jianjunz, 2016-09-26 01:56:55]

Description was changed from

==========
Ignores SecRandomCopyBytes() warning for GYP build.

BUG=webrtc:6396
==========

to

==========
r14326 added '-Wno-unused-result' to 'WARNING_CFLAGS!' which removes the flag.
This change moves it to WARNING_CFLAGS, and makes it work for both iOS and mac.

BUG=webrtc:6396
==========

[jianjunz, 2016-09-26 01:57:27]

Description was changed from

==========
r14326 added '-Wno-unused-result' to 'WARNING_CFLAGS!' which removes the flag.
This change moves it to WARNING_CFLAGS, and makes it work for both iOS and mac.

BUG=webrtc:6396
==========

to

==========
r14326 added '-Wno-unused-result' to 'WARNING_CFLAGS!' which removes the
flag. This change moves it to WARNING_CFLAGS, and makes it work for
both iOS and mac.

BUG=webrtc:6396
==========

[jianjunz, 2016-09-26 01:58:51]

Description was changed from

==========
r14326 added '-Wno-unused-result' to 'WARNING_CFLAGS!' which removes the
flag. This change moves it to WARNING_CFLAGS, and makes it work for
both iOS and mac.

BUG=webrtc:6396
==========

to

==========
r14326 added '-Wno-unused-result' to 'WARNING_CFLAGS!' which removes the
flag. This change moves it to WARNING_CFLAGS, and makes it work for 
both
iOS and mac.

BUG=webrtc:6396
==========

[jianjunz, 2016-09-26 01:58:51]

jianjun.zhu@intel.com changed reviewers:
+ henrika@webrtc.org

[jianjunz, 2016-09-26 01:59:45]

PTAL

[jianjunz, 2016-09-26 02:00:31]

Description was changed from

==========
r14326 added '-Wno-unused-result' to 'WARNING_CFLAGS!' which removes the
flag. This change moves it to WARNING_CFLAGS, and makes it work for 
both
iOS and mac.

BUG=webrtc:6396
==========

to

==========
r14326 added '-Wno-unused-result' to 'WARNING_CFLAGS!' which removes the
flag. This change moves it to WARNING_CFLAGS, and makes it work for both
iOS and mac.

BUG=webrtc:6396
==========

[henrika_webrtc, 2016-09-26 09:36:04]

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

[henrika_webrtc, 2016-09-26 09:36:36]

kjellander@: any input here?

[kjellander_webrtc, 2016-09-26 09:38:20]

Description was changed from

==========
r14326 added '-Wno-unused-result' to 'WARNING_CFLAGS!' which removes the
flag. This change moves it to WARNING_CFLAGS, and makes it work for both
iOS and mac.

BUG=webrtc:6396
==========

to

==========
r14326 added '-Wno-unused-result' to 'WARNING_CFLAGS!' which removes the
flag. This change moves it to WARNING_CFLAGS, and makes it work for both
iOS and mac.

BUG=webrtc:6396
NOTRY=True
==========

[kjellander_webrtc, 2016-09-26 09:40:03]

Description was changed from

==========
r14326 added '-Wno-unused-result' to 'WARNING_CFLAGS!' which removes the
flag. This change moves it to WARNING_CFLAGS, and makes it work for both
iOS and mac.

BUG=webrtc:6396
NOTRY=True
==========

to

==========
r14326 added '-Wno-unused-result' to 'WARNING_CFLAGS!' which removes the
flag. This change moves it to WARNING_CFLAGS, and makes it work for both
iOS and mac.

BUG=webrtc:6396
NOTRY=True
NOPRESUBMIT=True
NOTREECHECKS=True
==========

[henrika_webrtc, 2016-09-26 09:40:15]

LGTM from my side

[kjellander_webrtc, 2016-09-26 09:41:13]

lgtm
we didn't notice the error since we almost don't utilize GYP any more (it's
deprecated and mostly don't work anyway).

[kjellander_webrtc, 2016-09-26 09:41:18]

The CQ bit was checked by kjellander@webrtc.org

[commit-bot: I haz the power, 2016-09-26 09:41:30]

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-09-26 09:41:38]

Description was changed from

==========
r14326 added '-Wno-unused-result' to 'WARNING_CFLAGS!' which removes the
flag. This change moves it to WARNING_CFLAGS, and makes it work for both
iOS and mac.

BUG=webrtc:6396
NOTRY=True
NOPRESUBMIT=True
NOTREECHECKS=True
==========

to

==========
r14326 added '-Wno-unused-result' to 'WARNING_CFLAGS!' which removes the
flag. This change moves it to WARNING_CFLAGS, and makes it work for both
iOS and mac.

BUG=webrtc:6396
NOTRY=True
NOPRESUBMIT=True
NOTREECHECKS=True
==========

[commit-bot: I haz the power, 2016-09-26 09:41:41]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-26 09:41:46]

Description was changed from

==========
r14326 added '-Wno-unused-result' to 'WARNING_CFLAGS!' which removes the
flag. This change moves it to WARNING_CFLAGS, and makes it work for both
iOS and mac.

BUG=webrtc:6396
NOTRY=True
NOPRESUBMIT=True
NOTREECHECKS=True
==========

to

==========
r14326 added '-Wno-unused-result' to 'WARNING_CFLAGS!' which removes the
flag. This change moves it to WARNING_CFLAGS, and makes it work for both
iOS and mac.

BUG=webrtc:6396
NOTRY=True
NOPRESUBMIT=True
NOTREECHECKS=True

Committed: https://crrev.com/20684110fdbde0818579eea472ee376b31249fde
Cr-Commit-Position: refs/heads/master@{#14378}
==========

[commit-bot: I haz the power, 2016-09-26 09:41:47]

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