Plugin Placeholders: Refactor for platforms that don't support plugins

chrome/renderer/plugins/non_loadable_plugin_placeholder.h

Index: chrome/renderer/plugins/non_loadable_plugin_placeholder.h
diff --git a/chrome/renderer/plugins/non_loadable_plugin_placeholder.h b/chrome/renderer/plugins/non_loadable_plugin_placeholder.h
new file mode 100644
index 0000000000000000000000000000000000000000..af3dc7a499080e86bb4d77a5166832bb67eaacbb
--- /dev/null
+++ b/chrome/renderer/plugins/non_loadable_plugin_placeholder.h
@@ -0,0 +1,35 @@
+// Copyright 2015 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.
+
+#ifndef CHROME_RENDERER_PLUGINS_NON_LOADABLE_PLUGIN_PLACEHOLDER_H_
+#define CHROME_RENDERER_PLUGINS_NON_LOADABLE_PLUGIN_PLACEHOLDER_H_
+
+namespace base {
+class FilePath;
+}
+
+namespace blink {
+class WebLocalFrame;
+struct WebPluginParams;
+}
+
+namespace content {
+class RenderFrame;
+}
+
+namespace plugins {
+class PluginPlaceholder;
+}
+
+// Creates a non-loadable plugin placeholder for platforms without plugins.
+plugins::PluginPlaceholder* CreateMissingPluginPlaceholder(

[Bernhard Bauer, 2015/05/16 00:22:21]

Put this into a (static) class like NonLoadablePluginPlaceholder, to make it
more similar in usage to ChromePluginPlaceholder? It would also mean we didn't
clutter the global namespace.

[tommycli, 2015/05/18 21:12:42]

Done.

+    content::RenderFrame* render_frame,
+    blink::WebLocalFrame* frame,
+    const blink::WebPluginParams& params);
+
+plugins::PluginPlaceholder* CreateErrorPluginPlaceholder(
+    content::RenderFrame* render_frame,
+    const base::FilePath& file_path);
+
+#endif  // CHROME_RENDERER_PLUGINS_NON_LOADABLE_PLUGIN_PLACEHOLDER_H_