Store rotation/ui scale to local state.

chrome/browser/chromeos/display/display_preferences_unittest.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 "chrome/browser/chromeos/display/display_preferences.h"
 
 #include "ash/display/display_controller.h"
 #include "ash/screen_ash.h"
 #include "ash/shell.h"
 #include "ash/test/ash_test_base.h"
@@ skipping 95 matching lines @@
     local_state_.SetInteger(
         prefs::kSecondaryDisplayLayout, static_cast<int>(layout));
     local_state_.SetInteger(prefs::kSecondaryDisplayOffset, offset);
   }
 
   void StorePrimaryDisplayId(int64 display_id) {
     local_state_.SetInt64(prefs::kPrimaryDisplayID, display_id);
   }
 
   void StoreDisplayOverscan(int64 id, const gfx::Insets& insets) {
-    DictionaryPrefUpdate update(&local_state_, prefs::kDisplayOverscans);
+    DictionaryPrefUpdate update(&local_state_, prefs::kDisplayProperties);
     const std::string name = base::Int64ToString(id);
 
     base::DictionaryValue* pref_data = update.Get();
     base::DictionaryValue* insets_value = new base::DictionaryValue();
-    insets_value->SetInteger("top", insets.top());
-    insets_value->SetInteger("left", insets.left());
-    insets_value->SetInteger("bottom", insets.bottom());
-    insets_value->SetInteger("right", insets.right());
+    insets_value->SetInteger("insets_top", insets.top());
+    insets_value->SetInteger("insets_left", insets.left());
+    insets_value->SetInteger("insets_bottom", insets.bottom());
+    insets_value->SetInteger("insets_right", insets.right());
     pref_data->Set(name, insets_value);
   }
 
   std::string GetRegisteredDisplayLayoutStr(int64 id1, int64 id2) {
     ash::DisplayIdPair pair;
     pair.first = id1;
     pair.second = id2;
     return ash::Shell::GetInstance()->display_controller()->
         GetRegisteredDisplayLayout(pair).ToString();
   }
@@ skipping 15 matching lines @@
   int64 dummy_id = id2 + 1;
   ASSERT_NE(id1, dummy_id);
 
   StoreDisplayLayoutPrefForSecondary(id2, ash::DisplayLayout::BOTTOM, 20);
   StoreDisplayLayoutPrefForSecondary(dummy_id, ash::DisplayLayout::TOP, -10);
   StoreDefaultLayoutPref(ash::DisplayLayout::LEFT, 50);
   StorePrimaryDisplayId(id2);
   StoreDisplayOverscan(id1, gfx::Insets(10, 10, 10, 10));
   StoreDisplayOverscan(id2, gfx::Insets(20, 20, 20, 20));
 
-  NotifyDisplayLocalStatePrefChanged();
+  LoadDisplayPreferences();
   // Check if the layout settings are notified to the system properly.
   ash::DisplayController* display_controller =
       ash::Shell::GetInstance()->display_controller();
   gfx::Screen* screen = gfx::Screen::GetNativeScreen();
   EXPECT_EQ(id2, screen->GetPrimaryDisplay().id());
   // Display was swapped, so the layout was inverted.
   EXPECT_EQ("top, -20",
             display_controller->GetCurrentDisplayLayout().ToString());
 
   EXPECT_EQ("bottom, 20", GetRegisteredDisplayLayoutStr(id1, id2));
   EXPECT_EQ("top, -10", GetRegisteredDisplayLayoutStr(id1, dummy_id));
   EXPECT_EQ("left, 50",
             display_controller->default_display_layout().ToString());
   EXPECT_EQ("160x160", screen->GetPrimaryDisplay().bounds().size().ToString());
   EXPECT_EQ("80x80",
             ash::ScreenAsh::GetSecondaryDisplay().bounds().size().ToString());
 }
 
 TEST_F(DisplayPreferencesTest, PairedLayoutOverrides) {
   UpdateDisplay("100x100,200x200");
   int64 id1 = gfx::Screen::GetNativeScreen()->GetPrimaryDisplay().id();
   int64 id2 = ash::ScreenAsh::GetSecondaryDisplay().id();
   int64 dummy_id = id2 + 1;
   ASSERT_NE(id1, dummy_id);
 
   StoreDisplayLayoutPrefForPair(id1, id2, ash::DisplayLayout::TOP, 20);
   StoreDisplayLayoutPrefForPair(id1, dummy_id, ash::DisplayLayout::LEFT, 30);
   StoreDefaultLayoutPref(ash::DisplayLayout::LEFT, 50);
 
-  NotifyDisplayLocalStatePrefChanged();
+  LoadDisplayPreferences();
   // Check if the layout settings are notified to the system properly.
   // The paired layout overrides old layout.
   ash::DisplayController* display_controller =
       ash::Shell::GetInstance()->display_controller();
   // Inverted one of for specified pair (id1, id2).  Not used for the pair
   // (id1, dummy_id) since dummy_id is not connected right now.
   EXPECT_EQ("top, 20",
             display_controller->GetCurrentDisplayLayout().ToString());
   EXPECT_EQ("top, 20", GetRegisteredDisplayLayoutStr(id1, id2));
   EXPECT_EQ("left, 30", GetRegisteredDisplayLayoutStr(id1, dummy_id));
 }
 
 TEST_F(DisplayPreferencesTest, BasicStores) {
   UpdateDisplay("100x100,200x200");
   int64 id1 = gfx::Screen::GetNativeScreen()->GetPrimaryDisplay().id();
   int64 id2 = ash::ScreenAsh::GetSecondaryDisplay().id();
   int64 dummy_id = id2 + 1;
   ASSERT_NE(id1, dummy_id);
 
   LoggedInAsUser();
   ash::DisplayLayout layout(ash::DisplayLayout::TOP, 10);
   SetCurrentAndDefaultDisplayLayout(layout);
   StoreDisplayLayoutPrefForTest(
       id1, dummy_id, ash::DisplayLayout(ash::DisplayLayout::LEFT, 20));
   // Can't switch to a display that does not exist.
   GetDisplayController()->SetPrimaryDisplayId(dummy_id);
   EXPECT_NE(dummy_id, local_state()->GetInt64(prefs::kPrimaryDisplayID));
 
-  SetAndStoreDisplayOverscan(
-      ash::ScreenAsh::GetNativeScreen()->GetPrimaryDisplay(),
+  ash::Shell::GetInstance()->display_controller()->SetOverscanInsets(
+      ash::ScreenAsh::GetNativeScreen()->GetPrimaryDisplay().id(),
       gfx::Insets(10, 11, 12, 13));
 
   scoped_ptr<base::DictionaryValue> serialized_value(
       new base::DictionaryValue());
   ASSERT_TRUE(ash::DisplayLayout::ConvertToValue(layout,
                                                  serialized_value.get()));
 
   const base::DictionaryValue* displays =
       local_state()->GetDictionary(prefs::kSecondaryDisplays);
   const base::DictionaryValue* display_layout = NULL;
   std::string key = base::Int64ToString(id1) + "," + base::Int64ToString(id2);
   EXPECT_TRUE(displays->GetDictionary(key, &display_layout));
   EXPECT_TRUE(serialized_value->Equals(display_layout));
 
   // The default value is set for the last call of
   // SetCurrentAndDefaultDisplayLayout
   EXPECT_EQ(ash::DisplayLayout::TOP,
             local_state()->GetInteger(prefs::kSecondaryDisplayLayout));
   EXPECT_EQ(10, local_state()->GetInteger(prefs::kSecondaryDisplayOffset));
 
   const base::DictionaryValue* overscans =
-      local_state()->GetDictionary(prefs::kDisplayOverscans);
+      local_state()->GetDictionary(prefs::kDisplayProperties);
   const base::DictionaryValue* overscan = NULL;
   EXPECT_TRUE(overscans->GetDictionary(base::Int64ToString(id1), &overscan));
   int top = 0, left = 0, bottom = 0, right = 0;
-  EXPECT_TRUE(overscan->GetInteger("top", &top));
-  EXPECT_TRUE(overscan->GetInteger("left", &left));
-  EXPECT_TRUE(overscan->GetInteger("bottom", &bottom));
-  EXPECT_TRUE(overscan->GetInteger("right", &right));
+  EXPECT_TRUE(overscan->GetInteger("insets_top", &top));
+  EXPECT_TRUE(overscan->GetInteger("insets_left", &left));
+  EXPECT_TRUE(overscan->GetInteger("insets_bottom", &bottom));
+  EXPECT_TRUE(overscan->GetInteger("insets_right", &right));
   EXPECT_EQ(10, top);
   EXPECT_EQ(11, left);
   EXPECT_EQ(12, bottom);
   EXPECT_EQ(13, right);
 
   GetDisplayController()->SetPrimaryDisplayId(id2);
   EXPECT_EQ(id2, local_state()->GetInt64(prefs::kPrimaryDisplayID));
   // The layout remains the same.
   EXPECT_TRUE(displays->GetDictionary(key, &display_layout));
   EXPECT_TRUE(serialized_value->Equals(display_layout));
@@ skipping 42 matching lines @@
 }
 
 TEST_F(DisplayPreferencesTest, DontStoreInGuestMode) {
   UpdateDisplay("100x100,200x200");
 
   LoggedInAsGuest();
   int64 id2 = ash::ScreenAsh::GetSecondaryDisplay().id();
   ash::DisplayLayout layout(ash::DisplayLayout::TOP, 10);
   SetCurrentAndDefaultDisplayLayout(layout);
   GetDisplayController()->SetPrimaryDisplayId(id2);
-  SetAndStoreDisplayOverscan(
-      ash::ScreenAsh::GetNativeScreen()->GetPrimaryDisplay(),
+  ash::Shell::GetInstance()->display_controller()->SetOverscanInsets(
+      ash::ScreenAsh::GetNativeScreen()->GetPrimaryDisplay().id(),
       gfx::Insets(10, 11, 12, 13));
 
   // Does not store the preferences locally.
   EXPECT_FALSE(local_state()->FindPreference(
       prefs::kSecondaryDisplays)->HasUserSetting());
   EXPECT_FALSE(local_state()->FindPreference(
       prefs::kSecondaryDisplayLayout)->HasUserSetting());
   EXPECT_FALSE(local_state()->FindPreference(
       prefs::kSecondaryDisplayOffset)->HasUserSetting());
   EXPECT_FALSE(local_state()->FindPreference(
       prefs::kPrimaryDisplayID)->HasUserSetting());
   EXPECT_FALSE(local_state()->FindPreference(
-      prefs::kDisplayOverscans)->HasUserSetting());
+      prefs::kDisplayProperties)->HasUserSetting());
 
   // Settings are still notified to the system.
   ash::DisplayController* display_controller =
       ash::Shell::GetInstance()->display_controller();
   gfx::Screen* screen = gfx::Screen::GetNativeScreen();
   EXPECT_EQ(id2, screen->GetPrimaryDisplay().id());
   EXPECT_EQ(ash::DisplayLayout::BOTTOM,
             display_controller->GetCurrentDisplayLayout().position);
   EXPECT_EQ(-10, display_controller->GetCurrentDisplayLayout().offset);
   EXPECT_EQ("176x178", screen->GetPrimaryDisplay().bounds().size().ToString());
 }
 
 }  // namespace
 }  // namespace chromeos