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

ppapi/proxy/plugin_globals.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 PPAPI_PROXY_PLUGIN_GLOBALS_H_
 #define PPAPI_PROXY_PLUGIN_GLOBALS_H_
 
 #include <string>
 
 #include "base/compiler_specific.h"
@@ skipping 55 matching lines @@
   virtual void LogWithSource(PP_Instance instance,
                              PP_LogLevel level,
                              const std::string& source,
                              const std::string& value) OVERRIDE;
   virtual void BroadcastLogWithSource(PP_Module module,
                                       PP_LogLevel level,
                                       const std::string& source,
                                       const std::string& value) OVERRIDE;
   virtual MessageLoopShared* GetCurrentMessageLoop() OVERRIDE;
   base::TaskRunner* GetFileTaskRunner() OVERRIDE;
+  virtual void MarkPluginIsActive() OVERRIDE;
 
   // Returns the channel for sending to the browser.
   IPC::Sender* GetBrowserSender();
 
   // Returns the language code of the current UI language.
   std::string GetUILanguage();
 
   // Sets the active url which is reported by breakpad.
   void SetActiveURL(const std::string& url);
 
@@ skipping 35 matching lines @@
   // long as the plugin.
   MessageLoopResource* loop_for_main_thread();
 
   // The embedder should call this function when the name of the plugin module
   // is known. This will be used for error logging.
   void set_plugin_name(const std::string& name) { plugin_name_ = name; }
 
   // The embedder should call this function when the command line is known.
   void set_command_line(const std::string& c) { command_line_ = c; }
 
+  bool plugin_has_been_active() const { return plugin_has_been_active_; }
+  void set_plugin_has_been_active(bool b) { plugin_has_been_active_ = b; }
+
  private:
   class BrowserSender;
 
   // PpapiGlobals overrides.
   virtual bool IsPluginGlobals() const OVERRIDE;
 
   static PluginGlobals* plugin_globals_;
 
   PluginProxyDelegate* plugin_proxy_delegate_;
   PluginResourceTracker plugin_resource_tracker_;
@@ skipping 12 matching lines @@
   // Command line for the plugin. This will be empty until set_command_line is
   // called.
   std::string command_line_;
 
   scoped_ptr<BrowserSender> browser_sender_;
 
   // Thread for performing potentially blocking file operations. It's created
   // lazily, since it might not be needed.
   scoped_ptr<base::Thread> file_thread_;
 
+  // Indicates activity by the plugin. Used to monitor when a plugin can be
+  // shutdown due to idleness.
+  // TODO(scheib) Mark that this is OK to hit from any thread

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

Virtually all the ppapi/proxy stuff is guarded by ppapi::ProxyLock
(ppapi/shared_impl/proxy_lock.h). So this goes without saying.

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

Done.

+  bool plugin_has_been_active_;
+
   DISALLOW_COPY_AND_ASSIGN(PluginGlobals);
 };
 
 }  // namespace proxy
 }  // namespace ppapi
 
 #endif   // PPAPI_PROXY_PLUGIN_GLOBALS_H_