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

Unified Diff: cc/trees/single_thread_proxy.cc

Issue 251343002: Remove offscreen compositor contexts. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rm-offscreencontext: include Created 6 years, 8 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/single_thread_proxy.h ('k') | cc/trees/thread_proxy.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/trees/single_thread_proxy.cc
diff --git a/cc/trees/single_thread_proxy.cc b/cc/trees/single_thread_proxy.cc
index 72f16dc7c5fde15c663239499ddf639488dd35da..b5968fe43b86ffd990cc1a5cea50a511769b26e0 100644
--- a/cc/trees/single_thread_proxy.cc
+++ b/cc/trees/single_thread_proxy.cc
@@ -32,7 +32,6 @@ SingleThreadProxy::SingleThreadProxy(LayerTreeHost* layer_tree_host,
: Proxy(NULL),
layer_tree_host_(layer_tree_host),
client_(client),
- created_offscreen_context_provider_(false),
next_frame_is_newly_committed_frame_(false),
inside_draw_(false) {
TRACE_EVENT0("cc", "SingleThreadProxy::SingleThreadProxy");
@@ -122,17 +121,6 @@ void SingleThreadProxy::CreateAndInitializeOutputSurface() {
return;
}
- scoped_refptr<ContextProvider> offscreen_context_provider;
- if (created_offscreen_context_provider_) {
- offscreen_context_provider =
- layer_tree_host_->client()->OffscreenContextProvider();
- if (!offscreen_context_provider.get() ||
- !offscreen_context_provider->BindToCurrentThread()) {
- OnOutputSurfaceInitializeAttempted(false);
- return;
- }
- }
-
{
DebugScopedSetMainThreadBlocked main_thread_blocked(this);
DebugScopedSetImplThread impl(this);
@@ -147,13 +135,6 @@ void SingleThreadProxy::CreateAndInitializeOutputSurface() {
DCHECK(output_surface);
initialized = layer_tree_host_impl_->InitializeRenderer(
output_surface.Pass());
- if (!initialized && offscreen_context_provider.get()) {
- offscreen_context_provider->VerifyContexts();
- offscreen_context_provider = NULL;
- }
-
- layer_tree_host_impl_->SetOffscreenContextProvider(
- offscreen_context_provider);
}
OnOutputSurfaceInitializeAttempted(initialized);
@@ -469,25 +450,9 @@ bool SingleThreadProxy::CommitAndComposite(
layer_tree_host_->WillCommit();
- scoped_refptr<ContextProvider> offscreen_context_provider;
- if (renderer_capabilities_for_main_thread_.using_offscreen_context3d &&
- layer_tree_host_->needs_offscreen_context()) {
- offscreen_context_provider =
- layer_tree_host_->client()->OffscreenContextProvider();
- if (offscreen_context_provider.get() &&
- !offscreen_context_provider->BindToCurrentThread())
- offscreen_context_provider = NULL;
-
- if (offscreen_context_provider.get())
- created_offscreen_context_provider_ = true;
- }
-
DoCommit(queue.Pass());
- bool result = DoComposite(offscreen_context_provider,
- frame_begin_time,
- device_viewport_damage_rect,
- for_readback,
- frame);
+ bool result = DoComposite(
+ frame_begin_time, device_viewport_damage_rect, for_readback, frame);
layer_tree_host_->DidBeginMainFrame();
return result;
}
@@ -505,7 +470,6 @@ void SingleThreadProxy::UpdateBackgroundAnimateTicking() {
}
bool SingleThreadProxy::DoComposite(
- scoped_refptr<ContextProvider> offscreen_context_provider,
base::TimeTicks frame_begin_time,
const gfx::Rect& device_viewport_damage_rect,
bool for_readback,
@@ -518,9 +482,6 @@ bool SingleThreadProxy::DoComposite(
DebugScopedSetImplThread impl(this);
base::AutoReset<bool> mark_inside(&inside_draw_, true);
- layer_tree_host_impl_->SetOffscreenContextProvider(
- offscreen_context_provider);
-
bool can_do_readback = layer_tree_host_impl_->renderer()->CanReadPixels();
// We guard PrepareToDraw() with CanDraw() because it always returns a valid
@@ -550,10 +511,6 @@ bool SingleThreadProxy::DoComposite(
}
if (lost_output_surface) {
- ContextProvider* offscreen_contexts =
- layer_tree_host_impl_->offscreen_context_provider();
- if (offscreen_contexts)
- offscreen_contexts->VerifyContexts();
layer_tree_host_->DidLoseOutputSurface();
return false;
}
« no previous file with comments | « cc/trees/single_thread_proxy.h ('k') | cc/trees/thread_proxy.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698