Remove user-related data from local_state and add to user_preferences, i...

chrome/browser/prefs/browser_prefs.cc

 // Copyright (c) 2011 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/prefs/browser_prefs.h"
 
 #include "chrome/browser/about_flags.h"
 #include "chrome/browser/autofill/autofill_manager.h"
 #include "chrome/browser/background_contents_service.h"
 #include "chrome/browser/background_mode_manager.h"
@@ skipping 35 matching lines @@
 #include "chrome/browser/search_engines/template_url_model.h"
 #include "chrome/browser/search_engines/template_url_prepopulate_data.h"
 #include "chrome/browser/ssl/ssl_manager.h"
 #include "chrome/browser/sync/signin_manager.h"
 #include "chrome/browser/tab_contents/tab_contents.h"
 #include "chrome/browser/tabs/pinned_tab_codec.h"
 #include "chrome/browser/task_manager/task_manager.h"
 #include "chrome/browser/translate/translate_prefs.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/upgrade_detector.h"
+#include "chrome/common/pref_names.h"
 
 #if defined(TOOLKIT_VIEWS)  // TODO(port): whittle this down as we port
 #include "chrome/browser/ui/views/browser_actions_container.h"
 #include "chrome/browser/ui/views/frame/browser_view.h"
 #endif
 
 #if defined(TOOLKIT_GTK)
 #include "chrome/browser/ui/gtk/browser_window_gtk.h"
 #endif
 
@@ skipping 82 matching lines @@
   chromeos::Preferences::RegisterUserPrefs(user_prefs);
 #endif
   BackgroundContentsService::RegisterUserPrefs(user_prefs);
   SigninManager::RegisterUserPrefs(user_prefs);
   TemplateURLModel::RegisterUserPrefs(user_prefs);
   InstantController::RegisterUserPrefs(user_prefs);
   NetPrefObserver::RegisterPrefs(user_prefs);
   policy::ProfilePolicyContext::RegisterUserPrefs(user_prefs);
 }
 
+void MigrateBrowserPrefs(PrefService* user_prefs, PrefService* local_state) {
+  // Copy pref values which have been migrated to user_prefs from local_state,
+  // or remove them from local_state outright, if copying is not required.
+  int current_version =
+      local_state->GetInteger(prefs::kMultipleProfilePrefMigration);
+
+  if ((current_version & WINDOWS_PREFS) == 0) {
+    // Migrate the devtools split location preference.
+    local_state->RegisterIntegerPref(prefs::kDevToolsSplitLocation, -1);
+    DCHECK(user_prefs->FindPreference(prefs::kDevToolsSplitLocation));
+    if (local_state->HasPrefPath(prefs::kDevToolsSplitLocation)) {
+      user_prefs->SetInteger(prefs::kDevToolsSplitLocation,
+          local_state->GetInteger(prefs::kDevToolsSplitLocation));
+    }
+    local_state->ClearPref(prefs::kDevToolsSplitLocation);
+
+    // Migrate the browser window placement preference.
+    local_state->RegisterDictionaryPref(prefs::kBrowserWindowPlacement);
+    DCHECK(user_prefs->FindPreference(prefs::kBrowserWindowPlacement));
+    if (local_state->HasPrefPath(prefs::kBrowserWindowPlacement)) {
+      user_prefs->Set(prefs::kBrowserWindowPlacement,
+          *(local_state->FindPreference(prefs::kBrowserWindowPlacement)->
+              GetValue()->DeepCopy()));
+    }
+    local_state->ClearPref(prefs::kBrowserWindowPlacement);
+
+    local_state->SetInteger(prefs::kMultipleProfilePrefMigration,
+                            current_version | WINDOWS_PREFS);
+  }
+}
+
 }  // namespace browser