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

Side by Side Diff: cc/scheduler/scheduler.cc

Issue 2753843003: Create a new action triggered when a BeginMainFrame is not expected before vsync (Closed)
Patch Set: fix funnel initial state Created 3 years, 9 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/scheduler/scheduler.h" 5 #include "cc/scheduler/scheduler.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 8
9 #include "base/auto_reset.h" 9 #include "base/auto_reset.h"
10 #include "base/logging.h" 10 #include "base/logging.h"
(...skipping 611 matching lines...) Expand 10 before | Expand all | Expand 10 after
622 case SchedulerStateMachine::ACTION_NONE: 622 case SchedulerStateMachine::ACTION_NONE:
623 break; 623 break;
624 case SchedulerStateMachine::ACTION_SEND_BEGIN_MAIN_FRAME: 624 case SchedulerStateMachine::ACTION_SEND_BEGIN_MAIN_FRAME:
625 compositor_timing_history_->WillBeginMainFrame( 625 compositor_timing_history_->WillBeginMainFrame(
626 begin_main_frame_args_.on_critical_path, 626 begin_main_frame_args_.on_critical_path,
627 begin_main_frame_args_.frame_time); 627 begin_main_frame_args_.frame_time);
628 state_machine_.WillSendBeginMainFrame(); 628 state_machine_.WillSendBeginMainFrame();
629 // TODO(brianderson): Pass begin_main_frame_args_ directly to client. 629 // TODO(brianderson): Pass begin_main_frame_args_ directly to client.
630 client_->ScheduledActionSendBeginMainFrame(begin_main_frame_args_); 630 client_->ScheduledActionSendBeginMainFrame(begin_main_frame_args_);
631 break; 631 break;
632 case SchedulerStateMachine::ACTION_DO_IDLE_WORK:
633 state_machine_.WillDoIdleWork();
634 BeginImplFrameNotExpectedSoon();
Sami 2017/03/20 16:58:49 Let's introduce a new notification which kicks off
635 break;
632 case SchedulerStateMachine::ACTION_COMMIT: { 636 case SchedulerStateMachine::ACTION_COMMIT: {
633 bool commit_has_no_updates = false; 637 bool commit_has_no_updates = false;
634 state_machine_.WillCommit(commit_has_no_updates); 638 state_machine_.WillCommit(commit_has_no_updates);
635 client_->ScheduledActionCommit(); 639 client_->ScheduledActionCommit();
636 break; 640 break;
637 } 641 }
638 case SchedulerStateMachine::ACTION_ACTIVATE_SYNC_TREE: 642 case SchedulerStateMachine::ACTION_ACTIVATE_SYNC_TREE:
639 compositor_timing_history_->WillActivate(); 643 compositor_timing_history_->WillActivate();
640 state_machine_.WillActivate(); 644 state_machine_.WillActivate();
641 client_->ScheduledActionActivateSyncTree(); 645 client_->ScheduledActionActivateSyncTree();
(...skipping 157 matching lines...) Expand 10 before | Expand all | Expand 10 after
799 } 803 }
800 804
801 BeginFrameAck Scheduler::CurrentBeginFrameAckForActiveTree() const { 805 BeginFrameAck Scheduler::CurrentBeginFrameAckForActiveTree() const {
802 return BeginFrameAck( 806 return BeginFrameAck(
803 begin_main_frame_args_.source_id, begin_main_frame_args_.sequence_number, 807 begin_main_frame_args_.source_id, begin_main_frame_args_.sequence_number,
804 state_machine_.last_begin_frame_sequence_number_active_tree_was_fresh(), 808 state_machine_.last_begin_frame_sequence_number_active_tree_was_fresh(),
805 0, true); 809 0, true);
806 } 810 }
807 811
808 } // namespace cc 812 } // namespace cc
OLDNEW
« no previous file with comments | « no previous file | cc/scheduler/scheduler_state_machine.h » ('j') | cc/scheduler/scheduler_state_machine.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698