[compositorworker] Register compositor proxies with proxy client

third_party/WebKit/Source/web/CompositorProxyClientImpl.cpp

Index: third_party/WebKit/Source/web/CompositorProxyClientImpl.cpp
diff --git a/third_party/WebKit/Source/web/CompositorProxyClientImpl.cpp b/third_party/WebKit/Source/web/CompositorProxyClientImpl.cpp
index 785f096b53c210fa47430bbbf0c614d36fad27c0..1219cccdf69065dcfee9b2a21e1f1baa13e2718e 100644
--- a/third_party/WebKit/Source/web/CompositorProxyClientImpl.cpp
+++ b/third_party/WebKit/Source/web/CompositorProxyClientImpl.cpp
@@ -23,6 +23,7 @@ DEFINE_TRACE(CompositorProxyClientImpl)
     CompositorProxyClient::trace(visitor);
     visitor->trace(m_mutator);
     visitor->trace(m_globalScope);
+    visitor->trace(m_proxyMap);
 }
 
 void CompositorProxyClientImpl::setGlobalScope(WorkerGlobalScope* scope)
@@ -64,4 +65,38 @@ bool CompositorProxyClientImpl::executeAnimationFrameCallbacks(double monotonicT
     return shouldReinvoke;
 }
 
+void CompositorProxyClientImpl::registerCompositorProxy(CompositorProxy* proxy)
+{
+    uint64_t elementId = proxy->elementId();
+
+    TRACE_EVENT2("compositor-worker,compositor-proxy",

[haraken, 2016/06/08 00:10:36]

registerCompositorProxy should be cheap, so it won't make sense to add a trace
event.

[majidvp, 2016/06/09 20:34:21]

Removed. We were using them as a way to trace the call stack for debugging.

+        "CompositorProxyClientImpl::registerCompositorProxy",
+        "elementId", static_cast<int>(elementId),
+        "proxy", proxy);
+
+    ProxyMap::AddResult entry = m_proxyMap.add(elementId, new ProxyVector);
+    ProxyVector* proxies = entry.storedValue->value;
+    if (!proxies->contains(proxy))
+        proxies->append(proxy);
+}
+
+void CompositorProxyClientImpl::unregisterCompositorProxy(CompositorProxy* proxy)
+{
+    uint64_t elementId = proxy->elementId();
+    if (ProxyVector* proxies = m_proxyMap.get(elementId)) {
+        TRACE_EVENT2("compositor-worker,compositor-proxy",

[haraken, 2016/06/08 00:10:36]

Ditto.

+            "CompositorProxyClientImpl::unregisterCompositorProxy",
+            "elementId", static_cast<int>(elementId),
+            "proxy", proxy);
+
+        size_t position = proxies->find(proxy);
+        if (position == kNotFound)
+            return;
+
+        proxies->remove(position);
+        if (proxies->isEmpty())
+            m_proxyMap.remove(elementId);
+    }
+}
+
 } // namespace blink