gpu: Add async pixel transfer interface, stub and tests.

gpu/command_buffer/service/texture_manager.h

 // Copyright (c) 2012 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.
 
 #ifndef GPU_COMMAND_BUFFER_SERVICE_TEXTURE_MANAGER_H_
 #define GPU_COMMAND_BUFFER_SERVICE_TEXTURE_MANAGER_H_
 
 #include <string>
 #include <vector>
 #include "base/basictypes.h"
 #include "base/hash_tables.h"
 #include "base/logging.h"
 #include "base/memory/ref_counted.h"
 #include "gpu/command_buffer/service/feature_info.h"
 #include "gpu/command_buffer/service/gl_utils.h"
 #include "gpu/gpu_export.h"
+#include "ui/gl/async_pixel_transfer_delegate.h"
 #include "ui/gl/gl_image.h"
 
 namespace gpu {
 namespace gles2 {
 
 class GLES2Decoder;
 class Display;
 class TextureDefinition;
 class MemoryTracker;
 class MemoryTypeTracker;
@@ skipping 141 matching lines @@
     }
 
     void SetStreamTexture(bool stream_texture) {
       stream_texture_ = stream_texture;
     }
 
     int IsStreamTexture() {
       return stream_texture_;
     }
 
+    gfx::AsyncPixelTransferState* GetAsyncTransferState() const {
+      return async_transfer_state_.get();
+    }
+    void SetAsyncTransferState(gfx::AsyncPixelTransferState* state) {
+      async_transfer_state_ = state;
+    }
+    bool AsyncTransferIsInProgress() {
+      return async_transfer_state_

[greggman, 2012/12/12 03:51:36]

style: operators want to be at the end of the line

[epennerAtGoogle, 2012/12/12 04:49:49]

Done.

+          && async_transfer_state_->TransferIsInProgress();
+    }
+    bool BindAsyncTransferToTexture(GLenum target) {
+      DCHECK(async_transfer_state_);

[greggman, 2012/12/12 03:51:36]

It seems like this DCHECK is in conflict with the if below. Should this function
have the if, in which case it shouldn't have the DCHECK? Or should it not have
the if and should have the DCHECK?

[epennerAtGoogle, 2012/12/12 04:49:49]

Done.

+      if (async_transfer_state_)
+        return async_transfer_state_->BindAsyncTransferToTexture(target);
+      return false;
+    }
+
     void SetImmutable(bool immutable) {
       immutable_ = immutable;
     }
 
     bool IsImmutable() {
       return immutable_;
     }
 
     // Whether a particular level/face is cleared.
     bool IsLevelCleared(GLenum target, GLint level);
@@ skipping 142 matching lines @@
     // The number of framebuffers this texture is attached to.
     int framebuffer_attachment_count_;
 
     // Whether the associated context group owns this texture and should delete
     // it.
     bool owned_;
 
     // Whether this is a special streaming texture.
     bool stream_texture_;
 
+    // State to facilitate async transfers on this texture.
+    scoped_refptr<gfx::AsyncPixelTransferState> async_transfer_state_;
+
     // Whether the texture is immutable and no further changes to the format
     // or dimensions of the texture object can be made.
     bool immutable_;
 
     // Size in bytes this texture is assumed to take in memory.
     uint32 estimated_size_;
 
     DISALLOW_COPY_AND_ASSIGN(TextureInfo);
   };
 
@@ skipping 218 matching lines @@
   // The default textures for each target (texture name = 0)
   TextureInfo::Ref default_textures_[kNumDefaultTextures];
 
   DISALLOW_COPY_AND_ASSIGN(TextureManager);
 };
 
 }  // namespace gles2
 }  // namespace gpu
 
 #endif  // GPU_COMMAND_BUFFER_SERVICE_TEXTURE_MANAGER_H_