Don't mark JsonPrefStore as initialized in case of fatal failure.

chrome/common/json_pref_store.cc

Index: chrome/common/json_pref_store.cc
diff --git a/chrome/common/json_pref_store.cc b/chrome/common/json_pref_store.cc
index 65f4e107ddd889cc51f42c04a6e4884939459e95..ed30346ff048190d9ad004b210f14be3ffca8cf7 100644
--- a/chrome/common/json_pref_store.cc
+++ b/chrome/common/json_pref_store.cc
@@ -264,7 +264,6 @@ void JsonPrefStore::OnFileRead(Value* value_owned,
                                PersistentPrefStore::PrefReadError error,
                                bool no_dir) {
   scoped_ptr<Value> value(value_owned);
-  initialized_ = true;
   read_error_ = error;
 
   if (no_dir) {
@@ -274,6 +273,8 @@ void JsonPrefStore::OnFileRead(Value* value_owned,
     return;
   }
 
+  initialized_ = true;

[Mattias Nissler (ping if slow), 2012/05/10 15:44:04]

Good catch.

+
   switch (error) {
     case PREF_READ_ERROR_ACCESS_DENIED:
     case PREF_READ_ERROR_FILE_OTHER: