aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
...
* 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
* 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