commit 5093a539def3ae09df324018f2343827009b2e05
author Hal Canary <halcanary@google.com> Fri Dec 08 11:38:47 2017 -0500
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Fri Dec 08 19:30:48 2017 +0000
tree 0809099fad806dd9a8f00fe0e2dfcd8a958915af
parent 0b1b9c233a338ef26e7593c1423795c8ced53e1d

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>

samplecode/SampleSubpixelTranslate.cpp

diff --git a/samplecode/SampleSubpixelTranslate.cpp b/samplecode/SampleSubpixelTranslate.cpp
index 51b5ef6..7ce1885 100644
--- a/samplecode/SampleSubpixelTranslate.cpp
+++ b/samplecode/SampleSubpixelTranslate.cpp
@@ -24,15 +24,15 @@
     SubpixelTranslateView(const char imageFilename[],
                           float horizontalVelocity,
                           float verticalVelocity)
-      : fHorizontalVelocity(horizontalVelocity),
-        fVerticalVelocity(verticalVelocity) {
-      SkString resourcePath = GetResourcePath(imageFilename);
-      if (!decode_file(resourcePath.c_str(), &fBM)) {
-          fBM.allocN32Pixels(1, 1);
-          *(fBM.getAddr32(0,0)) = 0xFF0000FF; // red == bad
-      }
-      fCurPos = SkPoint::Make(0,0);
-      fSize = 200;
+        : fHorizontalVelocity(horizontalVelocity)
+        , fVerticalVelocity(verticalVelocity)
+    {
+        if (!DecodeDataToBitmap(GetResourceAsData(imageFilename), &fBM)) {
+            fBM.allocN32Pixels(1, 1);
+            *(fBM.getAddr32(0,0)) = 0xFF0000FF; // red == bad
+        }
+        fCurPos = SkPoint::Make(0,0);
+        fSize = 200;
     }
 
 protected: