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

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: Address comments Created 6 years, 5 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 189 matching lines...) Expand 10 before | Expand all | Expand 10 after
200 delegated_layer_->SetTransform(transform); 200 delegated_layer_->SetTransform(transform);
201 201
202 gl_surface_->SetBackingFrameBufferObject(framebuffer_binding_ext); 202 gl_surface_->SetBackingFrameBufferObject(framebuffer_binding_ext);
203 { 203 {
204 base::AutoReset<bool> frame_resetter(&viewport_clip_valid_for_dcheck_, 204 base::AutoReset<bool> frame_resetter(&viewport_clip_valid_for_dcheck_,
205 true); 205 true);
206 layer_tree_host_->SetNeedsRedrawRect(clip_); 206 layer_tree_host_->SetNeedsRedrawRect(clip_);
207 layer_tree_host_->Composite(gfx::FrameTime::Now()); 207 layer_tree_host_->Composite(gfx::FrameTime::Now());
208 } 208 }
209 gl_surface_->ResetBackingFrameBufferObject(); 209 gl_surface_->ResetBackingFrameBufferObject();
210
211 // Need to post the new transform matrix back to child compositor
212 // because there is no onDraw during a Render Thread animation, and child
213 // compositor might not have the tiles rasterized as the animation goes on.
214 if (transform != transform_) {
215 transform_ = transform;
216 shared_renderer_state_->PostExternalMatrixToChildCompositor(transform_);
217 }
210 } 218 }
211 219
212 scoped_ptr<cc::OutputSurface> HardwareRenderer::CreateOutputSurface( 220 scoped_ptr<cc::OutputSurface> HardwareRenderer::CreateOutputSurface(
213 bool fallback) { 221 bool fallback) {
214 // Android webview does not support losing output surface. 222 // Android webview does not support losing output surface.
215 DCHECK(!fallback); 223 DCHECK(!fallback);
216 scoped_refptr<cc::ContextProvider> context_provider = 224 scoped_refptr<cc::ContextProvider> context_provider =
217 CreateContext(gl_surface_, 225 CreateContext(gl_surface_,
218 DeferredGpuCommandService::GetInstance(), 226 DeferredGpuCommandService::GetInstance(),
219 shared_renderer_state_->GetSharedContext()); 227 shared_renderer_state_->GetSharedContext());
220 scoped_ptr<ParentOutputSurface> output_surface_holder( 228 scoped_ptr<ParentOutputSurface> output_surface_holder(
221 new ParentOutputSurface(context_provider)); 229 new ParentOutputSurface(context_provider));
222 output_surface_ = output_surface_holder.get(); 230 output_surface_ = output_surface_holder.get();
223 return output_surface_holder.PassAs<cc::OutputSurface>(); 231 return output_surface_holder.PassAs<cc::OutputSurface>();
224 } 232 }
225 233
226 void HardwareRenderer::UnusedResourcesAreAvailable() { 234 void HardwareRenderer::UnusedResourcesAreAvailable() {
227 cc::ReturnedResourceArray returned_resources; 235 cc::ReturnedResourceArray returned_resources;
228 resource_collection_->TakeUnusedResourcesForChildCompositor( 236 resource_collection_->TakeUnusedResourcesForChildCompositor(
229 &returned_resources); 237 &returned_resources);
230 shared_renderer_state_->InsertReturnedResources(returned_resources); 238 shared_renderer_state_->InsertReturnedResources(returned_resources);
231 } 239 }
232 240
233 } // namespace android_webview 241 } // namespace android_webview
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698