Chromium Code Reviews| Index: third_party/WebKit/Source/web/CompositorProxyClientImpl.h |
| diff --git a/third_party/WebKit/Source/web/CompositorProxyClientImpl.h b/third_party/WebKit/Source/web/CompositorProxyClientImpl.h |
| index 3390c4a965356cfb6abb1ee876835b7fa999ae44..fcc3a129c3852bb612a8c93cbb42bfb94f05567f 100644 |
| --- a/third_party/WebKit/Source/web/CompositorProxyClientImpl.h |
| +++ b/third_party/WebKit/Source/web/CompositorProxyClientImpl.h |
| @@ -21,7 +21,7 @@ class WorkerGlobalScope; |
| // mutator, e.g. if a single document creates multiple CompositorWorker objects. |
| // |
| // Should be accessed only on the compositor thread. |
| -class CompositorProxyClientImpl final : public GarbageCollected<CompositorProxyClientImpl>, public CompositorProxyClient { |
| +class CompositorProxyClientImpl final : public GarbageCollectedFinalized<CompositorProxyClientImpl>, public CompositorProxyClient { |
| USING_GARBAGE_COLLECTED_MIXIN(CompositorProxyClientImpl); |
| WTF_MAKE_NONCOPYABLE(CompositorProxyClientImpl); |
| public: |
| @@ -33,6 +33,7 @@ public: |
| bool mutate(double monotonicTimeNow, CompositorMutableStateProvider*); |
| // CompositorProxyClient: |
| + void dispose() override; |
| void setGlobalScope(WorkerGlobalScope*) override; |
| void requestAnimationFrame() override; |
| void registerCompositorProxy(CompositorProxy*) override; |
| @@ -41,9 +42,9 @@ public: |
| private: |
| bool executeAnimationFrameCallbacks(double monotonicTimeNow); |
| - Member<CompositorMutatorImpl> m_mutator; |
| + CrossThreadPersistent<CompositorMutatorImpl> m_mutator; |
| - Member<CompositorWorkerGlobalScope> m_globalScope; |
| + UntracedMember<CompositorWorkerGlobalScope> m_globalScope; |
|
haraken
2016/09/12 11:45:09
Can this be a CrossThreadPersistent?
keishi
2016/09/13 03:12:43
Done.
|
| bool m_requestedAnimationFrameCallbacks; |
| HeapHashSet<WeakMember<CompositorProxy>> m_proxies; |