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

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

Issue 1769913003: sync compositor: Add output_surface_id (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: unit test Created 4 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 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/shared_renderer_state.h" 5 #include "android_webview/browser/shared_renderer_state.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "android_webview/browser/browser_view_renderer.h" 9 #include "android_webview/browser/browser_view_renderer.h"
10 #include "android_webview/browser/child_frame.h" 10 #include "android_webview/browser/child_frame.h"
(...skipping 195 matching lines...) Expand 10 before | Expand all | Expand 10 after
206 void SharedRendererState::SetInsideHardwareRelease(bool inside) { 206 void SharedRendererState::SetInsideHardwareRelease(bool inside) {
207 base::AutoLock lock(lock_); 207 base::AutoLock lock(lock_);
208 inside_hardware_release_ = inside; 208 inside_hardware_release_ = inside;
209 } 209 }
210 210
211 bool SharedRendererState::IsInsideHardwareRelease() const { 211 bool SharedRendererState::IsInsideHardwareRelease() const {
212 base::AutoLock lock(lock_); 212 base::AutoLock lock(lock_);
213 return inside_hardware_release_; 213 return inside_hardware_release_;
214 } 214 }
215 215
216 SharedRendererState::ReturnedResources::ReturnedResources()
217 : output_surface_id(0u) {}
218
216 void SharedRendererState::InsertReturnedResourcesOnRT( 219 void SharedRendererState::InsertReturnedResourcesOnRT(
217 const cc::ReturnedResourceArray& resources, 220 const cc::ReturnedResourceArray& resources,
218 uint32_t compositor_id) { 221 uint32_t compositor_id,
222 uint32_t output_surface_id) {
219 base::AutoLock lock(lock_); 223 base::AutoLock lock(lock_);
220 cc::ReturnedResourceArray& returned_resources = 224 ReturnedResources& returned_resources =
221 returned_resources_map_[compositor_id]; 225 returned_resources_map_[compositor_id];
222 returned_resources.insert(returned_resources.end(), resources.begin(), 226 if (returned_resources.output_surface_id != output_surface_id) {
223 resources.end()); 227 returned_resources.resources.clear();
228 }
229 returned_resources.resources.insert(returned_resources.resources.end(),
230 resources.begin(), resources.end());
231 returned_resources.output_surface_id = output_surface_id;
224 } 232 }
225 233
226 void SharedRendererState::SwapReturnedResourcesOnUI( 234 void SharedRendererState::SwapReturnedResourcesOnUI(
227 std::map<uint32_t, cc::ReturnedResourceArray>* returned_resource_map) { 235 ReturnedResourcesMap* returned_resource_map) {
228 DCHECK(returned_resource_map->empty()); 236 DCHECK(returned_resource_map->empty());
229 base::AutoLock lock(lock_); 237 base::AutoLock lock(lock_);
230 returned_resource_map->swap(returned_resources_map_); 238 returned_resource_map->swap(returned_resources_map_);
231 } 239 }
232 240
233 bool SharedRendererState::ReturnedResourcesEmptyOnUI() const { 241 bool SharedRendererState::ReturnedResourcesEmptyOnUI() const {
234 base::AutoLock lock(lock_); 242 base::AutoLock lock(lock_);
235 return returned_resources_map_.empty(); 243 return returned_resources_map_.empty();
236 } 244 }
237 245
(...skipping 133 matching lines...) Expand 10 before | Expand all | Expand 10 after
371 : shared_renderer_state_(shared_renderer_state) { 379 : shared_renderer_state_(shared_renderer_state) {
372 DCHECK(!shared_renderer_state_->IsInsideHardwareRelease()); 380 DCHECK(!shared_renderer_state_->IsInsideHardwareRelease());
373 shared_renderer_state_->SetInsideHardwareRelease(true); 381 shared_renderer_state_->SetInsideHardwareRelease(true);
374 } 382 }
375 383
376 SharedRendererState::InsideHardwareReleaseReset::~InsideHardwareReleaseReset() { 384 SharedRendererState::InsideHardwareReleaseReset::~InsideHardwareReleaseReset() {
377 shared_renderer_state_->SetInsideHardwareRelease(false); 385 shared_renderer_state_->SetInsideHardwareRelease(false);
378 } 386 }
379 387
380 } // namespace android_webview 388 } // namespace android_webview
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698