Fix browser crash when parsing invalid Settings Override extension

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 "base/basictypes.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/values.h"
 #include "chrome/common/extensions/manifest_tests/extension_manifest_test.h"
@@ skipping 30 matching lines @@
       startup_pages->AppendString("http://startup.com/startup.html");
       settings_override->Set("startup_pages", startup_pages.release());
     }
     if (flags & kSearchProvider) {
       scoped_ptr<DictionaryValue> search_provider(new DictionaryValue);
       search_provider->SetString("search_url", "http://google.com/search.html");
       search_provider->SetString("name", "test");
       search_provider->SetString("keyword", "lock");
       search_provider->SetString("encoding", "UTF-8");
       search_provider->SetBoolean("is_default", true);
-      search_provider->SetString("favicon_url", "wikipedia.org/wiki/Favicon");
+      search_provider->SetString("favicon_url",
+                                 "http://wikipedia.org/wiki/Favicon");
       settings_override->Set("search_provider", search_provider.release());
     }
     ext_manifest.Set(
         manifest_keys::kSettingsOverride, settings_override.release());
 
     Manifest manifest(&ext_manifest, "test");
     return LoadAndExpectSuccess(manifest);
   }
 };
 
@@ skipping 66 matching lines @@
 
 
   EXPECT_TRUE(permission_set->HasAPIPermission(APIPermission::kSearchProvider));
   EXPECT_TRUE(permission_set->HasAPIPermission(APIPermission::kHomepage));
   EXPECT_FALSE(permission_set->HasAPIPermission(APIPermission::kStartupPages));
 }
 
 }  // namespace
 
 }  // namespace extensions