aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
* 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
* DM: go back to memcmp for BitmapsEqualGravatar mtklein2014-06-03
| | | | | | | | | | | | | | | Even when autovectorized, using MaxComponentDifference is slower than memcmp. And debug builds (most runs of DM) will never even be autovectorized. DM::MaxComponentDifference is the top function on DM profile, and memcmp moves to ~20th. BUG=skia: R=halcanary@google.com, mtklein@google.com Author: mtklein@chromium.org Review URL: https://codereview.chromium.org/315743002
* Whitespace change to test the new cron.chrome bugdroid.Gravatar Ravi Mistry2014-06-03
| | | | | | | (SkipBuildbotRuns) BUG=skia:2139 Review URL: https://codereview.chromium.org/309333003
* Fix neglected cleanup of SkTileGridPicture.hGravatar robertphillips2014-06-03
| | | | | | | | R=bsalomon@google.com Author: robertphillips@google.com Review URL: https://codereview.chromium.org/312803003
* Return NULL when building empty LayerRasterizer.Gravatar scroggo2014-06-03
| | | | | | | | | | | | | | | | | | In SkLayerRasterizer::snapshotRasterizer() and ::detachRasterizer(), if no layers have been added, do not attempt to create an SkLayerRasterizer. Instead, return NULL. This fixes an error when running tests on Android. Update dox to state that NULL may be returned. Add tests. R=reed@google.com Author: scroggo@google.com Review URL: https://codereview.chromium.org/313653006
* Initial KTX file decoderGravatar krajcevski2014-06-03
| | | | | | | | R=bsalomon@google.com, robertphillips@google.com, halcanary@google.com, reed@google.com Author: krajcevski@google.com Review URL: https://codereview.chromium.org/302333002
* Add LICENSE & README.google files to third_party/etc1Gravatar robertphillips2014-06-03
| | | | | | | | R=djsollen@google.com, krajcevski@google.com Author: robertphillips@google.com Review URL: https://codereview.chromium.org/309343004
* SK_CPU_ARM --> SK_CPU_ARM32Gravatar mtklein2014-06-03
| | | | | | | | | | | That's what it means. It keeps confusing us as named today. BUG=skia: R=djsollen@google.com, mtklein@google.com, reed@google.com Author: mtklein@chromium.org Review URL: https://codereview.chromium.org/314643004
* manual bench rebase after 55ada06Gravatar jvanverth2014-06-03
| | | | | | | | | | | | R=robertphillips@google.com TBR=robertphillips@google.com Bypassing trybots: NOTRY=true Author: jvanverth@google.com Review URL: https://codereview.chromium.org/315683004
* remove SkBounder -- unused and unlovedGravatar reed2014-06-03
| | | | | | | | | BUG=skia: R=scroggo@google.com Author: reed@google.com Review URL: https://codereview.chromium.org/312553006
* download_actuals.py: download JSON files from Google Storage instead of ↵Gravatar epoger2014-06-03
| | | | | | | | | | | | | skia-autogen uses google-api-python-client instead of gsutil binary to interact with Google Storage BUG=skia:553 R=rmistry@google.com, bensong@google.com Author: epoger@google.com Review URL: https://codereview.chromium.org/309653005
* a few rebaseline stragglers from image scaling changeGravatar humper2014-06-03
| | | | | | | | | | | BUG=skia: NOTRY=True NOTREECHECKS=True TBR=jvanverth Author: humper@google.com Review URL: https://codereview.chromium.org/311853004
* ARM Skia NEON patches - 38 - arm64 8888 blittersGravatar kevin.petit2014-06-03
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Enable NEON on arm64 for most 8888 blitters This patch enables NEON optimisation for the Color32, S32_Blend, S32A_Opaque blitters on arm64. Here are the perf improvements vs the existing code: Color32: ======== +-------+------------+------------+ | count | Cortex-A53 | Cortex-A57 | +-------+------------+------------+ | 1 | -2.39% | 23.78% | +-------+------------+------------+ | 2 | -5.46% | 8.88% | +-------+------------+------------+ | 4 | -4.74% | 4.89% | +-------+------------+------------+ | 8 | 67.74% | 107.12% | +-------+------------+------------+ | 16 | 40.03% | 101.20% | +-------+------------+------------+ | 64 | 11.09% | 98.40% | +-------+------------+------------+ | 256 | -2.20% | 74.81% | +-------+------------+------------+ | 1024 | -4.28% | 78.90% | +-------+------------+------------+ S32_Blend: ========== +-------+------------+------------+ | count | Cortex-A53 | Cortex-A57 | +-------+------------+------------+ | 1 | 7.84% | -6.75% | +-------+------------+------------+ | 2 | 28.95% | 39.77% | +-------+------------+------------+ | 4 | 5.80% | 8.26% | +-------+------------+------------+ | 8 | 1.35% | 33.80% | +-------+------------+------------+ | 16 | -2.13% | 41.13% | +-------+------------+------------+ | 64 | -4.91% | 42.84% | +-------+------------+------------+ | 256 | -6.53% | 48.72% | +-------+------------+------------+ | 1024 | -6.65% | 46.66% | +-------+------------+------------+ S32A_Opaque: ============ +-------+------------+------------+ | count | Cortex-A53 | Cortex-A57 | +-------+------------+------------+ | 1 | -7.51% | -19.06% | +-------+------------+------------+ | 2 | -5.02% | -27.70% | +-------+------------+------------+ | 4 | 15.38% | -21.66% | +-------+------------+------------+ | 8 | -0.98% | 1.05% | +-------+------------+------------+ | 16 | -7.35% | 3.34% | +-------+------------+------------+ | 64 | 50.53% | 94.63% | +-------+------------+------------+ | 256 | 71.17% | 164.10% | +-------+------------+------------+ | 1024 | 79.58% | 197.60% | +-------+------------+------------+ Signed-off-by: Kevin PETIT <kevin.petit@arm.com> BUG=skia: R=djsollen@google.com, mtklein@google.com Author: kevin.petit@arm.com Review URL: https://codereview.chromium.org/302283003
* Remove legacy picture recordingGravatar robertphillips2014-06-03
| | | | | | | | | | | | This is unblocked now that Android no longer uses the old interface. This is just the first step in cleaning this up. Future CLs will constify SkPicture access in SkCanvas and split up the SkPicture/SkPicturePlayback/SkPictureRecord trio. R=bsalomon@google.com, reed@google.com, mtklein@google.com Author: robertphillips@google.com Review URL: https://codereview.chromium.org/313613002
* Update SKP version to 16Gravatar borenet2014-06-03
| | | | | | | | | | Automatic commit by the RecreateSKPs bot. TBR= Author: borenet@google.com Review URL: https://codereview.chromium.org/313593002
* Fix etc1 support for SkFiddle.Gravatar jcgregorio2014-06-03
| | | | | | | | | | | Also update design doc to fix MySQL notation for auto incrementing columns. BUG=skia: R=mtklein@google.com Author: jcgregorio@google.com Review URL: https://codereview.chromium.org/315643002
* Fix memory leak in etc1bitmap GM.Gravatar mtklein2014-06-03
| | | | | | | | | | | | We never drop our first ref on fileData. (All our ref counts start at one). SkAutoTUnref<T> is almost always the right thing to use for refcounted types. BUG=skia:2635 R=krajcevski@google.com, mtklein@google.com Author: mtklein@chromium.org Review URL: https://codereview.chromium.org/310793003
* Allow chrome to use old version of installPixels.Gravatar scroggo2014-06-03
| | | | | | | | R=jvanverth@google.com, reed@google.com Author: scroggo@google.com Review URL: https://codereview.chromium.org/312703003
* Update link to Doxygen.Gravatar jcgregorio2014-06-03
| | | | | | | | | BUG=skia: R=mtklein@google.com Author: jcgregorio@google.com Review URL: https://codereview.chromium.org/302543007
* Rebase a few GMs and remove some tests from ignored-tests.txtGravatar Brian Salomon2014-06-03
| | | | Review URL: https://codereview.chromium.org/311813003
* Remove brackets from gm names.Gravatar scroggo2014-06-03
| | | | | | | | | BUG=skia:2634 R=epoger@google.com, borenet@google.com Author: scroggo@google.com Review URL: https://codereview.chromium.org/307373003
* Fall back to using clip effect for outer rect in drawdrrectGravatar bsalomon2014-06-03
| | | | | | | | R=jvanverth@google.com Author: bsalomon@google.com Review URL: https://codereview.chromium.org/310483010
* Whitespace change to test multiple master support in CQGravatar rmistry2014-06-03
| | | | | | | | | | | TBR= BUG=skia:2624 (SkipBuildbotRuns) Author: rmistry@google.com Review URL: https://codereview.chromium.org/315633002
* remove unneeded flagsGravatar reed2014-06-03
| | | | | | | | TBR= Author: reed@google.com Review URL: https://codereview.chromium.org/316463002
* Whitespace change to test new CQ trybotGravatar rmistry2014-06-03
| | | | | | | | | | TBR= BUG=skia:2203 (SkipBuildbotRuns) Author: rmistry@google.com Review URL: https://codereview.chromium.org/311813002
* Revert of Try TSAN blacklist for SkGLContextHelper::init. ↵Gravatar mtklein2014-06-02
| | | | | | | | | | | | | | | | | | | | | | | | | | (https://codereview.chromium.org/309823004/) Reason for revert: the bot's clang doesn't know -fsanitize-blacklist Original issue's description: > Try TSAN blacklist for SkGLContextHelper::init. > > Submitting to test this; this particular problem only happens on the bot. > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/ab2ec19b975a898d4ee2278ddad7d4268f134478 R=mtklein@chromium.org TBR=mtklein@chromium.org NOTREECHECKS=true NOTRY=true BUG=skia: Author: mtklein@google.com Review URL: https://codereview.chromium.org/314543002
* Try TSAN blacklist for SkGLContextHelper::init.Gravatar mtklein2014-06-02
| | | | | | | | | | | Submitting to test this; this particular problem only happens on the bot. BUG=skia: R=mtklein@google.com Author: mtklein@chromium.org Review URL: https://codereview.chromium.org/309823004
* Clean up another silly race in benches when run concurrently.Gravatar mtklein2014-06-02
| | | | | | | | | BUG=skia: R=mtklein@google.com Author: mtklein@chromium.org Review URL: https://codereview.chromium.org/309193003
* Tweak TSAN suppressions.Gravatar mtklein2014-06-02
| | | | | | | | | BUG=skia: R=mtklein@google.com Author: mtklein@chromium.org Review URL: https://codereview.chromium.org/312613002