Gyp->GN migration of RTPjitter.

Gyp->GN migration of RTPjitter.

BUG=6162
NOTRY=True

Committed: https://crrev.com/17dfa7425fbf8c54849c69b30e279cc8b31f004f
Cr-Commit-Position: refs/heads/master@{#13620}

[aleloi, 2016-08-01 15:01:05]

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

[aleloi, 2016-08-01 15:01:06]

Could you please take a look at this Gyp->GN migrated target?

https://codereview.webrtc.org/2196283002/diff/1/webrtc/modules/audio_coding/B...
File webrtc/modules/audio_coding/BUILD.gn (right):

https://codereview.webrtc.org/2196283002/diff/1/webrtc/modules/audio_coding/B...
webrtc/modules/audio_coding/BUILD.gn:1215: "../..:webrtc_common",
Added webrtc_common because linker couldn't find ntohs.

[kjellander_webrtc, 2016-08-01 19:10:13]

lgtm with a suggestion.

https://codereview.webrtc.org/2196283002/diff/1/webrtc/modules/audio_coding/B...
File webrtc/modules/audio_coding/BUILD.gn (right):

https://codereview.webrtc.org/2196283002/diff/1/webrtc/modules/audio_coding/B...
webrtc/modules/audio_coding/BUILD.gn:1215: "../..:webrtc_common",
On 2016/08/01 15:01:06, aleloi wrote:
> Added webrtc_common because linker couldn't find ntohs.

Please do the same for the GYP target, just for completeness.

[kjellander_webrtc, 2016-08-01 19:10:52]

On 2016/08/01 19:10:13, kjellander_webrtc wrote:
> lgtm with a suggestion.
> 
>
https://codereview.webrtc.org/2196283002/diff/1/webrtc/modules/audio_coding/B...
> File webrtc/modules/audio_coding/BUILD.gn (right):
> 
>
https://codereview.webrtc.org/2196283002/diff/1/webrtc/modules/audio_coding/B...
> webrtc/modules/audio_coding/BUILD.gn:1215: "../..:webrtc_common",
> On 2016/08/01 15:01:06, aleloi wrote:
> > Added webrtc_common because linker couldn't find ntohs.
> 
> Please do the same for the GYP target, just for completeness.

Add a link to a bug in the CL description as well (BUG= line)

[aleloi, 2016-08-02 08:48:47]

Description was changed from

==========
Gyp->GN migration of RTPjitter.
==========

to

==========
Gyp->GN migration of RTPjitter.

BUG=6162
==========

[aleloi, 2016-08-02 08:48:57]

The CQ bit was checked by aleloi@webrtc.org

[aleloi, 2016-08-02 08:48:58]

The patchset sent to the CQ was uploaded after l-g-t-m from
kjellander@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2196283002/#ps20001 (title:
"Added webrtc_common dependency to RTPjitter Gyp target.")

[commit-bot: I haz the power, 2016-08-02 08:49:04]

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-02 11:40:32]

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

[commit-bot: I haz the power, 2016-08-02 11:40:33]

Try jobs failed on following builders:
  android_arm64_rel on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/android_arm64_rel/build...)

[kjellander_webrtc, 2016-08-02 11:44:21]

On 2016/08/02 11:40:33, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   android_arm64_rel on master.tryserver.webrtc (JOB_FAILED,
>
http://build.chromium.org/p/tryserver.webrtc/builders/android_arm64_rel/build...)

Just set NOTRY=True and submit ignoring that bot. For changes like this I would
actually prefer you always do that and just run a few compile bots.

[aleloi, 2016-08-03 07:56:21]

Description was changed from

==========
Gyp->GN migration of RTPjitter.

BUG=6162
==========

to

==========
Gyp->GN migration of RTPjitter.

BUG=6162

NOTRY=True
==========

[aleloi, 2016-08-03 08:16:03]

The CQ bit was checked by aleloi@webrtc.org

[aleloi, 2016-08-03 08:16:03]

The patchset sent to the CQ was uploaded after l-g-t-m from
kjellander@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2196283002/#ps40001 (title:
"Rebase on top of master.")

[commit-bot: I haz the power, 2016-08-03 08:16:10]

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-03 08:17:28]

Description was changed from

==========
Gyp->GN migration of RTPjitter.

BUG=6162

NOTRY=True
==========

to

==========
Gyp->GN migration of RTPjitter.

BUG=6162

NOTRY=True
==========

[commit-bot: I haz the power, 2016-08-03 08:17:28]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-08-03 08:17:36]

Description was changed from

==========
Gyp->GN migration of RTPjitter.

BUG=6162

NOTRY=True
==========

to

==========
Gyp->GN migration of RTPjitter.

BUG=6162

NOTRY=True

Committed: https://crrev.com/17dfa7425fbf8c54849c69b30e279cc8b31f004f
Cr-Commit-Position: refs/heads/master@{#13620}
==========

[commit-bot: I haz the power, 2016-08-03 08:17:37]

Patchset 3 (id:??) landed as
https://crrev.com/17dfa7425fbf8c54849c69b30e279cc8b31f004f
Cr-Commit-Position: refs/heads/master@{#13620}