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

Unified Diff: android_webview/browser/render_thread_manager.cc

Issue 2375743002: [Merge M54] aw: Fix FBO restore in webview functor (Closed)
Patch Set: 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: android_webview/browser/render_thread_manager.cc
diff --git a/android_webview/browser/render_thread_manager.cc b/android_webview/browser/render_thread_manager.cc
index 9ffaec25ea3dfff81786f03e6eda12cc898e5df1..bb85bfbd5ce175eb5cda9d3874a35deee4d933ae 100644
--- a/android_webview/browser/render_thread_manager.cc
+++ b/android_webview/browser/render_thread_manager.cc
@@ -285,20 +285,10 @@ void RenderThreadManager::DrawGL(AwDrawGLInfo* draw_info) {
draw_info->mode == AwDrawGLInfo::kModeDraw
? ScopedAppGLStateRestore::MODE_DRAW
: ScopedAppGLStateRestore::MODE_RESOURCE_MANAGEMENT);
- // Set the correct FBO before kModeDraw. The GL commands run in kModeDraw
- // require a correctly bound FBO. The FBO remains until the next kModeDraw.
- // So kModeProcess between kModeDraws has correctly bound FBO, too.
- if (hardware_renderer_) {
- hardware_renderer_->SetBackingFrameBufferObject(
- state_restore.framebuffer_binding_ext());
- }
-
ScopedAllowGL allow_gl;
if (!hardware_renderer_ && draw_info->mode == AwDrawGLInfo::kModeDraw &&
!IsInsideHardwareRelease() && HasFrameForHardwareRendererOnRT()) {
hardware_renderer_.reset(new HardwareRenderer(this));
- hardware_renderer_->SetBackingFrameBufferObject(
- state_restore.framebuffer_binding_ext());
hardware_renderer_->CommitFrame();
}
@@ -322,7 +312,7 @@ void RenderThreadManager::DrawGL(AwDrawGLInfo* draw_info) {
}
if (hardware_renderer_)
- hardware_renderer_->DrawGL(draw_info, state_restore);
+ hardware_renderer_->DrawGL(draw_info);
DeferredGpuCommandService::GetInstance()->PerformIdleWork(false);
}
« no previous file with comments | « android_webview/browser/parent_output_surface.cc ('k') | android_webview/browser/scoped_app_gl_state_restore.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698