Fix WebUIMessageHandler unit tests.

content/browser/webui/web_ui_message_handler_unittest.cc

Index: content/browser/webui/web_ui_message_handler_unittest.cc
===================================================================
--- content/browser/webui/web_ui_message_handler_unittest.cc	(revision 127566)
+++ content/browser/webui/web_ui_message_handler_unittest.cc	(working copy)
@@ -1,4 +1,4 @@
-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// 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.
 
@@ -14,6 +14,9 @@
 TEST(WebUIMessageHandlerTest, ExtractIntegerValue) {
   ListValue list;
   int value, zero_value = 0, neg_value = -1234, pos_value = 1234;
+  string16 zero_string(UTF8ToUTF16("0"));
+  string16 neg_string(UTF8ToUTF16("-1234"));
+  string16 pos_string(UTF8ToUTF16("1234"));
 
   list.Append(Value::CreateIntegerValue(zero_value));
   EXPECT_TRUE(WebUIMessageHandler::ExtractIntegerValue(&list, &value));
@@ -28,25 +31,58 @@
   list.Append(Value::CreateIntegerValue(pos_value));
   EXPECT_TRUE(WebUIMessageHandler::ExtractIntegerValue(&list, &value));
   EXPECT_EQ(value, pos_value);
+  list.Clear();
+
+  list.Append(Value::CreateStringValue(zero_string));
+  EXPECT_TRUE(WebUIMessageHandler::ExtractIntegerValue(&list, &value));
+  EXPECT_EQ(value, zero_value);
+  list.Clear();
+
+  list.Append(Value::CreateStringValue(neg_string));
+  EXPECT_TRUE(WebUIMessageHandler::ExtractIntegerValue(&list, &value));
+  EXPECT_EQ(value, neg_value);
+  list.Clear();
+
+  list.Append(Value::CreateStringValue(pos_string));
+  EXPECT_TRUE(WebUIMessageHandler::ExtractIntegerValue(&list, &value));
+  EXPECT_EQ(value, pos_value);
 }
 
 TEST(WebUIMessageHandlerTest, ExtractDoubleValue) {
   ListValue list;
   double value, zero_value = 0.0, neg_value = -1234.5, pos_value = 1234.5;
+  string16 zero_string(UTF8ToUTF16("0"));
+  string16 neg_string(UTF8ToUTF16("-1234.5"));
+  string16 pos_string(UTF8ToUTF16("1234.5"));
 
   list.Append(Value::CreateDoubleValue(zero_value));
   EXPECT_TRUE(WebUIMessageHandler::ExtractDoubleValue(&list, &value));
-  EXPECT_EQ(value, zero_value);
+  EXPECT_DOUBLE_EQ(value, zero_value);
   list.Clear();
 
   list.Append(Value::CreateDoubleValue(neg_value));
   EXPECT_TRUE(WebUIMessageHandler::ExtractDoubleValue(&list, &value));
-  EXPECT_EQ(value, neg_value);
+  EXPECT_DOUBLE_EQ(value, neg_value);
   list.Clear();
 
   list.Append(Value::CreateDoubleValue(pos_value));
   EXPECT_TRUE(WebUIMessageHandler::ExtractDoubleValue(&list, &value));
-  EXPECT_EQ(value, pos_value);
+  EXPECT_DOUBLE_EQ(value, pos_value);
+  list.Clear();
+
+  list.Append(Value::CreateStringValue(zero_string));
+  EXPECT_TRUE(WebUIMessageHandler::ExtractDoubleValue(&list, &value));
+  EXPECT_DOUBLE_EQ(value, zero_value);
+  list.Clear();
+
+  list.Append(Value::CreateStringValue(neg_string));
+  EXPECT_TRUE(WebUIMessageHandler::ExtractDoubleValue(&list, &value));
+  EXPECT_DOUBLE_EQ(value, neg_value);
+  list.Clear();
+
+  list.Append(Value::CreateStringValue(pos_string));
+  EXPECT_TRUE(WebUIMessageHandler::ExtractDoubleValue(&list, &value));
+  EXPECT_DOUBLE_EQ(value, pos_value);
 }
 
 TEST(WebUIMessageHandlerTest, ExtractStringValue) {
@@ -55,7 +91,7 @@
       "The facts, though interesting, are irrelevant."));
   list.Append(Value::CreateStringValue(in_string));
   string16 out_string = WebUIMessageHandler::ExtractStringValue(&list);
-  EXPECT_STREQ(in_string.c_str(), out_string.c_str());
+  EXPECT_EQ(in_string, out_string);
 }
 
 }  // namespace content