| Index: chrome/common/extensions/manifest_handlers/settings_overrides_handler.cc
|
| diff --git a/chrome/common/extensions/manifest_handlers/settings_overrides_handler.cc b/chrome/common/extensions/manifest_handlers/settings_overrides_handler.cc
|
| index e6492a7e9b53682a829715784cd0964070a65ad9..357a2a744c21d79f574ce5333a8c7cc318570b16 100644
|
| --- a/chrome/common/extensions/manifest_handlers/settings_overrides_handler.cc
|
| +++ b/chrome/common/extensions/manifest_handlers/settings_overrides_handler.cc
|
| @@ -4,9 +4,9 @@
|
|
|
| #include "chrome/common/extensions/manifest_handlers/settings_overrides_handler.h"
|
|
|
| +#include <memory>
|
| #include <utility>
|
|
|
| -#include "base/memory/scoped_ptr.h"
|
| #include "base/strings/string_util.h"
|
| #include "base/strings/stringprintf.h"
|
| #include "base/strings/utf_string_conversions.h"
|
| @@ -31,19 +31,19 @@ namespace {
|
|
|
| const char kWwwPrefix[] = "www.";
|
|
|
| -scoped_ptr<GURL> CreateManifestURL(const std::string& url) {
|
| - scoped_ptr<GURL> manifest_url(new GURL(url));
|
| +std::unique_ptr<GURL> CreateManifestURL(const std::string& url) {
|
| + std::unique_ptr<GURL> manifest_url(new GURL(url));
|
| if (!manifest_url->is_valid() ||
|
| !manifest_url->SchemeIsHTTPOrHTTPS())
|
| - return scoped_ptr<GURL>();
|
| + return std::unique_ptr<GURL>();
|
| return manifest_url;
|
| }
|
|
|
| -scoped_ptr<GURL> ParseHomepage(const ChromeSettingsOverrides& overrides,
|
| - base::string16* error) {
|
| +std::unique_ptr<GURL> ParseHomepage(const ChromeSettingsOverrides& overrides,
|
| + base::string16* error) {
|
| if (!overrides.homepage)
|
| - return scoped_ptr<GURL>();
|
| - scoped_ptr<GURL> manifest_url = CreateManifestURL(*overrides.homepage);
|
| + return std::unique_ptr<GURL>();
|
| + std::unique_ptr<GURL> manifest_url = CreateManifestURL(*overrides.homepage);
|
| if (!manifest_url) {
|
| *error = extensions::ErrorUtils::FormatErrorMessageUTF16(
|
| manifest_errors::kInvalidHomepageOverrideURL, *overrides.homepage);
|
| @@ -60,7 +60,7 @@ std::vector<GURL> ParseStartupPage(const ChromeSettingsOverrides& overrides,
|
| for (std::vector<std::string>::const_iterator i =
|
| overrides.startup_pages->begin(); i != overrides.startup_pages->end();
|
| ++i) {
|
| - scoped_ptr<GURL> manifest_url = CreateManifestURL(*i);
|
| + std::unique_ptr<GURL> manifest_url = CreateManifestURL(*i);
|
| if (!manifest_url) {
|
| *error = extensions::ErrorUtils::FormatErrorMessageUTF16(
|
| manifest_errors::kInvalidStartupOverrideURL, *i);
|
| @@ -72,16 +72,16 @@ std::vector<GURL> ParseStartupPage(const ChromeSettingsOverrides& overrides,
|
| return urls;
|
| }
|
|
|
| -scoped_ptr<ChromeSettingsOverrides::Search_provider> ParseSearchEngine(
|
| +std::unique_ptr<ChromeSettingsOverrides::Search_provider> ParseSearchEngine(
|
| ChromeSettingsOverrides* overrides,
|
| base::string16* error) {
|
| if (!overrides->search_provider)
|
| - return scoped_ptr<ChromeSettingsOverrides::Search_provider>();
|
| + return std::unique_ptr<ChromeSettingsOverrides::Search_provider>();
|
| if (!CreateManifestURL(overrides->search_provider->search_url)) {
|
| *error = extensions::ErrorUtils::FormatErrorMessageUTF16(
|
| manifest_errors::kInvalidSearchEngineURL,
|
| overrides->search_provider->search_url);
|
| - return scoped_ptr<ChromeSettingsOverrides::Search_provider>();
|
| + return std::unique_ptr<ChromeSettingsOverrides::Search_provider>();
|
| }
|
| if (overrides->search_provider->prepopulated_id)
|
| return std::move(overrides->search_provider);
|
| @@ -91,13 +91,13 @@ scoped_ptr<ChromeSettingsOverrides::Search_provider> ParseSearchEngine(
|
| !overrides->search_provider->favicon_url) {
|
| *error =
|
| base::ASCIIToUTF16(manifest_errors::kInvalidSearchEngineMissingKeys);
|
| - return scoped_ptr<ChromeSettingsOverrides::Search_provider>();
|
| + return std::unique_ptr<ChromeSettingsOverrides::Search_provider>();
|
| }
|
| if (!CreateManifestURL(*overrides->search_provider->favicon_url)) {
|
| *error = extensions::ErrorUtils::FormatErrorMessageUTF16(
|
| manifest_errors::kInvalidSearchEngineURL,
|
| *overrides->search_provider->favicon_url);
|
| - return scoped_ptr<ChromeSettingsOverrides::Search_provider>();
|
| + return std::unique_ptr<ChromeSettingsOverrides::Search_provider>();
|
| }
|
| return std::move(overrides->search_provider);
|
| }
|
| @@ -131,12 +131,12 @@ bool SettingsOverridesHandler::Parse(Extension* extension,
|
| base::string16* error) {
|
| const base::Value* dict = NULL;
|
| CHECK(extension->manifest()->Get(manifest_keys::kSettingsOverride, &dict));
|
| - scoped_ptr<ChromeSettingsOverrides> settings(
|
| + std::unique_ptr<ChromeSettingsOverrides> settings(
|
| ChromeSettingsOverrides::FromValue(*dict, error));
|
| if (!settings)
|
| return false;
|
|
|
| - scoped_ptr<SettingsOverrides> info(new SettingsOverrides);
|
| + std::unique_ptr<SettingsOverrides> info(new SettingsOverrides);
|
| info->homepage = ParseHomepage(*settings, error);
|
| info->search_engine = ParseSearchEngine(settings.get(), error);
|
| info->startup_pages = ParseStartupPage(*settings, error);
|
|
|