aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/gpu
Commit message (Collapse)AuthorAge
* Revert "Implement caching of filled paths in the tessellated path renderer."Gravatar mtklein2015-08-04
| | | | | | | | | | | | | | | | | | This reverts commit 468dfa72eb6694145487be17876804dfca3b7adb. This CL caused signficant GM diffs. Revert "Fix resource cache test." This reverts commit b001c41ed37d2018d38ce787b412ed741585d75d. Revert "Fix SkData leaks at GrResourceKey::setCustomData() call sites." This reverts commit c369348aa596d7be05c9ce0ca5d349e5d1903789. These CLs depend on the first one. BUG=skia: TBR=bsalomon@google.com Review URL: https://codereview.chromium.org/1260363007
* Remove IGNORE_ROT_AA_RECT_OPT flagGravatar robertphillips2015-08-04
| | | | | | It looks like the positive version was added (SHADER_AA_FILL_RECT) but the negative version wasn't removed. Review URL: https://codereview.chromium.org/1271903003
* Added registerChild; transforms, textures, glKey automatically handled.Gravatar wangyix2015-08-04
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1266633003
* Fix SkData leaks at GrResourceKey::setCustomData() call sites.Gravatar Stephen White2015-08-03
| | | | | | | BUG=skia: TBR=bsalomon@google.com Review URL: https://codereview.chromium.org/1270103003 .
* Fix elliptical rrect clip shaders for large radii on devices with mediumpGravatar bsalomon2015-08-03
| | | | | | BUG=chromium:426217 Review URL: https://codereview.chromium.org/1256353004
* Implement caching of filled paths in the tessellated path renderer.Gravatar senorblanco2015-08-03
| | | | | | | | | | | | | | | | | | | | | | | Paths are cached as tessellated triangle meshes in vertex buffers on the GPU. Stroked paths are not (yet) cached. Paths containing no curved segments (linear paths) are reused at all scales. Paths containing curved segments are reused within a scale tolerance threshold. In order to invalidate the cache when an SkPath is changed or deleted, this required implementing genID change notification in SkPath. This is modelled almost exactly on SkPixelRef::GenIDChangeListener. However, It does not currently implement the check for unique genIDs, so notifiers will fire when the first instance of an SkPathRef using a given genID is destroyed. Another caveat is that you cannot successfully add a change notifier to an empty path, since it uses the "canonical" empty path which is never modified or destroyed. For this reason, we prevent adding listeners to it. BUG=skia:4121,skia:4122, 497403 DOCS_PREVIEW= https://skia.org/?cl=1114353004 Review URL: https://codereview.chromium.org/1114353004
* Use new API everywhere for GrDefaultGeoProcFactoryGravatar joshualitt2015-08-03
| | | | | | | | BUG=skia: Committed: https://skia.googlesource.com/skia/+/801823b3d825d96baadeb9077c5ed1a8ada05cf2 Review URL: https://codereview.chromium.org/1261083003
* Add abliity to set textblob cache budget to GrContextGravatar joshualitt2015-08-03
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1264283002
* Update assert to allow config conversion effect for all configs when not ↵Gravatar bsalomon2015-08-03
| | | | | | | | premul/unpremuling TBR=robertphillips@google.com Review URL: https://codereview.chromium.org/1258763006
* Revert of Use new API everywhere for GrDefaultGeoProcFactory (patchset #5 ↵Gravatar joshualitt2015-08-03
| | | | | | | | | | | | | | | | | | | | | | id:80001 of https://codereview.chromium.org/1261083003/) Reason for revert: breaking things Original issue's description: > Use new API everywhere for GrDefaultGeoProcFactory > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/801823b3d825d96baadeb9077c5ed1a8ada05cf2 TBR=robertphillips@google.com,bsalomon@google.com,joshualitt@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/1264283004
* Use new API everywhere for GrDefaultGeoProcFactoryGravatar joshualitt2015-08-03
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1261083003
* Move strike to subrun in GrAtlasTextContextGravatar joshualitt2015-07-31
| | | | | | | | BUG=skia: Committed: https://skia.googlesource.com/skia/+/77d89f7dd243a17452d3a5f16a98622993e6bdd9 Review URL: https://codereview.chromium.org/1257253005
* Revert of Move strike to subrun in GrAtlasTextContext (patchset #3 id:40001 ↵Gravatar joshualitt2015-07-31
| | | | | | | | | | | | | | | | | | | | | | of https://codereview.chromium.org/1257253005/) Reason for revert: breaking bots Original issue's description: > Move strike to subrun in GrAtlasTextContext > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/77d89f7dd243a17452d3a5f16a98622993e6bdd9 TBR=bsalomon@google.com,robertphillips@google.com,jvanverth@google.com,joshualitt@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/1266253002
* Some cleanup in GrTextureProvider and GrResourceProvider.Gravatar bsalomon2015-07-31
| | | | Review URL: https://codereview.chromium.org/1261643004
* Move strike to subrun in GrAtlasTextContextGravatar joshualitt2015-07-31
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1257253005
* Speculative fix for http://crbug.com/515966Gravatar bsalomon2015-07-31
| | | | | | BUG=chromium:515966 Review URL: https://codereview.chromium.org/1268723005
* Another small fix to GrFontScalerGravatar joshualitt2015-07-31
| | | | | | | TBR=bsalomon@google.com BUG=skia: Review URL: https://codereview.chromium.org/1261433006
* Modifying TextBlobCacheTest to use SkRandomScalerContextGravatar joshualitt2015-07-31
| | | | | | | | BUG=skia: Committed: https://skia.googlesource.com/skia/+/adcdca86ce425cf8c28bfad311cef028df756ee8 Review URL: https://codereview.chromium.org/1266003002
* Make ANGLE perf decisions be runtime rather than compile timeGravatar bsalomon2015-07-31
| | | | | | R=robertphillips@google.com Review URL: https://codereview.chromium.org/1268953002
* Restore read pixels perf on ANGLEGravatar bsalomon2015-07-31
| | | | | | BUG=chromium:513797 Review URL: https://codereview.chromium.org/1265003002
* bump the size of the atlas id to 64 bitsGravatar joshualitt2015-07-31
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1253003005
* Revert of Modifying TextBlobCacheTest to use SkRandomScalerContext (patchset ↵Gravatar joshualitt2015-07-31
| | | | | | | | | | | | | | | | | | | | | | #3 id:40001 of https://codereview.chromium.org/1266003002/) Reason for revert: breaking android Original issue's description: > Modifying TextBlobCacheTest to use SkRandomScalerContext > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/adcdca86ce425cf8c28bfad311cef028df756ee8 TBR=bungeman@google.com,bsalomon@google.com,joshualitt@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/1267623004
* Modifying TextBlobCacheTest to use SkRandomScalerContextGravatar joshualitt2015-07-31
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1266003002
* Also provides the ResourceProvider to onDrawPath which allows the DF PR to ↵Gravatar bsalomon2015-07-31
| | | | | | no longer require access to GrContext. Review URL: https://codereview.chromium.org/1265763002
* Remove unused field in micro cleanupGravatar joshualitt2015-07-31
| | | | | | | TBR=bsalomon@google.com BUG=skia: Review URL: https://codereview.chromium.org/1268893002
* Tweak the MaskFormat in one more place in GrBatchFontCacheGravatar joshualitt2015-07-31
| | | | | | BUG=510931 Review URL: https://codereview.chromium.org/1268743002
* Implement support for dual source blending in ESGravatar kkinnunen2015-07-30
| | | | | | | | | | | | | | | | | | | | | | | Use EXT_blend_func_extended to implement dual source blending in OpenGL ES. The extension is the ES version of ARB_blend_func_extended. The extension provides gl_SecondaryFragColorEXT for ES 2.0 contexts. The extension provides glBindFragDataLocationIndexed to bind a custom fragment shader output to the secondary color for ES 3.0 contexts. For ES 3.1 contexts, the extension would also give "layout (location=0, index=1)" output varible layout modifier syntax, but it is not used in this patch. The extension needs #extension GL_EXT_blend_func_extended : require directive for the variables to be available in ES 2.0. For ES 3.0, the directive relaxes the rules for the amount of output variables without layout location qualifiers. OpenGL continues to use GL_ARB_blend_func_extended for dual source blending. Review URL: https://codereview.chromium.org/1266773003
* Make it possible to toggle MSAA for ES (EXT_multisample_compatibility)Gravatar kkinnunen2015-07-30
| | | | | | | | | | | | | | | Make it possible to turn MSAA on and off for OpenGL ES, too. Use EXT_multisample_compatibility. The extension adds token: MULTISAMPLE_EXT 0x809D The token can be supplied to glEnable/glDisable/glGetInteger. This is needed in order to support mixed samples in OpenGL ES. This is important for path rendering Chromium command buffer integration. Review URL: https://codereview.chromium.org/1270533002
* Disable SRGB support on PowerVR Rogue due to SRGBReadWritePixels failureGravatar bsalomon2015-07-30
| | | | | | | TBR=jvanverth@google.com BUG=skia:4148 Review URL: https://codereview.chromium.org/1266933002
* Make TRecorder alloc_back return a void*Gravatar bsalomon2015-07-30
| | | | | | | | | | Speculative fix for this bug: BUG=chromium:515679 TBR=robertphillips@google.com Review URL: https://codereview.chromium.org/1268493003
* SRGB read and write pixels working and unit testGravatar bsalomon2015-07-30
| | | | Review URL: https://codereview.chromium.org/1264003002
* find image whenever we generateGlyph. This should ensure the glyph is fully ↵Gravatar joshualitt2015-07-30
| | | | | | | | initialized before we start using the maskformat. BUG=skia: Review URL: https://codereview.chromium.org/1270573003
* Move some parts of onReadPixels up to GrGpu readPixels.Gravatar egdaniel2015-07-30
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1270583003
* Dont try to draw glyphs with unexpected mask formatsGravatar joshualitt2015-07-30
| | | | | | BUG=510931 Review URL: https://codereview.chromium.org/1269743003
* Make GrGpu read/write pixels take GrSurfaceGravatar bsalomon2015-07-30
| | | | Review URL: https://codereview.chromium.org/1262473004
* Move headers with no dependencies.Gravatar mtklein2015-07-29
| | | | | | | | | | | | | | | | | C.f. https://codereview.chromium.org/1261013003/ BUG=skia:4126 Will follow up with two more CLs if this works: - one moving SkRecords.h - one moving SkMiniRecorder.h No public API changes. TBR=reed@google.com Committed: https://skia.googlesource.com/skia/+/117842223bd13325b6da26110d80e0590c1a742b Review URL: https://codereview.chromium.org/1266593002
* Fix for GrAtlasTextContext crashGravatar joshualitt2015-07-29
| | | | | | BUG=510931 Review URL: https://codereview.chromium.org/1256763005
* Revert "Move headers with no dependencies."Gravatar Mike Klein2015-07-29
| | | | | | | | | | | This reverts commit 117842223bd13325b6da26110d80e0590c1a742b. No good: https://uberchromegw.corp.google.com/i/client.skia/builders/Mac%20Builder/builds/3465/steps/compile/logs/stdio BUG=skia:4126 Review URL: https://codereview.chromium.org/1262173002 .
* Move headers with no dependencies.Gravatar mtklein2015-07-29
| | | | | | | | | | | | | | | C.f. https://codereview.chromium.org/1261013003/ BUG=skia:4126 Will follow up with two more CLs if this works: - one moving SkRecords.h - one moving SkMiniRecorder.h No public API changes. TBR=reed@google.com Review URL: https://codereview.chromium.org/1266593002
* Fix silly mistake in allocating geoms in TextBatchGravatar bsalomon2015-07-29
| | | | | | TBR=joshualitt@google.com Review URL: https://codereview.chromium.org/1259103004
* Skip temp texture when config conversion test has already failed.Gravatar bsalomon2015-07-29
| | | | Review URL: https://codereview.chromium.org/1261033005
* Don't read from unref'ed RT in GrContext::readSurfacePixelsGravatar bsalomon2015-07-28
| | | | | | | | BUG=chromium:514716 TBR=robertphillips@google.com Review URL: https://codereview.chromium.org/1258673009
* Move draw on upload decision in GrGpuGravatar bsalomon2015-07-28
| | | | | | | | BUG=skia: Committed: https://skia.googlesource.com/skia/+/0341b4427e5f037e3b501ed6e57dfdb7b40f150e Review URL: https://codereview.chromium.org/1257073003
* Revert "Move the rest of src headers used by include to include/private."Gravatar Mike Klein2015-07-28
| | | | | | | | | | | | This reverts commit d12e6ffa5cc1c1af47bf73c7c127d8d7f7443058. Our Chrome roll canaries are failing with the dreaded Ninja-says-there's-more-work-to-do message. I will break this up smaller (if possible) and try again tomorrow. BUG=skia:4126 Review URL: https://codereview.chromium.org/1258293004 .
* Revert of Move draw on upload decision in GrGpu (patchset #4 id:50001 of ↵Gravatar bsalomon2015-07-28
| | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/1257073003/) Reason for revert: breaking write pixels test on bots Original issue's description: > Move draw on upload decision in GrGpu > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/0341b4427e5f037e3b501ed6e57dfdb7b40f150e TBR=robertphillips@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/1260293004
* Move the rest of src headers used by include to include/private.Gravatar mtklein2015-07-28
| | | | | | | | | | | | | $ git grep "../../src/" | grep include now returns nothing. BUG=skia:4126 No public API changes. TBR=reed@google.com Review URL: https://codereview.chromium.org/1261013003
* Change GrDefaultGeoProcFactory to a namespaceGravatar joshualitt2015-07-28
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1257333002
* Cleanup GrDefaultGeoProcFactory localCoordsGravatar joshualitt2015-07-28
| | | | | | | TBR=bsalomon@google.com BUG=skia: Review URL: https://codereview.chromium.org/1257193002
* Rename GrInOrderDrawBuffer to GrBufferedDrawTargetGravatar joshualitt2015-07-28
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1261033002
* Move draw on upload decision in GrGpuGravatar bsalomon2015-07-28
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1257073003