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

Unified Diff: third_party/WebKit/Source/platform/graphics/OffscreenCanvasFrameDispatcherImpl.cpp

Issue 2588053003: Make OffscreenCanvas resizable (Closed)
Patch Set: fix Created 4 years 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/platform/graphics/OffscreenCanvasFrameDispatcherImpl.cpp
diff --git a/third_party/WebKit/Source/platform/graphics/OffscreenCanvasFrameDispatcherImpl.cpp b/third_party/WebKit/Source/platform/graphics/OffscreenCanvasFrameDispatcherImpl.cpp
index f1c74dcbd8e1291dd2d048516575a42535baae9a..27aea5fd3e7bc5ddf555d4f10a9e875fffdd12ee 100644
--- a/third_party/WebKit/Source/platform/graphics/OffscreenCanvasFrameDispatcherImpl.cpp
+++ b/third_party/WebKit/Source/platform/graphics/OffscreenCanvasFrameDispatcherImpl.cpp
@@ -36,6 +36,7 @@ OffscreenCanvasFrameDispatcherImpl::OffscreenCanvasFrameDispatcherImpl(
: m_frameSinkId(cc::FrameSinkId(clientId, sinkId)),
m_width(width),
m_height(height),
+ m_changeSizeForNextCommit(false),
m_nextResourceId(1u),
m_binding(this),
m_placeholderCanvasId(canvasId) {
@@ -357,6 +358,10 @@ void OffscreenCanvasFrameDispatcherImpl::dispatchFrame(
NOTREACHED();
}
+ if (m_changeSizeForNextCommit) {
+ m_currentLocalFrameId = m_surfaceIdAllocator.GenerateId();
+ m_changeSizeForNextCommit = false;
+ }
m_sink->SubmitCompositorFrame(m_currentLocalFrameId, std::move(frame));
}
@@ -406,8 +411,11 @@ bool OffscreenCanvasFrameDispatcherImpl::verifyImageSize(
}
void OffscreenCanvasFrameDispatcherImpl::reshape(int width, int height) {
- m_width = width;
- m_height = height;
+ if (m_width != width || m_height != height) {
+ m_width = width;
+ m_height = height;
+ m_changeSizeForNextCommit = true;
+ }
}
} // namespace blink

Powered by Google App Engine
This is Rietveld 408576698