Don't ping for extension blacklist when no extensions are installed....

chrome/browser/extensions/extension_browsertests_misc.cc

 // Copyright (c) 2009 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/ref_counted.h"
 #include "chrome/browser/browser.h"
 #include "chrome/browser/browser_list.h"
 #include "chrome/browser/renderer_host/render_view_host.h"
 #include "chrome/browser/extensions/autoupdate_interceptor.h"
 #include "chrome/browser/extensions/extension_browsertest.h"
@@ skipping 601 matching lines @@
   service->EnableExtension(service->disabled_extensions()->at(0)->id());
   EXPECT_EQ(0, contents->infobar_delegate_count());
   EXPECT_EQ(1u, service->extensions()->size());
   EXPECT_EQ(0u, service->disabled_extensions()->size());
 }
 #endif
 
 // Tests that we can uninstall a disabled extension.
 IN_PROC_BROWSER_TEST_F(ExtensionBrowserTest, UninstallDisabled) {
   // Install and upgrade, so that we have a disabled extension.
+  ExtensionsService* service = browser()->profile()->GetExtensionsService();
+  ASSERT_FALSE(service->HasInstalledExtensions());
   ASSERT_TRUE(InstallExtension(
       test_data_dir_.AppendASCII("permissions-low-v1.crx"), 1));
+  ASSERT_TRUE(service->HasInstalledExtensions());
   ASSERT_TRUE(UpdateExtension("pgdpcfcocojkjfbgpiianjngphoopgmo",
       test_data_dir_.AppendASCII("permissions-high-v2.crx"), -1));
 
-  ExtensionsService* service = browser()->profile()->GetExtensionsService();
   EXPECT_EQ(0u, service->extensions()->size());
   ASSERT_EQ(1u, service->disabled_extensions()->size());
+  ASSERT_TRUE(service->HasInstalledExtensions());
 
   // Now try uninstalling it.
   UninstallExtension(service->disabled_extensions()->at(0)->id());
   EXPECT_EQ(0u, service->extensions()->size());
   EXPECT_EQ(0u, service->disabled_extensions()->size());
+  ASSERT_FALSE(service->HasInstalledExtensions());
 }
 
 // Tests that disabling and re-enabling an extension works.
 IN_PROC_BROWSER_TEST_F(ExtensionBrowserTest, DisableEnable) {
   ExtensionsService* service = browser()->profile()->GetExtensionsService();
   ExtensionProcessManager* manager =
       browser()->profile()->GetExtensionProcessManager();
 
   // Load an extension, expect the toolstrip to be available.
+  ASSERT_FALSE(service->HasInstalledExtensions());
   ASSERT_TRUE(LoadExtension(
       test_data_dir_.AppendASCII("good").AppendASCII("Extensions")
                     .AppendASCII("bjafgdebaacbbbecmhlhpofkepfkgcpa")
                     .AppendASCII("1.0")));
   EXPECT_EQ(1u, service->extensions()->size());
   EXPECT_EQ(0u, service->disabled_extensions()->size());
   EXPECT_TRUE(FindHostWithPath(manager, "/toolstrip.html", 1));
+  ASSERT_TRUE(service->HasInstalledExtensions());
 
   // After disabling, the toolstrip should go away.
   service->DisableExtension("bjafgdebaacbbbecmhlhpofkepfkgcpa");
   EXPECT_EQ(0u, service->extensions()->size());
   EXPECT_EQ(1u, service->disabled_extensions()->size());
   EXPECT_FALSE(FindHostWithPath(manager, "/toolstrip.html", 0));
+  ASSERT_TRUE(service->HasInstalledExtensions());
 
   // And bring it back.
   service->EnableExtension("bjafgdebaacbbbecmhlhpofkepfkgcpa");
   EXPECT_EQ(1u, service->extensions()->size());
   EXPECT_EQ(0u, service->disabled_extensions()->size());
   EXPECT_TRUE(FindHostWithPath(manager, "/toolstrip.html", 1));
+  ASSERT_TRUE(service->HasInstalledExtensions());
 }
 
 // Helper function for common code shared by the 3 WindowOpen tests below.
 static TabContents* WindowOpenHelper(Browser* browser, const GURL& start_url,
                                     const std::string& newtab_url) {
   ui_test_utils::NavigateToURL(browser, start_url);
 
   bool result = false;
   ui_test_utils::ExecuteJavaScriptAndExtractBool(
       browser->GetSelectedTabContents()->render_view_host(), L"",
@@ skipping 123 matching lines @@
   // Note: This interceptor gets requests on the IO thread.
   scoped_refptr<AutoUpdateInterceptor> interceptor(new AutoUpdateInterceptor());
   URLFetcher::enable_interception_for_tests(true);
 
   interceptor->SetResponseOnIOThread("http://localhost/autoupdate/manifest",
                                      basedir.AppendASCII("manifest_v2.xml"));
   interceptor->SetResponseOnIOThread("http://localhost/autoupdate/v2.crx",
                                      basedir.AppendASCII("v2.crx"));
 
   // Install version 1 of the extension.
+  ExtensionsService* service = browser()->profile()->GetExtensionsService();
+  ASSERT_FALSE(service->HasInstalledExtensions());
   ASSERT_TRUE(InstallExtension(basedir.AppendASCII("v1.crx"), 1));
-  ExtensionsService* service = browser()->profile()->GetExtensionsService();
   const ExtensionList* extensions = service->extensions();
+  ASSERT_TRUE(service->HasInstalledExtensions());
   ASSERT_EQ(1u, extensions->size());
   ASSERT_EQ("ogjcoiohnmldgjemafoockdghcjciccf", extensions->at(0)->id());
   ASSERT_EQ("1.0", extensions->at(0)->VersionString());
 
   // We don't want autoupdate blacklist checks.
   service->updater()->set_blacklist_checks_enabled(false);
 
   // Run autoupdate and make sure version 2 of the extension was installed.
   service->updater()->CheckNow();
   ASSERT_TRUE(WaitForExtensionInstall());
@@ skipping 50 matching lines @@
 
   // If the options page hasn't already come up, wait for it.
   if (tab_strip->count() == 1) {
     ui_test_utils::WaitForNewTab(browser());
   }
   ASSERT_EQ(2, tab_strip->count());
 
   EXPECT_EQ(extension->GetResourceURL("options.html"),
             tab_strip->GetTabContentsAt(1)->GetURL());
 }