Change cc::ElementId to be a uint64_t

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

 // 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 "platform/graphics/paint/EffectPaintPropertyNode.h"
 
 #include "platform/graphics/paint/PropertyTreeState.h"
 
 namespace blink {
 
@@ skipping 18 matching lines @@
 cc::Layer* EffectPaintPropertyNode::EnsureDummyLayer() const {
   if (dummy_layer_)
     return dummy_layer_.get();
   dummy_layer_ = cc::Layer::Create();
   return dummy_layer_.get();
 }
 
 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());
 }
 
 #if DCHECK_IS_ON()
 
 String EffectPaintPropertyNode::ToTreeString() const {
   return blink::PropertyTreeStatePrinter<blink::EffectPaintPropertyNode>()
       .PathAsString(this);
 }
 
 #endif
 
 }  // namespace blink