From 9bfac759ae8db48965e426fb226a948ce5f6234f Mon Sep 17 00:00:00 2001 From: "evan@chromium.org" Date: Mon, 19 Apr 2010 17:26:23 +0000 Subject: extensions: test if the external provider exists before reading it Cut down on some WARNING spam in the common case of no file. BUG=41839 Review URL: http://codereview.chromium.org/1618031 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@44922 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/browser/extensions/external_pref_extension_provider.cc | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) (limited to 'chrome/browser/extensions') diff --git a/chrome/browser/extensions/external_pref_extension_provider.cc b/chrome/browser/extensions/external_pref_extension_provider.cc index c51f4f2..5fe39b4 100644 --- a/chrome/browser/extensions/external_pref_extension_provider.cc +++ b/chrome/browser/extensions/external_pref_extension_provider.cc @@ -23,8 +23,12 @@ ExternalPrefExtensionProvider::ExternalPrefExtensionProvider() { PathService::Get(app::DIR_EXTERNAL_EXTENSIONS, &json_file); json_file = json_file.Append(FILE_PATH_LITERAL("external_extensions.json")); - JSONFileValueSerializer serializer(json_file); - SetPreferences(&serializer); + if (file_util::PathExists(json_file)) { + JSONFileValueSerializer serializer(json_file); + SetPreferences(&serializer); + } else { + prefs_.reset(new DictionaryValue()); + } } ExternalPrefExtensionProvider::~ExternalPrefExtensionProvider() { -- cgit v1.1