mirror of
https://github.com/flutter/flutter.git
synced 2026-02-20 02:29:02 +08:00
81c79d6e1e
This fixes following three memory leaks in test utilities. There is no problem in production code and this CL doesn't affect any production behaviors. - SkTypeface leaks due to forget calling SkSafeUnref in dtor. - MinikinFontForTest leaks during constructing FontCollection. - FontCollection leaks due to unnecessary AddRef. Change-Id: I22e1e0307f1b2499296acb1aacc3ef66076a36e9
111 lines
4.2 KiB
C++
111 lines
4.2 KiB
C++
/*
|
|
* Copyright (C) 2015 The Android Open Source Project
|
|
*
|
|
* Licensed under the Apache License, Version 2.0 (the "License");
|
|
* you may not use this file except in compliance with the License.
|
|
* You may obtain a copy of the License at
|
|
*
|
|
* http://www.apache.org/licenses/LICENSE-2.0
|
|
*
|
|
* Unless required by applicable law or agreed to in writing, software
|
|
* distributed under the License is distributed on an "AS IS" BASIS,
|
|
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
* See the License for the specific language governing permissions and
|
|
* limitations under the License.
|
|
*/
|
|
|
|
#include <gtest/gtest.h>
|
|
|
|
#include <minikin/FontCollection.h>
|
|
#include "FontTestUtils.h"
|
|
#include "MinikinFontForTest.h"
|
|
#include "MinikinInternal.h"
|
|
|
|
namespace minikin {
|
|
|
|
// The test font has following glyphs.
|
|
// U+82A6
|
|
// U+82A6 U+FE00 (VS1)
|
|
// U+82A6 U+E0100 (VS17)
|
|
// U+82A6 U+E0101 (VS18)
|
|
// U+82A6 U+E0102 (VS19)
|
|
// U+845B
|
|
// U+845B U+FE01 (VS2)
|
|
// U+845B U+E0101 (VS18)
|
|
// U+845B U+E0102 (VS19)
|
|
// U+845B U+E0103 (VS20)
|
|
// U+537F
|
|
// U+717D U+FE02 (VS3)
|
|
// U+717D U+E0102 (VS19)
|
|
// U+717D U+E0103 (VS20)
|
|
const char kVsTestFont[] = kTestFontDir "/VarioationSelectorTest-Regular.ttf";
|
|
|
|
void expectVSGlyphs(const FontCollection* fc, uint32_t codepoint, const std::set<uint32_t>& vsSet) {
|
|
for (uint32_t vs = 0xFE00; vs <= 0xE01EF; ++vs) {
|
|
// Move to variation selectors supplements after variation selectors.
|
|
if (vs == 0xFF00) {
|
|
vs = 0xE0100;
|
|
}
|
|
if (vsSet.find(vs) == vsSet.end()) {
|
|
EXPECT_FALSE(fc->hasVariationSelector(codepoint, vs))
|
|
<< "Glyph for U+" << std::hex << codepoint << " U+" << vs;
|
|
} else {
|
|
EXPECT_TRUE(fc->hasVariationSelector(codepoint, vs))
|
|
<< "Glyph for U+" << std::hex << codepoint << " U+" << vs;
|
|
}
|
|
}
|
|
}
|
|
|
|
TEST(FontCollectionTest, hasVariationSelectorTest) {
|
|
MinikinAutoUnref<FontFamily> family(new FontFamily());
|
|
MinikinAutoUnref<MinikinFont> font(MinikinFontForTest::createFromFile(kVsTestFont));
|
|
family->addFont(font.get());
|
|
std::vector<FontFamily*> families({family.get()});
|
|
MinikinAutoUnref<FontCollection> fc(new FontCollection(families));
|
|
|
|
EXPECT_FALSE(fc->hasVariationSelector(0x82A6, 0));
|
|
expectVSGlyphs(fc.get(), 0x82A6, std::set<uint32_t>({0xFE00, 0xE0100, 0xE0101, 0xE0102}));
|
|
|
|
EXPECT_FALSE(fc->hasVariationSelector(0x845B, 0));
|
|
expectVSGlyphs(fc.get(), 0x845B, std::set<uint32_t>({0xFE01, 0xE0101, 0xE0102, 0xE0103}));
|
|
|
|
EXPECT_FALSE(fc->hasVariationSelector(0x537F, 0));
|
|
expectVSGlyphs(fc.get(), 0x537F, std::set<uint32_t>({}));
|
|
|
|
EXPECT_FALSE(fc->hasVariationSelector(0x717D, 0));
|
|
expectVSGlyphs(fc.get(), 0x717D, std::set<uint32_t>({0xFE02, 0xE0102, 0xE0103}));
|
|
}
|
|
|
|
const char kEmojiXmlFile[] = kTestFontDir "emoji.xml";
|
|
|
|
TEST(FontCollectionTest, hasVariationSelectorTest_emoji) {
|
|
MinikinAutoUnref<FontCollection> collection(getFontCollection(kTestFontDir, kEmojiXmlFile));
|
|
|
|
// Both text/color font have cmap format 14 subtable entry for VS15/VS16 respectively.
|
|
EXPECT_TRUE(collection->hasVariationSelector(0x2623, 0xFE0E));
|
|
EXPECT_TRUE(collection->hasVariationSelector(0x2623, 0xFE0F));
|
|
|
|
// The text font has cmap format 14 subtable entry for VS15 but the color font doesn't have for
|
|
// VS16
|
|
EXPECT_TRUE(collection->hasVariationSelector(0x2626, 0xFE0E));
|
|
EXPECT_FALSE(collection->hasVariationSelector(0x2626, 0xFE0F));
|
|
|
|
// The color font has cmap format 14 subtable entry for VS16 but the text font doesn't have for
|
|
// VS15.
|
|
EXPECT_TRUE(collection->hasVariationSelector(0x262A, 0xFE0E));
|
|
EXPECT_TRUE(collection->hasVariationSelector(0x262A, 0xFE0F));
|
|
|
|
// Neither text/color font have cmap format 14 subtable entry for VS15/VS16.
|
|
EXPECT_TRUE(collection->hasVariationSelector(0x262E, 0xFE0E));
|
|
EXPECT_FALSE(collection->hasVariationSelector(0x262E, 0xFE0F));
|
|
|
|
// Text font doesn't have U+262F U+FE0E or even its base code point U+262F.
|
|
EXPECT_FALSE(collection->hasVariationSelector(0x262F, 0xFE0E));
|
|
|
|
// VS15/VS16 is only for emoji, should return false for not an emoji code point.
|
|
EXPECT_FALSE(collection->hasVariationSelector(0x2229, 0xFE0E));
|
|
EXPECT_FALSE(collection->hasVariationSelector(0x2229, 0xFE0F));
|
|
}
|
|
|
|
} // namespace minikin
|