Roll WebRTC 7546:7549.

Roll WebRTC 7546:7549.

TBR=hellner@chromium.org
NOTRY=true
BUG=N/A

Committed: https://crrev.com/4e40bbe50a102784984852ca318dbcd2b03fc9fc
Cr-Commit-Position: refs/heads/master@{#301879}

Committed: https://crrev.com/ecb39f13449e606ca66b6d3401c3f306c65c2859
Cr-Commit-Position: refs/heads/master@{#302260}

[hellner1, 2014-10-15 22:41:27]

hellner@chromium.org changed reviewers:
+ dcheng@chromium.org, sergeyu@chromium.org

[hellner1, 2014-10-15 22:41:28]

PTAL

Synopsis: libjingle is moving into webrtc. This cl rolls chromium DEPS picking
up latest webrtc and libjingle and switches Chromium's usage of talk's p2p and
xmpp-folders to webrtc's

components/invalidation dcheng
everything else sergeyu
(content/browser/renderer_host/p2p, content/renderer/p2p, jingle, remoting,
third_party/libjingle)

Cl is similar to e.g. https://codereview.chromium.org/543433002/ where xmllite
was moved from libjingle to webrtc. Also tracked here:
https://code.google.com/p/webrtc/issues/detail?id=3379

[dcheng, 2014-10-15 23:04:34]

components/invalidation LGTM

[Jamie, 2014-10-16 00:31:37]

jamiewalch@chromium.org changed reviewers:
+ jamiewalch@chromium.org

[Jamie, 2014-10-16 00:31:39]

https://codereview.chromium.org/653203005/diff/1/remoting/remoting_nacl.gyp
File remoting/remoting_nacl.gyp (left):

https://codereview.chromium.org/653203005/diff/1/remoting/remoting_nacl.gyp#o...
remoting/remoting_nacl.gyp:158:
'>(tc_lib_dir_pnacl_newlib)/libjingle_p2p_constants_nacl.a',
I'm surprised to see this deleted rather than modified. Where are we getting the
symbols from that we used to get from here?

[hellner1, 2014-10-16 14:45:50]

https://codereview.chromium.org/653203005/diff/1/remoting/remoting_nacl.gyp
File remoting/remoting_nacl.gyp (left):

https://codereview.chromium.org/653203005/diff/1/remoting/remoting_nacl.gyp#o...
remoting/remoting_nacl.gyp:158:
'>(tc_lib_dir_pnacl_newlib)/libjingle_p2p_constants_nacl.a',
On 2014/10/16 00:31:39, Jamie wrote:
> I'm surprised to see this deleted rather than modified. Where are we getting
the
> symbols from that we used to get from here?

My mistake here. In the non-nacl, the different constants come from different
projects which removes the need for the workaround, however, nacl is still built
from one large target so it is still likely needed.

[hellner1, 2014-10-16 22:31:11]

hellner@chromium.org changed reviewers:
+ jamesr@chromium.org

[hellner1, 2014-10-16 22:31:12]

PTAL

Seems Sergey is on paternity leave.
content/browser/renderer_host/p2p hclam
jingle - hclam
third_party/libjingle - hclam
content/renderer/p2p - jamesr
remoting - jamiewalch

[hellner1, 2014-10-16 22:37:32]

hellner@chromium.org changed reviewers:
+ hclam@chromium.org

[hellner1, 2014-10-16 22:37:33]

Forgot to add hclam. PTAL

[Alpha Left Google, 2014-10-16 22:38:05]

The CQ bit was checked by hclam@chromium.org

[Alpha Left Google, 2014-10-16 22:38:06]

lgtm

[Alpha Left Google, 2014-10-16 22:38:12]

The CQ bit was unchecked by hclam@chromium.org

[Peter Kasting, 2014-10-21 20:44:50]

hellner, are you going to land this at some point?  Or has this been superseded?

[hellner1, 2014-10-22 14:28:51]

On 2014/10/21 20:44:50, Peter Kasting wrote:
> hellner, are you going to land this at some point?  Or has this been
superseded?

I plan to land this. I'm still waiting for reviews and I am currently doing
Android Bootcamp so it should happen sometime next week.

Note that I have temporarily rolled back the corresponding webrtc cl
(https://code.google.com/p/webrtc/source/detail?r=7459 in r7472) to unblock
regular DEPS rolls.

[hellner1, 2014-10-27 21:25:20]

On 2014/10/22 14:28:51, hellner1 wrote:
> On 2014/10/21 20:44:50, Peter Kasting wrote:
> > hellner, are you going to land this at some point?  Or has this been
> superseded?
> 
> I plan to land this. I'm still waiting for reviews and I am currently doing
> Android Bootcamp so it should happen sometime next week.
> 
> Note that I have temporarily rolled back the corresponding webrtc cl
> (https://code.google.com/p/webrtc/source/detail?r=7459 in r7472) to unblock
> regular DEPS rolls.

ping for jamesr and jamiewalch

Some more background: I will wait with re-submitting the webrtc cl which this cl
relates to until I have a complete set of lgtms here as that will minimize the
time that webrtc can't be rolled normally. Also note that I have a fix for the
broken win bots (tested locally). Also note I won't commit until all bots are
happy with the cl (which I won't know until re-landing the webrtc cl).

[jamesr, 2014-10-27 21:26:11]

On 2014/10/27 21:25:20, hellner1 wrote:
> On 2014/10/22 14:28:51, hellner1 wrote:
> > On 2014/10/21 20:44:50, Peter Kasting wrote:
> > > hellner, are you going to land this at some point?  Or has this been
> > superseded?
> > 
> > I plan to land this. I'm still waiting for reviews and I am currently doing
> > Android Bootcamp so it should happen sometime next week.
> > 
> > Note that I have temporarily rolled back the corresponding webrtc cl
> > (https://code.google.com/p/webrtc/source/detail?r=7459 in r7472) to unblock
> > regular DEPS rolls.
> 
> ping for jamesr and jamiewalch

Sorry, what do you want me to look at here?

[hellner1, 2014-10-27 21:31:29]

On 2014/10/27 21:26:11, jamesr wrote:
> On 2014/10/27 21:25:20, hellner1 wrote:
> > On 2014/10/22 14:28:51, hellner1 wrote:
> > > On 2014/10/21 20:44:50, Peter Kasting wrote:
> > > > hellner, are you going to land this at some point?  Or has this been
> > > superseded?
> > > 
> > > I plan to land this. I'm still waiting for reviews and I am currently
doing
> > > Android Bootcamp so it should happen sometime next week.
> > > 
> > > Note that I have temporarily rolled back the corresponding webrtc cl
> > > (https://code.google.com/p/webrtc/source/detail?r=7459 in r7472) to
unblock
> > > regular DEPS rolls.
> > 
> > ping for jamesr and jamiewalch
> 
> Sorry, what do you want me to look at here?

content/renderer/p2p - jamesr
remoting - jamiewalch

Libjingle's p2p and xmpp folders are being moved to webrtc in an effort to
consolidate the two into one single project. It is tracked here:
https://code.google.com/p/webrtc/issues/detail?id=3379.

You can also look at https://codereview.chromium.org/543433002/ which is
similar, but for xmllite.

[jamesr, 2014-10-27 21:32:15]

On 2014/10/27 21:31:29, hellner1 wrote:
> On 2014/10/27 21:26:11, jamesr wrote:
> > Sorry, what do you want me to look at here?
> 
> content/renderer/p2p - jamesr

There's a more specific owner for content/renderer/p2p, please go to them first
before looking at more general OWNERS.

[hellner1, 2014-10-27 21:34:18]

On 2014/10/27 21:32:15, jamesr wrote:
> On 2014/10/27 21:31:29, hellner1 wrote:
> > On 2014/10/27 21:26:11, jamesr wrote:
> > > Sorry, what do you want me to look at here?
> > 
> > content/renderer/p2p - jamesr
> 
> There's a more specific owner for content/renderer/p2p, please go to them
first
> before looking at more general OWNERS.

content/renderer/p2p's OWNER file only has sergeyu who is on parental leave.

[Jamie, 2014-10-28 00:28:23]

remoting/ lgtm

[jamesr, 2014-10-28 00:29:26]

content/renderer/ lgtm

[henrike, 2014-10-29 16:12:14]

henrike@webrtc.org changed reviewers:
+ henrike@webrtc.org

[henrike, 2014-10-29 16:12:15]

LGTM (to fix presubmit warnings for DEPS changes)

[hellner1, 2014-10-29 17:57:31]

tbr-ed myself to get past check deps since I am an owner of the target
directories (webrtc), even if the presubmit script doesn't recognize that. It
seems to do the trick.

[hellner1, 2014-10-29 18:19:10]

The CQ bit was checked by hellner@chromium.org

[commit-bot: I haz the power, 2014-10-29 18:21:02]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/653203005/140001

[commit-bot: I haz the power, 2014-10-29 18:23:30]

Committed patchset #8 (id:140001)

[commit-bot: I haz the power, 2014-10-29 18:24:12]

Patchset 8 (id:??) landed as
https://crrev.com/4e40bbe50a102784984852ca318dbcd2b03fc9fc
Cr-Commit-Position: refs/heads/master@{#301879}

[Mike Wittman, 2014-10-29 19:52:35]

Reverted this in 6ca1a64c4ab2d2e97d529808fd66d7bf63dd8b66. Appears to cause
timeouts on Mac 10.6 and 10.7 tests:

JingleSessionTest.TestFailedChannelAuth
JingleSessionTest.TestMultistepAuthStreamChannel
JingleSessionTest.TestMuxStreamChannel
JingleSessionTest.TestStreamChannel

PseudoTcpAdapterTest.DataTransfer
PseudoTcpAdapterTest.DeleteOnConnected
PseudoTcpAdapterTest.LimitedChannel
PseudoTcpAdapterTest.WriteWaitsForSendLetsDataThrough

http://build.chromium.org/p/chromium.mac/buildstatus?builder=Mac%2010.6%20Tes...

http://build.chromium.org/p/chromium.mac/buildstatus?builder=Mac%2010.7%20Tes...

[hellner1, 2014-10-31 14:19:27]

The CQ bit was checked by hellner@chromium.org

[commit-bot: I haz the power, 2014-10-31 14:21:36]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/653203005/160001

[commit-bot: I haz the power, 2014-10-31 14:23:15]

Committed patchset #9 (id:160001)

[commit-bot: I haz the power, 2014-10-31 14:24:03]

Patchset 9 (id:??) landed as
https://crrev.com/ecb39f13449e606ca66b6d3401c3f306c65c2859
Cr-Commit-Position: refs/heads/master@{#302260}