Don't expose a LayerImpl reference to CompositorProxy.

third_party/WebKit/Source/core/dom/CompositorProxy.h

 // Copyright 2015 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.
 
 #ifndef CompositorProxy_h
 #define CompositorProxy_h
 
 #include "bindings/core/v8/ScriptWrappable.h"
 #include "core/CoreExport.h"
 #include "core/dom/CompositorProxyClient.h"
@@ skipping 19 matching lines @@
                                  Element*,
                                  const Vector<String>& attributeArray,
                                  ExceptionState&);
   static CompositorProxy* create(ExecutionContext*,
                                  uint64_t element,
                                  uint32_t compositorMutableProperties);
   virtual ~CompositorProxy();
 
   DEFINE_INLINE_TRACE() {}
 
+  uint64_t proxyId() const { return (uint64_t)this; }

[smcgruer, 2017/03/20 14:31:50]

I think this should probably be a static uint64_t that is incremented and
assigned in the constructor, unless flackr has a better idea of how to assign an
id to each proxy.

[Scott Funkenhauser, 2017/03/21 19:31:11]

Done.

   uint64_t elementId() const { return m_elementId; }
   uint32_t compositorMutableProperties() const {
     return m_compositorMutableProperties;
   }
   bool supports(const String& attribute) const;
 
   bool initialized() const { return m_connected && m_state.get(); }
   bool connected() const { return m_connected; }
   void disconnect();
 
@@ skipping 25 matching lines @@
   const uint32_t m_compositorMutableProperties = 0;
 
   bool m_connected = true;
   CrossThreadPersistent<CompositorProxyClient> m_client;
   std::unique_ptr<CompositorMutableState> m_state;
 };
 
 }  // namespace blink
 
 #endif  // CompositorProxy_h