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

Unified Diff: cc/output/delegating_renderer.cc

Issue 12545018: Move context-related callbacks into OutputSurface (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: suppress lost context notification before renderer initialized Created 7 years, 9 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/output/delegating_renderer.cc
diff --git a/cc/output/delegating_renderer.cc b/cc/output/delegating_renderer.cc
index 313093a2cf1545945d65e5c1ec9da858eb889c04..49940dbf214f610407a83ce265ab0d505e37b05e 100644
--- a/cc/output/delegating_renderer.cc
+++ b/cc/output/delegating_renderer.cc
@@ -70,7 +70,6 @@ bool DelegatingRenderer::Initialize() {
if (!context3d->makeContextCurrent())
return false;
- context3d->setContextLostCallback(this);
context3d->pushGroupMarkerEXT("CompositorContext");
std::string extensionsString =
@@ -121,11 +120,7 @@ bool DelegatingRenderer::Initialize() {
return true;
}
-DelegatingRenderer::~DelegatingRenderer() {
- WebGraphicsContext3D* context3d = resource_provider_->GraphicsContext3D();
- if (context3d)
- context3d->setContextLostCallback(NULL);
-}
+DelegatingRenderer::~DelegatingRenderer() {}
const RendererCapabilities& DelegatingRenderer::Capabilities() const {
return capabilities_;
@@ -176,8 +171,6 @@ void DelegatingRenderer::GetFramebufferPixels(void* pixels, gfx::Rect rect) {
void DelegatingRenderer::ReceiveCompositorFrameAck(
const CompositorFrameAck& ack) {
resource_provider_->ReceiveFromParent(ack.resources);
- if (client_->HasImplThread())
- client_->OnSwapBuffersComplete();
}
@@ -192,8 +185,4 @@ void DelegatingRenderer::SetVisible(bool visible) {
visible_ = visible;
}
-void DelegatingRenderer::onContextLost() {
- client_->DidLoseOutputSurface();
-}
-
} // namespace cc

Powered by Google App Engine
This is Rietveld 408576698