Plugin Power Saver: Make PepperPluginInstanceThrottler interface public.

content/public/renderer/render_frame.h

Index: content/public/renderer/render_frame.h
diff --git a/content/public/renderer/render_frame.h b/content/public/renderer/render_frame.h
index 66cd7d211d2c57ffce208777427313bcd0ddbe41..b520edcf631e2a82a3d057e0156708965997a53b 100644
--- a/content/public/renderer/render_frame.h
+++ b/content/public/renderer/render_frame.h
@@ -36,6 +36,7 @@ class Isolate;
 
 namespace content {
 class ContextMenuClient;
+class PluginInstanceThrottler;
 class RenderView;
 class ServiceRegistry;
 struct ContextMenuParams;
@@ -48,15 +49,6 @@ struct WebPreferences;
 class CONTENT_EXPORT RenderFrame : public IPC::Listener,
                                    public IPC::Sender {
  public:
-  enum PluginPowerSaverMode {
-    // Plugin content is main content, and therefore never throttled.
-    POWER_SAVER_MODE_ESSENTIAL = 0,
-    // Plugin content is peripheral, but throttling is disabled.
-    POWER_SAVER_MODE_PERIPHERAL_UNTHROTTLED = 1,
-    // Plugin content is peripheral, and throttling is enabled.
-    POWER_SAVER_MODE_PERIPHERAL_THROTTLED = 2
-  };
-
   // Returns the RenderFrame given a WebFrame.
   static RenderFrame* FromWebFrame(blink::WebFrame* web_frame);
 
@@ -94,12 +86,12 @@ class CONTENT_EXPORT RenderFrame : public IPC::Listener,
   virtual blink::WebNode GetContextMenuNode() const = 0;
 
   // Create a new NPAPI/Pepper plugin depending on |info|. Returns NULL if no
-  // plugin was found.
+  // plugin was found. |throttler| may be NULL.

[Bernhard Bauer, 2015/01/13 23:29:36]

Nit: s/NULL/nullptr/? :)

[tommycli, 2015/01/14 01:44:59]

Done.

   virtual blink::WebPlugin* CreatePlugin(
       blink::WebFrame* frame,
       const WebPluginInfo& info,
       const blink::WebPluginParams& params,
-      PluginPowerSaverMode power_saver_mode) = 0;
+      PluginInstanceThrottler* throttler) = 0;

[Bernhard Bauer, 2015/01/13 23:29:36]

This takes ownership of |throttler|, right? In which case, pass a scoped_ptr?

[tommycli, 2015/01/14 01:45:00]

Done.

 
   // The client should handle the navigation externally.
   virtual void LoadURLExternally(blink::WebLocalFrame* frame,