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

Unified Diff: src/gpu/GrContext.cpp

Issue 1175553002: Make GrTextContext be owned by the GrDrawContext (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Fix nvpr config Created 5 years, 6 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/gpu/GrAtlasTextContext.cpp ('k') | src/gpu/GrDrawContext.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/gpu/GrContext.cpp
diff --git a/src/gpu/GrContext.cpp b/src/gpu/GrContext.cpp
index 64f4cb3deca237649047db0ababf9954813ca01e..d5f9f9ce4cf2d9f7b06e90e57e30699a805c4154 100755
--- a/src/gpu/GrContext.cpp
+++ b/src/gpu/GrContext.cpp
@@ -9,7 +9,6 @@
#include "GrContext.h"
#include "GrAARectRenderer.h"
-#include "GrAtlasTextContext.h"
#include "GrBatch.h"
#include "GrBatchFontCache.h"
#include "GrBatchTarget.h"
@@ -32,7 +31,6 @@
#include "GrResourceCache.h"
#include "GrResourceProvider.h"
#include "GrSoftwarePathRenderer.h"
-#include "GrStencilAndCoverTextContext.h"
#include "GrStrokeInfo.h"
#include "GrSurfacePriv.h"
#include "GrTextBlobCache.h"
@@ -62,24 +60,37 @@
////////////////////////////////////////////////////////////////////////////////
void GrContext::DrawingMgr::init(GrContext* context) {
+ fContext = context;
+
#ifdef IMMEDIATE_MODE
fDrawTarget = SkNEW_ARGS(GrImmediateDrawTarget, (context));
#else
fDrawTarget = SkNEW_ARGS(GrInOrderDrawBuffer, (context));
#endif
+}
- fDrawContext = SkNEW_ARGS(GrDrawContext, (context, fDrawTarget));
+void GrContext::DrawingMgr::cleanup() {
+ SkSafeSetNull(fDrawTarget);
+ for (int i = 0; i < kNumPixelGeometries; ++i) {
+ SkSafeSetNull(fDrawContext[i][0]);
+ SkSafeSetNull(fDrawContext[i][1]);
+ }
}
GrContext::DrawingMgr::~DrawingMgr() {
- SkSafeUnref(fDrawTarget);
- SkSafeUnref(fDrawContext);
+ this->cleanup();
}
void GrContext::DrawingMgr::abandon() {
SkSafeSetNull(fDrawTarget);
- SkSafeSetNull(fDrawContext->fDrawTarget);
- SkSafeSetNull(fDrawContext);
+ for (int i = 0; i < kNumPixelGeometries; ++i) {
+ for (int j = 0; j < kNumDFTOptions; ++j) {
+ if (fDrawContext[i][j]) {
+ SkSafeSetNull(fDrawContext[i][j]->fDrawTarget);
+ SkSafeSetNull(fDrawContext[i][j]);
+ }
+ }
+ }
}
void GrContext::DrawingMgr::purgeResources() {
@@ -100,11 +111,29 @@ void GrContext::DrawingMgr::flush() {
}
}
-GrDrawContext* GrContext::DrawingMgr::drawContext() {
+GrDrawContext* GrContext::DrawingMgr::drawContext(const SkDeviceProperties* devProps, bool useDFT) {
if (this->abandoned()) {
return NULL;
}
- return fDrawContext;
+
+ const SkDeviceProperties defProps;
+ if (!devProps) {
+ devProps = &defProps;
+ }
+
+ SkASSERT(devProps->pixelGeometry() < kNumPixelGeometries);
+ if (!fDrawContext[devProps->pixelGeometry()][useDFT]) {
+ fDrawContext[devProps->pixelGeometry()][useDFT] =
+ SkNEW_ARGS(GrDrawContext, (fContext, fDrawTarget, *devProps, useDFT));
+ }
+
+ SkASSERT(fDrawContext[devProps->pixelGeometry()][useDFT]->fDevProps->pixelGeometry() ==
+ devProps->pixelGeometry());
+ SkASSERT(fDrawContext[devProps->pixelGeometry()][useDFT]->fDevProps->gamma() ==
+ devProps->gamma());
+ SkASSERT(fDrawContext[devProps->pixelGeometry()][useDFT]->fUseDFT == useDFT);
+
+ return fDrawContext[devProps->pixelGeometry()][useDFT];
}
////////////////////////////////////////////////////////////////////////////////
@@ -185,6 +214,8 @@ GrContext::~GrContext() {
this->flush();
+ fDrawingMgr.cleanup();
+
for (int i = 0; i < fCleanUpData.count(); ++i) {
(*fCleanUpData[i].fFunc)(this, fCleanUpData[i].fInfo);
}
@@ -246,21 +277,6 @@ void GrContext::getResourceCacheUsage(int* resourceCount, size_t* resourceBytes)
}
}
-GrTextContext* GrContext::createTextContext(GrRenderTarget* renderTarget,
- const SkDeviceProperties&
- leakyProperties,
- bool enableDistanceFieldFonts) {
- if (fGpu->caps()->shaderCaps()->pathRenderingSupport() &&
- renderTarget->isStencilBufferMultisampled()) {
- GrStencilAttachment* sb = renderTarget->renderTargetPriv().attachStencilAttachment();
- if (sb) {
- return GrStencilAndCoverTextContext::Create(this, leakyProperties);
- }
- }
-
- return GrAtlasTextContext::Create(this, leakyProperties, enableDistanceFieldFonts);
-}
-
////////////////////////////////////////////////////////////////////////////////
void GrContext::OverBudgetCB(void* data) {
« no previous file with comments | « src/gpu/GrAtlasTextContext.cpp ('k') | src/gpu/GrDrawContext.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698