Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(592)

Side by Side Diff: chrome/renderer/plugins/non_loadable_plugin_placeholder.h

Issue 2855123003: Remove rendundant WebLocalFrame parameter in various plugin code. (Closed)
Patch Set: Fix Android Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_RENDERER_PLUGINS_NON_LOADABLE_PLUGIN_PLACEHOLDER_H_ 5 #ifndef CHROME_RENDERER_PLUGINS_NON_LOADABLE_PLUGIN_PLACEHOLDER_H_
6 #define CHROME_RENDERER_PLUGINS_NON_LOADABLE_PLUGIN_PLACEHOLDER_H_ 6 #define CHROME_RENDERER_PLUGINS_NON_LOADABLE_PLUGIN_PLACEHOLDER_H_
7 7
8 #include "base/macros.h" 8 #include "base/macros.h"
9 9
10 namespace base { 10 namespace base {
11 class FilePath; 11 class FilePath;
12 } 12 }
13 13
14 namespace blink { 14 namespace blink {
15 class WebLocalFrame;
16 struct WebPluginParams; 15 struct WebPluginParams;
17 } 16 }
18 17
19 namespace content { 18 namespace content {
20 class RenderFrame; 19 class RenderFrame;
21 } 20 }
22 21
23 namespace plugins { 22 namespace plugins {
24 class PluginPlaceholder; 23 class PluginPlaceholder;
25 } 24 }
26 25
27 class NonLoadablePluginPlaceholder { 26 class NonLoadablePluginPlaceholder {
28 public: 27 public:
29 // Creates a non-loadable plugin placeholder for platforms without plugins. 28 // Creates a non-loadable plugin placeholder for platforms without plugins.
30 static plugins::PluginPlaceholder* CreateNotSupportedPlugin( 29 static plugins::PluginPlaceholder* CreateNotSupportedPlugin(
31 content::RenderFrame* render_frame, 30 content::RenderFrame* render_frame,
32 blink::WebLocalFrame* frame,
33 const blink::WebPluginParams& params); 31 const blink::WebPluginParams& params);
34 32
35 static plugins::PluginPlaceholder* CreateErrorPlugin( 33 static plugins::PluginPlaceholder* CreateErrorPlugin(
36 content::RenderFrame* render_frame, 34 content::RenderFrame* render_frame,
37 const base::FilePath& file_path); 35 const base::FilePath& file_path);
38 36
39 private: 37 private:
40 DISALLOW_IMPLICIT_CONSTRUCTORS(NonLoadablePluginPlaceholder); 38 DISALLOW_IMPLICIT_CONSTRUCTORS(NonLoadablePluginPlaceholder);
41 }; 39 };
42 40
43 #endif // CHROME_RENDERER_PLUGINS_NON_LOADABLE_PLUGIN_PLACEHOLDER_H_ 41 #endif // CHROME_RENDERER_PLUGINS_NON_LOADABLE_PLUGIN_PLACEHOLDER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698