Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(481)

Side by Side Diff: dm/DMQuiltTask.cpp

Issue 375703002: Turn on threaded DMQuiltTask drawing for old-format SkPictures too. (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: borrow Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | src/core/SkPicturePlayback.cpp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #include "DMQuiltTask.h" 1 #include "DMQuiltTask.h"
2 #include "DMUtil.h" 2 #include "DMUtil.h"
3 #include "DMWriteTask.h" 3 #include "DMWriteTask.h"
4 4
5 #include "SkCommandLineFlags.h" 5 #include "SkCommandLineFlags.h"
6 #include "SkPicture.h" 6 #include "SkPicture.h"
7 #include "SkThreadPool.h" 7 #include "SkThreadPool.h"
8 8
9 DEFINE_bool(quilt, true, "If true, draw into a quilt of small tiles and compare. "); 9 DEFINE_bool(quilt, true, "If true, draw into a quilt of small tiles and compare. ");
10 DEFINE_int32(quiltTile, 256, "Dimension of (square) quilt tile."); 10 DEFINE_int32(quiltTile, 256, "Dimension of (square) quilt tile.");
11 DEFINE_bool(quiltThreaded, false, "If true, draw quilt tiles with multiple threa ds."); 11 DEFINE_bool(quiltThreaded, true, "If true, draw quilt tiles with multiple thread s.");
12 12
13 static const char* kSuffixes[] = { "quilt", "quilt_skr" }; 13 static const char* kSuffixes[] = { "quilt", "quilt_skr" };
14 14
15 namespace DM { 15 namespace DM {
16 16
17 QuiltTask::QuiltTask(const Task& parent, skiagm::GM* gm, SkBitmap reference, Qui ltTask::Mode mode) 17 QuiltTask::QuiltTask(const Task& parent, skiagm::GM* gm, SkBitmap reference, Qui ltTask::Mode mode)
18 : CpuTask(parent) 18 : CpuTask(parent)
19 , fMode(mode) 19 , fMode(mode)
20 , fName(UnderJoin(parent.name().c_str(), kSuffixes[mode])) 20 , fName(UnderJoin(parent.name().c_str(), kSuffixes[mode]))
21 , fGM(gm) 21 , fGM(gm)
(...skipping 62 matching lines...) Expand 10 before | Expand all | Expand 10 after
84 if (fGM->getFlags() & skiagm::GM::kSkipPicture_Flag) { 84 if (fGM->getFlags() & skiagm::GM::kSkipPicture_Flag) {
85 return true; 85 return true;
86 } 86 }
87 if (fGM->getFlags() & skiagm::GM::kSkipTiled_Flag) { 87 if (fGM->getFlags() & skiagm::GM::kSkipTiled_Flag) {
88 return true; 88 return true;
89 } 89 }
90 return !FLAGS_quilt; 90 return !FLAGS_quilt;
91 } 91 }
92 92
93 } // namespace DM 93 } // namespace DM
OLDNEW
« no previous file with comments | « no previous file | src/core/SkPicturePlayback.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698