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

Side by Side Diff: tests/TypefaceTest.cpp

Issue 2333713002: change SkStreams to work with sk_sp<SkData> instead of SkData* (Closed)
Patch Set: fix xpsdevice Created 4 years, 3 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
« no previous file with comments | « tests/StreamTest.cpp ('k') | tools/debugger/SkDrawCommand.cpp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright 2013 Google Inc. 2 * Copyright 2013 Google Inc.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license that can be 4 * Use of this source code is governed by a BSD-style license that can be
5 * found in the LICENSE file. 5 * found in the LICENSE file.
6 */ 6 */
7 7
8 #include "SkData.h" 8 #include "SkData.h"
9 #include "SkOTTable_OS_2.h" 9 #include "SkOTTable_OS_2.h"
10 #include "SkSFNTHeader.h" 10 #include "SkSFNTHeader.h"
11 #include "SkStream.h" 11 #include "SkStream.h"
12 #include "SkRefCnt.h" 12 #include "SkRefCnt.h"
13 #include "SkTypeface.h" 13 #include "SkTypeface.h"
14 #include "SkTypefaceCache.h" 14 #include "SkTypefaceCache.h"
15 #include "Resources.h" 15 #include "Resources.h"
16 #include "Test.h" 16 #include "Test.h"
17 17
18 #include <memory> 18 #include <memory>
19 19
20 static void TypefaceStyle_test(skiatest::Reporter* reporter, 20 static void TypefaceStyle_test(skiatest::Reporter* reporter,
21 uint16_t weight, uint16_t width, SkData* data) 21 uint16_t weight, uint16_t width, SkData* data)
22 { 22 {
23 sk_sp<SkData> dataCopy; 23 sk_sp<SkData> dataCopy;
24 SkData* dataToUse = data; 24 if (!data->unique()) {
25 if (!dataToUse->unique()) {
26 dataCopy = SkData::MakeWithCopy(data->data(), data->size()); 25 dataCopy = SkData::MakeWithCopy(data->data(), data->size());
27 dataToUse = dataCopy.get(); 26 data = dataCopy.get();
28 } 27 }
29 SkSFNTHeader* sfntHeader = static_cast<SkSFNTHeader*>(dataToUse->writable_da ta()); 28 SkSFNTHeader* sfntHeader = static_cast<SkSFNTHeader*>(data->writable_data()) ;
30 29
31 SkSFNTHeader::TableDirectoryEntry* tableEntry = 30 SkSFNTHeader::TableDirectoryEntry* tableEntry =
32 SkTAfter<SkSFNTHeader::TableDirectoryEntry>(sfntHeader); 31 SkTAfter<SkSFNTHeader::TableDirectoryEntry>(sfntHeader);
33 SkSFNTHeader::TableDirectoryEntry* os2TableEntry = nullptr; 32 SkSFNTHeader::TableDirectoryEntry* os2TableEntry = nullptr;
34 int numTables = SkEndian_SwapBE16(sfntHeader->numTables); 33 int numTables = SkEndian_SwapBE16(sfntHeader->numTables);
35 for (int tableEntryIndex = 0; tableEntryIndex < numTables; ++tableEntryIndex ) { 34 for (int tableEntryIndex = 0; tableEntryIndex < numTables; ++tableEntryIndex ) {
36 if (SkOTTableOS2::TAG == tableEntry[tableEntryIndex].tag) { 35 if (SkOTTableOS2::TAG == tableEntry[tableEntryIndex].tag) {
37 os2TableEntry = tableEntry + tableEntryIndex; 36 os2TableEntry = tableEntry + tableEntryIndex;
38 break; 37 break;
39 } 38 }
40 } 39 }
41 SkASSERT_RELEASE(os2TableEntry); 40 SkASSERT_RELEASE(os2TableEntry);
42 41
43 size_t os2TableOffset = SkEndian_SwapBE32(os2TableEntry->offset); 42 size_t os2TableOffset = SkEndian_SwapBE32(os2TableEntry->offset);
44 SkOTTableOS2_V0* os2Table = SkTAddOffset<SkOTTableOS2_V0>(sfntHeader, os2Tab leOffset); 43 SkOTTableOS2_V0* os2Table = SkTAddOffset<SkOTTableOS2_V0>(sfntHeader, os2Tab leOffset);
45 os2Table->usWeightClass.value = SkEndian_SwapBE16(weight); 44 os2Table->usWeightClass.value = SkEndian_SwapBE16(weight);
46 using WidthType = SkOTTableOS2_V0::WidthClass::Value; 45 using WidthType = SkOTTableOS2_V0::WidthClass::Value;
47 os2Table->usWidthClass.value = static_cast<WidthType>(SkEndian_SwapBE16(widt h)); 46 os2Table->usWidthClass.value = static_cast<WidthType>(SkEndian_SwapBE16(widt h));
48 47
49 sk_sp<SkTypeface> newTypeface(SkTypeface::MakeFromStream(new SkMemoryStream( dataToUse))); 48 sk_sp<SkTypeface> newTypeface(SkTypeface::MakeFromStream(new SkMemoryStream( sk_ref_sp(data))));
50 SkASSERT_RELEASE(newTypeface); 49 SkASSERT_RELEASE(newTypeface);
51 50
52 SkFontStyle newStyle = newTypeface->fontStyle(); 51 SkFontStyle newStyle = newTypeface->fontStyle();
53 52
54 //printf("%d, %f\n", weight, (newStyle.weight() - (float)0x7FFF) / (float)0x 7FFF); 53 //printf("%d, %f\n", weight, (newStyle.weight() - (float)0x7FFF) / (float)0x 7FFF);
55 //printf("%d, %f\n", width , (newStyle.width() - (float)0x7F) / (float)0x 7F); 54 //printf("%d, %f\n", width , (newStyle.width() - (float)0x7F) / (float)0x 7F);
56 //printf("%d, %d\n", weight, newStyle.weight()); 55 //printf("%d, %d\n", weight, newStyle.weight());
57 //printf("%d, %d\n", width , newStyle.width()); 56 //printf("%d, %d\n", width , newStyle.width());
58 57
59 // Some back-ends (CG, GDI, DW) support OS/2 version A which uses 0 - 10 (bu t all differently). 58 // Some back-ends (CG, GDI, DW) support OS/2 version A which uses 0 - 10 (bu t all differently).
(...skipping 106 matching lines...) Expand 10 before | Expand all | Expand 10 after
166 REPORTER_ASSERT(reporter, count(reporter, cache) == 2); 165 REPORTER_ASSERT(reporter, count(reporter, cache) == 2);
167 cache.purgeAll(); 166 cache.purgeAll();
168 REPORTER_ASSERT(reporter, count(reporter, cache) == 2); 167 REPORTER_ASSERT(reporter, count(reporter, cache) == 2);
169 } 168 }
170 REPORTER_ASSERT(reporter, count(reporter, cache) == 2); 169 REPORTER_ASSERT(reporter, count(reporter, cache) == 2);
171 cache.purgeAll(); 170 cache.purgeAll();
172 REPORTER_ASSERT(reporter, count(reporter, cache) == 1); 171 REPORTER_ASSERT(reporter, count(reporter, cache) == 1);
173 } 172 }
174 REPORTER_ASSERT(reporter, t1->unique()); 173 REPORTER_ASSERT(reporter, t1->unique());
175 } 174 }
OLDNEW
« no previous file with comments | « tests/StreamTest.cpp ('k') | tools/debugger/SkDrawCommand.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698