Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(188)

Unified Diff: third_party/WebKit/Source/core/dom/CompositorProxyClient.h

Issue 1900423004: [compositorworker] Register compositor proxies with proxy client (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: replace map of set with a set Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/dom/CompositorProxyClient.h
diff --git a/third_party/WebKit/Source/core/dom/CompositorProxyClient.h b/third_party/WebKit/Source/core/dom/CompositorProxyClient.h
index 9f9cea1630df55fab6cf18a20aba27a882066311..d5d5ca9615301f163cb27066698f0b20999bf55e 100644
--- a/third_party/WebKit/Source/core/dom/CompositorProxyClient.h
+++ b/third_party/WebKit/Source/core/dom/CompositorProxyClient.h
@@ -30,6 +30,12 @@ public:
virtual void setGlobalScope(WorkerGlobalScope*) = 0;
virtual void requestAnimationFrame() = 0;
+ virtual void registerCompositorProxy(CompositorProxy*) = 0;
+ // It is not guaranteed to receive an unregister call for every registered
+ // proxy. In fact we only receive one when a proxy is explicitly
+ // disconnected otherwise we rely on oilpan collection process to remove the
+ // proxy from our weak map.
jbroman 2016/06/13 17:15:05 There is no longer a weak map. Either be vague ("t
majidvp 2016/06/13 17:21:42 Done.
+ virtual void unregisterCompositorProxy(CompositorProxy*) = 0;
};
CORE_EXPORT void provideCompositorProxyClientTo(WorkerClients*, CompositorProxyClient*);

Powered by Google App Engine
This is Rietveld 408576698