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

Side by Side Diff: dm/DMSerializeTask.cpp

Issue 88543002: DM: some refactoring (Closed) Base URL: https://skia.googlecode.com/svn/trunk
Patch Set: Created 7 years 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 | Annotate | Revision Log
OLDNEW
1 #include "DMSerializeTask.h" 1 #include "DMSerializeTask.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 "SkPixelRef.h" 7 #include "SkPixelRef.h"
8 8
9 DEFINE_bool(serialize, false, "If true, run picture serialization tests."); 9 DEFINE_bool(serialize, true, "If true, run picture serialization tests.");
epoger 2013/11/26 17:55:22 better to separate
10 10
11 namespace DM { 11 namespace DM {
12 12
13 SerializeTask::SerializeTask(const Task& parent, 13 SerializeTask::SerializeTask(const Task& parent,
14 skiagm::GM* gm, 14 skiagm::GM* gm,
15 SkBitmap reference) 15 SkBitmap reference)
16 : Task(parent) 16 : Task(parent)
17 , fName(UnderJoin(parent.name().c_str(), "serialize")) 17 , fName(UnderJoin(parent.name().c_str(), "serialize"))
18 , fGM(gm) 18 , fGM(gm)
19 , fReference(reference) 19 , fReference(reference)
(...skipping 24 matching lines...) Expand all
44 this->fail(); 44 this->fail();
45 this->spawnChild(SkNEW_ARGS(WriteTask, (*this, bitmap))); 45 this->spawnChild(SkNEW_ARGS(WriteTask, (*this, bitmap)));
46 } 46 }
47 } 47 }
48 48
49 bool SerializeTask::shouldSkip() const { 49 bool SerializeTask::shouldSkip() const {
50 return !FLAGS_serialize || fGM->getFlags() & skiagm::GM::kSkipPicture_Flag; 50 return !FLAGS_serialize || fGM->getFlags() & skiagm::GM::kSkipPicture_Flag;
51 } 51 }
52 52
53 } // namespace DM 53 } // namespace DM
OLDNEW
« dm/DMReporter.cpp ('K') | « dm/DMReporter.cpp ('k') | dm/DMUtil.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698