Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit c34027ec authored by Alexandria Cornwall's avatar Alexandria Cornwall Committed by Android (Google) Code Review
Browse files

Merge "AAPT2: Fix resource type id symbol merging"

parents 9cce5d04 6a1f8dbd

tools/aapt2/link/TableMerger.cpp

+1 −1
Original line number Diff line number Diff line
@@ -142,7 +142,7 @@ bool TableMerger::doMerge(const Source& src, 
        ResourceTableType* dstType = mMasterPackage->findOrCreateType(srcType->type);

        if (srcType->symbolStatus.state == SymbolState::kPublic) {

            if (dstType->symbolStatus.state == SymbolState::kPublic && dstType->id && srcType->id

                    && dstType->id.value() == srcType->id.value()) {

                    && dstType->id.value() != srcType->id.value()) {

                // Both types are public and have different IDs.

                mContext->getDiagnostics()->error(DiagMessage(src)

                                                  << "can not merge type '"

tools/aapt2/link/TableMerger_test.cpp

+57 −0
Original line number Diff line number Diff line
@@ -164,6 +164,63 @@ TEST_F(TableMergerTest, OverrideResourceWithOverlay) { 
    EXPECT_EQ(0x0u, foo->value.data);

}



TEST_F(TableMergerTest, OverrideSameResourceIdsWithOverlay) {

    std::unique_ptr<ResourceTable> base = test::ResourceTableBuilder()

            .setPackageId("", 0x7f)

            .setSymbolState("bool/foo", ResourceId(0x7f, 0x01, 0x0001), SymbolState::kPublic)

            .build();

    std::unique_ptr<ResourceTable> overlay = test::ResourceTableBuilder()

            .setPackageId("", 0x7f)

            .setSymbolState("bool/foo", ResourceId(0x7f, 0x01, 0x0001), SymbolState::kPublic)

            .build();



    ResourceTable finalTable;

    TableMergerOptions tableMergerOptions;

    tableMergerOptions.autoAddOverlay = false;

    TableMerger merger(mContext.get(), &finalTable, tableMergerOptions);



    ASSERT_TRUE(merger.merge({}, base.get()));

    ASSERT_TRUE(merger.mergeOverlay({}, overlay.get()));

}



TEST_F(TableMergerTest, FailToOverrideConflictingTypeIdsWithOverlay) {

    std::unique_ptr<ResourceTable> base = test::ResourceTableBuilder()

            .setPackageId("", 0x7f)

            .setSymbolState("bool/foo", ResourceId(0x7f, 0x01, 0x0001), SymbolState::kPublic)

            .build();

    std::unique_ptr<ResourceTable> overlay = test::ResourceTableBuilder()

            .setPackageId("", 0x7f)

            .setSymbolState("bool/foo", ResourceId(0x7f, 0x02, 0x0001), SymbolState::kPublic)

            .build();



    ResourceTable finalTable;

    TableMergerOptions tableMergerOptions;

    tableMergerOptions.autoAddOverlay = false;

    TableMerger merger(mContext.get(), &finalTable, tableMergerOptions);



    ASSERT_TRUE(merger.merge({}, base.get()));

    ASSERT_FALSE(merger.mergeOverlay({}, overlay.get()));

}



TEST_F(TableMergerTest, FailToOverrideConflictingEntryIdsWithOverlay) {

    std::unique_ptr<ResourceTable> base = test::ResourceTableBuilder()

            .setPackageId("", 0x7f)

            .setSymbolState("bool/foo", ResourceId(0x7f, 0x01, 0x0001), SymbolState::kPublic)

            .build();

    std::unique_ptr<ResourceTable> overlay = test::ResourceTableBuilder()

            .setPackageId("", 0x7f)

            .setSymbolState("bool/foo", ResourceId(0x7f, 0x01, 0x0002), SymbolState::kPublic)

            .build();



    ResourceTable finalTable;

    TableMergerOptions tableMergerOptions;

    tableMergerOptions.autoAddOverlay = false;

    TableMerger merger(mContext.get(), &finalTable, tableMergerOptions);



    ASSERT_TRUE(merger.merge({}, base.get()));

    ASSERT_FALSE(merger.mergeOverlay({}, overlay.get()));

}



TEST_F(TableMergerTest, MergeAddResourceFromOverlay) {

    std::unique_ptr<ResourceTable> tableA = test::ResourceTableBuilder()

            .setPackageId("", 0x7f)