Don't close background pages if there is a pending response (callback).

chrome/browser/extensions/lazy_background_page_apitest.cc

 // Copyright (c) 2011 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/file_path.h"
 #include "chrome/browser/extensions/browser_action_test_util.h"
 #include "chrome/browser/extensions/extension_apitest.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/profiles/profile.h"
@@ skipping 69 matching lines @@
   ui_test_utils::WindowedNotificationObserver bg_pg_created(
       chrome::NOTIFICATION_EXTENSION_BACKGROUND_PAGE_READY,
       content::NotificationService::AllSources());
   ui_test_utils::WindowedNotificationObserver bg_pg_closed(
       chrome::NOTIFICATION_EXTENSION_HOST_DESTROYED,
       content::NotificationService::AllSources());
   BrowserActionTestUtil(browser()).Press(0);
   bg_pg_created.Wait();
   bg_pg_closed.Wait();
 
-  // Background page is closed before it created a new tab.
-  // TODO(tessamac): Implement! Close background page after callback.
+  // Background page is closed after creating a new tab.
   EXPECT_FALSE(pm->GetBackgroundHostForExtension(last_loaded_extension_id_));
-  EXPECT_EQ(num_tabs_before, browser()->tab_count());
+  EXPECT_EQ(num_tabs_before + 1, browser()->tab_count());
 }
 
 IN_PROC_BROWSER_TEST_F(LazyBackgroundPageApiTest,
                        BroadcastEvent) {
   ASSERT_TRUE(CommandLine::ForCurrentProcess()->HasSwitch(
       switches::kEnableLazyBackgroundPages));
 
   FilePath extdir = test_data_dir_.AppendASCII("lazy_background_page").
       AppendASCII("broadcast_event");
   ASSERT_TRUE(LoadExtension(extdir));
@@ skipping 15 matching lines @@
   //                 cause lazy background pages to be created.
   EXPECT_FALSE(pm->GetBackgroundHostForExtension(last_loaded_extension_id_));
   EXPECT_EQ(num_page_actions,   // should be + 1
             browser()->window()->GetLocationBar()->
             GetLocationBarForTesting()->PageActionVisibleCount());
 }
 
 // TODO: background page with timer.
 // TODO: background page that interacts with popup.
 // TODO: background page with menu.