mash: Add type names to PropertyConverter::Register*Property.

ui/aura/mus/property_converter_unittest.cc

 // Copyright 2016 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 "ui/aura/mus/property_converter.h"
 
 #include <stdint.h>
 #include <string>
 
 #include "base/logging.h"
@@ skipping 59 matching lines @@
 const char kTestString16PropertyServerKey[] = "test-string16-property-server";
 
 // Test registration, naming and value conversion for primitive property types.
 template <typename T>
 void TestPrimitiveProperty(PropertyConverter* property_converter,
                            Window* window,
                            const WindowProperty<T>* key,
                            const char* transport_name,
                            T value_1,
                            T value_2) {
-  property_converter->RegisterProperty(
+  property_converter->RegisterPrimitiveProperty(
       key, transport_name, PropertyConverter::CreateAcceptAnyValueCallback());
   EXPECT_EQ(transport_name,
             property_converter->GetTransportNameForPropertyKey(key));
   EXPECT_TRUE(property_converter->IsTransportNameRegistered(transport_name));
 
   window->SetProperty(key, value_1);
   EXPECT_EQ(value_1, window->GetProperty(key));
 
   std::string transport_name_out;
   std::unique_ptr<std::vector<uint8_t>> transport_value_out;
@@ skipping 76 matching lines @@
 
   const int64_t value_9a = INT64_MIN / 5, value_9b = INT64_MIN / 6;
   TestPrimitiveProperty(&property_converter, window.get(), kTestPropertyKey9,
                         kTestPropertyServerKey9, value_9a, value_9b);
 }
 
 TEST_F(PropertyConverterTest, TestPrimitiveVerifier) {
   std::unique_ptr<Window> window(CreateNormalWindow(1, root_window(), nullptr));
 
   PropertyConverter property_converter;
-  property_converter.RegisterProperty(kTestPropertyKey8,
-                                      kTestPropertyServerKey8,
-                                      base::Bind(&OnlyAllowNegativeNumbers));
+  property_converter.RegisterPrimitiveProperty(
+      kTestPropertyKey8, kTestPropertyServerKey8,
+      base::Bind(&OnlyAllowNegativeNumbers));
 
   // Test that we reject invalid TransportValues during
   // GetPropertyValueFromTransportValue().
   int64_t int_value = 5;
   std::vector<uint8_t> transport =
       mojo::ConvertTo<std::vector<uint8_t>>(int_value);
   EXPECT_FALSE(property_converter.GetPropertyValueFromTransportValue(
       kTestPropertyServerKey8, transport, &int_value));
 
   // Test that we reject invalid TransportValues during
   // SetPropertyFromTransportValue().
   EXPECT_EQ(-1, window->GetProperty(kTestPropertyKey8));
   property_converter.SetPropertyFromTransportValue(
       window.get(), kTestPropertyServerKey8, &transport);
   EXPECT_EQ(-1, window->GetProperty(kTestPropertyKey8));
 }
 
 // Verifies property setting behavior for a gfx::ImageSkia* property.
 TEST_F(PropertyConverterTest, ImageSkiaProperty) {
   PropertyConverter property_converter;
-  property_converter.RegisterProperty(kTestImagePropertyKey,
-                                      kTestImagePropertyServerKey);
+  property_converter.RegisterImageProperty(kTestImagePropertyKey,
+                                           kTestImagePropertyServerKey);
   EXPECT_EQ(
       kTestImagePropertyServerKey,
       property_converter.GetTransportNameForPropertyKey(kTestImagePropertyKey));
   EXPECT_TRUE(property_converter.IsTransportNameRegistered(
       kTestImagePropertyServerKey));
 
   SkBitmap bitmap_1;
   bitmap_1.allocN32Pixels(16, 32);
   bitmap_1.eraseARGB(255, 11, 22, 33);
   gfx::ImageSkia value_1 = gfx::ImageSkia::CreateFrom1xBitmap(bitmap_1);
@@ skipping 20 matching lines @@
       mojo::ConvertTo<std::vector<uint8_t>>(bitmap_2);
   property_converter.SetPropertyFromTransportValue(
       window.get(), kTestImagePropertyServerKey, &transport_value);
   gfx::ImageSkia* image_out_2 = window->GetProperty(kTestImagePropertyKey);
   EXPECT_TRUE(gfx::BitmapsAreEqual(bitmap_2, *image_out_2->bitmap()));
 }
 
 // Verifies property setting behavior for a gfx::Rect* property.
 TEST_F(PropertyConverterTest, RectProperty) {
   PropertyConverter property_converter;
-  property_converter.RegisterProperty(kTestRectPropertyKey,
-                                      kTestRectPropertyServerKey);
+  property_converter.RegisterRectProperty(kTestRectPropertyKey,
+                                          kTestRectPropertyServerKey);
   EXPECT_EQ(
       kTestRectPropertyServerKey,
       property_converter.GetTransportNameForPropertyKey(kTestRectPropertyKey));
   EXPECT_TRUE(
       property_converter.IsTransportNameRegistered(kTestRectPropertyServerKey));
 
   gfx::Rect value_1(1, 2, 3, 4);
   std::unique_ptr<Window> window(CreateNormalWindow(1, root_window(), nullptr));
   window->SetProperty(kTestRectPropertyKey, new gfx::Rect(value_1));
   EXPECT_EQ(value_1, *window->GetProperty(kTestRectPropertyKey));
@@ skipping 11 matching lines @@
   std::vector<uint8_t> transport_value =
       mojo::ConvertTo<std::vector<uint8_t>>(value_2);
   property_converter.SetPropertyFromTransportValue(
       window.get(), kTestRectPropertyServerKey, &transport_value);
   EXPECT_EQ(value_2, *window->GetProperty(kTestRectPropertyKey));
 }
 
 // Verifies property setting behavior for a gfx::Size* property.
 TEST_F(PropertyConverterTest, SizeProperty) {
   PropertyConverter property_converter;
-  property_converter.RegisterProperty(kTestSizePropertyKey,
-                                      kTestSizePropertyServerKey);
+  property_converter.RegisterSizeProperty(kTestSizePropertyKey,
+                                          kTestSizePropertyServerKey);
   EXPECT_EQ(
       kTestSizePropertyServerKey,
       property_converter.GetTransportNameForPropertyKey(kTestSizePropertyKey));
   EXPECT_TRUE(
       property_converter.IsTransportNameRegistered(kTestSizePropertyServerKey));
 
   gfx::Size value_1(1, 2);
   std::unique_ptr<Window> window(CreateNormalWindow(1, root_window(), nullptr));
   window->SetProperty(kTestSizePropertyKey, new gfx::Size(value_1));
   EXPECT_EQ(value_1, *window->GetProperty(kTestSizePropertyKey));
@@ skipping 11 matching lines @@
   std::vector<uint8_t> transport_value =
       mojo::ConvertTo<std::vector<uint8_t>>(value_2);
   property_converter.SetPropertyFromTransportValue(
       window.get(), kTestSizePropertyServerKey, &transport_value);
   EXPECT_EQ(value_2, *window->GetProperty(kTestSizePropertyKey));
 }
 
 // Verifies property setting behavior for a std::string* property.
 TEST_F(PropertyConverterTest, StringProperty) {
   PropertyConverter property_converter;
-  property_converter.RegisterProperty(kTestStringPropertyKey,
-                                      kTestStringPropertyServerKey);
+  property_converter.RegisterStringProperty(kTestStringPropertyKey,
+                                            kTestStringPropertyServerKey);
   EXPECT_EQ(kTestStringPropertyServerKey,
             property_converter.GetTransportNameForPropertyKey(
                 kTestStringPropertyKey));
   EXPECT_TRUE(property_converter.IsTransportNameRegistered(
       kTestStringPropertyServerKey));
 
   std::string value_1 = "test value";
   std::unique_ptr<Window> window(CreateNormalWindow(1, root_window(), nullptr));
   window->SetProperty(kTestStringPropertyKey, new std::string(value_1));
   EXPECT_EQ(value_1, *window->GetProperty(kTestStringPropertyKey));
@@ skipping 11 matching lines @@
   std::vector<uint8_t> transport_value =
       mojo::ConvertTo<std::vector<uint8_t>>(value_2);
   property_converter.SetPropertyFromTransportValue(
       window.get(), kTestStringPropertyServerKey, &transport_value);
   EXPECT_EQ(value_2, *window->GetProperty(kTestStringPropertyKey));
 }
 
 // Verifies property setting behavior for a base::string16* property.
 TEST_F(PropertyConverterTest, String16Property) {
   PropertyConverter property_converter;
-  property_converter.RegisterProperty(kTestString16PropertyKey,
-                                      kTestString16PropertyServerKey);
+  property_converter.RegisterString16Property(kTestString16PropertyKey,
+                                              kTestString16PropertyServerKey);
   EXPECT_EQ(kTestString16PropertyServerKey,
             property_converter.GetTransportNameForPropertyKey(
                 kTestString16PropertyKey));
   EXPECT_TRUE(property_converter.IsTransportNameRegistered(
       kTestString16PropertyServerKey));
 
   base::string16 value_1 = base::ASCIIToUTF16("test value");
   std::unique_ptr<Window> window(CreateNormalWindow(1, root_window(), nullptr));
   window->SetProperty(kTestString16PropertyKey, new base::string16(value_1));
   EXPECT_EQ(value_1, *window->GetProperty(kTestString16PropertyKey));
 
   std::string transport_name_out;
   std::unique_ptr<std::vector<uint8_t>> transport_value_out;
   EXPECT_TRUE(property_converter.ConvertPropertyForTransport(
       window.get(), kTestString16PropertyKey, &transport_name_out,
       &transport_value_out));
   EXPECT_EQ(kTestString16PropertyServerKey, transport_name_out);
   EXPECT_EQ(mojo::ConvertTo<std::vector<uint8_t>>(value_1),
             *transport_value_out.get());
 
   base::string16 value_2 = base::ASCIIToUTF16("another test value");
   std::vector<uint8_t> transport_value =
       mojo::ConvertTo<std::vector<uint8_t>>(value_2);
   property_converter.SetPropertyFromTransportValue(
       window.get(), kTestString16PropertyServerKey, &transport_value);
   EXPECT_EQ(value_2, *window->GetProperty(kTestString16PropertyKey));
 }
 
 }  // namespace aura