bench/TextBench.cpp - Issue 1933393002: Move SkTypeface to sk_sp. - Code Review

Chromium Code Reviews

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

(1290)

My Issues | Starred Open | Closed | All

Unified Diff: bench/TextBench.cpp

Issue 1933393002: Move SkTypeface to sk_sp. (Closed) Base URL: https://skia.googlesource.com/skia.git@master

Patch Set: Make vc++ happy. Created 4 years, 8 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 side-by-side diff with in-line comments

« no previous file with comments | « bench/SkGlyphCacheBench.cpp ('k') | bench/TextBlobBench.cpp » ('j') | gm/fontmgr.cpp » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: bench/TextBench.cpp

diff --git a/bench/TextBench.cpp b/bench/TextBench.cpp

index 04f824c700c055106de209b2f8e6e630edd1a724..a048ffd4a12ff2265f3a1e75d9ab59fd88dfe996 100644

--- a/bench/TextBench.cpp

+++ b/bench/TextBench.cpp

@@ -47,7 +47,7 @@ class TextBench : public Benchmark {

FontQuality fFQ;

bool fDoPos;

bool fDoColorEmoji;

- SkAutoTUnref<SkTypeface> fColorEmojiTypeface;

+ sk_sp<SkTypeface> fColorEmojiTypeface;

SkPoint* fPos;

public:

TextBench(const char text[], int ps,

@@ -71,7 +71,7 @@ protected:

void onDelayedSetup() override {

if (fDoColorEmoji) {

SkASSERT(kBW == fFQ);

- fColorEmojiTypeface.reset(GetResourceAsTypeface("/fonts/Funkster.ttf"));

+ fColorEmojiTypeface = MakeResourceAsTypeface("/fonts/Funkster.ttf");

}

if (fDoPos) {

« no previous file with comments | « bench/SkGlyphCacheBench.cpp ('k') | bench/TextBlobBench.cpp » ('j') | gm/fontmgr.cpp » ('J')

Powered by Google App Engine

This is Rietveld 408576698