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

Unified Diff: cc/trees/proxy_impl.cc

Issue 2409923002: cc: Rename SwapBuffers on CompositorFrameSink to SubmitCompositorFrame (Closed)
Patch Set: swap-to-submit: rebase Created 4 years, 2 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
Index: cc/trees/proxy_impl.cc
diff --git a/cc/trees/proxy_impl.cc b/cc/trees/proxy_impl.cc
index 659ff5c9ab13a2c1fea5c57a61b620943f47858e..90b8d531e2e378fe85cb57285129cb6a61b083f9 100644
--- a/cc/trees/proxy_impl.cc
+++ b/cc/trees/proxy_impl.cc
@@ -268,11 +268,12 @@ void ProxyImpl::SetBeginFrameSource(BeginFrameSource* source) {
}
}
-void ProxyImpl::DidSwapBuffersCompleteOnImplThread() {
- TRACE_EVENT0("cc,benchmark", "ProxyImpl::DidSwapBuffersCompleteOnImplThread");
+void ProxyImpl::DidReceiveCompositorFrameAckOnImplThread() {
+ TRACE_EVENT0("cc,benchmark",
+ "ProxyImpl::DidReceiveCompositorFrameAckOnImplThread");
DCHECK(IsImplThread());
- scheduler_->DidSwapBuffersComplete();
- channel_impl_->DidCompleteSwapBuffers();
+ scheduler_->DidReceiveCompositorFrameAck();
+ channel_impl_->DidReceiveCompositorFrameAck();
}
void ProxyImpl::OnCanDrawStateChanged(bool can_draw) {
@@ -448,8 +449,8 @@ void ProxyImpl::ScheduledActionSendBeginMainFrame(const BeginFrameArgs& args) {
devtools_instrumentation::DidRequestMainThreadFrame(layer_tree_host_id_);
}
-DrawResult ProxyImpl::ScheduledActionDrawAndSwapIfPossible() {
- TRACE_EVENT0("cc", "ProxyImpl::ScheduledActionDrawAndSwap");
+DrawResult ProxyImpl::ScheduledActionSubmitCompositorFrameIfPossible() {
+ TRACE_EVENT0("cc", "ProxyImpl::ScheduledActionSubmitCompositorFrame");
DCHECK(IsImplThread());
// SchedulerStateMachine::DidDrawIfPossibleCompleted isn't set up to
@@ -458,14 +459,14 @@ DrawResult ProxyImpl::ScheduledActionDrawAndSwapIfPossible() {
DCHECK(layer_tree_host_impl_->CanDraw());
bool forced_draw = false;
- return DrawAndSwapInternal(forced_draw);
+ return SubmitCompositorFrameInternal(forced_draw);
}
-DrawResult ProxyImpl::ScheduledActionDrawAndSwapForced() {
- TRACE_EVENT0("cc", "ProxyImpl::ScheduledActionDrawAndSwapForced");
+DrawResult ProxyImpl::ScheduledActionSubmitCompositorFrameForced() {
+ TRACE_EVENT0("cc", "ProxyImpl::ScheduledActionSubmitCompositorFrameForced");
DCHECK(IsImplThread());
bool forced_draw = true;
- return DrawAndSwapInternal(forced_draw);
+ return SubmitCompositorFrameInternal(forced_draw);
}
void ProxyImpl::ScheduledActionCommit() {
@@ -538,8 +539,8 @@ void ProxyImpl::SendBeginMainFrameNotExpectedSoon() {
channel_impl_->BeginMainFrameNotExpectedSoon();
}
-DrawResult ProxyImpl::DrawAndSwapInternal(bool forced_draw) {
- TRACE_EVENT_SYNTHETIC_DELAY("cc.DrawAndSwap");
+DrawResult ProxyImpl::SubmitCompositorFrameInternal(bool forced_draw) {
+ TRACE_EVENT_SYNTHETIC_DELAY("cc.SubmitCompositorFrame");
DCHECK(IsImplThread());
DCHECK(layer_tree_host_impl_.get());
@@ -576,7 +577,7 @@ DrawResult ProxyImpl::DrawAndSwapInternal(bool forced_draw) {
if (draw_frame) {
if (layer_tree_host_impl_->DrawLayers(&frame))
- scheduler_->DidSwapBuffers();
+ scheduler_->DidSubmitCompositorFrame();
result = DRAW_SUCCESS;
} else {
DCHECK_NE(DRAW_SUCCESS, result);

Powered by Google App Engine
This is Rietveld 408576698