diff options
author | caryclark <caryclark@google.com> | 2015-07-20 13:38:56 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-07-20 13:38:56 -0700 |
commit | 6531c3619f08cc83aad34883c5f00f6a358c9fad (patch) | |
tree | 5dd35fd7ef486272a17c424bdb4702279d264631 /gm/fontmgr.cpp | |
parent | 973ae86a824c6c1ece428d6777d0bce648f3f20c (diff) |
Generate platform specific fontmgr gm.
The gm output on different font platforms is so different
that comparing images in Gold has little value. Separate the
images by appending platform information to the gm name to
group somewhat similar images together.
Note that this does not attempt to make sure that all images
generated by Gold are nearly pixel identical; it only reduces
the number of nonsensical comparisons.
R=bungeman@google.com
Review URL: https://codereview.chromium.org/1245643002
Diffstat (limited to 'gm/fontmgr.cpp')
-rw-r--r-- | gm/fontmgr.cpp | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/gm/fontmgr.cpp b/gm/fontmgr.cpp index 14ea7ea377..35914e798c 100644 --- a/gm/fontmgr.cpp +++ b/gm/fontmgr.cpp @@ -65,6 +65,8 @@ public: } else { fFM.reset(SkFontMgr::RefDefault()); } + fName.append(sk_tool_utils::platform_os_name()); + fName.append(sk_tool_utils::platform_extra_config("GDI")); } protected: @@ -131,7 +133,10 @@ public: protected: SkString onShortName() override { - return SkString("fontmgr_match"); + SkString name("fontmgr_match"); + name.append(sk_tool_utils::platform_os_name()); + name.append(sk_tool_utils::platform_extra_config("GDI")); + return name; } SkISize onISize() override { @@ -218,6 +223,8 @@ public: if (scale != 1 || skew != 0) { fName.appendf("_%g_%g", scale, skew); } + fName.append(sk_tool_utils::platform_os_name()); + fName.append(sk_tool_utils::platform_extra_config("GDI")); fFM.reset(SkFontMgr::RefDefault()); } |