diff options
author | commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> | 2014-05-12 15:48:38 +0000 |
---|---|---|
committer | commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> | 2014-05-12 15:48:38 +0000 |
commit | 261d0153e2ce1828355221f8dad0855eb6e6e243 (patch) | |
tree | 0ed3bc740aed516680f13f87cc6533563173a787 /bench/FontScalerBench.cpp | |
parent | f31fa24914c683abcc2c860093b142725c43fbe6 (diff) |
Clean up global gSkSuppressFontCachePurgeSpew.
No one is using it except to flick it to true to shut up the spew,
but the spew is already guarded by #ifdef SPEW_PURGE_STATUS. Let's
leave it to SPEW_PURGE_STATUS.
Noticed this specificially when TSAN complained about FontScalerBench
modifying it in a thread-unsafe way.
BUG=skia:1792
R=reed@google.com, mtklein@google.com
Author: mtklein@chromium.org
Review URL: https://codereview.chromium.org/264333008
git-svn-id: http://skia.googlecode.com/svn/trunk@14697 2bbb7eff-a529-9590-31e7-b0007b416f81
Diffstat (limited to 'bench/FontScalerBench.cpp')
-rw-r--r-- | bench/FontScalerBench.cpp | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/bench/FontScalerBench.cpp b/bench/FontScalerBench.cpp index 6ef0703346..a14723c0ff 100644 --- a/bench/FontScalerBench.cpp +++ b/bench/FontScalerBench.cpp @@ -12,8 +12,6 @@ #include "SkRandom.h" #include "SkString.h" -extern bool gSkSuppressFontCachePurgeSpew; - class FontScalerBench : public SkBenchmark { SkString fName; SkString fText; @@ -32,9 +30,6 @@ protected: this->setupPaint(&paint); paint.setLCDRenderText(fDoLCD); - bool prev = gSkSuppressFontCachePurgeSpew; - gSkSuppressFontCachePurgeSpew = true; - for (int i = 0; i < loops; i++) { // this is critical - we want to time the creation process, so we // explicitly flush our cache before each run @@ -46,8 +41,6 @@ protected: 0, SkIntToScalar(20), paint); } } - - gSkSuppressFontCachePurgeSpew = prev; } private: typedef SkBenchmark INHERITED; |