Only build PepperVideoSource/DestinationHost when the webrtc is enabled.

content/renderer/pepper/content_renderer_pepper_host_factory.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/renderer/pepper/content_renderer_pepper_host_factory.h"
 
 #include "base/logging.h"
 #include "base/string_util.h"
 #include "content/renderer/pepper/pepper_audio_input_host.h"
 #include "content/renderer/pepper/pepper_file_chooser_host.h"
@@ skipping 97 matching lines @@
             host_, instance, params.pp_resource());
         if (!host->Init()) {
           delete host;
           return scoped_ptr<ResourceHost>();
         }
         return scoped_ptr<ResourceHost>(host);
       }
     }
   }
 
+#if defined(ENABLE_WEBRTC)
   // Private interfaces.
   if (GetPermissions().HasPermission(ppapi::PERMISSION_PRIVATE)) {
     switch (message.type()) {
       case PpapiHostMsg_VideoDestination_Create::ID:
         return scoped_ptr<ResourceHost>(new PepperVideoDestinationHost(
             host_, instance, params.pp_resource()));
       case PpapiHostMsg_VideoSource_Create::ID:
         return scoped_ptr<ResourceHost>(new PepperVideoSourceHost(
             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>();
 }
 
 const ppapi::PpapiPermissions&
 ContentRendererPepperHostFactory::GetPermissions() const {
   return host_->GetPpapiHost()->permissions();
 }
 
 }  // namespace content