cc: Partial tile update for one-copy raster.

cc/resources/resource_provider.h

Index: cc/resources/resource_provider.h
diff --git a/cc/resources/resource_provider.h b/cc/resources/resource_provider.h
index 76f09a1035fbafb3528c6876a1f853f9638d8237..1e7fb5172291e60d2c28429f4992c964df3b3167 100644
--- a/cc/resources/resource_provider.h
+++ b/cc/resources/resource_provider.h
@@ -85,7 +85,8 @@ class CC_EXPORT ResourceProvider {
       BlockingTaskRunner* blocking_main_thread_task_runner,
       int highp_threshold_min,
       bool use_rgba_4444_texture_format,
-      size_t id_allocation_chunk_size);
+      size_t id_allocation_chunk_size,
+      bool use_persistent_map_for_gpu_memory_buffers);
   virtual ~ResourceProvider();
 
   void DidLoseOutputSurface() { lost_output_surface_ = true; }
@@ -97,6 +98,7 @@ class CC_EXPORT ResourceProvider {
   ResourceFormat best_texture_format() const { return best_texture_format_; }
   ResourceFormat yuv_resource_format() const { return yuv_resource_format_; }
   bool use_sync_query() const { return use_sync_query_; }
+  bool use_persistent_map() const { return use_persistent_map_; }
   size_t num_resources() const { return resources_.size(); }
 
   // Checks whether a resource is in use by a consumer.
@@ -446,7 +448,8 @@ class CC_EXPORT ResourceProvider {
                    BlockingTaskRunner* blocking_main_thread_task_runner,
                    int highp_threshold_min,
                    bool use_rgba_4444_texture_format,
-                   size_t id_allocation_chunk_size);
+                   size_t id_allocation_chunk_size,
+                   bool use_persistent_map);
   void Initialize();
 
  private:
@@ -602,6 +605,7 @@ class CC_EXPORT ResourceProvider {
   scoped_ptr<IdAllocator> buffer_id_allocator_;
 
   bool use_sync_query_;
+  bool use_persistent_map_;

[reveman, 2015/05/28 05:07:25]

There are 4 different ways of referring to this in this patch:

- have_persistent_gpu_memory_buffers
- use_persistent_map_for_gpu_memory_buffers
- use_persistent_map
- use_persistent_map_gpu_memory_buffer_usage

use_persistent_map_gpu_memory_buffer_usage exists since before and is the most
appropriate term IMO. Would be nice if we could use one term consistently
throughout the code.

[danakj, 2015/05/28 18:54:47]

I think I disagree. The workerpool isn't being told to to "use persistent map".
It's being told that its gpu memory buffers will be persistent, ie "it has
persistent buffers".

I have consolidated the last 3 as use_persistent_map_for_gpu_memory_buffers,
which means exactly what it says.

   // Fence used for CopyResource if CHROMIUM_sync_query is not supported.
   scoped_refptr<SynchronousFence> synchronous_fence_;