socket_dispatcher_host.h uses webrtc, skip when webrtc is disabled

socket_dispatcher_host.h uses webrtc, skip when webrtc is disabled

https://codereview.chromium.org/264793017 added some webrtc specific
code to socket_dispatcher_host.h, and it only appears to be used for
webrtc as far as I can see.  So we should only include this file
elsewhere when webrtc is enabled.

BUG=363459
TEST=build with enable_webrtc=0

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

[Mostyn Bramley-Moore, 2014-06-19 23:09:53]

@jiayl, mallinath2, sky: how does this look to you?

(I don't have access to the referenced bug, but I suspect this CL should be
linked to it.)

[jiayl, 2014-06-19 23:20:18]

On 2014/06/19 23:09:53, Mostyn Bramley-Moore wrote:
> @jiayl, mallinath2, sky: how does this look to you?
> 
> (I don't have access to the referenced bug, but I suspect this CL should be
> linked to it.)

lgtm

[sky, 2014-06-19 23:30:28]

https://codereview.chromium.org/345823003/diff/1/content/browser/renderer_hos...
File content/browser/renderer_host/render_process_host_impl.cc (right):

https://codereview.chromium.org/345823003/diff/1/content/browser/renderer_hos...
content/browser/renderer_host/render_process_host_impl.cc:89: #include
"content/browser/renderer_host/p2p/socket_dispatcher_host.h"
Move this with other ENABLE_WEBRTC includes below.

[Mallinath (Gone from Chromium), 2014-06-20 00:58:51]

On 2014/06/19 23:30:28, sky wrote:
>
https://codereview.chromium.org/345823003/diff/1/content/browser/renderer_hos...
> File content/browser/renderer_host/render_process_host_impl.cc (right):
> 
>
https://codereview.chromium.org/345823003/diff/1/content/browser/renderer_hos...
> content/browser/renderer_host/render_process_host_impl.cc:89: #include
> "content/browser/renderer_host/p2p/socket_dispatcher_host.h"
> Move this with other ENABLE_WEBRTC includes below.

lgtm, if you are address comments from sky.

[Mostyn Bramley-Moore, 2014-06-20 07:50:55]

https://codereview.chromium.org/345823003/diff/1/content/browser/renderer_hos...
File content/browser/renderer_host/render_process_host_impl.cc (right):

https://codereview.chromium.org/345823003/diff/1/content/browser/renderer_hos...
content/browser/renderer_host/render_process_host_impl.cc:89: #include
"content/browser/renderer_host/p2p/socket_dispatcher_host.h"
On 2014/06/19 23:30:28, sky wrote:
> Move this with other ENABLE_WEBRTC includes below.

Done.

[Mostyn Bramley-Moore, 2014-06-25 08:19:28]

@sky: *ping*

[sky, 2014-06-25 16:44:58]

LGTM

[Mostyn Bramley-Moore, 2014-06-25 16:52:10]

The CQ bit was checked by mostynb@opera.com

[commit-bot: I haz the power, 2014-06-25 16:52:51]

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

[commit-bot: I haz the power, 2014-06-25 21:04:45]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_dbg_triggered_tests on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_dbg_triggered...)

[commit-bot: I haz the power, 2014-06-25 22:20:42]

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

[commit-bot: I haz the power, 2014-06-25 22:20:43]

Try jobs failed on following builders:
  android_dbg_triggered_tests on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_dbg_triggered...)

[Mostyn Bramley-Moore, 2014-06-25 22:26:12]

The CQ bit was checked by mostynb@opera.com

[commit-bot: I haz the power, 2014-06-25 22:27:01]

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

[commit-bot: I haz the power, 2014-06-25 23:43:19]

Change committed as 279857