Don't keep the lazy background page alive after a view with a Pepper plugin is closed

chrome/browser/extensions/lazy_background_page_apitest.cc

 // 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.
 
 #include "base/command_line.h"
 #include "base/files/file_path.h"
 #include "base/path_service.h"
 #include "base/scoped_observer.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/bookmarks/bookmark_model_factory.h"
@@ skipping 292 matching lines @@
   EXPECT_TRUE(result);
   page_complete.Wait();
 
   // Lazy Background Page has been shut down.
   EXPECT_FALSE(pm->GetBackgroundHostForExtension(last_loaded_extension_id()));
 }
 
 // Tests that the lazy background page stays alive while a NaCl module exists in
 // its DOM.
 #if !defined(DISABLE_NACL) && !defined(DISABLE_NACL_BROWSERTESTS)
-IN_PROC_BROWSER_TEST_F(LazyBackgroundPageApiTest, NaCl) {
+
+IN_PROC_BROWSER_TEST_F(LazyBackgroundPageApiTest, NaClInBackgroundPage) {
   {
     base::FilePath extdir;
     ASSERT_TRUE(PathService::Get(chrome::DIR_GEN_TEST_DATA, &extdir));
     extdir = extdir.AppendASCII("ppapi/tests/extensions/load_unload/newlib");
     LazyBackgroundObserver page_complete;
     ASSERT_TRUE(LoadExtension(extdir));
     page_complete.Wait();
   }
 
   // The NaCl module is loaded, and the Lazy Background Page stays alive.
@@ skipping 10 matching lines @@
   // down.
   {
     LazyBackgroundObserver page_complete;
     BrowserActionTestUtil(browser()).Press(0);
     page_complete.WaitUntilClosed();
   }
 
   // The Lazy Background Page has been shut down.
   EXPECT_FALSE(IsBackgroundPageAlive(last_loaded_extension_id()));
 }
+
+// Tests that the lazy background page shuts down when all visible views with
+// NaCl modules are closed.
+IN_PROC_BROWSER_TEST_F(LazyBackgroundPageApiTest, NaClInView) {
+  // The extension is loaded and should've opened a new tab to an extension
+  // page, and the Lazy Background Page stays alive.
+  {
+    base::FilePath extdir;
+    ASSERT_TRUE(PathService::Get(chrome::DIR_GEN_TEST_DATA, &extdir));
+    extdir = extdir.AppendASCII("ppapi/tests/extensions/popup/newlib");
+    ResultCatcher catcher;
+    const Extension* extension = LoadExtension(extdir);
+    ASSERT_TRUE(extension);
+    EXPECT_TRUE(catcher.GetNextResult()) << catcher.message();
+    EXPECT_EQ(
+        extension->GetResourceURL("popup.html").spec(),
+        browser()->tab_strip_model()->GetActiveWebContents()->GetURL().spec());
+    EXPECT_TRUE(IsBackgroundPageAlive(last_loaded_extension_id()));
+  }
+
+  // Close the new tab.
+  {
+    LazyBackgroundObserver page_complete;
+    browser()->tab_strip_model()->CloseWebContentsAt(
+        browser()->tab_strip_model()->active_index(),
+        TabStripModel::CLOSE_NONE);
+    page_complete.WaitUntilClosed();
+  }
+
+  // The Lazy Background Page has been shut down.
+  EXPECT_FALSE(IsBackgroundPageAlive(last_loaded_extension_id()));
+}
 #endif
 
 // Tests that the lazy background page stays alive until all visible views are
 // closed.
 // http://crbug.com/175778; test fails frequently on OS X
 #if defined(OS_MACOSX)
 #define MAYBE_WaitForNTP DISABLED_WaitForNTP
 #else
 #define MAYBE_WaitForNTP WaitForNTP
 #endif
@@ skipping 262 matching lines @@
 // handler calls an API function such as chrome.storage.local.set().
 // See: http://crbug.com/296834
 IN_PROC_BROWSER_TEST_F(LazyBackgroundPageApiTest, OnSuspendUseStorageApi) {
   EXPECT_TRUE(LoadExtensionAndWait("on_suspend"));
 }
 
 // TODO: background page with timer.
 // TODO: background page that interacts with popup.
 
 }  // namespace extensions