No longer register app window placement preference keys on the fly.

chrome/browser/prefs/pref_service_browsertest.cc

 // Copyright (c) 2012 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 <string>
 
 #include "base/command_line.h"
 #include "base/files/file_util.h"
 #include "base/files/scoped_temp_dir.h"
 #include "base/json/json_file_value_serializer.h"
 #include "base/path_service.h"
 #include "base/test/test_file_util.h"
 #include "base/values.h"
 #include "build/build_config.h"
+#include "chrome/browser/devtools/devtools_window.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_window.h"
 #include "chrome/browser/ui/browser_window_state.h"
 #include "chrome/common/chrome_constants.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/test/base/in_process_browser_test.h"
 #include "chrome/test/base/test_switches.h"
 #include "chrome/test/base/testing_profile.h"
 #include "chrome/test/base/ui_test_utils.h"
@@ skipping 195 matching lines @@
   EXPECT_EQ(right, bounds.x() + bounds.width());
 
   // Find if launched window is maximized.
   bool is_window_maximized = browser()->window()->IsMaximized();
   bool is_maximized = false;
   EXPECT_TRUE(root_dict->GetBoolean(kBrowserWindowPlacement + ".maximized",
       &is_maximized));
   EXPECT_EQ(is_maximized, is_window_maximized);
 }
 #endif
+
+class AppWindowPlacementIsMigrated : public InProcessBrowserTest {
+ public:
+  virtual bool SetUpUserDataDirectory() OVERRIDE {
+    base::FilePath user_data_directory;
+    PathService::Get(chrome::DIR_USER_DATA, &user_data_directory);
+
+    base::FilePath original_pref_file = ui_test_utils::GetTestFilePath(
+        base::FilePath(FILE_PATH_LITERAL("profiles")).
+                   AppendASCII("app_window_placement").
+                   AppendASCII("Default"),
+        base::FilePath().Append(chrome::kPreferencesFilename));
+    base::FilePath tmp_pref_file =
+        user_data_directory.AppendASCII(TestingProfile::kTestUserProfileDir);
+    CHECK(base::CreateDirectory(tmp_pref_file));
+    tmp_pref_file = tmp_pref_file.Append(chrome::kPreferencesFilename);
+    CHECK(base::PathExists(original_pref_file));
+    CHECK(base::CopyFile(original_pref_file, tmp_pref_file));

[gab, 2014/09/08 21:29:26]

s/tmp_pref_file/test_pref_file ? (it's not really a temp file, it's the file
under test -- which happens to be in a temp dir)

+    return true;
+  }
+};
+
+IN_PROC_BROWSER_TEST_F(AppWindowPlacementIsMigrated, Test) {
+  PrefService* prefs = browser()->profile()->GetPrefs();
+  const base::DictionaryValue* browser_placement =
+      prefs->GetDictionary(prefs::kBrowserWindowPlacement);
+  EXPECT_NE(static_cast<base::DictionaryValue*>(NULL), browser_placement);
+
+  const base::DictionaryValue* app_windows =
+      prefs->GetDictionary(prefs::kAppWindowPlacement);
+  EXPECT_NE(static_cast<base::DictionaryValue*>(NULL), browser_placement);

[gab, 2014/09/08 21:29:25]

Also verify that things in this dictionary appear as expected from the test
file.

+  const base::DictionaryValue* dev_tools_dict = NULL;
+  app_windows->GetDictionary(DevToolsWindow::kDevToolsApp, &dev_tools_dict);
+  int right;
+  EXPECT_TRUE(dev_tools_dict->GetInteger("right", &right));
+  EXPECT_EQ(721, right);
+}

[gab, 2014/09/08 21:29:25]

I think you also want to verify that other kBrowserWindowPlacement + "_*"
entries were deleted.