Change cc::ElementId to be a uint64_t

third_party/WebKit/Source/web/tests/VisualViewportTest.cpp

Index: third_party/WebKit/Source/web/tests/VisualViewportTest.cpp
diff --git a/third_party/WebKit/Source/web/tests/VisualViewportTest.cpp b/third_party/WebKit/Source/web/tests/VisualViewportTest.cpp
index cd28ae53a6ec76c93cba0289fb43813867d827d2..fce377163161d019eee4ee09c7cb442f8e3e3809 100644
--- a/third_party/WebKit/Source/web/tests/VisualViewportTest.cpp
+++ b/third_party/WebKit/Source/web/tests/VisualViewportTest.cpp
@@ -791,8 +791,9 @@ TEST_P(VisualViewportTest, TestAttachingNewFrameSetsInnerScrollLayerSize) {
   // Ensure the scroll layer matches the frame view's size.
   EXPECT_SIZE_EQ(FloatSize(320, 240), visualViewport.ScrollLayer()->Size());
 
-  EXPECT_EQ(static_cast<int>(CompositorSubElementId::kViewport),
-            visualViewport.ScrollLayer()->GetElementId().secondaryId);
+  EXPECT_EQ(static_cast<uint64_t>(CompositorSubElementId::kViewport),

[suzyh_UTC10 (ex-contributor), 2017/05/08 01:04:15]

Is this cast necessary?

[chrishtr, 2017/05/08 16:32:13]

Not sure. Compiles locally.. Removed it for now, will see what the bots say.

+            SubElementIdFromCompositorElementId(
+                visualViewport.ScrollLayer()->GetElementId()));
 
   // Ensure the location and scale were reset.
   EXPECT_SIZE_EQ(FloatSize(), visualViewport.GetScrollOffset());