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

Unified Diff: cc/trees/proxy_impl.cc

Issue 2470333002: Move PendingTree's UpdateDrawProperties to the back of SubmitCompositorFrame. (Closed)
Patch Set: Created 4 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
« 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/trees/proxy_impl.cc
diff --git a/cc/trees/proxy_impl.cc b/cc/trees/proxy_impl.cc
index 38adc8606f1172b06ab09a3c672f777df58182ae..93f45ec772d34e244bcbc1a0415434f4615fbdae 100644
--- a/cc/trees/proxy_impl.cc
+++ b/cc/trees/proxy_impl.cc
@@ -546,12 +546,6 @@ DrawResult ProxyImpl::DrawInternal(bool forced_draw) {
base::AutoReset<bool> mark_inside(&inside_draw_, true);
- if (layer_tree_host_impl_->pending_tree()) {
- bool update_lcd_text = false;
- layer_tree_host_impl_->pending_tree()->UpdateDrawProperties(
- update_lcd_text);
- }
-
// This method is called on a forced draw, regardless of whether we are able
// to produce a frame, as the calling site on main thread is blocked until its
// request completes, and we signal completion here. If CanDraw() is false, we
@@ -594,6 +588,12 @@ DrawResult ProxyImpl::DrawInternal(bool forced_draw) {
channel_impl_->DidCommitAndDrawFrame();
}
+ if (layer_tree_host_impl_->pending_tree()) {
+ bool update_lcd_text = false;
+ layer_tree_host_impl_->pending_tree()->UpdateDrawProperties(
+ update_lcd_text);
+ }
+
DCHECK_NE(INVALID_RESULT, result);
return result;
}
« 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