diff options
-rw-r--r-- | libresourceqt/src/resource-engine.cpp | 22 | ||||
-rw-r--r-- | libresourceqt/src/resource.cpp | 4 | ||||
-rw-r--r-- | tests/test-resource/test-resource.cpp | 97 | ||||
-rw-r--r-- | tests/test-resource/test-resource.h | 6 |
4 files changed, 6 insertions, 123 deletions
diff --git a/libresourceqt/src/resource-engine.cpp b/libresourceqt/src/resource-engine.cpp index 1478e33..e2d6113 100644 --- a/libresourceqt/src/resource-engine.cpp +++ b/libresourceqt/src/resource-engine.cpp @@ -5,7 +5,6 @@ using namespace ResourcePolicy; static inline quint32 allResourcesToBitmask(const ResourceSet *resourceSet); static inline quint32 optionalResourcesToBitmask(const ResourceSet *resourceSet); -static inline quint32 sharedResourcesToBitmask(const ResourceSet *resourceSet); static inline quint32 resourceTypeToLibresourceType(ResourceType type); static void connectionIsUp(resconn_t *connection); @@ -102,7 +101,7 @@ bool ResourceEngine::connect() messageMap.insert(requestId, RESMSG_REGISTER); - uint32_t allResources, optionalResources, sharedResources; + uint32_t allResources, optionalResources; allResources = allResourcesToBitmask(resourceSet); optionalResources = optionalResourcesToBitmask(resourceSet); @@ -132,15 +131,14 @@ bool ResourceEngine::disconnect() messageMap.insert(requestId, RESMSG_UNREGISTER); - uint32_t allResources, optionalResources, sharedResources; + uint32_t allResources, optionalResources; allResources = allResourcesToBitmask(resourceSet); optionalResources = optionalResourcesToBitmask(resourceSet); - sharedResources = sharedResourcesToBitmask(resourceSet); resourceMessage.record.rset.all = allResources; resourceMessage.record.rset.opt = optionalResources; - resourceMessage.record.rset.share = sharedResources; - resourceMessage.record.rset.mask = 0; //find out what it is + resourceMessage.record.rset.share = 0; + resourceMessage.record.rset.mask = mode; QByteArray ba = resourceSet->applicationClass().toLatin1(); resourceMessage.record.klass = ba.data(); @@ -224,18 +222,6 @@ static inline quint32 optionalResourcesToBitmask(const ResourceSet *resourceSet) return bitmask; } -static inline quint32 sharedResourcesToBitmask(const ResourceSet *resourceSet) -{ - QList<Resource *> resourceList = resourceSet->resources(); - quint32 bitmask=0; - for(int i=0; i < resourceList.size(); i++) { - if(resourceList[i]->isShared()) { - bitmask += resourceTypeToLibresourceType(resourceList[i]->type()); - } - } - return bitmask; -} - static void statusCallbackHandler(resset_t *libresourceSet, resmsg_t *message) { ResourceEngine *resourceEngine = reinterpret_cast<ResourceEngine *>(libresourceSet->userdata); diff --git a/libresourceqt/src/resource.cpp b/libresourceqt/src/resource.cpp index 796a345..c9bd79e 100644 --- a/libresourceqt/src/resource.cpp +++ b/libresourceqt/src/resource.cpp @@ -3,14 +3,14 @@ using namespace ResourcePolicy; Resource::Resource() - : optional(false), shared(false), + : optional(false), identifier(0), granted(false) { identifier = (quint32)this; } Resource::Resource(const Resource &other) - : optional(other.optional), shared(other.shared), + : optional(other.optional), identifier(other.identifier), granted(other.granted) { } diff --git a/tests/test-resource/test-resource.cpp b/tests/test-resource/test-resource.cpp index 2098a88..e943fa2 100644 --- a/tests/test-resource/test-resource.cpp +++ b/tests/test-resource/test-resource.cpp @@ -167,101 +167,6 @@ void TestResource::testOptional() QVERIFY(result == expected); } -void TestResource::testShared_data() -{ - QTest::addColumn<ResourceType>("type"); - QTest::addColumn<bool>("shared"); - QTest::addColumn<bool>("expected"); - - QTest::newRow("Resource is shared") << AudioPlaybackType << true << true; - QTest::newRow("Resource is not shared") << AudioPlaybackType << false << false; - QTest::newRow("Resource is shared") << AudioRecorderType << true << true; - QTest::newRow("Resource is not shared") << AudioRecorderType << false << false; - QTest::newRow("Resource is shared") << VideoPlaybackType << true << true; - QTest::newRow("Resource is not shared") << VideoPlaybackType << false << false; - QTest::newRow("Resource is shared") << VideoRecorderType << true << true; - QTest::newRow("Resource is not shared") << VideoRecorderType << false << false; - QTest::newRow("Resource is shared") << VibraType << true << true; - QTest::newRow("Resource is not shared") << VibraType<< false << false; - QTest::newRow("Resource is shared") << BacklightType << true << true; - QTest::newRow("Resource is not shared") << BacklightType << false << false; - QTest::newRow("Resource is shared") << SystemButtonType << true << true; - QTest::newRow("Resource is not shared") << SystemButtonType << false << false; - QTest::newRow("Resource is shared") << LedsType << true << true; - QTest::newRow("Resource is not shared") << LedsType << false << false; - QTest::newRow("Resource is shared") << LockButtonType << true << true; - QTest::newRow("Resource is not shared") << LockButtonType << false << false; - QTest::newRow("Resource is shared") << ScaleButtonType << true << true; - QTest::newRow("Resource is not shared") << ScaleButtonType << false << false; - QTest::newRow("Resource is shared") << SnapButtonType << true << true; - QTest::newRow("Resource is not shared") << SnapButtonType << false << false; - QTest::newRow("Resource is shared") << LensCoverType << true << true; - QTest::newRow("Resource is not shared") << LensCoverType << false << false; -} - -void TestResource::testShared() -{ - QFETCH(ResourceType, type); - QFETCH(bool, shared); - QFETCH(bool, expected); - - Resource *resource = resourceFromType(type); - resource->setShared(shared); - - bool result = resource->isShared(); - - QCOMPARE(result, expected); -} - -void TestResource::testGranted_data() -{ - QTest::addColumn<ResourceType>("type"); - QTest::addColumn<bool>("granted"); - QTest::addColumn<bool>("expected"); - - QTest::newRow("Resource is granted") << AudioPlaybackType << true << true; - QTest::newRow("Resource is denied") << AudioPlaybackType << false << false; - QTest::newRow("Resource is granted") << AudioRecorderType << true << true; - QTest::newRow("Resource is denied") << AudioRecorderType << false << false; - QTest::newRow("Resource is granted") << VideoPlaybackType << true << true; - QTest::newRow("Resource is denied") << VideoPlaybackType << false << false; - QTest::newRow("Resource is granted") << VideoRecorderType << true << true; - QTest::newRow("Resource is denied") << VideoRecorderType << false << false; - QTest::newRow("Resource is granted") << VibraType << true << true; - QTest::newRow("Resource is denied") << VibraType<< false << false; - QTest::newRow("Resource is granted") << BacklightType << true << true; - QTest::newRow("Resource is denied") << BacklightType << false << false; - QTest::newRow("Resource is granted") << SystemButtonType << true << true; - QTest::newRow("Resource is denied") << SystemButtonType << false << false; - QTest::newRow("Resource is granted") << LedsType << true << true; - QTest::newRow("Resource is denied") << LedsType << false << false; - QTest::newRow("Resource is granted") << LockButtonType << true << true; - QTest::newRow("Resource is denied") << LockButtonType << false << false; - QTest::newRow("Resource is granted") << ScaleButtonType << true << true; - QTest::newRow("Resource is denied") << ScaleButtonType << false << false; - QTest::newRow("Resource is granted") << SnapButtonType << true << true; - QTest::newRow("Resource is denied") << SnapButtonType << false << false; - QTest::newRow("Resource is granted") << LensCoverType << true << true; - QTest::newRow("Resource is denied") << LensCoverType << false << false; -} - -void TestResource::testGranted() -{ - QFETCH(ResourceType, type); - QFETCH(bool, granted); - QFETCH(bool, expected); - - Resource *resource = resourceFromType(type); - - if(granted) - resource->setGranted(); - else - resource->unsetGranted(); - bool result = resource->isGranted(); - - QVERIFY(result == expected); -} - void TestResource::testClone() { for(quint32 type=AudioPlaybackType; type < NumberOfTypes; type++) { @@ -270,8 +175,6 @@ void TestResource::testClone() Resource *copy = resource->clone(); QCOMPARE(copy->isOptional(), resource->isOptional()); - QCOMPARE(copy->isShared(), resource->isShared()); - QCOMPARE(copy->isGranted(), resource->isGranted()); QCOMPARE(copy->type(), resource->type()); delete copy; } diff --git a/tests/test-resource/test-resource.h b/tests/test-resource/test-resource.h index 61a48ea..d80f828 100644 --- a/tests/test-resource/test-resource.h +++ b/tests/test-resource/test-resource.h @@ -42,12 +42,6 @@ private slots: void testOptional_data(); void testOptional(); - void testShared_data(); - void testShared(); - - void testGranted_data(); - void testGranted(); - void testClone(); }; |