aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
* Turn on Sk4px xfermodes when we have NEON too.Gravatar mtklein2015-05-13
| | | | | | | | | | | For SSE, Sk4px is better than Sk4f is better than SkXfermodes_opts_SSE2 (where implemented). For NEON, Sk4px is better than SkXfermodes_opts_arm_neon is better than Sk4f (where implemented). This is a 1.6-1.9x speedup for Plus,Modulate, and Screen for NEON. BUG=skia: Review URL: https://codereview.chromium.org/1128053004
* fix bounds for BW lines, AA HairlinesGravatar joshualitt2015-05-13
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1123253003
* Fix assert in GrBatchAtlas.Gravatar jvanverth2015-05-13
| | | | | | Stops this assert from triggering on the iOS build. Review URL: https://codereview.chromium.org/1131003004
* simplify RawIterGravatar reed2015-05-13
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1137783006
* Revert of Remove workaround to for lower glsl support for qualcomm (patchset ↵Gravatar egdaniel2015-05-13
| | | | | | | | | | | | | | | | | | | | | | #1 id:1 of https://codereview.chromium.org/1133543004/) Reason for revert: breaks many qualcomm gms Original issue's description: > Remove workaround to for lower glsl support for qualcomm > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/9770bdce776f45367c419b8dd82219eece26ca5c TBR=bsalomon@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/1135393003
* Remove documentation for missing SkiaExamples.Gravatar scroggo2015-05-13
| | | | | | | | | SkiaExamples were removed in February, so no need to instruct newbies to try building them. NOTRY=true DOCS_PREVIEW= https://skia.org/?cl=1135303002 Review URL: https://codereview.chromium.org/1135303002
* Documentation: open GM in SampleAppGravatar halcanary2015-05-13
| | | | | | | NOTRY=true DOCS_PREVIEW= https://skia.org/dev/testing/tests?cl=1138353002 Review URL: https://codereview.chromium.org/1138353002
* Remove workaround to for lower glsl support for qualcommGravatar egdaniel2015-05-13
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1133543004
* Document NO_MERGE_BUILDS keywordGravatar borenet2015-05-13
| | | | | | | | BUG=skia:3584 NOTRY=true DOCS_PREVIEW= https://skia.org/?cl=1135313004 Review URL: https://codereview.chromium.org/1135313004
* Fix SkStrokeRec == to report true for all fillsGravatar kkinnunen2015-05-13
| | | | | | | Fix SkStrokeRec == to report true for all fills, regardless of the stale stroking data the object might contain. Review URL: https://codereview.chromium.org/1130153002
* Plus xfermode using Sk4px.Gravatar mtklein2015-05-12
| | | | | | | | | | | | | | | | | | | Xfermode_Plus runs 4-5x faster. We expect mixed_xfermodes to have a small diff. This is because kFoldCoverageIntoSrcAlpha was incorrectly set to true. This implementation handily beats the Sk4f impl, the portable impl, and the existing SSE2 impl. Reading the SkXfermodes_opts_SSE2.cpp file, I'm pretty confident that we'll be able to beat all SSE2 impls. I believe this impl will beat or match the existing NEON impl too, but that may not be true for more complicated xfermodes. They can take advantage of transposing ARGBARGB... to AAAARRRR.... cheaply and I haven't figured out an abstraction for that yet that doesn't screw SSE. Adds: - MapDstSrc() to Sk4px - saturatedAdd() to SkNi (only implemented as far as it's used). - div255Narrow() BUG=skia: Review URL: https://codereview.chromium.org/1138893002
* Add check in GrGpu createTexture to make sure we don't create multisampled ↵Gravatar egdaniel2015-05-12
| | | | | | | | textures. BUG=skia: Review URL: https://codereview.chromium.org/1138223003
* Make Noto Sans Mono CJK JP an alias to MS GothicGravatar jshin2015-05-12
| | | | | | | | | | | This CL works together with the following CrOS CL: https://chromium-review.googlesource.com/#/c/269899/ BUG=chromium:486164 TEST=With the above CrOS CL, 'MS Gothic' in CSS is replaced by 'Noto Sans Mono CJK JP'. Review URL: https://codereview.chromium.org/1134853002
* add option to supress prints on contextGravatar joshualitt2015-05-12
| | | | | | | | BUG=skia: Committed: https://skia.googlesource.com/skia/+/55661337667a8305ebb100e36af23d34c2fb83ba Review URL: https://codereview.chromium.org/1128903008
* Fix valgrind bug in Porter Duff LCD.Gravatar egdaniel2015-05-12
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1135283002
* PerGlyphInfo is on SkTypefaceGravatar reed2015-05-12
| | | | | | | | | BUG=skia: NOTRY=True NOTREECHECKS=True TBR= Review URL: https://codereview.chromium.org/1128023004
* Revert of Revert of stop calling SkScalarDiv (patchset #1 id:1 of ↵Gravatar reed2015-05-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/1138263002/) Reason for revert: android patched, blink has rolled Original issue's description: > Revert of stop calling SkScalarDiv (patchset #4 id:60001 of https://codereview.chromium.org/1135053002/) > > Reason for revert: > need to wait for Blink roll (and patch android) > > Original issue's description: > > stop calling SkScalarDiv > > > > BUG=skia: > > TBR= > > > > Committed: https://skia.googlesource.com/skia/+/67d71c898249a7af3523b16c6a69895a63bfae0a > > TBR= > NOPRESUBMIT=true > NOTREECHECKS=true > NOTRY=true > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/2629697933b5cc975e45d2a45c48f803fc6cbcec TBR= NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/1135693003
* fix reference to _PerGlyphInfoGravatar reed2015-05-12
| | | | | | | | | BUG=skia: TBR=bungeman@google.com NOTRY=True NOTREECHECKS=True Review URL: https://codereview.chromium.org/1136283004
* opt into SkScalarDiv until we can fix android call-sitesGravatar reed2015-05-12
| | | | | | | | BUG=skia: TBR= NOTRY=True Review URL: https://codereview.chromium.org/1138273002
* Revert of stop calling SkScalarDiv (patchset #4 id:60001 of ↵Gravatar reed2015-05-12
| | | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/1135053002/) Reason for revert: need to wait for Blink roll (and patch android) Original issue's description: > stop calling SkScalarDiv > > BUG=skia: > TBR= > > Committed: https://skia.googlesource.com/skia/+/67d71c898249a7af3523b16c6a69895a63bfae0a TBR= NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/1138263002
* Make SkAdvancedTypefaceMetrics private.Gravatar reed2015-05-12
| | | | Review URL: https://codereview.chromium.org/1129283003
* stop calling SkScalarDivGravatar reed2015-05-12
| | | | | | | BUG=skia: TBR= Review URL: https://codereview.chromium.org/1135053002
* quickie tool to exercise chrome filter fuzz filesGravatar caryclark2015-05-12
| | | | | | | | | | | This is a quick Skia transcription of the Chromium tool at src/skia/tools/filter_fuzz_stub.cc to read and decode filters captured as .fil files. R=joshualitt@google.com,mtklein@google.com,reed@google.com,robertphillips@google.com BUG=487213 Review URL: https://codereview.chromium.org/1126423005
* Revert of add option to supress prints on context (patchset #5 id:80001 of ↵Gravatar joshualitt2015-05-12
| | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/1128903008/) Reason for revert: breaks windows rollbots Original issue's description: > add option to supress prints on context > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/55661337667a8305ebb100e36af23d34c2fb83ba TBR=bsalomon@google.com,robertphillips@google.com,joshualitt@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/1125193006
* add option to supress prints on contextGravatar joshualitt2015-05-12
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1128903008
* Make Porter Duff LCD XP its own XferProcessorGravatar egdaniel2015-05-12
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1134093003
* Sk4pxGravatar mtklein2015-05-12
| | | | | | | | | | Xfermode_SrcOver: SSE: 2.08ms -> 2.03ms (~2% faster) NEON: my N5 is noisy, but there appears to be no perf change BUG=skia: Review URL: https://codereview.chromium.org/1132273004
* Remove quickstart docs.Gravatar tfarina2015-05-11
| | | | | | | | | | It is now obsolete by skia.org. All the relevant doc is hosted there now. BUG=None R=mtklein@google.com Review URL: https://codereview.chromium.org/1138833005
* Fix blur on large glyphs in runs < SkGlyphCache::maxGravatar joshualitt2015-05-11
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1135113002
* remove near one check for arcsGravatar caryclark2015-05-11
| | | | | | | | | Small arcs are pinned by SkSinCos and do not need to be additionally pinned by SkConic::BuildUnitArc. R=reed@google.com, djsollen@google.com Review URL: https://codereview.chromium.org/1133113003
* fix for cached textblobs look garbledGravatar joshualitt2015-05-11
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1135813007
* Add virtual destructor for GrDrawTargetCapsGravatar bsalomon2015-05-11
| | | | | | TBR=jvanverth@google.com Review URL: https://codereview.chromium.org/1134043004
* Add assignment op to enum bitfield opsGravatar cdalton2015-05-11
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1139513002
* Add function for logging blend info on XP.Gravatar bsalomon2015-05-11
| | | | Review URL: https://codereview.chromium.org/1132373003
* convex path renderer unit testsGravatar joshualitt2015-05-11
| | | | | | | TBR=bsalomon@google.com BUG=skia: Review URL: https://codereview.chromium.org/1134753002
* Add unit tests to text contextGravatar joshualitt2015-05-11
| | | | | | | TBR=bsalomon@google.com BUG=skia: Review URL: https://codereview.chromium.org/1128153005
* Add unit tests for stroke rects and drawVerticesGravatar joshualitt2015-05-11
| | | | | | | TBR=bsalomon@google.com BUG=skia: Review URL: https://codereview.chromium.org/1133613004
* AADistanceFieldPathRenderer unit testsGravatar joshualitt2015-05-11
| | | | | | | TBR=bsalomon@google.com BUG=skia: Review URL: https://codereview.chromium.org/1129083005
* Path ops formerly found the topmost unprocessed edge and determined its ↵Gravatar caryclark2015-05-11
| | | | | | | | | | | | | | | angle sort order to initialize the winding. This never worked correctly with cubics and was flaky with paths consisting mostly of vertical edges. This replacement shoots axis-aligned rays through all intersecting edges to find the outermost one either horizontally or vertically. The resulting code is smaller and twice as fast. To support this, most of the horizontal / vertical intersection code was rewritten and standardized, and old code supporting the top-directed winding was deleted. Contours were pointed to by an SkTDArray. Instead, put them in a linked list, and designate the list head with its own class to ensure that methods that take lists of contours start at the top. This change removed a large percentage of memory allocations used by path ops. TBR=reed@google.com BUG=skia:3588 Review URL: https://codereview.chromium.org/1111333002
* print on unused uniforms in debug buildsGravatar joshualitt2015-05-11
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1136803002
* Add android_launch_app script to allow command line options to be passed to ↵Gravatar Derek Sollenberger2015-05-11
| | | | | | | | | | the sampleApp when starting. BUG=skia:3815 DOCS_PREVIEW= https://skia.org/?cl=1136753003 R=bungeman@google.com, tomhudson@google.com Review URL: https://codereview.chromium.org/1136753003
* SkPictureShader: floor scaledSize not to exceed maxTextureSizeGravatar gen.kim2015-05-10
| | | | | | | | | scaledSize becomes larger than maxTextureSize by zoom level because it is rounded. BUG=chromium:485916 Review URL: https://codereview.chromium.org/1129413002
* Only discard for XP dstCopies if we have a coverage value.Gravatar egdaniel2015-05-10
| | | | | | | | TBR=bsalomon@google.com BUG=skia: Review URL: https://codereview.chromium.org/1125283005
* Update SKP versionGravatar skia.buildbots2015-05-10
| | | | | | | | Automatic commit by the RecreateSKPs bot. TBR= Review URL: https://codereview.chromium.org/1123233004
* Disable advance blend equations.Gravatar egdaniel2015-05-08
| | | | | | | | | This is a temporary disabling of advanced blend equations till we can fix the various bugs associated with it. BUG=skia:3822 Review URL: https://codereview.chromium.org/1130003003
* Remove coverage multiplies when it is known to be fully opaque.Gravatar egdaniel2015-05-08
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1132883002
* Remove unused SK_SUPPORT_LEGACY_ROUND_MIPMAP_LEVEL_CHOICE guardGravatar fmalita2015-05-08
| | | | | | R=reed@google.com,robertphillips@google.com Review URL: https://codereview.chromium.org/1135823003
* whitespace change (testing flaky bots)Gravatar halcanary2015-05-08
| | | | | | | NOTRY=true TBR= Review URL: https://codereview.chromium.org/1132043004
* whitespace changeGravatar halcanary2015-05-08
| | | | | | | NOTRY=true TBR= Review URL: https://codereview.chromium.org/1131053004
* Trim whitespace from parsed filename in Android v21.Gravatar bungeman2015-05-08
| | | | | | | | | | | | | | | | | | | | | The entire text content of the 'font' element is currently used as the file name. This wasn't an issue in earlier versions, as the 'file' element was dedicated to only containing the file name. The new 'font' element also contains a number of attributes and potentially other tags. This means that a 'font' element can become quite long, making it desireable to be able to split it across multiple lines. However, splitting the 'font' element across multiple lines is currently difficult and awkward as any whitespace outside of tags will be considered part of the file name. This change means that any leading or trailing whitespace will not be considered part of the file name. This only applies to v21 and later files, so while this restricts font file names from beginning and ending with whitespace, it is unlikely to break any users in practice. It is probably also undesireable to have font files with names that begin or end with whitespace in any event. Review URL: https://codereview.chromium.org/1125413003