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

Unified Diff: third_party/WebKit/Source/modules/compositorworker/AnimationWorklet.cpp

Issue 2312493002: [worklets] Introduce ThreadedWorkletMessagingProxy and AnimationWorkletMessagaingProxy. (Closed)
Patch Set: rebase. Created 4 years, 3 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/modules/compositorworker/AnimationWorklet.cpp
diff --git a/third_party/WebKit/Source/modules/compositorworker/AnimationWorklet.cpp b/third_party/WebKit/Source/modules/compositorworker/AnimationWorklet.cpp
index 6f36ac9f30688348eb1babd3202dcd3fa5c7d7fa..5c14e4a1baeeccca3c64b18a418a27e4e10ab960 100644
--- a/third_party/WebKit/Source/modules/compositorworker/AnimationWorklet.cpp
+++ b/third_party/WebKit/Source/modules/compositorworker/AnimationWorklet.cpp
@@ -5,8 +5,10 @@
#include "modules/compositorworker/AnimationWorklet.h"
#include "bindings/core/v8/V8Binding.h"
+#include "core/dom/Document.h"
#include "core/frame/LocalFrame.h"
-#include "core/workers/ThreadedWorkletGlobalScopeProxy.h"
+#include "modules/compositorworker/AnimationWorkletMessagingProxy.h"
+#include "modules/compositorworker/AnimationWorkletThread.h"
namespace blink {
@@ -20,17 +22,36 @@ AnimationWorklet* AnimationWorklet::create(LocalFrame* frame)
AnimationWorklet::AnimationWorklet(LocalFrame* frame)
: Worklet(frame)
- , m_workletGlobalScopeProxy(new ThreadedWorkletGlobalScopeProxy())
+ , m_workletMessagingProxy(nullptr)
{
}
AnimationWorklet::~AnimationWorklet()
{
+ if (m_workletMessagingProxy)
+ m_workletMessagingProxy->parentObjectDestroyed();
+}
+
+void AnimationWorklet::initialize()
+{
+ AnimationWorkletThread::ensureSharedBackingThread();
+
+ DCHECK(!m_workletMessagingProxy);
+ DCHECK(getExecutionContext());
+
+ m_workletMessagingProxy = new AnimationWorkletMessagingProxy(getExecutionContext());
+ m_workletMessagingProxy->initialize();
+}
+
+bool AnimationWorklet::isInitialized() const
+{
+ return m_workletMessagingProxy;
}
WorkletGlobalScopeProxy* AnimationWorklet::workletGlobalScopeProxy() const
{
- return m_workletGlobalScopeProxy.get();
+ DCHECK(m_workletMessagingProxy);
+ return m_workletMessagingProxy;
}
DEFINE_TRACE(AnimationWorklet)

Powered by Google App Engine
This is Rietveld 408576698