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

Unified Diff: cc/layer_tree_host.cc

Issue 12185008: cc: Disable partial updates when impl-side painting is used. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 11 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/layer_tree_host.cc
diff --git a/cc/layer_tree_host.cc b/cc/layer_tree_host.cc
index 74d3cae3e4f7febc4049be9e91e0e3e34a9c6fdf..7ce7eb2c4aa6a81c4bf16ac02873f4e2c6d4ff28 100644
--- a/cc/layer_tree_host.cc
+++ b/cc/layer_tree_host.cc
@@ -157,7 +157,7 @@ void LayerTreeHost::initializeRenderer()
// Update m_settings based on partial update capability.
size_t maxPartialTextureUpdates = 0;
- if (m_proxy->rendererCapabilities().allowPartialTextureUpdates)
+ if (m_proxy->rendererCapabilities().allowPartialTextureUpdates && !m_settings.implSidePainting)
maxPartialTextureUpdates = std::min(m_settings.maxPartialTextureUpdates, m_proxy->maxPartialTextureUpdates());
danakj 2013/02/03 18:24:15 Would it be better to have the embedder set maxPar
enne (OOO) 2013/02/11 20:01:57 I disagree; the caller is specifying a maximum, no
m_settings.maxPartialTextureUpdates = maxPartialTextureUpdates;
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698