Change ui cursor identifiers to an enum class.

services/ui/public/interfaces/cursor/cursor_struct_traits_unittest.cc

Index: services/ui/public/interfaces/cursor/cursor_struct_traits_unittest.cc
diff --git a/services/ui/public/interfaces/cursor/cursor_struct_traits_unittest.cc b/services/ui/public/interfaces/cursor/cursor_struct_traits_unittest.cc
index fa0d041782398e210ca32f6dc815a5bb18722afe..e5b1dd26687953fbe2b224b41d4ecb41d29cbe65 100644
--- a/services/ui/public/interfaces/cursor/cursor_struct_traits_unittest.cc
+++ b/services/ui/public/interfaces/cursor/cursor_struct_traits_unittest.cc
@@ -48,11 +48,12 @@ std::vector<SkBitmap> CreateTestCursorFrames(const gfx::Size& size,
 // Tests numeric cursor ids.
 TEST_F(CursorStructTraitsTest, TestBuiltIn) {
   for (int i = 0; i < 43; ++i) {

[Tom Sepez, 2017/04/24 20:06:44]

nit: is there a symbolic constant for 43 we can use?

[Elliot Glaysher, 2017/04/25 17:16:45]

Done.

-    ui::CursorData input(i);
+    ui::CursorType type = static_cast<ui::CursorType>(i);
+    ui::CursorData input(type);
 
     ui::CursorData output;
     ASSERT_TRUE(EchoCursorData(input, &output));
-    EXPECT_TRUE(output.IsType(i));
+    EXPECT_TRUE(output.IsType(type));
   }
 }
 
@@ -68,7 +69,7 @@ TEST_F(CursorStructTraitsTest, TestBitmapCursor) {
   ui::CursorData output;
   ASSERT_TRUE(EchoCursorData(input, &output));
 
-  EXPECT_EQ(kCursorCustom, output.cursor_type());
+  EXPECT_EQ(CursorType::CUSTOM, output.cursor_type());
   EXPECT_EQ(kScale, output.scale_factor());
   EXPECT_EQ(kFrameDelay, output.frame_delay());
   EXPECT_EQ(kHotspot, output.hotspot_in_pixels());