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

Unified Diff: cc/trees/remote_channel_impl.cc

Issue 2267263002: cc: Delete all the RendererCapabilities. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@remove-prepare-mailbox-param
Patch Set: renderer-caps: rebase 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 3f8754c1c17c479c7fbf05691681b13f5ee742ce..f981df9dfd9bf162cd0ede80e10dab8f6bf83a43 100644
--- a/cc/trees/remote_channel_impl.cc
+++ b/cc/trees/remote_channel_impl.cc
@@ -193,11 +193,6 @@ void RemoteChannelImpl::SetVisible(bool visible) {
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";
}
@@ -324,9 +319,6 @@ void RemoteChannelImpl::DidCompleteSwapBuffers() {
impl().remote_channel_weak_ptr));
}
-void RemoteChannelImpl::SetRendererCapabilitiesMainCopy(
- const RendererCapabilities& capabilities) {}
-
void RemoteChannelImpl::BeginMainFrameNotExpectedSoon() {}
void RemoteChannelImpl::DidCommitAndDrawFrame() {
@@ -355,15 +347,13 @@ void RemoteChannelImpl::RequestNewOutputSurface() {
impl().remote_channel_weak_ptr));
}
-void RemoteChannelImpl::DidInitializeOutputSurface(
- bool success,
- const RendererCapabilities& capabilities) {
+void RemoteChannelImpl::DidInitializeOutputSurface(bool success) {
DCHECK(task_runner_provider_->IsImplThread());
MainThreadTaskRunner()->PostTask(
FROM_HERE,
base::Bind(&RemoteChannelImpl::DidInitializeOutputSurfaceOnMain,
- impl().remote_channel_weak_ptr, success, capabilities));
+ impl().remote_channel_weak_ptr, success));
}
void RemoteChannelImpl::DidCompletePageScaleAnimation() {}
@@ -416,9 +406,7 @@ void RemoteChannelImpl::RequestNewOutputSurfaceOnMain() {
main().layer_tree_host->RequestNewOutputSurface();
}
-void RemoteChannelImpl::DidInitializeOutputSurfaceOnMain(
- bool success,
- const RendererCapabilities& capabilities) {
+void RemoteChannelImpl::DidInitializeOutputSurfaceOnMain(bool success) {
DCHECK(task_runner_provider_->IsMainThread());
if (!success) {
@@ -427,7 +415,6 @@ void RemoteChannelImpl::DidInitializeOutputSurfaceOnMain(
}
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