Extensions: Only display host name for the overridden home and start-up pages.

chrome/common/extensions/permissions/settings_override_permission_unittest.cc

 // Copyright 2013 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.
 
 // These tests make sure SettingsOverridePermission values are set correctly.
 
 #include <stdint.h>
 
 #include <memory>
 
@@ skipping 31 matching lines @@
 
   scoped_refptr<Extension> GetPermissionSet(uint32_t flags) {
     base::DictionaryValue ext_manifest;
     ext_manifest.SetString(manifest_keys::kName, "test");
     ext_manifest.SetString(manifest_keys::kVersion, "0.1");
     ext_manifest.SetInteger(manifest_keys::kManifestVersion, 2);
 
     std::unique_ptr<base::DictionaryValue> settings_override(
         new base::DictionaryValue);
     if (flags & kHomepage)
-      settings_override->SetString("homepage", "http://www.google.com");
+      settings_override->SetString("homepage", "http://www.google.com/home");
     if (flags & kStartupPages) {
       std::unique_ptr<base::ListValue> startup_pages(new base::ListValue);
       startup_pages->AppendString("http://startup.com/startup.html");
       settings_override->Set("startup_pages", startup_pages.release());
     }
     if (flags & kSearchProvider) {
       std::unique_ptr<base::DictionaryValue> search_provider(
           new base::DictionaryValue);
       search_provider->SetString("search_url", "http://google.com/search.html");
       search_provider->SetString("name", "test");
@@ skipping 18 matching lines @@
 };
 
 TEST_F(SettingsOverridePermissionTest, HomePage) {
   scoped_refptr<Extension> extension(GetPermissionSet(kHomepage));
   const PermissionSet& permission_set =
       extension->permissions_data()->active_permissions();
 
 #if defined(OS_WIN) || defined(OS_MACOSX)
   EXPECT_TRUE(permission_set.HasAPIPermission(APIPermission::kHomepage));
   VerifyOnePermissionMessage(extension->permissions_data(),
-                             "Change your home page to: google.com/");
+                             "Change your home page to: google.com");
 #else
   EXPECT_FALSE(permission_set.HasAPIPermission(APIPermission::kHomepage));
 #endif
 
   EXPECT_FALSE(permission_set.HasAPIPermission(APIPermission::kStartupPages));
   EXPECT_FALSE(permission_set.HasAPIPermission(APIPermission::kSearchProvider));
 }
 
 TEST_F(SettingsOverridePermissionTest, StartupPages) {
   scoped_refptr<Extension> extension(GetPermissionSet(kStartupPages));
   const PermissionSet& permission_set =
       extension->permissions_data()->active_permissions();
 
 #if defined(OS_WIN) || defined(OS_MACOSX)
   EXPECT_TRUE(permission_set.HasAPIPermission(APIPermission::kStartupPages));
-  VerifyOnePermissionMessage(
-      extension->permissions_data(),
-      "Change your start page to: startup.com/startup.html");
+  VerifyOnePermissionMessage(extension->permissions_data(),
+                             "Change your start page to: startup.com");
 #else
   EXPECT_FALSE(permission_set.HasAPIPermission(APIPermission::kStartupPages));
 #endif
 
   EXPECT_FALSE(permission_set.HasAPIPermission(APIPermission::kHomepage));
   EXPECT_FALSE(permission_set.HasAPIPermission(APIPermission::kSearchProvider));
 }
 
 TEST_F(SettingsOverridePermissionTest, SearchSettings) {
   scoped_refptr<Extension> extension(GetPermissionSet(kSearchProvider));
@@ skipping 42 matching lines @@
   EXPECT_FALSE(permission_set.HasAPIPermission(APIPermission::kHomepage));
   EXPECT_FALSE(permission_set.HasAPIPermission(APIPermission::kSearchProvider));
 #endif
 
   EXPECT_FALSE(permission_set.HasAPIPermission(APIPermission::kStartupPages));
 }
 
 }  // namespace
 
 }  // namespace extensions