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

Side by Side Diff: dm/DMGpuGMTask.cpp

Issue 693933004: Revert of Turn on NVPR 4x MSAA by default when supported in DM and nanobench. (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Created 6 years, 1 month 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 | dm/DMGpuSupport.h » ('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 "DMGpuGMTask.h" 1 #include "DMGpuGMTask.h"
2 #include "DMUtil.h" 2 #include "DMUtil.h"
3 #include "DMWriteTask.h" 3 #include "DMWriteTask.h"
4 #include "SkCommonFlags.h" 4 #include "SkCommonFlags.h"
5 #include "SkSurface.h" 5 #include "SkSurface.h"
6 #include "SkTLS.h" 6 #include "SkTLS.h"
7 7
8 namespace DM { 8 namespace DM {
9 9
10 GpuGMTask::GpuGMTask(const char* config, 10 GpuGMTask::GpuGMTask(const char* config,
11 Reporter* reporter, 11 Reporter* reporter,
12 TaskRunner* taskRunner, 12 TaskRunner* taskRunner,
13 skiagm::GMRegistry::Factory gmFactory, 13 skiagm::GMRegistry::Factory gmFactory,
14 GrContextFactory::GLContextType contextType, 14 GrContextFactory::GLContextType contextType,
15 GrGLStandard gpuAPI, 15 GrGLStandard gpuAPI,
16 int sampleCount) 16 int sampleCount)
17 : GpuTask(reporter, taskRunner) 17 : GpuTask(reporter, taskRunner)
18 , fGM(gmFactory(NULL)) 18 , fGM(gmFactory(NULL))
19 , fName(UnderJoin(fGM->getName(), config)) 19 , fName(UnderJoin(fGM->getName(), config))
20 , fContextType(contextType) 20 , fContextType(contextType)
21 , fGpuAPI(gpuAPI) 21 , fGpuAPI(gpuAPI)
22 , fSampleCount(sampleCount) 22 , fSampleCount(sampleCount)
23 {} 23 {}
24 24
25 static bool gAlreadyWarned[GrContextFactory::kGLContextTypeCnt][kGrGLStandardCnt ];
26
27 void GpuGMTask::draw(GrContextFactory* grFactory) { 25 void GpuGMTask::draw(GrContextFactory* grFactory) {
28 SkImageInfo info = SkImageInfo::Make(SkScalarCeilToInt(fGM->width()), 26 SkImageInfo info = SkImageInfo::Make(SkScalarCeilToInt(fGM->width()),
29 SkScalarCeilToInt(fGM->height()), 27 SkScalarCeilToInt(fGM->height()),
30 kN32_SkColorType, 28 kN32_SkColorType,
31 kPremul_SkAlphaType); 29 kPremul_SkAlphaType);
32 SkAutoTUnref<SkSurface> surface(NewGpuSurface(grFactory, fContextType, fGpuA PI, info, 30 SkAutoTUnref<SkSurface> surface(NewGpuSurface(grFactory, fContextType, fGpuA PI, info,
33 fSampleCount)); 31 fSampleCount));
34 if (!surface) { 32 if (!surface) {
35 if (!gAlreadyWarned[fContextType][fGpuAPI]) { 33 this->fail("Could not create context for the config and the api.");
36 SkDebugf("FYI: couldn't create GPU context, type %d API %d. Will sk ip.\n",
37 fContextType, fGpuAPI);
38 gAlreadyWarned[fContextType][fGpuAPI] = true;
39 }
40 return; 34 return;
41 } 35 }
42 SkCanvas* canvas = surface->getCanvas(); 36 SkCanvas* canvas = surface->getCanvas();
43 CanvasPreflight(canvas); 37 CanvasPreflight(canvas);
44 38
45 canvas->concat(fGM->getInitialTransform()); 39 canvas->concat(fGM->getInitialTransform());
46 fGM->draw(canvas); 40 fGM->draw(canvas);
47 canvas->flush(); 41 canvas->flush();
48 #if GR_CACHE_STATS && SK_SUPPORT_GPU 42 #if GR_CACHE_STATS && SK_SUPPORT_GPU
49 if (FLAGS_veryVerbose) { 43 if (FLAGS_veryVerbose) {
50 grFactory->get(fContextType)->printCacheStats(); 44 grFactory->get(fContextType)->printCacheStats();
51 } 45 }
52 #endif 46 #endif
53 47
54 SkBitmap bitmap; 48 SkBitmap bitmap;
55 bitmap.setInfo(info); 49 bitmap.setInfo(info);
56 canvas->readPixels(&bitmap, 0, 0); 50 canvas->readPixels(&bitmap, 0, 0);
57 51
58 this->spawnChild(SkNEW_ARGS(WriteTask, (*this, "GM", bitmap))); 52 this->spawnChild(SkNEW_ARGS(WriteTask, (*this, "GM", bitmap)));
59 } 53 }
60 54
61 bool GpuGMTask::shouldSkip() const { 55 bool GpuGMTask::shouldSkip() const {
62 return kGPUDisabled || SkToBool(fGM->getFlags() & skiagm::GM::kSkipGPU_Flag) ; 56 return kGPUDisabled || SkToBool(fGM->getFlags() & skiagm::GM::kSkipGPU_Flag) ;
63 } 57 }
64 58
65 } // namespace DM 59 } // namespace DM
OLDNEW
« no previous file with comments | « no previous file | dm/DMGpuSupport.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698