aboutsummaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
-rw-r--r--bench/ETCBitmapBench.cpp5
-rw-r--r--bench/SkBenchmark.cpp9
-rw-r--r--bench/SkBenchmark.h7
-rw-r--r--bench/SkipZeroesBench.cpp7
-rw-r--r--gm/copyTo4444.cpp3
-rw-r--r--gm/etc1bitmap.cpp6
-rw-r--r--gm/factory.cpp3
-rw-r--r--gm/gm.cpp12
-rw-r--r--gm/gm.h11
-rw-r--r--tests/Test.cpp9
-rw-r--r--tests/Test.h2
11 files changed, 44 insertions, 30 deletions
diff --git a/bench/ETCBitmapBench.cpp b/bench/ETCBitmapBench.cpp
index 4b9cd6ea93..e9aee76a4a 100644
--- a/bench/ETCBitmapBench.cpp
+++ b/bench/ETCBitmapBench.cpp
@@ -89,8 +89,9 @@ protected:
private:
SkData *loadPKM() {
- SkString filename = SkOSPath::SkPathJoin(
- INHERITED::GetResourcePath().c_str(), "mandrill_128.pkm");
+ SkString resourcePath = GetResourcePath();
+ SkString filename = SkOSPath::SkPathJoin(resourcePath.c_str(),
+ "mandrill_128.pkm");
// Expand the data
SkAutoDataUnref fileData(SkData::NewFromFileName(filename.c_str()));
diff --git a/bench/SkBenchmark.cpp b/bench/SkBenchmark.cpp
index d995415632..1f12ed339f 100644
--- a/bench/SkBenchmark.cpp
+++ b/bench/SkBenchmark.cpp
@@ -12,7 +12,7 @@ const char* SkTriState::Name[] = { "default", "true", "false" };
template BenchRegistry* BenchRegistry::gHead;
-SkString SkBenchmark::gResourcePath;
+const char* SkBenchmark::gResourcePath;
SkBenchmark::SkBenchmark() {
fForceAlpha = 0xFF;
@@ -55,6 +55,13 @@ void SkBenchmark::setupPaint(SkPaint* paint) {
}
}
+void SkBenchmark::SetResourcePath(const char* resourcePath) {
+ gResourcePath = resourcePath;
+}
+
+SkString SkBenchmark::GetResourcePath() {
+ return SkString(gResourcePath);
+}
///////////////////////////////////////////////////////////////////////////////
diff --git a/bench/SkBenchmark.h b/bench/SkBenchmark.h
index bf28689236..00a816cdba 100644
--- a/bench/SkBenchmark.h
+++ b/bench/SkBenchmark.h
@@ -107,9 +107,8 @@ public:
fClearMask = clearMask;
}
- static void SetResourcePath(const char* resPath) { gResourcePath.set(resPath); }
-
- static SkString& GetResourcePath() { return gResourcePath; }
+ static void SetResourcePath(const char*);
+ static SkString GetResourcePath();
protected:
virtual void setupPaint(SkPaint* paint);
@@ -129,7 +128,7 @@ private:
bool fForceFilter;
SkTriState::State fDither;
uint32_t fOrMask, fClearMask;
- static SkString gResourcePath;
+ static const char* gResourcePath;
typedef SkRefCnt INHERITED;
};
diff --git a/bench/SkipZeroesBench.cpp b/bench/SkipZeroesBench.cpp
index 3d846bf73d..b84cf78169 100644
--- a/bench/SkipZeroesBench.cpp
+++ b/bench/SkipZeroesBench.cpp
@@ -45,13 +45,14 @@ protected:
}
virtual void onPreDraw() SK_OVERRIDE {
- const char* resPath = GetResourcePath().c_str();
- if (NULL == resPath) {
+ SkString resourcePath = GetResourcePath();
+ if (resourcePath.isEmpty()) {
fValid = false;
return;
}
- SkString fullPath = SkOSPath::SkPathJoin(resPath, fFilename.c_str());
+ SkString fullPath = SkOSPath::SkPathJoin(resourcePath.c_str(),
+ fFilename.c_str());
SkFILEStream fileStream(fullPath.c_str());
fValid = fileStream.isValid() && fileStream.getLength() > 0;
if (fValid) {
diff --git a/gm/copyTo4444.cpp b/gm/copyTo4444.cpp
index 7e2c2798e4..6233301fed 100644
--- a/gm/copyTo4444.cpp
+++ b/gm/copyTo4444.cpp
@@ -30,8 +30,7 @@ protected:
virtual void onDraw(SkCanvas* canvas) {
SkBitmap bm, bm4444;
- SkString filename = SkOSPath::SkPathJoin(
- INHERITED::gResourcePath.c_str(), "mandrill_512.png");
+ SkString filename = SkOSPath::SkPathJoin(INHERITED::gResourcePath, "mandrill_512.png");
if (!SkImageDecoder::DecodeFile(filename.c_str(), &bm,
SkBitmap::kARGB_8888_Config,
SkImageDecoder::kDecodePixels_Mode)) {
diff --git a/gm/etc1bitmap.cpp b/gm/etc1bitmap.cpp
index cdf8617a35..ce0aa7dbad 100644
--- a/gm/etc1bitmap.cpp
+++ b/gm/etc1bitmap.cpp
@@ -93,8 +93,7 @@ protected:
virtual void onDraw(SkCanvas* canvas) SK_OVERRIDE {
SkBitmap bm;
- SkString filename = SkOSPath::SkPathJoin(
- INHERITED::gResourcePath.c_str(), "mandrill_128.");
+ SkString filename = SkOSPath::SkPathJoin(INHERITED::gResourcePath, "mandrill_128.");
filename.append(this->fileExtension());
SkAutoTUnref<SkData> fileData(SkData::NewFromFileName(filename.c_str()));
@@ -169,8 +168,7 @@ protected:
virtual void onDraw(SkCanvas* canvas) SK_OVERRIDE {
SkBitmap bm;
- SkString filename = SkOSPath::SkPathJoin(
- INHERITED::gResourcePath.c_str(), "mandrill_128.pkm");
+ SkString filename = SkOSPath::SkPathJoin(INHERITED::gResourcePath, "mandrill_128.pkm");
SkAutoDataUnref fileData(SkData::NewFromFileName(filename.c_str()));
if (NULL == fileData) {
diff --git a/gm/factory.cpp b/gm/factory.cpp
index bd79c9d81e..aa643a2c5b 100644
--- a/gm/factory.cpp
+++ b/gm/factory.cpp
@@ -28,8 +28,7 @@ public:
protected:
virtual void onOnceBeforeDraw() SK_OVERRIDE {
// Copyright-free file from http://openclipart.org/detail/29213/paper-plane-by-ddoo
- SkString filename = SkOSPath::SkPathJoin(INHERITED::gResourcePath.c_str(),
- "plane.png");
+ SkString filename = SkOSPath::SkPathJoin(INHERITED::gResourcePath, "plane.png");
SkAutoDataUnref data(SkData::NewFromFileName(filename.c_str()));
if (NULL != data.get()) {
// Create a cache which will boot the pixels out anytime the
diff --git a/gm/gm.cpp b/gm/gm.cpp
index 803874f5bc..8da45c5fcb 100644
--- a/gm/gm.cpp
+++ b/gm/gm.cpp
@@ -6,9 +6,10 @@
*/
#include "gm.h"
+
using namespace skiagm;
-SkString GM::gResourcePath;
+const char* GM::gResourcePath;
GM::GM() {
fMode = kGM_Mode;
@@ -17,6 +18,7 @@ GM::GM() {
fHaveCalledOnceBeforeDraw = false;
fStarterMatrix.reset();
}
+
GM::~GM() {}
void GM::draw(SkCanvas* canvas) {
@@ -64,5 +66,13 @@ void GM::drawSizeBounds(SkCanvas* canvas, SkColor color) {
canvas->drawRect(r, paint);
}
+void GM::SetResourcePath(const char* resourcePath) {
+ gResourcePath = resourcePath;
+}
+
+SkString GM::GetResourcePath() {
+ return SkString(gResourcePath);
+}
+
// need to explicitly declare this, or we get some weird infinite loop llist
template GMRegistry* SkTRegistry<GM*(*)(void*)>::gHead;
diff --git a/gm/gm.h b/gm/gm.h
index 90de96f7e8..a48976b20e 100644
--- a/gm/gm.h
+++ b/gm/gm.h
@@ -96,13 +96,8 @@ namespace skiagm {
// GM's getISize bounds.
void drawSizeBounds(SkCanvas*, SkColor);
- static void SetResourcePath(const char* resourcePath) {
- gResourcePath = resourcePath;
- }
-
- static SkString& GetResourcePath() {
- return gResourcePath;
- }
+ static void SetResourcePath(const char*);
+ static SkString GetResourcePath();
bool isCanvasDeferred() const { return fCanvasIsDeferred; }
void setCanvasIsDeferred(bool isDeferred) {
@@ -115,7 +110,7 @@ namespace skiagm {
}
protected:
- static SkString gResourcePath;
+ static const char* gResourcePath;
virtual void onOnceBeforeDraw() {}
virtual void onDraw(SkCanvas*) = 0;
diff --git a/tests/Test.cpp b/tests/Test.cpp
index e57427a354..b904d5a76d 100644
--- a/tests/Test.cpp
+++ b/tests/Test.cpp
@@ -1,10 +1,10 @@
-
/*
* Copyright 2011 Google Inc.
*
* Use of this source code is governed by a BSD-style license that can be
* found in the LICENSE file.
*/
+
#include "Test.h"
#include "SkCommandLineFlags.h"
@@ -41,6 +41,8 @@ void Reporter::endTest(Test* test) {
///////////////////////////////////////////////////////////////////////////////
+const char* Test::gResourcePath;
+
Test::Test() : fReporter(NULL), fPassed(true) {}
Test::~Test() {
@@ -121,8 +123,9 @@ SkString Test::GetTmpDir() {
return SkString(tmpDir);
}
-static const char* gResourcePath = NULL;
-void Test::SetResourcePath(const char* resourcePath) { gResourcePath = resourcePath; }
+void Test::SetResourcePath(const char* resourcePath) {
+ gResourcePath = resourcePath;
+}
SkString Test::GetResourcePath() {
return SkString(gResourcePath);
diff --git a/tests/Test.h b/tests/Test.h
index 4d2cf2b639..90d072ef66 100644
--- a/tests/Test.h
+++ b/tests/Test.h
@@ -73,6 +73,8 @@ namespace skiatest {
virtual void onRun(Reporter*) = 0;
private:
+ static const char* gResourcePath;
+
Reporter* fReporter;
SkString fName;
bool fPassed;