cc/resources/resource_provider_unittest.cc - Issue 105743004: Add gpu::MailboxHolder to hold state for a gpu::Mailbox

Unified Diff: cc/resources/resource_provider_unittest.cc

Issue 105743004: Add gpu::MailboxHolder to hold state for a gpu::Mailbox (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src

Patch Set: c301e01d Rebase. Created 6 years, 12 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View side-by-side diff with in-line comments

Download patch

Index: cc/resources/resource_provider_unittest.cc

diff --git a/cc/resources/resource_provider_unittest.cc b/cc/resources/resource_provider_unittest.cc

index e40c9a7e6fe8fe8fe276e549e5e4170831b27dea..29627cd6c566fb02812914cb75df88e06e18b3a6 100644

--- a/cc/resources/resource_provider_unittest.cc

+++ b/cc/resources/resource_provider_unittest.cc

@@ -37,15 +37,15 @@ using testing::_;

namespace cc {

namespace {

-static void EmptyReleaseCallback(unsigned sync_point, bool lost_resource) {}

+static void EmptyReleaseCallback(uint32 sync_point, bool lost_resource) {}

static void SharedMemoryReleaseCallback(scoped_ptr<base::SharedMemory> memory,

- unsigned sync_point,

+ uint32 sync_point,

bool lost_resource) {}

-static void ReleaseTextureMailbox(unsigned* release_sync_point,

+static void ReleaseTextureMailbox(uint32* release_sync_point,

bool* release_lost_resource,

- unsigned sync_point,

+ uint32 sync_point,

bool lost_resource) {

*release_sync_point = sync_point;

*release_lost_resource = lost_resource;

@@ -54,9 +54,9 @@ static void ReleaseTextureMailbox(unsigned* release_sync_point,

static void ReleaseSharedMemoryCallback(

scoped_ptr<base::SharedMemory> shared_memory,

bool* release_called,

- unsigned* release_sync_point,

+ uint32* release_sync_point,

bool* lost_resource_result,

- unsigned sync_point,

+ uint32 sync_point,

bool lost_resource) {

*release_called = true;

*release_sync_point = sync_point;

@@ -78,8 +78,8 @@ class TextureStateTrackingContext : public TestWebGraphicsContext3D {

public:

MOCK_METHOD2(bindTexture, void(GLenum target, GLuint texture));

MOCK_METHOD3(texParameteri, void(GLenum target, GLenum pname, GLint param));

- MOCK_METHOD1(waitSyncPoint, void(unsigned sync_point));

- MOCK_METHOD0(insertSyncPoint, unsigned(void));

+ MOCK_METHOD1(waitSyncPoint, void(GLuint sync_point));

+ MOCK_METHOD0(insertSyncPoint, GLuint(void));

MOCK_METHOD2(produceTextureCHROMIUM,

void(GLenum target, const GLbyte* mailbox));

MOCK_METHOD2(consumeTextureCHROMIUM,

@@ -102,7 +102,7 @@ class ContextSharedData {

return make_scoped_ptr(new ContextSharedData());

}

- unsigned InsertSyncPoint() { return next_sync_point_++; }

+ GLuint InsertSyncPoint() { return next_sync_point_++; }

danakj 2014/01/06 20:13:42 I think for consistency, things that aren't part o

sheu 2014/01/06 22:44:44 Sure.

void GenMailbox(GLbyte* mailbox) {

memset(mailbox, 0, sizeof(GLbyte[64]));

@@ -111,7 +111,7 @@ class ContextSharedData {

}

void ProduceTexture(const GLbyte* mailbox_name,

- unsigned sync_point,

+ GLuint sync_point,

scoped_refptr<TestTexture> texture) {

unsigned mailbox = 0;

memcpy(&mailbox, mailbox_name, sizeof(mailbox));

@@ -122,7 +122,7 @@ class ContextSharedData {

}

scoped_refptr<TestTexture> ConsumeTexture(const GLbyte* mailbox_name,

- unsigned sync_point) {

+ GLuint sync_point) {

unsigned mailbox = 0;

memcpy(&mailbox, mailbox_name, sizeof(mailbox));

DCHECK(mailbox && mailbox < next_mailbox_);

@@ -140,11 +140,11 @@ class ContextSharedData {

private:

ContextSharedData() : next_sync_point_(1), next_mailbox_(1) {}

- unsigned next_sync_point_;

+ GLuint next_sync_point_;

unsigned next_mailbox_;

typedef base::hash_map<unsigned, scoped_refptr<TestTexture> > TextureMap;

TextureMap textures_;

- base::hash_map<unsigned, unsigned> sync_point_for_mailbox_;

+ base::hash_map<unsigned, GLuint> sync_point_for_mailbox_;

};

class ResourceProviderContext : public TestWebGraphicsContext3D {

@@ -154,8 +154,8 @@ class ResourceProviderContext : public TestWebGraphicsContext3D {

return make_scoped_ptr(new ResourceProviderContext(shared_data));

}

- virtual unsigned insertSyncPoint() OVERRIDE {

- unsigned sync_point = shared_data_->InsertSyncPoint();

+ virtual GLuint insertSyncPoint() OVERRIDE {

+ GLuint sync_point = shared_data_->InsertSyncPoint();

// Commit the produceTextureCHROMIUM calls at this point, so that

// they're associated with the sync point.

for (PendingProduceTextureList::iterator it =

@@ -169,7 +169,7 @@ class ResourceProviderContext : public TestWebGraphicsContext3D {

return sync_point;

}

- virtual void waitSyncPoint(unsigned sync_point) OVERRIDE {

+ virtual void waitSyncPoint(GLuint sync_point) OVERRIDE {

last_waited_sync_point_ = std::max(sync_point, last_waited_sync_point_);

}

@@ -323,7 +323,7 @@ class ResourceProviderContext : public TestWebGraphicsContext3D {

};

typedef ScopedPtrDeque<PendingProduceTexture> PendingProduceTextureList;

ContextSharedData* shared_data_;

- unsigned last_waited_sync_point_;

+ GLuint last_waited_sync_point_;

PendingProduceTextureList pending_produce_textures_;

};

@@ -473,10 +473,10 @@ class ResourceProviderTest

ResourceProviderContext* context() { return context3d_; }

- ResourceProvider::ResourceId CreateChildMailbox(unsigned* release_sync_point,

+ ResourceProvider::ResourceId CreateChildMailbox(uint32* release_sync_point,

bool* lost_resource,

bool* release_called,

- unsigned* sync_point) {

+ uint32* sync_point) {

if (GetParam() == ResourceProvider::GLTexture) {

unsigned texture = child_context_->createTexture();

gpu::Mailbox gpu_mailbox;

@@ -494,7 +494,8 @@ class ResourceProviderTest

release_sync_point,

lost_resource));

return child_resource_provider_->CreateResourceFromTextureMailbox(

- TextureMailbox(gpu_mailbox, *sync_point), callback.Pass());

+ TextureMailbox(gpu_mailbox, GL_TEXTURE_2D, *sync_point),

+ callback.Pass());

} else {

gfx::Size size(64, 64);

scoped_ptr<base::SharedMemory> shared_memory(

@@ -654,7 +655,7 @@ TEST_P(ResourceProviderTest, TransferGLResources) {

child_context_->genMailboxCHROMIUM(external_mailbox.name);

child_context_->produceTextureCHROMIUM(GL_TEXTURE_EXTERNAL_OES,

external_mailbox.name);

- const unsigned external_sync_point = child_context_->insertSyncPoint();

+ const GLuint external_sync_point = child_context_->insertSyncPoint();

ResourceProvider::ResourceId id3 =

child_resource_provider_->CreateResourceFromTextureMailbox(

TextureMailbox(

@@ -674,12 +675,15 @@ TEST_P(ResourceProviderTest, TransferGLResources) {

child_resource_provider_->PrepareSendToParent(resource_ids_to_transfer,

&list);

ASSERT_EQ(3u, list.size());

- EXPECT_NE(0u, list[0].sync_point);

- EXPECT_NE(0u, list[1].sync_point);

- EXPECT_EQ(external_sync_point, list[2].sync_point);

- EXPECT_EQ(static_cast<GLenum>(GL_TEXTURE_2D), list[0].target);

- EXPECT_EQ(static_cast<GLenum>(GL_TEXTURE_2D), list[1].target);

- EXPECT_EQ(static_cast<GLenum>(GL_TEXTURE_EXTERNAL_OES), list[2].target);

+ EXPECT_NE(0u, list[0].mailbox_holder.sync_point);

+ EXPECT_NE(0u, list[1].mailbox_holder.sync_point);

+ EXPECT_EQ(external_sync_point, list[2].mailbox_holder.sync_point);

+ EXPECT_EQ(static_cast<GLenum>(GL_TEXTURE_2D),

+ list[0].mailbox_holder.texture_target);

+ EXPECT_EQ(static_cast<GLenum>(GL_TEXTURE_2D),

+ list[1].mailbox_holder.texture_target);

+ EXPECT_EQ(static_cast<GLenum>(GL_TEXTURE_EXTERNAL_OES),

+ list[2].mailbox_holder.texture_target);

EXPECT_TRUE(child_resource_provider_->InUseByConsumer(id1));

EXPECT_TRUE(child_resource_provider_->InUseByConsumer(id2));

EXPECT_TRUE(child_resource_provider_->InUseByConsumer(id3));

@@ -720,7 +724,8 @@ TEST_P(ResourceProviderTest, TransferGLResources) {

&list);

EXPECT_EQ(1u, list.size());

EXPECT_EQ(id1, list[0].id);

- EXPECT_EQ(static_cast<GLenum>(GL_TEXTURE_2D), list[0].target);

+ EXPECT_EQ(static_cast<GLenum>(GL_TEXTURE_2D),

+ list[0].mailbox_holder.texture_target);

ReturnedResourceArray returned;

TransferableResource::ReturnResources(list, &returned);

child_resource_provider_->ReceiveReturnsFromParent(returned);

@@ -779,12 +784,15 @@ TEST_P(ResourceProviderTest, TransferGLResources) {

EXPECT_EQ(id1, list[0].id);

EXPECT_EQ(id2, list[1].id);

EXPECT_EQ(id3, list[2].id);

- EXPECT_NE(0u, list[0].sync_point);

- EXPECT_NE(0u, list[1].sync_point);

- EXPECT_NE(0u, list[2].sync_point);

- EXPECT_EQ(static_cast<GLenum>(GL_TEXTURE_2D), list[0].target);

- EXPECT_EQ(static_cast<GLenum>(GL_TEXTURE_2D), list[1].target);

- EXPECT_EQ(static_cast<GLenum>(GL_TEXTURE_EXTERNAL_OES), list[2].target);

+ EXPECT_NE(0u, list[0].mailbox_holder.sync_point);

+ EXPECT_NE(0u, list[1].mailbox_holder.sync_point);

+ EXPECT_NE(0u, list[2].mailbox_holder.sync_point);

+ EXPECT_EQ(static_cast<GLenum>(GL_TEXTURE_2D),

+ list[0].mailbox_holder.texture_target);

+ EXPECT_EQ(static_cast<GLenum>(GL_TEXTURE_2D),

+ list[1].mailbox_holder.texture_target);

+ EXPECT_EQ(static_cast<GLenum>(GL_TEXTURE_EXTERNAL_OES),

+ list[2].mailbox_holder.texture_target);

EXPECT_TRUE(child_resource_provider_->InUseByConsumer(id1));

EXPECT_TRUE(child_resource_provider_->InUseByConsumer(id2));

EXPECT_TRUE(child_resource_provider_->InUseByConsumer(id3));

@@ -850,9 +858,9 @@ TEST_P(ResourceProviderTest, TransferSoftwareResources) {

child_resource_provider_->PrepareSendToParent(resource_ids_to_transfer,

&list);

ASSERT_EQ(3u, list.size());

- EXPECT_EQ(0u, list[0].sync_point);

- EXPECT_EQ(0u, list[1].sync_point);

- EXPECT_EQ(0u, list[2].sync_point);

+ EXPECT_EQ(0u, list[0].mailbox_holder.sync_point);

+ EXPECT_EQ(0u, list[1].mailbox_holder.sync_point);

+ EXPECT_EQ(0u, list[2].mailbox_holder.sync_point);

EXPECT_TRUE(child_resource_provider_->InUseByConsumer(id1));

EXPECT_TRUE(child_resource_provider_->InUseByConsumer(id2));

EXPECT_TRUE(child_resource_provider_->InUseByConsumer(id3));

@@ -1078,8 +1086,9 @@ TEST_P(ResourceProviderTest, TransferGLToSoftware) {

child_resource_provider->PrepareSendToParent(resource_ids_to_transfer,

&list);

ASSERT_EQ(1u, list.size());

- EXPECT_NE(0u, list[0].sync_point);

- EXPECT_EQ(static_cast<GLenum>(GL_TEXTURE_2D), list[0].target);

+ EXPECT_NE(0u, list[0].mailbox_holder.sync_point);

+ EXPECT_EQ(static_cast<GLenum>(GL_TEXTURE_2D),

+ list[0].mailbox_holder.texture_target);

EXPECT_TRUE(child_resource_provider->InUseByConsumer(id1));

resource_provider_->ReceiveFromChild(child_id, list);

}

@@ -1125,7 +1134,7 @@ TEST_P(ResourceProviderTest, TransferInvalidSoftware) {

&list);

ASSERT_EQ(1u, list.size());

// Make invalid.

- list[0].mailbox.name[1] = 5;

+ list[0].mailbox_holder.mailbox.name[1] = 5;

EXPECT_TRUE(child_resource_provider_->InUseByConsumer(id1));

resource_provider_->ReceiveFromChild(child_id, list);

}

@@ -1175,8 +1184,8 @@ TEST_P(ResourceProviderTest, DeleteExportedResources) {

&list);

ASSERT_EQ(2u, list.size());

if (GetParam() == ResourceProvider::GLTexture) {

- EXPECT_NE(0u, list[0].sync_point);

- EXPECT_NE(0u, list[1].sync_point);

+ EXPECT_NE(0u, list[0].mailbox_holder.sync_point);

+ EXPECT_NE(0u, list[1].mailbox_holder.sync_point);

}

EXPECT_TRUE(child_resource_provider_->InUseByConsumer(id1));

EXPECT_TRUE(child_resource_provider_->InUseByConsumer(id2));

@@ -1205,8 +1214,8 @@ TEST_P(ResourceProviderTest, DeleteExportedResources) {

ASSERT_EQ(2u, list.size());

if (GetParam() == ResourceProvider::GLTexture) {

- EXPECT_NE(0u, list[0].sync_point);

- EXPECT_NE(0u, list[1].sync_point);

+ EXPECT_NE(0u, list[0].mailbox_holder.sync_point);

+ EXPECT_NE(0u, list[1].mailbox_holder.sync_point);

}

EXPECT_TRUE(resource_provider_->InUseByConsumer(id1));

EXPECT_TRUE(resource_provider_->InUseByConsumer(id2));

@@ -1269,8 +1278,8 @@ TEST_P(ResourceProviderTest, DestroyChildWithExportedResources) {

&list);

ASSERT_EQ(2u, list.size());

if (GetParam() == ResourceProvider::GLTexture) {

- EXPECT_NE(0u, list[0].sync_point);

- EXPECT_NE(0u, list[1].sync_point);

+ EXPECT_NE(0u, list[0].mailbox_holder.sync_point);

+ EXPECT_NE(0u, list[1].mailbox_holder.sync_point);

}

EXPECT_TRUE(child_resource_provider_->InUseByConsumer(id1));

EXPECT_TRUE(child_resource_provider_->InUseByConsumer(id2));

@@ -1299,8 +1308,8 @@ TEST_P(ResourceProviderTest, DestroyChildWithExportedResources) {

ASSERT_EQ(2u, list.size());

if (GetParam() == ResourceProvider::GLTexture) {

- EXPECT_NE(0u, list[0].sync_point);

- EXPECT_NE(0u, list[1].sync_point);

+ EXPECT_NE(0u, list[0].mailbox_holder.sync_point);

+ EXPECT_NE(0u, list[1].mailbox_holder.sync_point);

}

EXPECT_TRUE(resource_provider_->InUseByConsumer(id1));

EXPECT_TRUE(resource_provider_->InUseByConsumer(id2));

@@ -1374,7 +1383,7 @@ TEST_P(ResourceProviderTest, DeleteTransferredResources) {

&list);

ASSERT_EQ(1u, list.size());

if (GetParam() == ResourceProvider::GLTexture)

- EXPECT_NE(0u, list[0].sync_point);

+ EXPECT_NE(0u, list[0].mailbox_holder.sync_point);

EXPECT_TRUE(child_resource_provider_->InUseByConsumer(id));

resource_provider_->ReceiveFromChild(child_id, list);

resource_provider_->DeclareUsedResourcesFromChild(child_id,

@@ -1588,18 +1597,18 @@ TEST_P(ResourceProviderTest, TransferMailboxResources) {

gpu::Mailbox mailbox;

context()->genMailboxCHROMIUM(mailbox.name);

context()->produceTextureCHROMIUM(GL_TEXTURE_2D, mailbox.name);

- unsigned sync_point = context()->insertSyncPoint();

+ uint32 sync_point = context()->insertSyncPoint();

// All the logic below assumes that the sync points are all positive.

EXPECT_LT(0u, sync_point);

- unsigned release_sync_point = 0;

+ uint32 release_sync_point = 0;

bool lost_resource = false;

ReleaseCallback callback =

base::Bind(ReleaseTextureMailbox, &release_sync_point, &lost_resource);

ResourceProvider::ResourceId resource =

resource_provider_->CreateResourceFromTextureMailbox(

- TextureMailbox(mailbox, sync_point),

+ TextureMailbox(mailbox, GL_TEXTURE_2D, sync_point),

SingleReleaseCallback::Create(callback));

EXPECT_EQ(1u, context()->NumTextures());

EXPECT_EQ(0u, release_sync_point);

@@ -1610,12 +1619,14 @@ TEST_P(ResourceProviderTest, TransferMailboxResources) {

TransferableResourceArray list;

resource_provider_->PrepareSendToParent(resource_ids_to_transfer, &list);

ASSERT_EQ(1u, list.size());

- EXPECT_LE(sync_point, list[0].sync_point);

+ EXPECT_LE(sync_point, list[0].mailbox_holder.sync_point);

EXPECT_EQ(0,

- memcmp(mailbox.name, list[0].mailbox.name, sizeof(mailbox.name)));

+ memcmp(mailbox.name,

+ list[0].mailbox_holder.mailbox.name,

+ sizeof(mailbox.name)));

EXPECT_EQ(0u, release_sync_point);

- context()->waitSyncPoint(list[0].sync_point);

+ context()->waitSyncPoint(list[0].mailbox_holder.sync_point);

unsigned other_texture = context()->createTexture();

context()->bindTexture(GL_TEXTURE_2D, other_texture);

context()->consumeTextureCHROMIUM(GL_TEXTURE_2D, mailbox.name);

@@ -1625,8 +1636,8 @@ TEST_P(ResourceProviderTest, TransferMailboxResources) {

EXPECT_EQ(0, memcmp(data, test_data, sizeof(data)));

context()->produceTextureCHROMIUM(GL_TEXTURE_2D, mailbox.name);

context()->deleteTexture(other_texture);

- list[0].sync_point = context()->insertSyncPoint();

- EXPECT_LT(0u, list[0].sync_point);

+ list[0].mailbox_holder.sync_point = context()->insertSyncPoint();

+ EXPECT_LT(0u, list[0].mailbox_holder.sync_point);

// Receive the resource, then delete it, expect the sync points to be

// consistent.

@@ -1637,7 +1648,7 @@ TEST_P(ResourceProviderTest, TransferMailboxResources) {

EXPECT_EQ(0u, release_sync_point);

resource_provider_->DeleteResource(resource);

- EXPECT_LE(list[0].sync_point, release_sync_point);

+ EXPECT_LE(list[0].mailbox_holder.sync_point, release_sync_point);

EXPECT_FALSE(lost_resource);

}

@@ -1647,7 +1658,7 @@ TEST_P(ResourceProviderTest, TransferMailboxResources) {

EXPECT_LT(0u, sync_point);

release_sync_point = 0;

resource = resource_provider_->CreateResourceFromTextureMailbox(

- TextureMailbox(mailbox, sync_point),

+ TextureMailbox(mailbox, GL_TEXTURE_2D, sync_point),

SingleReleaseCallback::Create(callback));

EXPECT_EQ(1u, context()->NumTextures());

EXPECT_EQ(0u, release_sync_point);

@@ -1658,12 +1669,14 @@ TEST_P(ResourceProviderTest, TransferMailboxResources) {

TransferableResourceArray list;

resource_provider_->PrepareSendToParent(resource_ids_to_transfer, &list);

ASSERT_EQ(1u, list.size());

- EXPECT_LE(sync_point, list[0].sync_point);

+ EXPECT_LE(sync_point, list[0].mailbox_holder.sync_point);

EXPECT_EQ(0,

- memcmp(mailbox.name, list[0].mailbox.name, sizeof(mailbox.name)));

+ memcmp(mailbox.name,

+ list[0].mailbox_holder.mailbox.name,

+ sizeof(mailbox.name)));

EXPECT_EQ(0u, release_sync_point);

- context()->waitSyncPoint(list[0].sync_point);

+ context()->waitSyncPoint(list[0].mailbox_holder.sync_point);

unsigned other_texture = context()->createTexture();

context()->bindTexture(GL_TEXTURE_2D, other_texture);

context()->consumeTextureCHROMIUM(GL_TEXTURE_2D, mailbox.name);

@@ -1673,8 +1686,8 @@ TEST_P(ResourceProviderTest, TransferMailboxResources) {

EXPECT_EQ(0, memcmp(data, test_data, sizeof(data)));

context()->produceTextureCHROMIUM(GL_TEXTURE_2D, mailbox.name);

context()->deleteTexture(other_texture);

- list[0].sync_point = context()->insertSyncPoint();

- EXPECT_LT(0u, list[0].sync_point);

+ list[0].mailbox_holder.sync_point = context()->insertSyncPoint();

+ EXPECT_LT(0u, list[0].mailbox_holder.sync_point);

// Delete the resource, which shouldn't do anything.

resource_provider_->DeleteResource(resource);

@@ -1686,7 +1699,7 @@ TEST_P(ResourceProviderTest, TransferMailboxResources) {

ReturnedResourceArray returned;

TransferableResource::ReturnResources(list, &returned);

resource_provider_->ReceiveReturnsFromParent(returned);

- EXPECT_LE(list[0].sync_point, release_sync_point);

+ EXPECT_LE(list[0].mailbox_holder.sync_point, release_sync_point);

EXPECT_FALSE(lost_resource);

}

@@ -1826,10 +1839,10 @@ TEST_P(ResourceProviderTest, LostResourceInGrandParent) {

}

TEST_P(ResourceProviderTest, LostMailboxInParent) {

- unsigned release_sync_point = 0;

+ uint32 release_sync_point = 0;

bool lost_resource = false;

bool release_called = false;

- unsigned sync_point = 0;

+ uint32 sync_point = 0;

ResourceProvider::ResourceId resource = CreateChildMailbox(

&release_sync_point, &lost_resource, &release_called, &sync_point);

@@ -1876,10 +1889,10 @@ TEST_P(ResourceProviderTest, LostMailboxInParent) {

}

TEST_P(ResourceProviderTest, LostMailboxInGrandParent) {

- unsigned release_sync_point = 0;

+ uint32 release_sync_point = 0;

bool lost_resource = false;

bool release_called = false;

- unsigned sync_point = 0;

+ uint32 sync_point = 0;

ResourceProvider::ResourceId resource = CreateChildMailbox(

&release_sync_point, &lost_resource, &release_called, &sync_point);

@@ -1944,10 +1957,10 @@ TEST_P(ResourceProviderTest, LostMailboxInGrandParent) {

}

TEST_P(ResourceProviderTest, Shutdown) {

- unsigned release_sync_point = 0;

+ uint32 release_sync_point = 0;

bool lost_resource = false;

bool release_called = false;

- unsigned sync_point = 0;

+ uint32 sync_point = 0;

CreateChildMailbox(

&release_sync_point, &lost_resource, &release_called, &sync_point);

@@ -1964,10 +1977,10 @@ TEST_P(ResourceProviderTest, Shutdown) {

}

TEST_P(ResourceProviderTest, ShutdownWithExportedResource) {

- unsigned release_sync_point = 0;

+ uint32 release_sync_point = 0;

bool lost_resource = false;

bool release_called = false;

- unsigned sync_point = 0;

+ uint32 sync_point = 0;

ResourceProvider::ResourceId resource = CreateChildMailbox(

&release_sync_point, &lost_resource, &release_called, &sync_point);

@@ -1997,17 +2010,16 @@ TEST_P(ResourceProviderTest, LostContext) {

gpu::Mailbox mailbox;

context()->genMailboxCHROMIUM(mailbox.name);

context()->produceTextureCHROMIUM(GL_TEXTURE_2D, mailbox.name);

- unsigned sync_point = context()->insertSyncPoint();

+ uint32 sync_point = context()->insertSyncPoint();

EXPECT_LT(0u, sync_point);

- unsigned release_sync_point = 0;

+ uint32 release_sync_point = 0;

bool lost_resource = false;

scoped_ptr<SingleReleaseCallback> callback = SingleReleaseCallback::Create(

base::Bind(ReleaseTextureMailbox, &release_sync_point, &lost_resource));

resource_provider_->CreateResourceFromTextureMailbox(

- TextureMailbox(mailbox, sync_point),

- callback.Pass());

+ TextureMailbox(mailbox, GL_TEXTURE_2D, sync_point), callback.Pass());

EXPECT_EQ(0u, release_sync_point);

EXPECT_FALSE(lost_resource);

@@ -2256,7 +2268,7 @@ TEST_P(ResourceProviderTest, TextureMailbox_GLTexture2D) {

ResourceProvider::Create(output_surface.get(), NULL, 0, false, 1));

unsigned texture_id = 1;

- unsigned sync_point = 30;

+ uint32 sync_point = 30;

unsigned target = GL_TEXTURE_2D;

EXPECT_CALL(*context, bindTexture(_, _)).Times(0);

@@ -2270,7 +2282,7 @@ TEST_P(ResourceProviderTest, TextureMailbox_GLTexture2D) {

scoped_ptr<SingleReleaseCallback> callback = SingleReleaseCallback::Create(

base::Bind(&EmptyReleaseCallback));

- TextureMailbox mailbox(gpu_mailbox, sync_point);

+ TextureMailbox mailbox(gpu_mailbox, target, sync_point);

ResourceProvider::ResourceId id =

resource_provider->CreateResourceFromTextureMailbox(

@@ -2320,7 +2332,7 @@ TEST_P(ResourceProviderTest, TextureMailbox_GLTextureExternalOES) {

ResourceProvider::Create(output_surface.get(), NULL, 0, false, 1));

unsigned texture_id = 1;

- unsigned sync_point = 30;

+ uint32 sync_point = 30;

unsigned target = GL_TEXTURE_EXTERNAL_OES;

EXPECT_CALL(*context, bindTexture(_, _)).Times(0);

« cc/resources/resource_provider.cc ('K') | « cc/resources/resource_provider.cc ('k') | cc/resources/single_release_callback.h » ('j') | cc/resources/texture_mailbox.h » ('J')