aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
* 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
* remove unneeded invalGravatar reed2014-06-02
| | | | | | | | TBR= Author: reed@google.com Review URL: https://codereview.chromium.org/307993012
* add makeOffset and makeInset to SkRectGravatar reed2014-06-02
| | | | | | | | | | TBR=bsalomon precursor for https://codereview.chromium.org/306013010 Author: reed@google.com Review URL: https://codereview.chromium.org/306893012
* use most innocuous characters possible within skimage output filenamesGravatar epoger2014-06-02
| | | | | | | | | BUG=skia:2634 R=scroggo@google.com Author: epoger@google.com Review URL: https://codereview.chromium.org/302323003
* Change #def to be opt-outGravatar krajcevski2014-06-02
| | | | | | | | R=jvanverth@google.com, robertphillips@google.com Author: krajcevski@google.com Review URL: https://codereview.chromium.org/308143008
* Revert of stop using brackets within skimage output filenames ↵Gravatar epoger2014-06-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | (https://codereview.chromium.org/303263012/) Reason for revert: Broke http://108.170.220.120:10117/builders/Test-ChromeOS-Link-HD4000-x86_64-Release/builds/2806/steps/UploadSKImageResults/logs/stdio : [13:38:43.401942] /bin/sh: 1: Syntax error: "(" unexpected [13:38:43.902632] Command /home/chrome-bot/buildbot/skiabot-shuttle-ubuntu12-003/buildbot/third_party/chromium_buildbot/scripts/slave/../../third_party/gsutil/gsutil cp -a private /home/chrome-bot/buildbot/skiabot-shuttle-ubuntu12-003/buildbot/third_party/chromium_buildbot/slave/Test-ChromeOS-Link-HD4000-x86_64-Release/build/skia/out/Release/skimage_out/images/bitmap-64bitMD5/webp-test-webp-(123,263,587,394)/16263897892077508819.png gs://chromium-skia-gm/skimage/output/images/bitmap-64bitMD5/webp-test-webp-(123,263,587,394)/16263897892077508819.png failed with retcode 2, try 1. Original issue's description: > stop using brackets within skimage output filenames > > BUG=skia:2634 > > Committed: https://skia.googlesource.com/skia/+/f07892413cd2359549dc187f1d10bc48d7d263ba R=scroggo@google.com TBR=scroggo@google.com NOTREECHECKS=true NOTRY=true BUG=skia:2634 Author: epoger@google.com Review URL: https://codereview.chromium.org/314443002
* Update TSAN suppressions.Gravatar mtklein2014-06-02
| | | | | | | | | | | | | | | | | | TSAN suppressions need to be on their own line, even free of comments. Also, temporarily add SK_ANNOTATE_UNPROTECTED_READ to sk_acquire_load. Will remove this when we're done iterating on SkBarriers_tsan.h: TSAN has an atomic load that makes the annotation moot. BUG=skia: Committed: https://skia.googlesource.com/skia/+/be7de4fc4229b70b0954f3ea5e3b053dae4c1a9d R=bungeman@google.com, mtklein@google.com Author: mtklein@chromium.org Review URL: https://codereview.chromium.org/308073013
* stop using brackets within skimage output filenamesGravatar epoger2014-06-02
| | | | | | | | | BUG=skia:2634 R=scroggo@google.com Author: epoger@google.com Review URL: https://codereview.chromium.org/303263012
* Update expectations to fix self tests.Gravatar scroggo2014-06-02
| | | | | | | | | R=epoger@google.com TBR=epoger@google.com Author: scroggo@google.com Review URL: https://codereview.chromium.org/306313004
* Update TSAN suppressions.Gravatar mtklein2014-06-02
| | | | | | | | | | | | | | | TSAN suppressions need to be on their own line, even free of comments. Also, temporarily add SK_ANNOTATE_UNPROTECTED_READ to sk_acquire_load. Will remove this when we're done iterating on SkBarriers_tsan.h: TSAN has an atomic load that makes the annotation moot. BUG=skia: R=bungeman@google.com, mtklein@google.com Author: mtklein@chromium.org Review URL: https://codereview.chromium.org/308073013
* use colortype instead of configGravatar reed2014-06-02
| | | | | | | | | | | | clone of https://codereview.chromium.org/305133006/ TBR= BUG=skia: Author: reed@google.com Review URL: https://codereview.chromium.org/301233011
* TSAN caught us racing in ScalarBench.cppGravatar mtklein2014-06-02
| | | | | | | | | | | | | | http://108.170.220.102:10117/builders/Test-Ubuntu13-ShuttleA-HD2000-x86_64-Debug-TSAN/builds/914/steps/RunDM/logs/stdio BUG=skia: Committed: http://code.google.com/p/skia/source/detail?r=14992 R=reed@google.com, mtklein@google.com Author: mtklein@chromium.org Review URL: https://codereview.chromium.org/305033002
* Allow access to device config for Moz2d.Gravatar jvanverth2014-06-02
| | | | | | | | | BUG=skia:2633 R=robertphillips@google.com, borenet@google.com, george@mozilla.com Author: jvanverth@google.com Review URL: https://codereview.chromium.org/304273009
* Revert of Update SKP version to 13 (https://codereview.chromium.org/309753004/)Gravatar borenet2014-06-02
| | | | | | | | | | | | | | | | | | | | | Reason for revert: Did not include perf baselines. Original issue's description: > Update SKP version to 13 > > TBR= > > Committed: https://skia.googlesource.com/skia/+/7574153e9cc52b1b560ded02a2b219cda4b1816c R=rmistry@google.com TBR=rmistry@google.com NOTREECHECKS=true NOTRY=true Author: borenet@google.com Review URL: https://codereview.chromium.org/309813003
* Adding resource filesGravatar Robert Phillips2014-06-02
| | | | | | | | | The ktx file is to test the new ktx reader The pkm shrinks the size of the pkm test file TBR=jvanverth@google.com Review URL: https://codereview.chromium.org/312563002