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

Unified Diff: third_party/WebKit/Source/core/offscreencanvas/OffscreenCanvas.cpp

Issue 2318383002: Submit CompositorFrame from worker (Closed)
Patch Set: Remove a TODO comment 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/core/offscreencanvas/OffscreenCanvas.cpp
diff --git a/third_party/WebKit/Source/core/offscreencanvas/OffscreenCanvas.cpp b/third_party/WebKit/Source/core/offscreencanvas/OffscreenCanvas.cpp
index 1d62bf46088754aaa1d7afd728e2f571bcf2cf46..ed310c146db6bf4be137a786f85e66a59b2183b5 100644
--- a/third_party/WebKit/Source/core/offscreencanvas/OffscreenCanvas.cpp
+++ b/third_party/WebKit/Source/core/offscreencanvas/OffscreenCanvas.cpp
@@ -144,8 +144,12 @@ bool OffscreenCanvas::isPaintable() const
OffscreenCanvasFrameDispatcher* OffscreenCanvas::getOrCreateFrameDispatcher()
{
- if (!m_frameDispatcher)
+ if (!m_frameDispatcher) {
+ // The frame dispatcher connects the current thread of OffscreenCanvas
+ // (either main or worker) to browser process and remains unchanged
Justin Novosad 2016/09/07 21:40:15 *the* browser process
xlai (Olivia) 2016/09/08 15:27:15 Done.
+ // throughout the lifetime of this OffscreenCanvas.
m_frameDispatcher = wrapUnique(new OffscreenCanvasFrameDispatcherImpl(m_clientId, m_localId, m_nonce, width(), height()));
+ }
return m_frameDispatcher.get();
}

Powered by Google App Engine
This is Rietveld 408576698