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

Unified Diff: cc/trees/proxy_main.cc

Issue 2337913003: Fork cc::OutputSurface into cc::CompositorFrameSink. (Closed)
Patch Set: cfsfork: rebase Created 4 years, 3 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_main.cc
diff --git a/cc/trees/proxy_main.cc b/cc/trees/proxy_main.cc
index 414e014c6e48f38301e5331219910a62f29adee2..d366c69c125aebade81f3970e6c9ad6b23beedf1 100644
--- a/cc/trees/proxy_main.cc
+++ b/cc/trees/proxy_main.cc
@@ -13,7 +13,7 @@
#include "cc/animation/animation_events.h"
#include "cc/debug/benchmark_instrumentation.h"
#include "cc/debug/devtools_instrumentation.h"
-#include "cc/output/output_surface.h"
+#include "cc/output/compositor_frame_sink.h"
#include "cc/output/swap_promise.h"
#include "cc/resources/ui_resource_manager.h"
#include "cc/trees/blocking_task_runner.h"
@@ -94,26 +94,26 @@ void ProxyMain::SetAnimationEvents(std::unique_ptr<AnimationEvents> events) {
layer_tree_host_->SetAnimationEvents(std::move(events));
}
-void ProxyMain::DidLoseOutputSurface() {
- TRACE_EVENT0("cc", "ProxyMain::DidLoseOutputSurface");
+void ProxyMain::DidLoseCompositorFrameSink() {
+ TRACE_EVENT0("cc", "ProxyMain::DidLoseCompositorFrameSink");
DCHECK(IsMainThread());
- layer_tree_host_->DidLoseOutputSurface();
+ layer_tree_host_->DidLoseCompositorFrameSink();
}
-void ProxyMain::RequestNewOutputSurface() {
- TRACE_EVENT0("cc", "ProxyMain::RequestNewOutputSurface");
+void ProxyMain::RequestNewCompositorFrameSink() {
+ TRACE_EVENT0("cc", "ProxyMain::RequestNewCompositorFrameSink");
DCHECK(IsMainThread());
- layer_tree_host_->RequestNewOutputSurface();
+ layer_tree_host_->RequestNewCompositorFrameSink();
}
-void ProxyMain::DidInitializeOutputSurface(bool success) {
- TRACE_EVENT0("cc", "ProxyMain::DidInitializeOutputSurface");
+void ProxyMain::DidInitializeCompositorFrameSink(bool success) {
+ TRACE_EVENT0("cc", "ProxyMain::DidInitializeCompositorFrameSink");
DCHECK(IsMainThread());
if (!success)
- layer_tree_host_->DidFailToInitializeOutputSurface();
+ layer_tree_host_->DidFailToInitializeCompositorFrameSink();
else
- layer_tree_host_->DidInitializeOutputSurface();
+ layer_tree_host_->DidInitializeCompositorFrameSink();
}
void ProxyMain::DidCompletePageScaleAnimation() {
@@ -252,8 +252,9 @@ bool ProxyMain::CommitToActiveTree() const {
return false;
}
-void ProxyMain::SetOutputSurface(OutputSurface* output_surface) {
- channel_main_->InitializeOutputSurfaceOnImpl(output_surface);
+void ProxyMain::SetCompositorFrameSink(
+ CompositorFrameSink* compositor_frame_sink) {
+ channel_main_->InitializeCompositorFrameSinkOnImpl(compositor_frame_sink);
}
void ProxyMain::SetVisible(bool visible) {
@@ -395,11 +396,11 @@ bool ProxyMain::MainFrameWillHappenForTesting() {
return main_frame_will_happen;
}
-void ProxyMain::ReleaseOutputSurface() {
+void ProxyMain::ReleaseCompositorFrameSink() {
DCHECK(IsMainThread());
DebugScopedSetMainThreadBlocked main_thread_blocked(task_runner_provider_);
CompletionEvent completion;
- channel_main_->ReleaseOutputSurfaceOnImpl(&completion);
+ channel_main_->ReleaseCompositorFrameSinkOnImpl(&completion);
completion.Wait();
}

Powered by Google App Engine
This is Rietveld 408576698