aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/gpu/GrAtlasTextContext.h
Commit message (Collapse)AuthorAge
* Detach cache in DF routines rather than passing it in.Gravatar jvanverth2015-09-09
| | | | | | | This removes nesting of cache accesses, which avoids a possible deadlock condition when the caches are the same. Review URL: https://codereview.chromium.org/1310513008
* Make GrTextContext no longer store a GrDrawContextGravatar robertphillips2015-09-01
| | | | | | This, at least, makes GrTextContexts no longer bound to a single GrDrawContext. (Passing in a RenderTarget and praying it matched the DrawContext was always a bit fishy too). Review URL: https://codereview.chromium.org/1320323004
* Style Change: SkNEW->new; SkDELETE->deleteGravatar halcanary2015-08-26
| | | | | | DOCS_PREVIEW= https://skia.org/?cl=1316123003 Review URL: https://codereview.chromium.org/1316123003
* Fix for distance field draw large glyph as pathsGravatar joshualitt2015-08-25
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1315933002
* Regenerate LCD text blobs if GrPaint's color changes, not SkPaint.Gravatar jvanverth2015-08-18
| | | | | | | | | The GrPaint's color takes into account shaders and color filters, so is a more accurate picture of the color state. BUG=chromium:511787 Review URL: https://codereview.chromium.org/1297053004
* Introduce GrBatch subclasses GrDrawBatch and GrVertexBatch to prepare for ↵Gravatar bsalomon2015-08-13
| | | | | | non-drawing batches Review URL: https://codereview.chromium.org/1293583002
* Minimize retrieving SkGlyph in GrTextContextGravatar joshualitt2015-07-24
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1257603005
* Remove some redundant fields from BitmapTextBatch (and rename to TextBatch).Gravatar bsalomon2015-07-22
| | | | Review URL: https://codereview.chromium.org/1244093004
* rename BitmapTextBlob and move it to its own fileGravatar joshualitt2015-07-21
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1249663002
* remove SkDevicePropertiesGravatar robertphillips2015-06-19
| | | | | | There is a lot more clean up to do here but this is probably a big enough bite. Review URL: https://codereview.chromium.org/1196683003
* Remove gamma field from SkDevicePropertiesGravatar robertphillips2015-06-19
| | | | Review URL: https://codereview.chromium.org/1189223002
* Add useDFT field to SkDevicePropertiesGravatar robertphillips2015-06-18
| | | | Review URL: https://codereview.chromium.org/1191943002
* Make GrTextContext be owned by the GrDrawContextGravatar robertphillips2015-06-17
| | | | | | | | | | | | | | This CL makes the GrTextContext be owned (and hidden) by the GrDrawContext. This funnels all the drawText* calls through the GrDrawContext and hides the (dispreferred) GrPipelineBuilder drawText variant. Some consequences of this are: GrDrawContext now has to get the text drawing settings (i.e., SkDeviceProperties & useDFT). This means that we need a separate GrDrawContext for each combination of pixel geometry and DFT-use. All the GrTextContext-derived classes now get a back pointer to the originating GrDrawContext so their method calls no longer take one. Committed: https://skia.googlesource.com/skia/+/5b16e740fe6ab6d679083d06f07651602265081b Review URL: https://codereview.chromium.org/1175553002
* Revert of Make GrTextContext be owned by the GrDrawContext (patchset #7 ↵Gravatar bsalomon2015-06-16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | id:120001 of https://codereview.chromium.org/1175553002/) Reason for revert: Breaking Test-Win8-MSVC-ShuttleA-GPU-GTX660-x86_64-Debug ? https://build.chromium.org/p/client.skia/builders/Test-Win8-MSVC-ShuttleA-GPU-GTX660-x86_64-Debug/builds/436/steps/dm/logs/stdio Original issue's description: > Make GrTextContext be owned by the GrDrawContext > > This CL makes the GrTextContext be owned (and hidden) by the GrDrawContext. This funnels all the drawText* calls through the GrDrawContext and hides the (dispreferred) GrPipelineBuilder drawText variant. > > Some consequences of this are: > > GrDrawContext now has to get the text drawing settings (i.e., SkDeviceProperties & useDFT). This means that we need a separate GrDrawContext for each combination of pixel geometry and DFT-use. > > All the GrTextContext-derived classes now get a back pointer to the originating GrDrawContext so their method calls no longer take one. > > Committed: https://skia.googlesource.com/skia/+/5b16e740fe6ab6d679083d06f07651602265081b TBR=joshualitt@chromium.org,joshualitt@google.com,jvanverth@google.com,reed@google.com,robertphillips@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/1178383003
* Make GrTextContext be owned by the GrDrawContextGravatar robertphillips2015-06-16
| | | | | | | | | | | | This CL makes the GrTextContext be owned (and hidden) by the GrDrawContext. This funnels all the drawText* calls through the GrDrawContext and hides the (dispreferred) GrPipelineBuilder drawText variant. Some consequences of this are: GrDrawContext now has to get the text drawing settings (i.e., SkDeviceProperties & useDFT). This means that we need a separate GrDrawContext for each combination of pixel geometry and DFT-use. All the GrTextContext-derived classes now get a back pointer to the originating GrDrawContext so their method calls no longer take one. Review URL: https://codereview.chromium.org/1175553002
* Break GrTextContext's reliance on SkGpuDeviceGravatar robertphillips2015-05-28
| | | | | | | | | | | This CL seems to have 2 main downsides: 1) It duplicates some code in SkBaseDevice::filterTextFlags 2) It makes it tougher to derive from SkGpuDevice It seems reasonable (at least to me) that the TextContexts get the power to reset the LCD flags. Review URL: https://codereview.chromium.org/1159973002
* Move SkGpuDevice::internalDrawPath to GrBlurUtils::drawPathWithMaskFilterGravatar robertphillips2015-05-27
| | | | | | | | | | | This CL is ugly but it: removes the stored SkGpuDevice back pointer from GrTextContext (at the cost of passing more parameters) moves SkGpuDevice::internalDrawPath to GrDrawContext::drawPathFull Unfortunately, the GrTextContext-derived classes still need the SkGpuDevice for filterTextFlags calls but I will try removing that in a separate CL. Review URL: https://codereview.chromium.org/1157773003
* Split drawing functionality out of GrContext and into new GrDrawContextGravatar robertphillips2015-05-26
| | | | | | | | This is mainly a mechanical CL. There were some fiddly bits in GrContext.cpp where it no longer had access to the GrDrawTarget (and had to use the new GrDrawContext). I've converted GrAARectRenderer & GrOvalRenderer into static classes so I could stop allocating them. Review URL: https://codereview.chromium.org/1151283004
* Keep Big Glyph coordinates in float in GrAtlasTextContextGravatar joshualitt2015-05-13
| | | | | | | TBR=jvanverth@google.com BUG=skia: Review URL: https://codereview.chromium.org/1132853004
* GLProgramsTest 3.0Gravatar joshualitt2015-05-13
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1139743002
* Fix blur on large glyphs in runs < SkGlyphCache::maxGravatar joshualitt2015-05-11
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1135113002
* fix for cached textblobs look garbledGravatar joshualitt2015-05-11
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1135813007
* Add unit tests to text contextGravatar joshualitt2015-05-11
| | | | | | | TBR=bsalomon@google.com BUG=skia: Review URL: https://codereview.chromium.org/1128153005
* Move state management to GrInOrderDrawBufferGravatar joshualitt2015-05-04
| | | | | | | | BUG=skia: Committed: https://skia.googlesource.com/skia/+/5d6bb6f795143ca360b868560b52165de51fa269 Review URL: https://codereview.chromium.org/1120143002
* Revert of Move state management to GrInOrderDrawBuffer (patchset #4 id:60001 ↵Gravatar joshualitt2015-05-04
| | | | | | | | | | | | | | | | | | | | | | of https://codereview.chromium.org/1120143002/) Reason for revert: Breaks windows Original issue's description: > Move state management to GrInOrderDrawBuffer > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/5d6bb6f795143ca360b868560b52165de51fa269 TBR=bsalomon@google.com,robertphillips@google.com,joshualitt@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/1119353002
* Move state management to GrInOrderDrawBufferGravatar joshualitt2015-05-04
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1120143002
* fix a bug in scrolling BitmapText gpu cached textblobsGravatar joshualitt2015-04-27
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1103243002
* Fix for segfault on destruction of BitmapTextBlobGravatar joshualitt2015-04-22
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1065773005
* Fix for GrAtlasTextContext memory leakGravatar joshualitt2015-04-22
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1072193009
* fix some small bugs with GrAtlasTextContextGravatar joshualitt2015-04-21
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1098093003
* Add support for rotating / scaling / translating gpu cached distance field ↵Gravatar joshualitt2015-04-21
| | | | | | | | textblobs BUG=skia: Review URL: https://codereview.chromium.org/1092283002
* Store pointers to GrGlyph directly in BitmapTextBlob. This patch improves ↵Gravatar joshualitt2015-04-21
| | | | | | | | performance by avoiding hashmap lookups under normal use BUG=skia: Review URL: https://codereview.chromium.org/1087203004
* Pack BitmapTextBlob a bit tighterGravatar joshualitt2015-04-21
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1092973004
* A small patch to enable distance field text in textblobsGravatar joshualitt2015-04-21
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1098653005
* Small change to allow DistanceField and BMP text to coexist in a blobGravatar joshualitt2015-04-17
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1096753002
* Initial CL to add distance field support to GrAtlasTextContextGravatar joshualitt2015-04-17
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1082843002
* Convert BitmapTextBlob from using STArray to AutoSTMallocGravatar joshualitt2015-04-14
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1083703002
* Start canonicalizing color for all A8 textblobsGravatar joshualitt2015-04-14
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1076593002
* Start caching masks / stroke fills for textblobsGravatar joshualitt2015-04-13
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1065293003
* Avoid regenerating cached textblobs on integer scrollsGravatar joshualitt2015-04-13
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1062863002
* Adding a cache + memory pool for GPU TextBlobsGravatar joshualitt2015-04-08
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1055843002
* Adding bulk plot reffer to cached textblobsGravatar joshualitt2015-04-08
| | | | | | | | | | | This change will prevent the atlas from evicting glyphs the TextBlob needs. BUG=skia: Committed: https://skia.googlesource.com/skia/+/7281c61e7bc689d484dcbda49be3cef4ce4f11c2 Review URL: https://codereview.chromium.org/1050113004
* Revert of Adding bulk plot reffer to cached textblobs (patchset #7 id:110001 ↵Gravatar joshualitt2015-04-07
| | | | | | | | | | | | | | | | | | | | | | | | | of https://codereview.chromium.org/1050113004/) Reason for revert: Breaks linux builder Original issue's description: > Adding bulk plot reffer to cached textblobs > > This change will prevent the atlas from evicting glyphs the TextBlob > needs. > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/7281c61e7bc689d484dcbda49be3cef4ce4f11c2 TBR=bsalomon@google.com,jvanverth@google.com,robertphillips@google.com,joshualitt@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/1061713003
* Adding bulk plot reffer to cached textblobsGravatar joshualitt2015-04-07
| | | | | | | | | This change will prevent the atlas from evicting glyphs the TextBlob needs. BUG=skia: Review URL: https://codereview.chromium.org/1050113004
* Handle large paths in textblobsGravatar joshualitt2015-04-06
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1057613002
* BUG=skia:Gravatar joshualitt2015-04-03
| | | | Review URL: https://codereview.chromium.org/1031423002
* Rename GrBitmapTextContextB to GrAtlasTextContextGravatar joshualitt2015-04-02
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1050173002
* move Atlas Text Context to its own fileGravatar joshualitt2015-04-01
BUG=skia: Review URL: https://codereview.chromium.org/1045723010
generated by cgit v1.2.3 (git 2.39.1) at 2025-01-22 03:37:36 +0000