Index: cc/resources/resource_provider_unittest.cc |
diff --git a/cc/resources/resource_provider_unittest.cc b/cc/resources/resource_provider_unittest.cc |
index 9c47cb2dc437b9c782d3b5626236d24791dab825..4782bd57a30558b4f4cd322174e568d3f560eb18 100644 |
--- a/cc/resources/resource_provider_unittest.cc |
+++ b/cc/resources/resource_provider_unittest.cc |
@@ -609,8 +609,8 @@ TEST_P(ResourceProviderTest, TransferGLResources) { |
ResourceProvider::ResourceId id3 = child_resource_provider_->CreateResource( |
size, GL_CLAMP_TO_EDGE, ResourceProvider::TextureUsageAny, format); |
- child_resource_provider_->MapImageRasterBuffer(id3); |
- child_resource_provider_->UnmapImageRasterBuffer(id3); |
+ child_resource_provider_->AcquireImageRasterBuffer(id3); |
+ child_resource_provider_->ReleaseImageRasterBuffer(id3); |
GLuint external_texture_id = child_context_->createExternalTexture(); |
child_context_->bindTexture(GL_TEXTURE_EXTERNAL_OES, external_texture_id); |
@@ -952,9 +952,12 @@ TEST_P(ResourceProviderTest, TransferSoftwareResources) { |
size, GL_CLAMP_TO_EDGE, ResourceProvider::TextureUsageAny, format); |
uint8_t data3[4] = { 6, 7, 8, 9 }; |
SkImageInfo info = SkImageInfo::MakeN32Premul(size.width(), size.height()); |
- SkCanvas* raster_canvas = child_resource_provider_->MapImageRasterBuffer(id3); |
- raster_canvas->writePixels(info, data3, info.minRowBytes(), 0, 0); |
- child_resource_provider_->UnmapImageRasterBuffer(id3); |
+ RasterBuffer* raster_buffer = |
+ child_resource_provider_->AcquireImageRasterBuffer(id3); |
+ SkCanvas* canvas = raster_buffer->AcquireSkCanvas(); |
+ canvas->writePixels(info, data3, info.minRowBytes(), 0, 0); |
+ raster_buffer->ReleaseSkCanvas(); |
+ child_resource_provider_->ReleaseImageRasterBuffer(id3); |
scoped_ptr<base::SharedMemory> shared_memory(new base::SharedMemory()); |
shared_memory->CreateAndMapAnonymous(1); |
@@ -2938,8 +2941,10 @@ TEST_P(ResourceProviderTest, PixelBufferLostContext) { |
GL_INNOCENT_CONTEXT_RESET_ARB); |
resource_provider->AcquirePixelRasterBuffer(id); |
- SkCanvas* raster_canvas = resource_provider->MapPixelRasterBuffer(id); |
- EXPECT_TRUE(raster_canvas == NULL); |
+ RasterBuffer* raster_buffer = resource_provider->MapPixelRasterBuffer(id); |
+ SkCanvas* canvas = raster_buffer->AcquireSkCanvas(); |
+ EXPECT_TRUE(canvas == NULL); |
+ raster_buffer->ReleaseSkCanvas(); |
resource_provider->UnmapPixelRasterBuffer(id); |
resource_provider->ReleasePixelRasterBuffer(id); |
Mock::VerifyAndClearExpectations(context); |
@@ -2987,12 +2992,12 @@ TEST_P(ResourceProviderTest, Image_GLTexture) { |
EXPECT_CALL(*context, mapImageCHROMIUM(kImageId)) |
.WillOnce(Return(dummy_mapped_buffer_address)) |
.RetiresOnSaturation(); |
- resource_provider->MapImageRasterBuffer(id); |
+ resource_provider->AcquireImageRasterBuffer(id); |
EXPECT_CALL(*context, unmapImageCHROMIUM(kImageId)) |
.Times(1) |
.RetiresOnSaturation(); |
- resource_provider->UnmapImageRasterBuffer(id); |
+ resource_provider->ReleaseImageRasterBuffer(id); |
EXPECT_CALL(*context, NextTextureId()) |
.WillOnce(Return(kTextureId)) |
@@ -3017,12 +3022,12 @@ TEST_P(ResourceProviderTest, Image_GLTexture) { |
EXPECT_CALL(*context, mapImageCHROMIUM(kImageId)) |
.WillOnce(Return(dummy_mapped_buffer_address)) |
.RetiresOnSaturation(); |
- resource_provider->MapImageRasterBuffer(id); |
+ resource_provider->AcquireImageRasterBuffer(id); |
EXPECT_CALL(*context, unmapImageCHROMIUM(kImageId)) |
.Times(1) |
.RetiresOnSaturation(); |
- resource_provider->UnmapImageRasterBuffer(id); |
+ resource_provider->ReleaseImageRasterBuffer(id); |
EXPECT_CALL(*context, bindTexture(GL_TEXTURE_2D, kTextureId)).Times(1) |
.RetiresOnSaturation(); |
@@ -3069,10 +3074,12 @@ TEST_P(ResourceProviderTest, Image_Bitmap) { |
SkBitmap bitmap; |
bitmap.allocN32Pixels(size.width(), size.height()); |
*(bitmap.getAddr32(0, 0)) = kBadBeef; |
- SkCanvas* canvas = resource_provider->MapImageRasterBuffer(id); |
+ RasterBuffer* raster_buffer = resource_provider->AcquireImageRasterBuffer(id); |
+ SkCanvas* canvas = raster_buffer->AcquireSkCanvas(); |
ASSERT_TRUE(!!canvas); |
canvas->writePixels(bitmap, 0, 0); |
- resource_provider->UnmapImageRasterBuffer(id); |
+ raster_buffer->ReleaseSkCanvas(); |
+ resource_provider->ReleaseImageRasterBuffer(id); |
{ |
ResourceProvider::ScopedReadLockSoftware lock(resource_provider.get(), id); |
@@ -3129,13 +3136,13 @@ TEST_P(ResourceProviderTest, CopyResource_GLTexture) { |
EXPECT_CALL(*context, mapImageCHROMIUM(kImageId)) |
.WillOnce(Return(dummy_mapped_buffer_address)) |
.RetiresOnSaturation(); |
- resource_provider->MapImageRasterBuffer(source_id); |
+ resource_provider->AcquireImageRasterBuffer(source_id); |
Mock::VerifyAndClearExpectations(context); |
EXPECT_CALL(*context, unmapImageCHROMIUM(kImageId)) |
.Times(1) |
.RetiresOnSaturation(); |
- resource_provider->UnmapImageRasterBuffer(source_id); |
+ resource_provider->ReleaseImageRasterBuffer(source_id); |
Mock::VerifyAndClearExpectations(context); |
dest_id = resource_provider->CreateResource( |
@@ -3195,10 +3202,13 @@ TEST_P(ResourceProviderTest, CopyResource_Bitmap) { |
SkBitmap bitmap; |
bitmap.allocN32Pixels(size.width(), size.height()); |
*(bitmap.getAddr32(0, 0)) = kBadBeef; |
- SkCanvas* canvas = resource_provider->MapImageRasterBuffer(source_id); |
+ RasterBuffer* raster_buffer = |
+ resource_provider->AcquireImageRasterBuffer(source_id); |
+ SkCanvas* canvas = raster_buffer->AcquireSkCanvas(); |
ASSERT_TRUE(!!canvas); |
canvas->writePixels(bitmap, 0, 0); |
- resource_provider->UnmapImageRasterBuffer(source_id); |
+ raster_buffer->ReleaseSkCanvas(); |
+ resource_provider->ReleaseImageRasterBuffer(source_id); |
dest_id = resource_provider->CreateResource( |
size, GL_CLAMP_TO_EDGE, ResourceProvider::TextureUsageAny, format); |