Partial raster for GPU

cc/raster/one_copy_raster_buffer_provider.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "cc/raster/one_copy_raster_buffer_provider.h"
 
 #include <stdint.h>
 
 #include <algorithm>
 #include <limits>
@@ skipping 136 matching lines @@
 
   return resource_provider_->best_texture_format();
 }
 
 bool OneCopyRasterBufferProvider::IsResourceSwizzleRequired(
     bool must_support_alpha) const {
   return ResourceFormatRequiresSwizzle(GetResourceFormat(must_support_alpha));
 }
 
 bool OneCopyRasterBufferProvider::IsPartialRasterSupported() const {
-  return true;
+  return false;

[ericrk, 2016/07/11 18:55:29]

One copy doesn't support partial raster into a buffer provided by TileManager
(which is what this fn controls). Instead, it handles partial raster internally
in its staging buffers.

Could rename to "CanPartialRasterIntoProvidedBuffer"?

[enne (OOO), 2016/07/11 21:16:24]

I'm not sure I know what the best answer is, but I think it's weird to have this
return false.

[ericrk, 2016/07/12 17:31:56]

Renamed this function to hopefully be clearer. 

 }
 
 void OneCopyRasterBufferProvider::Shutdown() {
   staging_pool_.Shutdown();
   pending_raster_buffers_.clear();
 }
 
 void OneCopyRasterBufferProvider::PlaybackAndCopyOnWorkerThread(
     const Resource* resource,
     ResourceProvider::ScopedWriteLockGL* resource_lock,
@@ skipping 206 matching lines @@
   gl->OrderingBarrierCHROMIUM();
 
   // Generate sync token after the barrier for cross context synchronization.
   gpu::SyncToken resource_sync_token;
   gl->GenUnverifiedSyncTokenCHROMIUM(fence_sync, resource_sync_token.GetData());
   resource_lock->set_sync_token(resource_sync_token);
   resource_lock->set_synchronized(!async_worker_context_enabled_);
 }
 
 }  // namespace cc