Adds MapBufferSubData and MapTexSubImage2D....

Adds MapBufferSubData and MapTexSubImage2D.

This is only the internal command buffer implementation. 
Connecting these functions publicly to PPAPI or whatever 
will happen separate from this CL.

It's not clear to me where to put GL_READ_ONLY and GL_WRITE_ONLY
Probably the same place we expose these 2 functions to the public.

TEST=unit test
BUG=none



Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=52263

[apatrick_chromium, 2010-07-12 23:34:38]

LGTM

http://codereview.chromium.org/2956005/diff/1/4
File gpu/command_buffer/client/fenced_allocator.cc (right):

http://codereview.chromium.org/2956005/diff/1/4#newcode162
gpu/command_buffer/client/fenced_allocator.cc:162: Block& block = blocks_[i];
const Block&

http://codereview.chromium.org/2956005/diff/1/6
File gpu/command_buffer/client/fenced_allocator_test.cc (right):

http://codereview.chromium.org/2956005/diff/1/6#newcode226
gpu/command_buffer/client/fenced_allocator_test.cc:226: // This no memory should
be available.
Typo

[greggman, 2010-07-13 00:12:38]

I forgot 3 files. Can you check over them? They are

mapped_memory.cc
mapped_memory.h
mapped_memory_unittest.cc

http://codereview.chromium.org/2956005/diff/1/4
File gpu/command_buffer/client/fenced_allocator.cc (right):

http://codereview.chromium.org/2956005/diff/1/4#newcode162
gpu/command_buffer/client/fenced_allocator.cc:162: Block& block = blocks_[i];
On 2010/07/12 23:34:38, apatrick_chromium wrote:
> const Block&

I modify block on line 164

http://codereview.chromium.org/2956005/diff/1/6
File gpu/command_buffer/client/fenced_allocator_test.cc (right):

http://codereview.chromium.org/2956005/diff/1/6#newcode226
gpu/command_buffer/client/fenced_allocator_test.cc:226: // This no memory should
be available.
On 2010/07/12 23:34:38, apatrick_chromium wrote:
> Typo

Done.

[apatrick_chromium, 2010-07-13 00:34:00]

LGTM

http://codereview.chromium.org/2956005/diff/7002/17011
File gpu/command_buffer/client/mapped_memory.h (right):

http://codereview.chromium.org/2956005/diff/7002/17011#newcode92
gpu/command_buffer/client/mapped_memory.h:92: pointer < reinterpret_cast<const
char*>(shm_.ptr) + shm_.size;
char->int8

http://codereview.chromium.org/2956005/diff/7002/17012
File gpu/command_buffer/client/mapped_memory_unittest.cc (right):

http://codereview.chromium.org/2956005/diff/7002/17012#newcode75
gpu/command_buffer/client/mapped_memory_unittest.cc:75: #ifndef COMPILER_MSVC
Can you change this to _MSC_VER so it works with my upcoming CL?