retry r123804 - ntp theme fixes

chrome/browser/themes/theme_service_unittest.cc

Index: chrome/browser/themes/theme_service_unittest.cc
diff --git a/chrome/browser/themes/theme_service_unittest.cc b/chrome/browser/themes/theme_service_unittest.cc
index 7b39c2e06b91324c32a70148d1003d939950fa85..04293bed214474056392133b1f98aa3ee32abca7 100644
--- a/chrome/browser/themes/theme_service_unittest.cc
+++ b/chrome/browser/themes/theme_service_unittest.cc
@@ -9,43 +9,44 @@
 
 TEST(ThemeServiceTest, AlignmentConversion) {
   // Verify that we get out what we put in.
-  std::string top_left = "top left";
+  std::string top_left = "left top";
   int alignment = ThemeService::StringToAlignment(top_left);
   EXPECT_EQ(ThemeService::ALIGN_TOP | ThemeService::ALIGN_LEFT,
             alignment);
   EXPECT_EQ(top_left, ThemeService::AlignmentToString(alignment));
 
+  // We get back a normalized version of what we put in.
   alignment = ThemeService::StringToAlignment("top");
   EXPECT_EQ(ThemeService::ALIGN_TOP, alignment);
-  EXPECT_EQ("top", ThemeService::AlignmentToString(alignment));
+  EXPECT_EQ("center top", ThemeService::AlignmentToString(alignment));

[Evan Stade, 2012/02/28 02:17:23]

we can't make it spit out the same thing we put in, because both "center top"
and "top" will be converted to ALIGN_TOP.

also, we can't leave it how it was, because plain ALIGN_CENTER was converted to
"", which defaults to "left top", not "center center".

 
   alignment = ThemeService::StringToAlignment("left");
   EXPECT_EQ(ThemeService::ALIGN_LEFT, alignment);
-  EXPECT_EQ("left", ThemeService::AlignmentToString(alignment));
+  EXPECT_EQ("left center", ThemeService::AlignmentToString(alignment));
 
   alignment = ThemeService::StringToAlignment("right");
   EXPECT_EQ(ThemeService::ALIGN_RIGHT, alignment);
-  EXPECT_EQ("right", ThemeService::AlignmentToString(alignment));
+  EXPECT_EQ("right center", ThemeService::AlignmentToString(alignment));
 
   alignment = ThemeService::StringToAlignment("righttopbottom");
   EXPECT_EQ(ThemeService::ALIGN_CENTER, alignment);
-  EXPECT_EQ("", ThemeService::AlignmentToString(alignment));
+  EXPECT_EQ("center center", ThemeService::AlignmentToString(alignment));
 }
 
 TEST(ThemeServiceTest, AlignmentConversionInput) {
   // Verify that we output in an expected format.
-  int alignment = ThemeService::StringToAlignment("right bottom");
-  EXPECT_EQ("bottom right", ThemeService::AlignmentToString(alignment));
+  int alignment = ThemeService::StringToAlignment("bottom right");
+  EXPECT_EQ("right bottom", ThemeService::AlignmentToString(alignment));
 
   // Verify that bad strings don't cause explosions.
   alignment = ThemeService::StringToAlignment("new zealand");
-  EXPECT_EQ("", ThemeService::AlignmentToString(alignment));
+  EXPECT_EQ("center center", ThemeService::AlignmentToString(alignment));
 
   // Verify that bad strings don't cause explosions.
   alignment = ThemeService::StringToAlignment("new zealand top");
-  EXPECT_EQ("top", ThemeService::AlignmentToString(alignment));
+  EXPECT_EQ("center top", ThemeService::AlignmentToString(alignment));
 
   // Verify that bad strings don't cause explosions.
   alignment = ThemeService::StringToAlignment("new zealandtop");
-  EXPECT_EQ("", ThemeService::AlignmentToString(alignment));
+  EXPECT_EQ("center center", ThemeService::AlignmentToString(alignment));
 }