aboutsummaryrefslogtreecommitdiffhomepage
path: root/dm
diff options
context:
space:
mode:
authorGravatar Hal Canary <halcanary@google.com>2017-12-08 11:38:47 -0500
committerGravatar Skia Commit-Bot <skia-commit-bot@chromium.org>2017-12-08 19:30:48 +0000
commit5093a539def3ae09df324018f2343827009b2e05 (patch)
tree0809099fad806dd9a8f00fe0e2dfcd8a958915af /dm
parent0b1b9c233a338ef26e7593c1423795c8ced53e1d (diff)
resources: remove most uses of GetResourcePath()
Going forward, we will standardize on GetResourceAsData(), which will make it easier to run tests in environments without access to the filesystem. Also: GetResourceAsData() complains when a resource is missing. This is usually an error. Change-Id: Iaf70b71b0ca5ed8cd1a5538a60ef185ae8736188 Reviewed-on: https://skia-review.googlesource.com/82642 Reviewed-by: Hal Canary <halcanary@google.com> Commit-Queue: Hal Canary <halcanary@google.com>
Diffstat (limited to 'dm')
-rw-r--r--dm/DM.cpp8
-rw-r--r--dm/DMSrcSink.cpp3
2 files changed, 3 insertions, 8 deletions
diff --git a/dm/DM.cpp b/dm/DM.cpp
index d2c127c133..ebd2fe4b56 100644
--- a/dm/DM.cpp
+++ b/dm/DM.cpp
@@ -1357,12 +1357,8 @@ int main(int argc, char** argv) {
SkTaskGroup::Enabler enabled(FLAGS_threads);
gCreateTypefaceDelegate = &create_from_name;
- {
- SkString testResourcePath = GetResourcePath("images/color_wheel.png");
- SkFILEStream testResource(testResourcePath.c_str());
- if (!testResource.isValid()) {
- info("Some resources are missing. Do you need to set --resourcePath?\n");
- }
+ if (nullptr == GetResourceAsData("images/color_wheel.png")) {
+ info("Some resources are missing. Do you need to set --resourcePath?\n");
}
gather_gold();
gather_uninteresting_hashes();
diff --git a/dm/DMSrcSink.cpp b/dm/DMSrcSink.cpp
index 89a4b531e2..96817e0749 100644
--- a/dm/DMSrcSink.cpp
+++ b/dm/DMSrcSink.cpp
@@ -1066,8 +1066,7 @@ Error ColorCodecSrc::draw(SkCanvas* canvas) const {
}
// Load the dst ICC profile. This particular dst is fairly similar to Adobe RGB.
- sk_sp<SkData> dstData = SkData::MakeFromFileName(
- GetResourcePath("icc_profiles/HP_ZR30w.icc").c_str());
+ sk_sp<SkData> dstData = GetResourceAsData("icc_profiles/HP_ZR30w.icc");
if (!dstData) {
return "Cannot read monitor profile. Is the resource path set correctly?";
}