Keep NaCl plugins used in app background pages alive when active.

content/public/browser/browser_ppapi_host.h

 // 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.
 
 #ifndef CONTENT_PUBLIC_BROWSER_BROWSER_PPAPI_HOST_H_
 #define CONTENT_PUBLIC_BROWSER_BROWSER_PPAPI_HOST_H_
 
 #include "base/callback_forward.h"
 #include "base/process/process.h"
 #include "content/common/content_export.h"
 #include "content/public/browser/render_view_host.h"
 #include "ppapi/c/pp_instance.h"
 #include "url/gurl.h"
 
 namespace IPC {
 class ChannelProxy;
 struct ChannelHandle;
 class Sender;
 }
 
 namespace ppapi {
 class PpapiPermissions;
 namespace host {
 class PpapiHost;
 }
 }
 
 namespace content {
 
+struct PepperRendererInstanceData;

[yzshen1, 2013/11/06 22:43:15]

It is not needed here.

[scheib, 2013/11/15 01:22:17]

Done.

+
 // Interface that allows components in the embedder app to talk to the
 // PpapiHost in the browser process.
 //
 // There will be one of these objects in the browser per plugin process. It
 // lives entirely on the I/O thread.
-class CONTENT_EXPORT BrowserPpapiHost {
+class CONTENT_EXPORT BrowserPpapiHost :
+    public base::RefCountedThreadSafe<BrowserPpapiHost> {

[yzshen1, 2013/11/06 22:43:15]

If possible, please don't make it a ref-counted object.

This should only be accessed on the IO thread.

[scheib, 2013/11/15 01:22:17]

Done.

  public:
+  class Delegate {
+   public:
+    virtual ~Delegate() {}
+
+    // Called to handle a plugin instance transitioning in or out of idle state.
+    virtual void OnIdleState(bool idle,
+                             const base::FilePath& profile_data_directory,
+                             const RenderViewHost* render_view_host,
+                             const GURL& document_url) = 0;
+  };
+
   // Creates a browser host and sets up an out-of-process proxy for an external
   // pepper plugin process.
-  static BrowserPpapiHost* CreateExternalPluginProcess(
+  // |delegate| will be owned by the BrowserPpapiHost, is optional and may be
+  // NULL, and is used only for OnIdleState for NaCl extensions.

[yzshen1, 2013/11/06 22:43:15]

A general rule is not to mention "NaCl" in content/, because it is chrome
specific, please consider "external plugin".

[scheib, 2013/11/15 01:22:17]

Done.

+  static scoped_refptr<BrowserPpapiHost> CreateExternalPluginProcess(
+      Delegate* delegate,
       IPC::Sender* sender,
       ppapi::PpapiPermissions permissions,
       base::ProcessHandle plugin_child_process,
       IPC::ChannelProxy* channel,
       int render_process_id,
       int render_view_id,
       const base::FilePath& profile_directory);
 
-  virtual ~BrowserPpapiHost() {}
-
   // Returns the PpapiHost object.
   virtual ppapi::host::PpapiHost* GetPpapiHost() = 0;
 
   // Returns the handle to the plugin process.
   virtual base::ProcessHandle GetPluginProcessHandle() const = 0;
 
   // Returns true if the given PP_Instance is valid.
   virtual bool IsValidInstance(PP_Instance instance) const = 0;
 
   // Retrieves the process/view Ids associated with the RenderView containing
@@ skipping 14 matching lines @@
 
   // Returns the path of the plugin.
   virtual const base::FilePath& GetPluginPath() = 0;
 
   // Returns the user's profile data directory.
   virtual const base::FilePath& GetProfileDataDirectory() = 0;
 
   // Get the Document/Plugin URLs for the given PP_Instance.
   virtual GURL GetDocumentURLForInstance(PP_Instance instance) = 0;
   virtual GURL GetPluginURLForInstance(PP_Instance instance) = 0;
+
+ protected:
+  virtual ~BrowserPpapiHost() {}
+
+ private:
+  friend class base::RefCountedThreadSafe<BrowserPpapiHost>;
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_BROWSER_PPAPI_HOST_H_