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

Side by Side Diff: android_webview/browser/hardware_renderer.cc

Issue 394113002: Tiling priorities in Android Webview. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: better fix Created 6 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "android_webview/browser/hardware_renderer.h" 5 #include "android_webview/browser/hardware_renderer.h"
6 6
7 #include "android_webview/browser/aw_gl_surface.h" 7 #include "android_webview/browser/aw_gl_surface.h"
8 #include "android_webview/browser/deferred_gpu_command_service.h" 8 #include "android_webview/browser/deferred_gpu_command_service.h"
9 #include "android_webview/browser/parent_output_surface.h" 9 #include "android_webview/browser/parent_output_surface.h"
10 #include "android_webview/browser/shared_renderer_state.h" 10 #include "android_webview/browser/shared_renderer_state.h"
(...skipping 173 matching lines...) Expand 10 before | Expand all | Expand 10 after
184 184
185 if (!delegated_layer_.get()) { 185 if (!delegated_layer_.get()) {
186 DLOG(ERROR) << "No frame committed"; 186 DLOG(ERROR) << "No frame committed";
187 return; 187 return;
188 } 188 }
189 189
190 // TODO(boliu): Handle context loss. 190 // TODO(boliu): Handle context loss.
191 if (last_egl_context_ != current_context) 191 if (last_egl_context_ != current_context)
192 DLOG(WARNING) << "EGLContextChanged"; 192 DLOG(WARNING) << "EGLContextChanged";
193 193
194 gfx::Transform transform(gfx::Transform::kSkipInitialization);
195 transform.matrix().setColMajorf(draw_info->transform);
196 transform.Translate(scroll_offset_.x(), scroll_offset_.y());
197
198 // Need to post the new transform matrix back to child compositor
199 // because there is no onDraw during a Render Thread animation, and child
200 // compositor might not have the tiles rasterized as the animation goes on.
201 ParentCompositorDrawConstraints draw_constraints(
202 draw_info->is_layer, transform, gfx::Rect(viewport_));
203 if (!draw_constraints_.Equals(draw_constraints)) {
204 draw_constraints_ = draw_constraints;
205 shared_renderer_state_->PostExternalDrawConstraintsToChildCompositor(
206 draw_constraints);
207 }
208
194 viewport_.SetSize(draw_info->width, draw_info->height); 209 viewport_.SetSize(draw_info->width, draw_info->height);
195 layer_tree_host_->SetViewportSize(viewport_); 210 layer_tree_host_->SetViewportSize(viewport_);
196 clip_.SetRect(draw_info->clip_left, 211 clip_.SetRect(draw_info->clip_left,
197 draw_info->clip_top, 212 draw_info->clip_top,
198 draw_info->clip_right - draw_info->clip_left, 213 draw_info->clip_right - draw_info->clip_left,
199 draw_info->clip_bottom - draw_info->clip_top); 214 draw_info->clip_bottom - draw_info->clip_top);
200 stencil_enabled_ = stencil_enabled; 215 stencil_enabled_ = stencil_enabled;
201 216
202 gfx::Transform transform(gfx::Transform::kSkipInitialization);
203 transform.matrix().setColMajorf(draw_info->transform);
204 transform.Translate(scroll_offset_.x(), scroll_offset_.y());
205 delegated_layer_->SetTransform(transform); 217 delegated_layer_->SetTransform(transform);
206 218
207 gl_surface_->SetBackingFrameBufferObject(framebuffer_binding_ext); 219 gl_surface_->SetBackingFrameBufferObject(framebuffer_binding_ext);
208 { 220 {
209 base::AutoReset<bool> frame_resetter(&viewport_clip_valid_for_dcheck_, 221 base::AutoReset<bool> frame_resetter(&viewport_clip_valid_for_dcheck_,
210 true); 222 true);
211 layer_tree_host_->SetNeedsRedrawRect(clip_); 223 layer_tree_host_->SetNeedsRedrawRect(clip_);
212 layer_tree_host_->Composite(gfx::FrameTime::Now()); 224 layer_tree_host_->Composite(gfx::FrameTime::Now());
213 } 225 }
214 gl_surface_->ResetBackingFrameBufferObject(); 226 gl_surface_->ResetBackingFrameBufferObject();
(...skipping 14 matching lines...) Expand all
229 } 241 }
230 242
231 void HardwareRenderer::UnusedResourcesAreAvailable() { 243 void HardwareRenderer::UnusedResourcesAreAvailable() {
232 cc::ReturnedResourceArray returned_resources; 244 cc::ReturnedResourceArray returned_resources;
233 resource_collection_->TakeUnusedResourcesForChildCompositor( 245 resource_collection_->TakeUnusedResourcesForChildCompositor(
234 &returned_resources); 246 &returned_resources);
235 shared_renderer_state_->InsertReturnedResources(returned_resources); 247 shared_renderer_state_->InsertReturnedResources(returned_resources);
236 } 248 }
237 249
238 } // namespace android_webview 250 } // namespace android_webview
OLDNEW
« no previous file with comments | « android_webview/browser/hardware_renderer.h ('k') | android_webview/browser/parent_compositor_draw_constraints.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698