Change cc::ElementId to be a uint64_t

third_party/WebKit/Source/platform/graphics/paint/EffectPaintPropertyNode.cpp

Index: third_party/WebKit/Source/platform/graphics/paint/EffectPaintPropertyNode.cpp
diff --git a/third_party/WebKit/Source/platform/graphics/paint/EffectPaintPropertyNode.cpp b/third_party/WebKit/Source/platform/graphics/paint/EffectPaintPropertyNode.cpp
index 8ee5702ec6883ceaef2beca929d8d82f37607da1..ad5b72defac095a2d56e2640d5eea4ece095bb1d 100644
--- a/third_party/WebKit/Source/platform/graphics/paint/EffectPaintPropertyNode.cpp
+++ b/third_party/WebKit/Source/platform/graphics/paint/EffectPaintPropertyNode.cpp
@@ -36,12 +36,12 @@ cc::Layer* EffectPaintPropertyNode::EnsureDummyLayer() const {
 String EffectPaintPropertyNode::ToString() const {
   return String::Format(
       "parent=%p localTransformSpace=%p outputClip=%p opacity=%f filter=%s "
-      "blendMode=%s directCompositingReasons=%s compositorElementId=(%d, %d) "
+      "blendMode=%s directCompositingReasons=%s compositorElementId=%lu "
       "paintOffset=%s",
       parent_.Get(), local_transform_space_.Get(), output_clip_.Get(), opacity_,
       filter_.ToString().Ascii().data(), SkBlendMode_Name(blend_mode_),
       CompositingReasonsAsString(direct_compositing_reasons_).Ascii().data(),
-      compositor_element_id_.primaryId, compositor_element_id_.secondaryId,
+      static_cast<unsigned long>(compositor_element_id_.id),

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

Is this cast necessary? Is it not possible to specify a uint64_t value in the
format string?

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

Yes it is, sadly. I ran in to some annoying compile errors in earlier
iterations of the patch because apparently on some platforms uint64 is an
unsigned long long and some an unsigned long. Forcing the static cast
to match the printf % directive makes it work everywhere.

       paint_offset_.ToString().Ascii().data());
 }