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

Side by Side Diff: cc/trees/single_thread_proxy.cc

Issue 935143002: cc: Remove UpdateDrawProperties during sync composite (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@lcd_text_compositor_side_only
Patch Set: Created 5 years, 10 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 unified diff | Download patch
OLDNEW
1 // Copyright 2011 The Chromium Authors. All rights reserved. 1 // Copyright 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "cc/trees/single_thread_proxy.h" 5 #include "cc/trees/single_thread_proxy.h"
6 6
7 #include "base/auto_reset.h" 7 #include "base/auto_reset.h"
8 #include "base/trace_event/trace_event.h" 8 #include "base/trace_event/trace_event.h"
9 #include "cc/debug/benchmark_instrumentation.h" 9 #include "cc/debug/benchmark_instrumentation.h"
10 #include "cc/debug/devtools_instrumentation.h" 10 #include "cc/debug/devtools_instrumentation.h"
(...skipping 514 matching lines...) Expand 10 before | Expand all | Expand 10 after
525 DoCommit(); 525 DoCommit();
526 526
527 DCHECK_EQ(0u, layer_tree_host_->num_queued_swap_promises()) 527 DCHECK_EQ(0u, layer_tree_host_->num_queued_swap_promises())
528 << "Commit should always succeed and transfer promises."; 528 << "Commit should always succeed and transfer promises.";
529 } 529 }
530 530
531 { 531 {
532 DebugScopedSetImplThread impl(const_cast<SingleThreadProxy*>(this)); 532 DebugScopedSetImplThread impl(const_cast<SingleThreadProxy*>(this));
533 if (layer_tree_host_impl_->settings().impl_side_painting) { 533 if (layer_tree_host_impl_->settings().impl_side_painting) {
534 layer_tree_host_impl_->ActivateSyncTree(); 534 layer_tree_host_impl_->ActivateSyncTree();
535 bool update_lcd_text = false; 535 DCHECK(!layer_tree_host_impl_->active_tree()
536 layer_tree_host_impl_->active_tree()->UpdateDrawProperties( 536 ->needs_update_draw_properties());
537 update_lcd_text);
538 layer_tree_host_impl_->PrepareTiles(); 537 layer_tree_host_impl_->PrepareTiles();
539 layer_tree_host_impl_->SynchronouslyInitializeAllTiles(); 538 layer_tree_host_impl_->SynchronouslyInitializeAllTiles();
540 } 539 }
541 540
542 DoAnimate(); 541 DoAnimate();
543 542
544 LayerTreeHostImpl::FrameData frame; 543 LayerTreeHostImpl::FrameData frame;
545 DoComposite(frame_begin_time, &frame); 544 DoComposite(frame_begin_time, &frame);
546 545
547 // DoComposite could abort, but because this is a synchronous composite 546 // DoComposite could abort, but because this is a synchronous composite
(...skipping 284 matching lines...) Expand 10 before | Expand all | Expand 10 after
832 831
833 void SingleThreadProxy::DidBeginImplFrameDeadline() { 832 void SingleThreadProxy::DidBeginImplFrameDeadline() {
834 layer_tree_host_impl_->ResetCurrentBeginFrameArgsForNextFrame(); 833 layer_tree_host_impl_->ResetCurrentBeginFrameArgsForNextFrame();
835 } 834 }
836 835
837 void SingleThreadProxy::SendBeginFramesToChildren(const BeginFrameArgs& args) { 836 void SingleThreadProxy::SendBeginFramesToChildren(const BeginFrameArgs& args) {
838 layer_tree_host_->SendBeginFramesToChildren(args); 837 layer_tree_host_->SendBeginFramesToChildren(args);
839 } 838 }
840 839
841 } // namespace cc 840 } // namespace cc
OLDNEW
« cc/test/layer_tree_pixel_test.cc ('K') | « cc/test/layer_tree_pixel_test.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698