Only build PepperVideoSource/DestinationHost when the webrtc is enabled.

content/renderer/pepper/content_renderer_pepper_host_factory.cc

Index: content/renderer/pepper/content_renderer_pepper_host_factory.cc
===================================================================
--- content/renderer/pepper/content_renderer_pepper_host_factory.cc	(revision 198749)
+++ content/renderer/pepper/content_renderer_pepper_host_factory.cc	(working copy)
@@ -115,6 +115,7 @@
     }
   }
 
+#if defined(ENABLE_WEBRTC)
   // Private interfaces.
   if (GetPermissions().HasPermission(ppapi::PERMISSION_PRIVATE)) {
     switch (message.type()) {
@@ -126,6 +127,7 @@
             host_, instance, params.pp_resource()));
     }
   }
+#endif

[bbudge, 2013/05/07 18:00:55]

It would be better if the #if only affected the webrtc resources. We will likely
add more private interfaces here in the future. Is there a problem with an empty
switch statement triggering build errors?

[Ronghua Wu (Left Chromium), 2013/05/07 18:03:48]

What about make that change when we actually have some non-webrtc private
interfaces? It's a bit odd if the switch only has an empty body.

 
   return scoped_ptr<ResourceHost>();
 }