Allow Plugin placeholders to be shown when plugins are disabled.

chrome/renderer/chrome_content_renderer_client.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 "chrome/renderer/chrome_content_renderer_client.h"
 
 #include <string>
 #include <vector>
 
 #include "base/command_line.h"
@@ skipping 342 matching lines @@
       return false;
     WebDocument document = frame->document();
     const extensions::Extension* extension =
         GetExtension(document.securityOrigin());
     if (extension && extension->HasAPIPermission(
         extensions::APIPermission::kWebView))
       return false;
   }
 
   ChromeViewHostMsg_GetPluginInfo_Output output;
+#if defined(ENABLE_PLUGINS)
   render_view->Send(new ChromeViewHostMsg_GetPluginInfo(
       render_view->GetRoutingID(), GURL(params.url),
       frame->top()->document().url(), orig_mime_type, &output));
+#else
+  output.status.value = ChromeViewHostMsg_GetPluginInfo_Status::kNotFound;

[Bernhard Bauer, 2013/01/03 06:51:37]

Not: I think it would probably not be unreasonable to make kNotFound the default
value in the struct, then you wouldn't need the #else.

[nilesh, 2013/01/03 19:37:59]

It looks like there is code which relies on the default value to be kAllowed
(e.g. PluginInfoMessageFilter::Context::FindEnabledPlugin).

+#endif
   *plugin = CreatePlugin(render_view, frame, params, output);
   return true;
 }
 
 WebPlugin* ChromeContentRendererClient::CreatePluginReplacement(
     content::RenderView* render_view,
     const FilePath& plugin_path) {
   PluginPlaceholder* placeholder =
       PluginPlaceholder::CreateErrorPlugin(render_view, plugin_path);
   return placeholder->plugin();
@@ skipping 643 matching lines @@
 }
 
 void ChromeContentRendererClient::RegisterPPAPIInterfaceFactories(
     webkit::ppapi::PpapiInterfaceFactoryManager* factory_manager) {
 #if defined(ENABLE_PLUGINS)
   factory_manager->RegisterFactory(ChromePPAPIInterfaceFactory);
 #endif
 }
 
 }  // namespace chrome