From 261d0153e2ce1828355221f8dad0855eb6e6e243 Mon Sep 17 00:00:00 2001 From: "commit-bot@chromium.org" Date: Mon, 12 May 2014 15:48:38 +0000 Subject: 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 --- bench/FontScalerBench.cpp | 7 ------- 1 file changed, 7 deletions(-) (limited to 'bench/FontScalerBench.cpp') 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; -- cgit v1.2.3