| Index: cc/trees/single_thread_proxy.cc
|
| diff --git a/cc/trees/single_thread_proxy.cc b/cc/trees/single_thread_proxy.cc
|
| index 725d6123d4781f44b26e6ec0ca89b5cde05cb584..4ce74a6dd718ca51ca6e736e81e05153d649dcce 100644
|
| --- a/cc/trees/single_thread_proxy.cc
|
| +++ b/cc/trees/single_thread_proxy.cc
|
| @@ -188,21 +188,12 @@ void SingleThreadProxy::DoCommit() {
|
| TRACE_EVENT0("cc", "SingleThreadProxy::DoCommit");
|
| DCHECK(task_runner_provider_->IsMainThread());
|
|
|
| - // TODO(robliao): Remove ScopedTracker below once https://crbug.com/461509 is
|
| - // fixed.
|
| - tracked_objects::ScopedTracker tracking_profile1(
|
| - FROM_HERE_WITH_EXPLICIT_FUNCTION("461509 SingleThreadProxy::DoCommit1"));
|
| layer_tree_host_->WillCommit();
|
| devtools_instrumentation::ScopedCommitTrace commit_task(
|
| layer_tree_host_->id());
|
|
|
| // Commit immediately.
|
| {
|
| - // TODO(robliao): Remove ScopedTracker below once https://crbug.com/461509
|
| - // is fixed.
|
| - tracked_objects::ScopedTracker tracking_profile2(
|
| - FROM_HERE_WITH_EXPLICIT_FUNCTION(
|
| - "461509 SingleThreadProxy::DoCommit2"));
|
| DebugScopedSetMainThreadBlocked main_thread_blocked(task_runner_provider_);
|
| DebugScopedSetImplThread impl(task_runner_provider_);
|
|
|
| @@ -215,19 +206,9 @@ void SingleThreadProxy::DoCommit() {
|
| layer_tree_host_impl_->ReadyToCommit();
|
| layer_tree_host_impl_->BeginCommit();
|
|
|
| - // TODO(robliao): Remove ScopedTracker below once https://crbug.com/461509
|
| - // is fixed.
|
| - tracked_objects::ScopedTracker tracking_profile6(
|
| - FROM_HERE_WITH_EXPLICIT_FUNCTION(
|
| - "461509 SingleThreadProxy::DoCommit6"));
|
| if (layer_tree_host_impl_->EvictedUIResourcesExist())
|
| layer_tree_host_->RecreateUIResources();
|
|
|
| - // TODO(robliao): Remove ScopedTracker below once https://crbug.com/461509
|
| - // is fixed.
|
| - tracked_objects::ScopedTracker tracking_profile7(
|
| - FROM_HERE_WITH_EXPLICIT_FUNCTION(
|
| - "461509 SingleThreadProxy::DoCommit7"));
|
| layer_tree_host_->FinishCommitOnImplThread(layer_tree_host_impl_.get());
|
|
|
| if (scheduler_on_impl_thread_)
|
| @@ -235,11 +216,6 @@ void SingleThreadProxy::DoCommit() {
|
|
|
| layer_tree_host_impl_->CommitComplete();
|
|
|
| - // TODO(robliao): Remove ScopedTracker below once https://crbug.com/461509
|
| - // is fixed.
|
| - tracked_objects::ScopedTracker tracking_profile8(
|
| - FROM_HERE_WITH_EXPLICIT_FUNCTION(
|
| - "461509 SingleThreadProxy::DoCommit8"));
|
| // Commit goes directly to the active tree, but we need to synchronously
|
| // "activate" the tree still during commit to satisfy any potential
|
| // SetNextCommitWaitsForActivation calls. Unfortunately, the tree
|
| @@ -575,12 +551,6 @@ DrawResult SingleThreadProxy::DoComposite(LayerTreeHostImpl::FrameData* frame) {
|
| DebugScopedSetImplThread impl(task_runner_provider_);
|
| base::AutoReset<bool> mark_inside(&inside_draw_, true);
|
|
|
| - // TODO(robliao): Remove ScopedTracker below once https://crbug.com/461509
|
| - // is fixed.
|
| - tracked_objects::ScopedTracker tracking_profile1(
|
| - FROM_HERE_WITH_EXPLICIT_FUNCTION(
|
| - "461509 SingleThreadProxy::DoComposite1"));
|
| -
|
| // We guard PrepareToDraw() with CanDraw() because it always returns a valid
|
| // frame, so can only be used when such a frame is possible. Since
|
| // DrawLayers() depends on the result of PrepareToDraw(), it is guarded on
|
| @@ -600,53 +570,21 @@ DrawResult SingleThreadProxy::DoComposite(LayerTreeHostImpl::FrameData* frame) {
|
| BlockingTaskRunner::CapturePostTasks blocked(
|
| task_runner_provider_->blocking_main_thread_task_runner());
|
|
|
| - // TODO(robliao): Remove ScopedTracker below once https://crbug.com/461509
|
| - // is fixed.
|
| - tracked_objects::ScopedTracker tracking_profile2(
|
| - FROM_HERE_WITH_EXPLICIT_FUNCTION(
|
| - "461509 SingleThreadProxy::DoComposite2"));
|
| draw_result = layer_tree_host_impl_->PrepareToDraw(frame);
|
| draw_frame = draw_result == DRAW_SUCCESS;
|
| if (draw_frame) {
|
| - // TODO(robliao): Remove ScopedTracker below once https://crbug.com/461509
|
| - // is fixed.
|
| - tracked_objects::ScopedTracker tracking_profile3(
|
| - FROM_HERE_WITH_EXPLICIT_FUNCTION(
|
| - "461509 SingleThreadProxy::DoComposite3"));
|
| layer_tree_host_impl_->DrawLayers(frame);
|
| -
|
| - // TODO(robliao): Remove ScopedTracker below once https://crbug.com/461509
|
| - // is fixed.
|
| - tracked_objects::ScopedTracker tracking_profile8(
|
| - FROM_HERE_WITH_EXPLICIT_FUNCTION(
|
| - "461509 SingleThreadProxy::DoComposite8"));
|
| if (layer_tree_host_impl_->SwapBuffers(*frame)) {
|
| if (scheduler_on_impl_thread_)
|
| scheduler_on_impl_thread_->DidSwapBuffers();
|
| client_->DidPostSwapBuffers();
|
| }
|
| }
|
| -
|
| - // TODO(robliao): Remove ScopedTracker below once https://crbug.com/461509
|
| - // is fixed.
|
| - tracked_objects::ScopedTracker tracking_profile4(
|
| - FROM_HERE_WITH_EXPLICIT_FUNCTION(
|
| - "461509 SingleThreadProxy::DoComposite4"));
|
| layer_tree_host_impl_->DidDrawAllLayers(*frame);
|
|
|
| - // TODO(robliao): Remove ScopedTracker below once https://crbug.com/461509
|
| - // is fixed.
|
| - tracked_objects::ScopedTracker tracking_profile5(
|
| - FROM_HERE_WITH_EXPLICIT_FUNCTION(
|
| - "461509 SingleThreadProxy::DoComposite5"));
|
| bool start_ready_animations = draw_frame;
|
| layer_tree_host_impl_->UpdateAnimationState(start_ready_animations);
|
| }
|
| - // TODO(robliao): Remove ScopedTracker below once https://crbug.com/461509 is
|
| - // fixed.
|
| - tracked_objects::ScopedTracker tracking_profile9(
|
| - FROM_HERE_WITH_EXPLICIT_FUNCTION(
|
| - "461509 SingleThreadProxy::DoComposite9"));
|
| DidCommitAndDrawFrame();
|
|
|
| return draw_result;
|
|
|