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

Unified Diff: sky/engine/core/html/canvas/WebGLRenderingContextBase.cpp

Issue 735033003: Remove a bunch of dead code from RenderLayer (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Created 6 years, 1 month 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: sky/engine/core/html/canvas/WebGLRenderingContextBase.cpp
diff --git a/sky/engine/core/html/canvas/WebGLRenderingContextBase.cpp b/sky/engine/core/html/canvas/WebGLRenderingContextBase.cpp
index ed20cb1c84d20c3c80c76ff9f6e83afc99c19b6c..163108cd600942d7f33d9566367f183ab78ebf1b 100644
--- a/sky/engine/core/html/canvas/WebGLRenderingContextBase.cpp
+++ b/sky/engine/core/html/canvas/WebGLRenderingContextBase.cpp
@@ -784,7 +784,6 @@ void WebGLRenderingContextBase::markContextChanged(ContentChangeType changeType)
if (renderBox && renderBox->hasAcceleratedCompositing()) {
m_markedCanvasDirty = true;
canvas()->clearCopiedImage();
- renderBox->contentChanged(changeType);
} else {
if (!m_markedCanvasDirty) {
m_markedCanvasDirty = true;
@@ -949,9 +948,7 @@ void WebGLRenderingContextBase::reshape(int width, int height)
height = clamp(height, 1, maxHeight);
if (m_needsUpdate) {
- RenderBox* renderBox = canvas()->renderBox();
- if (renderBox && renderBox->hasAcceleratedCompositing())
- renderBox->contentChanged(CanvasChanged);
+ // This seems wrongs, but was already a no-op because hasAcceleratedCompositing returns false.
ojan 2014/11/29 01:48:15 FIXME(sky) ? I expect this bool doesn't do anythi
m_needsUpdate = false;
}

Powered by Google App Engine
This is Rietveld 408576698