Protect WebRTC code in content/renderer

Protect WebRTC code in content/renderer

When compiling in Debug mode Win7/MSVS 2010 using
GYP_DEFINES="enable_webrtc=0 component=shared_library"
the compilation fails. This CL ensures these files are
not included when enable_webrtc=0.

BUG=284083
TEST=local compile.

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

[kjellander_chromium, 2013-09-04 13:24:49]

I'm pretty sure those trybot test failures have nothing to do with this change.

[Tommy Widenflycht, 2013-09-05 07:34:41]

lgtm

[commit-bot: I haz the power, 2013-09-05 08:04:48]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kjellander@chromium.org/23749003/1

[kjellander_chromium, 2013-09-05 08:09:17]

James: may I have rubberstamp lg?

[jamesr, 2013-09-06 22:39:57]

lgtm if you fix the include order

https://codereview.chromium.org/23749003/diff/1/content/renderer/render_frame...
File content/renderer/render_frame_impl.cc (right):

https://codereview.chromium.org/23749003/diff/1/content/renderer/render_frame...
content/renderer/render_frame_impl.cc:25: #if defined(ENABLE_WEBRTC)
move this include out below the rest of the block since it's conditional

https://codereview.chromium.org/23749003/diff/1/content/renderer/render_view_...
File content/renderer/render_view_impl.cc (right):

https://codereview.chromium.org/23749003/diff/1/content/renderer/render_view_...
content/renderer/render_view_impl.cc:103: #if defined(ENABLE_WEBRTC)
same here. notice there are already a set of conditional #includes starting
around line 209

[kjellander_chromium, 2013-09-09 09:26:08]

Thanks, sorry for not paying more attention in the first patch set. Now I've
moved the conditional #includes down below the main ones.

https://codereview.chromium.org/23749003/diff/1/content/renderer/render_frame...
File content/renderer/render_frame_impl.cc (right):

https://codereview.chromium.org/23749003/diff/1/content/renderer/render_frame...
content/renderer/render_frame_impl.cc:25: #if defined(ENABLE_WEBRTC)
On 2013/09/06 22:39:57, jamesr wrote:
> move this include out below the rest of the block since it's conditional

Done.

https://codereview.chromium.org/23749003/diff/1/content/renderer/render_view_...
File content/renderer/render_view_impl.cc (right):

https://codereview.chromium.org/23749003/diff/1/content/renderer/render_view_...
content/renderer/render_view_impl.cc:103: #if defined(ENABLE_WEBRTC)
On 2013/09/06 22:39:57, jamesr wrote:
> same here. notice there are already a set of conditional #includes starting
> around line 209

Done.

[commit-bot: I haz the power, 2013-09-09 09:26:18]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kjellander@chromium.org/23749003/13001

[commit-bot: I haz the power, 2013-09-09 12:49:23]

Change committed as 222015