aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
* Try fix the mac build bot after removing Android SampleAppGravatar liyuqian2016-07-06
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2117793002 Review-Url: https://codereview.chromium.org/2117793002
* Increase batching for AA fill rects.Gravatar bsalomon2016-07-06
| | | | | | | | | | This allows batching of rects provided without a local matrix when local coords are required and when the view matrix changes. It also allows batching of rects with a local matrix with rects without a local matrix. GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2116823002 Committed: https://skia.googlesource.com/skia/+/e525ecaf63f225f1da6e9834f7a291c06ad44d23 Review-Url: https://codereview.chromium.org/2116823002
* fix for Clang 3.8 warning in SkSLGravatar ethannicholas2016-07-06
| | | | | | GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2118033002 Review-Url: https://codereview.chromium.org/2118033002
* Blacklist fontscalerdistortable on Win8.Gravatar benjaminwagner2016-07-06
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=2109583003 Review-Url: https://codereview.chromium.org/2109583003
* Enlarge strokerect GM size to reveal hidden contentGravatar bsalomon2016-07-06
| | | | | | GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2126723002 Review-Url: https://codereview.chromium.org/2126723002
* Document SkSurface::MakeRaster's memory initializationGravatar fmalita2016-07-06
| | | | | | | | | So clients don't go clearing w/ SK_ColorTRANSPARENT unnecessarily. R=reed@google.com GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2066903003 Review-Url: https://codereview.chromium.org/2066903003
* Update dev docs to reference the new skia debugger (skiaserve)Gravatar pdr2016-07-06
| | | | | | | | | | | | | | | | This patch adds a new markdown file describing the new debugger along with instructions on running it locally and a sweet screenshot. The screenshot has been sized to not be clipped when viewed in production. The old documentation for the qt-based debugger has been marked as deprecated and all docs referencing the old debugger have been updated. BUG=skia:5493 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2121673002 NOTRY=true DOCS_PREVIEW= https://skia.org/?cl=2121673002 Review-Url: https://codereview.chromium.org/2121673002
* make setScaleTranslate publicGravatar reed2016-07-05
| | | | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2122093002 TBR= Review-Url: https://codereview.chromium.org/2122093002
* Revert of Begin instanced rendering for simple shapes (patchset #20 ↵Gravatar ksakamoto2016-07-05
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | id:380001 of https://codereview.chromium.org/2066993003/ ) Reason for revert: This caused static initializer regressions in Chromium (crbug.com/625728). Relevant build logs here: Linux: https://build.chromium.org/p/chromium/builders/Linux%20x64/builds/21849 Mac: https://build.chromium.org/p/chromium/builders/Mac/builds/17350 Relevant lines from the error log: Linux: # InstanceProcessor.cpp GrUniqueKey::GenerateDomain() # InstanceProcessor.cpp gr_instanced::kShapeBufferDomain FAILED linux-release-64/sizes/nacl_helper-si/initializers: actual 8, expected 7, better lower FAILED linux-release-64/sizes/chrome-si/initializers: actual 8, expected 7, better lower Mac: FAILED mac-release/sizes/chrome-si/initializers: actual 2, expected 0, better lower Original issue's description: > Begin instanced rendering for simple shapes > > Adds a module that performs instanced rendering and starts using it > for a select subset of draws on Mac GL platforms. The instance > processor can currently handle rects, ovals, round rects, and double > round rects. It can generalize shapes as round rects in order to > improve batching. The instance processor also employs new drawing > algorithms, irrespective of instanced rendering, that improve GPU-side > performance (e.g. sample mask, different triangle layouts, etc.). > > This change only scratches the surface of instanced rendering. The > majority of draws still only have one instance. Future work may > include: > > * Passing coord transforms through the texel buffer. > * Sending FP uniforms through instanced vertex attribs. > * Using instanced rendering for more draws (stencil writes, > drawAtlas, etc.). > * Adding more shapes to the instance processor’s repertoire. > * Batching draws that have mismatched scissors (analyzing draw > bounds, inserting clip planes, etc.). > * Bindless textures. > * Uber shaders. > > BUG=skia: > GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2066993003 > > Committed: https://skia.googlesource.com/skia/+/42eafa4bc00354b132ad114d22ed6b95d8849891 NOTREECHECKS=true TBR=bsalomon@google.com,egdaniel@google.com,robertphillips@google.com,csmartdalton@google.com # Not skipping CQ checks because original CL landed more than 1 days ago. BUG=skia: Review-Url: https://codereview.chromium.org/2123693002
* Revert of Fix caching of sample locations (patchset #3 id:40001 of ↵Gravatar rmistry2016-07-04
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/2111423002/ ) Reason for revert: Seems to be causing Chromium roll failures: * https://codereview.chromium.org/2120373003 * https://codereview.chromium.org/2117193002 * https://codereview.chromium.org/2124653002 Original issue's description: > Fix caching of sample locations > > The original caching logic for sample locations wishfully assumed that > the GPU would always use the same sample pattern for render targets > that had the same number of samples. It turns out we can't rely on > that. This change improves the caching logic to handle mismatched > simple patterns with the same count, and adds a unit test that > emulates different sample patterns observed on real hardware. > > BUG=skia: > GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2111423002 > > Committed: https://skia.googlesource.com/skia/+/09d49a3bfe2d1e652a648ce1ea0962b38d10d166 TBR=bsalomon@google.com,csmartdalton@google.com # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review-Url: https://codereview.chromium.org/2120403002
* Fix caching of sample locationsGravatar csmartdalton2016-07-04
| | | | | | | | | | | | | | The original caching logic for sample locations wishfully assumed that the GPU would always use the same sample pattern for render targets that had the same number of samples. It turns out we can't rely on that. This change improves the caching logic to handle mismatched simple patterns with the same count, and adds a unit test that emulates different sample patterns observed on real hardware. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2111423002 Review-Url: https://codereview.chromium.org/2111423002
* all dumpcanvas to be built in releaseGravatar reed2016-07-04
| | | | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2123633002 TBR= Review-Url: https://codereview.chromium.org/2123633002
* Update SKP versionGravatar update-skps2016-07-04
| | | | | | | | | | Automatic commit by the RecreateSKPs bot. TBR= NO_MERGE_BUILDS GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2119053002 Review-Url: https://codereview.chromium.org/2119053002
* Revert of Increase batching for AA fill rects. (patchset #4 id:60001 of ↵Gravatar benjaminwagner2016-07-04
| | | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/2116823002/ ) Reason for revert: I believe this is causing the Chromium DEPS roll to fail due to linux_blink_rel. Reverting for now to get the roll going again. Original issue's description: > Increase batching for AA fill rects. > > This allows batching of rects provided without a local matrix when local coords are required and when the view matrix changes. > > It also allows batching of rects with a local matrix with rects without a local matrix. > GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2116823002 > > Committed: https://skia.googlesource.com/skia/+/e525ecaf63f225f1da6e9834f7a291c06ad44d23 TBR=robertphillips@google.com,bsalomon@google.com NOTREECHECKS=true # Not skipping CQ checks because original CL landed more than 1 days ago. Review-Url: https://codereview.chromium.org/2124603002
* Exclude SkSL in public.bzl.Gravatar benjaminwagner2016-07-02
| | | | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2122463002 TBR=ethannicholas NOTREECHECKS=true Review-Url: https://codereview.chromium.org/2122463002
* fixed SkSL memory leakGravatar ethannicholas2016-07-01
| | | | | | | GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2119903002 NOTREECHECKS=true Review-Url: https://codereview.chromium.org/2119903002
* fixed non-GPU buildGravatar ethannicholas2016-07-01
| | | | | | | GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2120543003 NOTREECHECKS=true Review-Url: https://codereview.chromium.org/2120543003
* Update Win toolchainGravatar skia.buildbots2016-07-01
| | | | | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2116173002 NOTREECHECKS=true NOTRY=true TBR=rmistry@google.com Review-Url: https://codereview.chromium.org/2116173002
* Specify new Windows Toolchain isolated hash.Gravatar benjaminwagner2016-07-01
| | | | | | | | | NOTREECHECKS=true NOTRY=true TBR=rmistry@google.com GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2121443002 Review-Url: https://codereview.chromium.org/2121443002
* Increase batching for AA fill rects.Gravatar bsalomon2016-07-01
| | | | | | | | | This allows batching of rects provided without a local matrix when local coords are required and when the view matrix changes. It also allows batching of rects with a local matrix with rects without a local matrix. GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2116823002 Review-Url: https://codereview.chromium.org/2116823002
* Split apart the GP construction for the NonAAFillRect BatchesGravatar robertphillips2016-07-01
| | | | | | | | This sets up for adding yet another batch type that specializes for rectStaysRect GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2106063005 Review-Url: https://codereview.chromium.org/2106063005
* using_skia_and_harfbuzz: use default typefaceGravatar halcanary2016-07-01
| | | | | | | | | Also: add little script to generate utf-8 test text. BUG=skia:5434 TBR=bungeman@google.com Review-Url: https://codereview.chromium.org/2118833002
* initial checkin of SkSL compilerGravatar ethannicholas2016-07-01
| | | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1984363002 CQ_EXTRA_TRYBOTS=client.skia.compile:Build-Ubuntu-GCC-x86_64-Release-CMake-Trybot,Build-Mac-Clang-x86_64-Release-CMake-Trybot Review-Url: https://codereview.chromium.org/1984363002
* Use bounds of batches for render pass boundsGravatar egdaniel2016-07-01
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2113303002 Review-Url: https://codereview.chromium.org/2113303002
* Fix vulkan ImageView swizzling to match specGravatar egdaniel2016-07-01
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2115903002 Review-Url: https://codereview.chromium.org/2115903002
* Cleanup non-AA fill rect batch construction.Gravatar bsalomon2016-07-01
| | | | | | GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2119853002 Review-Url: https://codereview.chromium.org/2119853002
* Simplify AA fill rect batch construction and rename Geometry->RectInfoGravatar bsalomon2016-07-01
| | | | | | GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2116723003 Review-Url: https://codereview.chromium.org/2116723003
* Remove unneeded helpers in AA and non-AA rect GrBatch subclasses.Gravatar bsalomon2016-07-01
| | | | | | GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2115673002 Review-Url: https://codereview.chromium.org/2115673002
* Cleanup PLSPathBatchGravatar bsalomon2016-06-30
| | | | | | GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2116543002 Review-Url: https://codereview.chromium.org/2116543002
* Disable GPU dithering in Android framework buildsGravatar bsalomon2016-06-30
| | | | | | GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2115683002 Review-Url: https://codereview.chromium.org/2115683002
* Remove unused batch rect append functionsGravatar bsalomon2016-06-30
| | | | | | GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2106283003 Review-Url: https://codereview.chromium.org/2106283003
* Remove GrTInstanceBatchGravatar bsalomon2016-06-30
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2114663003 Review-Url: https://codereview.chromium.org/2114663003
* Fix bug where ovals' AA exceed bounds by .5 pixelGravatar vjiaoblack2016-06-30
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2106953009 Review-Url: https://codereview.chromium.org/2106953009
* Begin instanced rendering for simple shapesGravatar csmartdalton2016-06-30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds a module that performs instanced rendering and starts using it for a select subset of draws on Mac GL platforms. The instance processor can currently handle rects, ovals, round rects, and double round rects. It can generalize shapes as round rects in order to improve batching. The instance processor also employs new drawing algorithms, irrespective of instanced rendering, that improve GPU-side performance (e.g. sample mask, different triangle layouts, etc.). This change only scratches the surface of instanced rendering. The majority of draws still only have one instance. Future work may include: * Passing coord transforms through the texel buffer. * Sending FP uniforms through instanced vertex attribs. * Using instanced rendering for more draws (stencil writes, drawAtlas, etc.). * Adding more shapes to the instance processor’s repertoire. * Batching draws that have mismatched scissors (analyzing draw bounds, inserting clip planes, etc.). * Bindless textures. * Uber shaders. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2066993003 Review-Url: https://codereview.chromium.org/2066993003
* Hide GrNonAANinePatchBatch::Geometry and rename to PatchGravatar bsalomon2016-06-30
| | | | | | GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2111943002 Review-Url: https://codereview.chromium.org/2111943002
* Hide MSAAPathBatch::Geometry and rename to PathInfoGravatar bsalomon2016-06-30
| | | | | | GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2116533002 Review-Url: https://codereview.chromium.org/2116533002
* SkPDF: Glyph Useage Map improvementsGravatar halcanary2016-06-30
| | | | | | | | | | | | | | | | | | | | Instead of having a fFontGlyphUsage on each device and one on each document, just have the one on the document, and never merge. Make fGlyphUsage accesible on SkPDFDocument. Remove SkPDFGlyphSetMap::merge, ::reset, and SkPDFGlyphSet::merge. SkPDFGlyphSetMap has an TArray of SkPDFGlyphSet, not TDArray of SkPDFGlyphSet pointers. SkPDFGlyphSet and SkPDFBitset get move constructors. All tests produce exactly identical output PDFs. BUG=skia:5434 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2112943002 Review-Url: https://codereview.chromium.org/2112943002
* Remove support for GL discardsGravatar egdaniel2016-06-30
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2112693003 Review-Url: https://codereview.chromium.org/2112693003
* Fix png encoding in skia debuggerGravatar msarett2016-06-30
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2119513002 Review-Url: https://codereview.chromium.org/2119513002
* Add cubic example and draw original pathGravatar hstern2016-06-30
| | | | | | | | | | | - Add cubic example - Draw original path - Also added dumpHex to stderr option on 'D' BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2106063004 Review-Url: https://codereview.chromium.org/2106063004
* skiaserve: Use snakeCase for JSON identifiers.Gravatar jcgregorio2016-06-30
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2108823006 Review-Url: https://codereview.chromium.org/2108823006
* Add static buffer supportGravatar jvanverth2016-06-30
| | | | | | | | | Put resource tracking check in the right place BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2112653002 Review-Url: https://codereview.chromium.org/2112653002
* Add discard support to VulkanGravatar egdaniel2016-06-30
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2104353002 Review-Url: https://codereview.chromium.org/2104353002
* SkPDF: allow overriding Producer metadataGravatar halcanary2016-06-30
| | | | | | | | | | | | I recommend not using this functionality. Also, some documentation. BUG=skia:5436 GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2074583003 Review-Url: https://codereview.chromium.org/2074583003
* use mapRectScaleTranslate for common case in clipRectGravatar reed2016-06-30
| | | | | | | BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2119473002 Review-Url: https://codereview.chromium.org/2119473002
* Hide GrDrawVerticesBatch::Geometry and rename to MeshGravatar bsalomon2016-06-30
| | | | | | GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2104383003 Review-Url: https://codereview.chromium.org/2104383003
* debugger: Allow for larger images.Gravatar jcgregorio2016-06-30
| | | | | | | | | Also differentiate between max dimensions and default dimensions. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2113693002 Review-Url: https://codereview.chromium.org/2113693002
* inline very common intersect+rect_rgnGravatar reed2016-06-30
| | | | | | | | | | | | | Extracted from previous compound CL (with other changes). This is the very common case for clips, and inlining this eliminates a series of function calls. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2116513002 TBR=mtklein Review-Url: https://codereview.chromium.org/2116513002
* Update skia milestone to nextGravatar hcm2016-06-30
| | | | | | | | | | No API changes, just moving version fwd. TBR=reed@google.com GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2115573002 Review-Url: https://codereview.chromium.org/2115573002
* Even more hiding of Geometry structs in GrBatch subclasses.Gravatar bsalomon2016-06-30
| | | | | | GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2110853004 Review-Url: https://codereview.chromium.org/2110853004