Add a scoped notifier for pref dictionaries / lists.

chrome/browser/session_startup_pref.cc

 // Copyright (c) 2006-2008 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.
 
 #include "chrome/browser/session_startup_pref.h"
 
 #include <string>
 
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/defaults.h"
 #include "chrome/browser/pref_service.h"
 #include "chrome/browser/profile.h"
+#include "chrome/browser/scoped_pref_update.h"
 #include "chrome/common/pref_names.h"
 
 namespace {
 
 // For historical reasons the enum and value registered in the prefs don't line
 // up. These are the values registered in prefs.
 const int kPrefValueDefault = 0;
 const int kPrefValueLast = 1;
 const int kPrefValueURLs = 4;
 
@@ skipping 34 matching lines @@
 
 // static
 void SessionStartupPref::SetStartupPref(PrefService* prefs,
                                         const SessionStartupPref& pref) {
   DCHECK(prefs);
   prefs->SetInteger(prefs::kRestoreOnStartup, TypeToPrefValue(pref.type));
 
   // Always save the URLs, that way the UI can remain consistent even if the
   // user changes the startup type pref.
   // Ownership of the ListValue retains with the pref service.
+  ScopedPrefUpdate update(prefs, prefs::kURLsToRestoreOnStartup);
   ListValue* url_pref_list =
       prefs->GetMutableList(prefs::kURLsToRestoreOnStartup);
   DCHECK(url_pref_list);
   url_pref_list->Clear();
   for (size_t i = 0; i < pref.urls.size(); ++i) {
     url_pref_list->Set(static_cast<int>(i),
                        new StringValue(UTF8ToWide(pref.urls[i].spec())));
   }
 }
 
@@ skipping 18 matching lines @@
     Value* value = NULL;
     if (url_pref_list->Get(i, &value)) {
       std::string url_text;
       if (value->GetAsString(&url_text))
         pref.urls.push_back(GURL(url_text));
     }
   }
 
   return pref;
 }