aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
...
* Add SkBarriers_tsan.h.Gravatar mtklein2014-06-06
| | | | | | | | | | | | | | Slight counterproposal to crrev.com/310663002. BUG=skia: No API changes. R=bungeman@google.com, mtklein@google.com, reed@google.com Author: mtklein@chromium.org Review URL: https://codereview.chromium.org/306373002
* Use vertex attribute for text color in GrBitmapTextContextGravatar bsalomon2014-06-06
| | | | | | | | R=jvanverth@google.com Author: bsalomon@google.com Review URL: https://codereview.chromium.org/316063002
* Initial KTX encoderGravatar krajcevski2014-06-06
| | | | | | | | | | | The encoder comes with tests to check that the encoding/decoding operations between ETC encoded bitmaps and ARGB bitmaps are sane. R=bsalomon@google.com, robertphillips@google.com Author: krajcevski@google.com Review URL: https://codereview.chromium.org/312353003
* Fix SK_API for DirectWrite port.Gravatar Ben Wagner2014-06-05
| | | | | | | | The declaration and definition should have the same calling convention. TBR=reed@google.com Review URL: https://codereview.chromium.org/317993002
* Rename SkFontHost_win_dw.cpp SkScalerContext.Gravatar Ben Wagner2014-06-05
| | | | | | | | This is a continuation of https://codereview.chromium.org/314193002/ . TBR=reed@google.com Review URL: https://codereview.chromium.org/315173004
* Split SkFontHost_win_dw.Gravatar bungeman2014-06-05
| | | | | | | | | | | | Split SkFontHost_win_dw into FontMgr, Typeface, and ScalerContext. This makes working on these files easier, and moves away from the legacy FontHost naming. R=reed@google.com Author: bungeman@google.com Review URL: https://codereview.chromium.org/314193002
* SkClipStack::Element tweaks.Gravatar fmalita2014-06-05
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | ( This is intended to facilitate efficient kMatrix_SaveFlags emulation on restore(): * collect all clip stack elements for the current save count into a side clip stack * canvas.restore(everything) * replay the collected clip ops to restore the initial clip state => we restored the matrix but the clip state is unchanged ) Two main changes: * expose the save count for SkClipStack::Element * expose a replay method for the same (logic relocated from SkCanvas::replayClips) The SkCanvas::ClipVisitor shuffling is to enable forward decl in SkClipStack.h (cannot fwdecl a nested class). R=reed@google.com, robertphillips@google.com TBR=reed@google.com Author: fmalita@chromium.org Review URL: https://codereview.chromium.org/269693003
* Update gyp for msvs-ninja clean.Gravatar bungeman2014-06-05
| | | | | | | | | BUG=skia:2439 R=borenet@google.com, epoger@google.com Author: bungeman@google.com Review URL: https://codereview.chromium.org/317833005
* Fixes for SkAlphaTypes.Gravatar scroggo2014-06-05
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | When using allocPixels to set the info and create a new pixelRef in one shot, create the pixelRef with the corrected info of the bitmap (i.e. alphatype). Simplify the check to determine whether to copy the genID in SkBitmap:: copyTo's same configs/same sizes case. The old logic (only checking the height and assuming the others must be the same) relied on a deep (mis?)understanding of the rest of the function, and I cannot convince myself that it has to be true, given that readPixels may have switched src to a different SkBitmap. Instead, just compare the SkImageInfos, which is much clearer. (This also caught a bug where a 565 bitmap had the wrong alphatype, leading to the allocPixels fixes.) If readPixels succeeds but the source is unpremultiplied, fail out of copyTo, since readPixels assumes premultiplied. When copying from 8888 to 4444 or in the fallback case of drawing, return false if the alphatype is unpremul, which would have failed anyway. BUG=skia:2012 R=reed@google.com, bsalomon@google.com Author: scroggo@google.com Review URL: https://codereview.chromium.org/317053003
* reland "rebaseline_server: download actual-results.json files from GCS ↵Gravatar epoger2014-06-05
| | | | | | | | | | | | | instead of SVN" relands https://codereview.chromium.org/310093003 with modifications. BUG=skia:2641 R=jcgregorio@google.com Author: epoger@google.com Review URL: https://codereview.chromium.org/313343003
* Add mips64 to the makefile.Gravatar scroggo2014-06-05
| | | | | | | | | | | | | Currently we have no optimizations for mips64, so it correctly contains the various _none.cpps. Corresponds to https://android-review.googlesource.com/#/c/96536/1 R=halcanary@google.com, djsollen@google.com Author: scroggo@google.com Review URL: https://codereview.chromium.org/311163003
* add SK_SUPPORT_LEGACY_COMPUTE_CONFIG_SIZEGravatar reed2014-06-05
| | | | | | | | | TBR=scroggo NOTRY=True Author: reed@google.com Review URL: https://codereview.chromium.org/315263002
* Rename skimage expectations for ASAN bot to match bot renaming.Gravatar mtklein2014-06-05
| | | | | | | | | BUG=skia: R=borenet@google.com, mtklein@google.com Author: mtklein@chromium.org Review URL: https://codereview.chromium.org/316063004
* remove SkBitmap::allocConfigPixels and update doxGravatar reed2014-06-05
| | | | | | | | | | I see no callers in android R=scroggo@google.com Author: reed@google.com Review URL: https://codereview.chromium.org/310283004
* Revert of Add SSE4 optimization of S32A_Opaque_Blitrow ↵Gravatar jvanverth2014-06-05
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (https://codereview.chromium.org/289473009/) Reason for revert: Buildbot failures on Mac 10.6 and Mac 10.7. R=reed@google.com, mtklein@google.com, tomhudson@google.com, djsollen@google.com, joakim.landberg@intel.com, henrik.smiding@intel.com TBR=reed@google.com NOTRY=True Original issue's description: > Add SSE4 optimization of S32A_Opaque_Blitrow > > Adds optimization of Skia S32A_Opaque_Blitrow blitter using SSE4.2 SIMD > instruction set. Special case for when alpha is zero or opaque. > > Performance increase of 10%-400% compared to the existing SSE2 > optimization (measured on Silvermont architecture). > Noticeable in ~25 different skia bench subtests, especially in > bitmap_8888_*, repeatTile_*, and morph_*. > > bitmap_8888_A - 100% faster > bitmap_8888_A_source_transparent - 250% faster > bitmap_8888_A_source_opaque - 25% faster > bitmap_8888_A_scale_bicubic - 75% faster > > Signed-off-by: Henrik Smiding <henrik.smiding@intel.com> > > Committed: https://skia.googlesource.com/skia/+/e2527b147679b0c43019fae7d59cc3777d2d097e Author: jvanverth@google.com Review URL: https://codereview.chromium.org/311053009
* remove ashmem from skiaGravatar halcanary2014-06-05
| | | | | | | | R=scroggo@google.com, djsollen@google.com Author: halcanary@google.com Review URL: https://codereview.chromium.org/319533004
* Rebaselines for drrect GM on GPUGravatar bsalomon2014-06-05
| | | | | | | | TBR= Author: bsalomon@google.com Review URL: https://codereview.chromium.org/318773007
* Add SSE4 optimization of S32A_Opaque_BlitrowGravatar henrik.smiding2014-06-05
| | | | | | | | | | | | | | | | | | | | | | | Adds optimization of Skia S32A_Opaque_Blitrow blitter using SSE4.2 SIMD instruction set. Special case for when alpha is zero or opaque. Performance increase of 10%-400% compared to the existing SSE2 optimization (measured on Silvermont architecture). Noticeable in ~25 different skia bench subtests, especially in bitmap_8888_*, repeatTile_*, and morph_*. bitmap_8888_A - 100% faster bitmap_8888_A_source_transparent - 250% faster bitmap_8888_A_source_opaque - 25% faster bitmap_8888_A_scale_bicubic - 75% faster Signed-off-by: Henrik Smiding <henrik.smiding@intel.com> R=reed@google.com, mtklein@google.com, tomhudson@google.com, djsollen@google.com, joakim.landberg@intel.com Author: henrik.smiding@intel.com Review URL: https://codereview.chromium.org/289473009
* Use the correct homophone of 'there'.Gravatar scroggo2014-06-05
| | | | | | | | | R=reed@google.com TBR=reed@google.com Author: scroggo@google.com Review URL: https://codereview.chromium.org/319493004
* Fix submit_tryGravatar borenet2014-06-05
| | | | | | | | | BUG=skia:2643 R=epoger@google.com, tfarina@chromium.org Author: borenet@google.com Review URL: https://codereview.chromium.org/317823003
* Add bench for rectanizersGravatar robertphillips2014-06-05
| | | | | | | | R=bsalomon@google.com, jvanverth@google.com Author: robertphillips@google.com Review URL: https://codereview.chromium.org/319523003
* Change default generators on windows to include msvs-ninja rather than msvsGravatar bsalomon2014-06-05
| | | | | | | | | | BUG=skia:2439 R=epoger@google.com TBR=epoger@google.com Author: bsalomon@google.com Review URL: https://codereview.chromium.org/317903002
* Two new benches for ETC1 bitmaps.Gravatar krajcevski2014-06-05
| | | | | | | | | | | | | | | | | First bench: Simply render from a compressed ETC1 bitmap. This is roughly equal between compressed and uncompressed data. Second bench: Notify the bitmap that the pixels have changed (that's right, like a liar) to force a re-upload to the GPU. On this bench, decompressed textures seem to inexplicably do an order of magnitude better than compressed textures when it should be the other way around, but that investigation is reserved for a future CL. R=robertphillips@google.com Author: krajcevski@google.com Review URL: https://codereview.chromium.org/317023002
* On Windows, fail if there are missing files specified in the project.Gravatar jvanverth2014-06-04
| | | | | | | | | BUG=skia:2638 R=robertphillips@google.com, borenet@google.com, rmistry@google.com, epoger@google.com Author: jvanverth@google.com Review URL: https://codereview.chromium.org/319503003
* Remove SK_IGNORE_DWRITE_BITMAP_FIX.Gravatar bungeman2014-06-04
| | | | | | | | | | | This define has been removed from Blink and is no longer used anywhere. R=reed@google.com TBR=reed1 Author: bungeman@google.com Review URL: https://codereview.chromium.org/319513006
* Revert "rebaseline_server: download actual-results.json files from GCS ↵Gravatar epoger2014-06-04
| | | | | | | | | | | | | | | | instead of SVN" This reverts commit f4666045f378dee22ce40108e90bf5195634ebfc. BUG=skia:2641 NOTREECHECKS=True NOTRY=True R=jcgregorio@google.com TBR=jcgregorio Author: epoger@google.com Review URL: https://codereview.chromium.org/317783004
* Update expectations for gyp change.Gravatar bungeman2014-06-04
| | | | | | | | | | | | | It appears that with the gyp update, floating point operations on x86_32 are ever so slightly different when drawing some paths. R=jvanverth@google.com TBR=jvanverth@google.com NOTRY=True Author: bungeman@google.com Review URL: https://codereview.chromium.org/315043006
* Remove Android flag for legacy SkCanvas::drawPicture APIGravatar robertphillips2014-06-04
| | | | | | | | | | Android should no longer need this flag R=scroggo@google.com, djsollen@google.com Author: robertphillips@google.com Review URL: https://codereview.chromium.org/316933005
* Remove SkImageRef and related functionality.Gravatar Hal Canary2014-06-04
| | | | | | | | From here on out, use SkDecodingImageGenerator+SkDiscardablePixelRef. R=scroggo@google.com Review URL: https://codereview.chromium.org/100183002
* Update gyp for msvs-ninja.Gravatar bungeman2014-06-04
| | | | | | | | | BUG=skia:2439 R=borenet@google.com, epoger@google.com Author: bungeman@google.com Review URL: https://codereview.chromium.org/311213002
* Add utils gypi fileGravatar egdaniel2014-06-04
| | | | | | | | | BUG=skia: R=bsalomon@google.com, robertphillips@google.com Author: egdaniel@google.com Review URL: https://codereview.chromium.org/317733003
* Relocate SK_IGNORE_CORRECT_HIGH_QUALITY_IMAGE_SCALEGravatar fmalita2014-06-04
| | | | | | | | | | | | | Temporarily move the flag into Chromium's repo, to facilitate atomic remove-and-suppress-failures changes. BUG=380686 R=humper@google.com TBR=humper@google.com Author: fmalita@chromium.org Review URL: https://codereview.chromium.org/319533002
* rebaseline_server: download actual-results.json files from GCS instead of SVNGravatar epoger2014-06-04
| | | | | | | | | BUG=skia:553 R=borenet@google.com Author: epoger@google.com Review URL: https://codereview.chromium.org/310093003
* Revert of Move Dashing filterPath to a dashing utils file ↵Gravatar egdaniel2014-06-04
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (https://codereview.chromium.org/314623004/) Reason for revert: Need to create utils.gypi file and update chrome Original issue's description: > Move Dashing filterPath to a dashing utils file > > From inside GrContext, we have a need to create an SkPath an original path and > some dashing info. We do not have access to the original path effect so we need > a way to make the FilterPath function accessible outside of the effect. So I moved > the core filterPath code (and all need helper functions) out of SkDashPathEffect > and created a SkDashPath in utils to store these helper functions. > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/576dcdc793a762ec63fbecdbfd5768066b548fe5 R=bsalomon@google.com, reed@google.com, rmistry@google.com TBR=bsalomon@google.com, reed@google.com, rmistry@google.com NOTREECHECKS=true NOTRY=true BUG=skia: Author: egdaniel@google.com Review URL: https://codereview.chromium.org/317663003
* Add non power of two gm for ETC1 bitmapGravatar krajcevski2014-06-04
| | | | | | | | R=robertphillips@google.com Author: krajcevski@google.com Review URL: https://codereview.chromium.org/316813005
* Use tests tag for Skia tools on Android.Gravatar scroggo2014-06-04
| | | | | | | | | | | When generating the makefiles for Skia testing tools on Android, use the tag 'tests' and do not use the tags 'optional' or 'eng'. R=djsollen@google.com Author: scroggo@google.com Review URL: https://codereview.chromium.org/315013003
* Use Identity matrix for SkTransparentShader.Gravatar scroggo2014-06-04
| | | | | | | | | | | | | | | The SkTransparentShader is just a placeholder that should not actually be used, so its CTM is not meaningful. However, it needs to be invertible in order to not assert in SkShader::Context's constructor. So use SkMatrix::I(). This fixes a CTS test on Android. R=reed@google.com Author: scroggo@google.com Review URL: https://codereview.chromium.org/315743003
* Is this SkDebug important? It drives me nuts.Gravatar mtklein2014-06-04
| | | | | | | | | BUG=skia: R=bsalomon@google.com, mtklein@google.com Author: mtklein@chromium.org Review URL: https://codereview.chromium.org/310113006
* Whitespace change to test bugdroidGravatar Ravi Mistry2014-06-04
| | | | | | | (SkipBuildbotRuns) BUG=skia:2139 BUG=chromium:380617
* Move Dashing filterPath to a dashing utils fileGravatar egdaniel2014-06-04
| | | | | | | | | | | | | | | From inside GrContext, we have a need to create an SkPath an original path and some dashing info. We do not have access to the original path effect so we need a way to make the FilterPath function accessible outside of the effect. So I moved the core filterPath code (and all need helper functions) out of SkDashPathEffect and created a SkDashPath in utils to store these helper functions. BUG=skia: R=bsalomon@google.com, reed@google.com, rmistry@google.com Author: egdaniel@google.com Review URL: https://codereview.chromium.org/314623004
* Add bench to measure drawing a dashed gridGravatar egdaniel2014-06-04
| | | | | | | | | BUG=skia: R=bsalomon@google.com Author: egdaniel@google.com Review URL: https://codereview.chromium.org/310083005
* Alter SkCanvas::drawPicture (devirtualize, take const SkPicture, take pointer)Gravatar robertphillips2014-06-04
| | | | | | | | R=reed@google.com, bsalomon@google.com, mtklein@google.com Author: robertphillips@google.com Review URL: https://codereview.chromium.org/313613004
* Whitespace change to test bugdroidGravatar Ravi Mistry2014-06-04
| | | | | | (SkipBuildbotRuns) BUG=skia:2139,chromium:380617
* Update SKP version to 17Gravatar borenet2014-06-03
| | | | | | | | | | Automatic commit by the RecreateSKPs bot. TBR= Author: borenet@google.com Review URL: https://codereview.chromium.org/312043004
* readd SK_SUPPORT_LEGACY_SETCONFIG_INFO -- missed a place in a blink testGravatar reed2014-06-03
| | | | | | | | | | R=jvanverth@google.com TBR=jvanverth NOTRY=True Author: reed@google.com Review URL: https://codereview.chromium.org/316753002
* Disable PDF in dm for tonight. It's crashy.Gravatar mtklein2014-06-03
| | | | | | | | | BUG=skia:2598 R=mtklein@google.com Author: mtklein@chromium.org Review URL: https://codereview.chromium.org/317533003
* Added grid data to logging outputGravatar kelvinly2014-06-03
| | | | | | | | | BUG=skia: R=bensong@google.com, jcgregorio@google.com Author: kelvinly@google.com Review URL: https://codereview.chromium.org/318433003
* remove unneeded flags SK_SUPPORT_LEGACY_SETCONFIG_INFO and ↵Gravatar reed2014-06-03
| | | | | | | | | | SK_SUPPORT_LEGACY_DEVICE_CONFIG TBR=scroggo Author: reed@google.com Review URL: https://codereview.chromium.org/315733002
* Android: remove -lcutils dependencyGravatar mtklein2014-06-03
| | | | | | | | | | | | | I believe we no longer depend on this after we ported our atomics to __sync. Tested by running android_ninja. BUG=skia: R=scroggo@google.com, mtklein@google.com Author: mtklein@chromium.org Review URL: https://codereview.chromium.org/318493002
* DM: add pdfGravatar mtklein2014-06-03
| | | | | | | | | BUG=skia:2598 R=halcanary@google.com, mtklein@google.com Author: mtklein@chromium.org Review URL: https://codereview.chromium.org/312873002