webrtc deps in remoting should be inside a webrtc condition

webrtc deps in remoting should be inside a webrtc condition

If you try to build with enable_webrtc=0 and remoting=1, gyp will fail:
libjingle.gyp:libpeerconnection#target' when processing
remoting.gyp:remoting_host#target.  This happens because the
libpeerconnnection target is defined inside an enable_webrtc==1 condition.
Therefore, these dependencies in remoting.gyp should be too.

BUG=399413
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=287193

[Mostyn Bramley-Moore, 2014-07-31 20:19:35]

@Sergey, aiguha: does this look OK?

[aiguha, 2014-08-01 18:35:56]

LGTM

You might have already seen this: modules.gyp:desktop_capture#target is not
defined inside an enable_webrtc==1 condition (though perhaps it should be, I'm
not sure).

[garykac, 2014-08-01 18:38:12]

lgtm

[Mostyn Bramley-Moore, 2014-08-01 18:49:23]

On 2014/08/01 18:35:56, aiguha wrote:
> LGTM
> 
> You might have already seen this: modules.gyp:desktop_capture#target is not
> defined inside an enable_webrtc==1 condition (though perhaps it should be, I'm
> not sure).

I hadn't noticed that, but I don't think it causes trouble (since it can't break
in the way that the libpeerconnection dependency did before this patch).

[Mostyn Bramley-Moore, 2014-08-01 18:49:41]

The CQ bit was checked by mostynb@opera.com

[commit-bot: I haz the power, 2014-08-01 18:50:50]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/mostynb@opera.com/431163002/1

[Mostyn Bramley-Moore, 2014-08-02 07:56:29]

The CQ bit was checked by mostynb@opera.com

[commit-bot: I haz the power, 2014-08-02 07:58:28]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/mostynb@opera.com/431163002/1

[commit-bot: I haz the power, 2014-08-02 11:19:03]

Change committed as 287193