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;
+
 // 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> {
  public:
+  class Delegate {

[dmichael (off chromium), 2013/11/06 22:39:21]

How about instead having something like this on BrowserPpapiHost:
virtual void SetIdleStateChangeCallback(
    const base::Callback<void (bool, /* idle */
                               base::FilePath, /* profile_data_dir */
                               const RenderViewHost*,
                               GURL /*document_url*/)>& callback) = 0;

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

Done.

+   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.
+  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() {}
 
@@ skipping 29 matching lines @@
   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;
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_BROWSER_PPAPI_HOST_H_