Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(167)

Side by Side Diff: cc/resources/resource_provider_unittest.cc

Issue 1142343008: cc: Rework overlays to not use the ResourceProvider and pass texture size (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix more tests Created 5 years, 6 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 unified diff | Download patch
OLDNEW
1 // Copyright 2012 The Chromium Authors. All rights reserved. 1 // Copyright 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "cc/resources/resource_provider.h" 5 #include "cc/resources/resource_provider.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <map> 8 #include <map>
9 #include <set> 9 #include <set>
10 10
(...skipping 1027 matching lines...) Expand 10 before | Expand all | Expand 10 after
1038 id1); 1038 id1);
1039 child_resource_provider_->DeleteResource(id1); 1039 child_resource_provider_->DeleteResource(id1);
1040 EXPECT_EQ(1u, child_resource_provider_->num_resources()); 1040 EXPECT_EQ(1u, child_resource_provider_->num_resources());
1041 EXPECT_TRUE(child_resource_provider_->InUseByConsumer(id1)); 1041 EXPECT_TRUE(child_resource_provider_->InUseByConsumer(id1));
1042 } 1042 }
1043 1043
1044 EXPECT_EQ(0u, child_resource_provider_->num_resources()); 1044 EXPECT_EQ(0u, child_resource_provider_->num_resources());
1045 resource_provider_->DestroyChild(child_id); 1045 resource_provider_->DestroyChild(child_id);
1046 } 1046 }
1047 1047
1048 TEST_P(ResourceProviderTest, AllowOverlayTransfersToParent) {
1049 // Overlays only supported on the GL path.
1050 if (GetParam() != ResourceProvider::RESOURCE_TYPE_GL_TEXTURE)
1051 return;
1052
1053 uint32 sync_point = 0;
1054 TextureMailbox mailbox(gpu::Mailbox::Generate(), GL_TEXTURE_2D, sync_point);
1055 mailbox.set_allow_overlay(true);
1056 scoped_ptr<SingleReleaseCallbackImpl> release_callback =
1057 SingleReleaseCallbackImpl::Create(base::Bind(&EmptyReleaseCallback));
1058 ResourceId id1 = child_resource_provider_->CreateResourceFromTextureMailbox(
1059 mailbox, release_callback.Pass());
1060
1061 TextureMailbox mailbox2(gpu::Mailbox::Generate(), GL_TEXTURE_2D, sync_point);
1062 mailbox2.set_allow_overlay(false);
1063 scoped_ptr<SingleReleaseCallbackImpl> release_callback2 =
1064 SingleReleaseCallbackImpl::Create(base::Bind(&EmptyReleaseCallback));
1065 ResourceId id2 = child_resource_provider_->CreateResourceFromTextureMailbox(
1066 mailbox2, release_callback2.Pass());
1067
1068 ReturnedResourceArray returned_to_child;
1069 int child_id =
1070 resource_provider_->CreateChild(GetReturnCallback(&returned_to_child));
1071
1072 // Transfer some resources to the parent.
1073 ResourceProvider::ResourceIdArray resource_ids_to_transfer;
1074 resource_ids_to_transfer.push_back(id1);
1075 resource_ids_to_transfer.push_back(id2);
1076 TransferableResourceArray list;
1077 child_resource_provider_->PrepareSendToParent(resource_ids_to_transfer,
1078 &list);
1079 ASSERT_EQ(2u, list.size());
1080 resource_provider_->ReceiveFromChild(child_id, list);
1081 EXPECT_TRUE(resource_provider_->AllowOverlay(list[0].id));
1082 EXPECT_FALSE(resource_provider_->AllowOverlay(list[1].id));
1083
1084 resource_provider_->DeclareUsedResourcesFromChild(
1085 child_id, ResourceProvider::ResourceIdSet());
1086
1087 EXPECT_EQ(2u, returned_to_child.size());
1088 child_resource_provider_->ReceiveReturnsFromParent(returned_to_child);
1089
1090 child_resource_provider_->DeleteResource(id1);
1091 child_resource_provider_->DeleteResource(id2);
1092 EXPECT_EQ(0u, child_resource_provider_->num_resources());
1093
1094 resource_provider_->DestroyChild(child_id);
1095 }
1096
1097 TEST_P(ResourceProviderTest, TransferSoftwareResources) { 1048 TEST_P(ResourceProviderTest, TransferSoftwareResources) {
1098 if (GetParam() != ResourceProvider::RESOURCE_TYPE_BITMAP) 1049 if (GetParam() != ResourceProvider::RESOURCE_TYPE_BITMAP)
1099 return; 1050 return;
1100 1051
1101 gfx::Size size(1, 1); 1052 gfx::Size size(1, 1);
1102 ResourceFormat format = RGBA_8888; 1053 ResourceFormat format = RGBA_8888;
1103 size_t pixel_size = TextureSizeBytes(size, format); 1054 size_t pixel_size = TextureSizeBytes(size, format);
1104 ASSERT_EQ(4U, pixel_size); 1055 ASSERT_EQ(4U, pixel_size);
1105 1056
1106 ResourceId id1 = child_resource_provider_->CreateResource( 1057 ResourceId id1 = child_resource_provider_->CreateResource(
(...skipping 2570 matching lines...) Expand 10 before | Expand all | Expand 10 after
3677 resource_provider->AllocateForTesting(id); 3628 resource_provider->AllocateForTesting(id);
3678 Mock::VerifyAndClearExpectations(context); 3629 Mock::VerifyAndClearExpectations(context);
3679 3630
3680 DCHECK_EQ(10u, context->PeekTextureId()); 3631 DCHECK_EQ(10u, context->PeekTextureId());
3681 resource_provider->DeleteResource(id); 3632 resource_provider->DeleteResource(id);
3682 } 3633 }
3683 } 3634 }
3684 3635
3685 } // namespace 3636 } // namespace
3686 } // namespace cc 3637 } // namespace cc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698