gpu: Rename AllocateGpuMemoryBuffer to CreateGpuMemoryBuffer.

media/video/gpu_memory_buffer_video_frame_pool_unittest.cc

Index: media/video/gpu_memory_buffer_video_frame_pool_unittest.cc
diff --git a/media/video/gpu_memory_buffer_video_frame_pool_unittest.cc b/media/video/gpu_memory_buffer_video_frame_pool_unittest.cc
index 67ece214ec13598a8f1db4d93400b23d1dbd8bae..b85319997daec1e3aecc9628db691b8c93e4a063 100644
--- a/media/video/gpu_memory_buffer_video_frame_pool_unittest.cc
+++ b/media/video/gpu_memory_buffer_video_frame_pool_unittest.cc
@@ -264,7 +264,7 @@ TEST_F(GpuMemoryBufferVideoFramePoolTest, CreateOneHardwareNV12Frame2) {
       media::VideoFrameMetadata::READ_LOCK_FENCES_ENABLED));
 }
 
-// AllocateGpuMemoryBuffer can return null (e.g: when the GPU process is down).
+// CreateGpuMemoryBuffer can return null (e.g: when the GPU process is down).

[reveman, 2016/11/23 15:21:59]

This comment is incorrect. Gpu process being down is not a valid reason for
failure. Just FYI. No need to change it now.

[sadrul, 2016/11/23 16:01:27]

I have left the comment as is (I am not sure if I should just remove the example
comment, or replace it with something else). Happy to make this change if you
have a suggestion though.

 // This test checks that in that case we don't crash and still create the
 // textures.
 TEST_F(GpuMemoryBufferVideoFramePoolTest, AllocateGpuMemoryBufferFail) {

[reveman, 2016/11/23 15:21:59]

Maybe change the name of this test too. Not very important though.

[sadrul, 2016/11/23 16:01:27]

I have changed this.