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

Unified Diff: cc/trees/remote_channel_impl.cc

Issue 2282433002: Revert of cc: Delete all the RendererCapabilities. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@remove-prepare-mailbox-param
Patch Set: Created 4 years, 4 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
« no previous file with comments | « cc/trees/remote_channel_impl.h ('k') | cc/trees/single_thread_proxy.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/trees/remote_channel_impl.cc
diff --git a/cc/trees/remote_channel_impl.cc b/cc/trees/remote_channel_impl.cc
index f981df9dfd9bf162cd0ede80e10dab8f6bf83a43..3f8754c1c17c479c7fbf05691681b13f5ee742ce 100644
--- a/cc/trees/remote_channel_impl.cc
+++ b/cc/trees/remote_channel_impl.cc
@@ -193,6 +193,11 @@
base::Bind(&ProxyImpl::SetVisibleOnImpl, proxy_impl_weak_ptr_, visible));
}
+const RendererCapabilities& RemoteChannelImpl::GetRendererCapabilities() const {
+ NOTREACHED() << "Should not be called on the remote client LayerTreeHost";
+ return main().renderer_capabilities;
+}
+
void RemoteChannelImpl::SetNeedsAnimate() {
NOTREACHED() << "Should not be called on the remote client LayerTreeHost";
}
@@ -319,6 +324,9 @@
impl().remote_channel_weak_ptr));
}
+void RemoteChannelImpl::SetRendererCapabilitiesMainCopy(
+ const RendererCapabilities& capabilities) {}
+
void RemoteChannelImpl::BeginMainFrameNotExpectedSoon() {}
void RemoteChannelImpl::DidCommitAndDrawFrame() {
@@ -347,13 +355,15 @@
impl().remote_channel_weak_ptr));
}
-void RemoteChannelImpl::DidInitializeOutputSurface(bool success) {
+void RemoteChannelImpl::DidInitializeOutputSurface(
+ bool success,
+ const RendererCapabilities& capabilities) {
DCHECK(task_runner_provider_->IsImplThread());
MainThreadTaskRunner()->PostTask(
FROM_HERE,
base::Bind(&RemoteChannelImpl::DidInitializeOutputSurfaceOnMain,
- impl().remote_channel_weak_ptr, success));
+ impl().remote_channel_weak_ptr, success, capabilities));
}
void RemoteChannelImpl::DidCompletePageScaleAnimation() {}
@@ -406,7 +416,9 @@
main().layer_tree_host->RequestNewOutputSurface();
}
-void RemoteChannelImpl::DidInitializeOutputSurfaceOnMain(bool success) {
+void RemoteChannelImpl::DidInitializeOutputSurfaceOnMain(
+ bool success,
+ const RendererCapabilities& capabilities) {
DCHECK(task_runner_provider_->IsMainThread());
if (!success) {
@@ -415,6 +427,7 @@
}
VLOG(1) << "OutputSurface initialized successfully";
+ main().renderer_capabilities = capabilities;
main().layer_tree_host->DidInitializeOutputSurface();
// If we were waiting for output surface initialization, we might have queued
« no previous file with comments | « cc/trees/remote_channel_impl.h ('k') | cc/trees/single_thread_proxy.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698