aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
* Suggested version with 'undo'.Gravatar mtklein2015-02-17
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/931483002
* use common impl for drawTextOnPathGravatar reed2015-02-17
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/925343003
* Remove SK_SUPPORT_LEGACY_ADDRRECTGravatar fmalita2015-02-17
| | | | | | | | | The flag is no longer used in Chromium. BUG=458898 R=reed@google.com,robertphillips@google.com Review URL: https://codereview.chromium.org/929173002
* Remove SK_SUPPORT_LEGACY_CONIC_COMPUTE_QUAD_POW2Gravatar fmalita2015-02-17
| | | | | | | | | The flag is no longer used in Chromium. BUG=458898 R=reed@google.com,robertphillips@google.com Review URL: https://codereview.chromium.org/931663004
* Improve GrInOrderDrawBuffer::Cmd encapsulationGravatar robertphillips2015-02-17
| | | | | | In preparation for moving all the Cmds out of GrInOrderDrawBuffer. Review URL: https://codereview.chromium.org/931673002
* Fix broken build due to updated #defineGravatar djsollen2015-02-17
| | | | | | TBR=scroggo@google.com Review URL: https://codereview.chromium.org/934803003
* Use uint16s for texture coordinates when rendering text.Gravatar jvanverth2015-02-17
| | | | | | | Allows us to push more vertices into a given vertex buffer, with a slight performance improvement. Review URL: https://codereview.chromium.org/917373002
* Remove android specific srcPath from SkPathGravatar djsollen2015-02-17
| | | | Review URL: https://codereview.chromium.org/926693002
* Fix skia bug 2845Gravatar kui.zheng2015-02-17
| | | | | | | | | | Shouldn't call Fast Blur path(DoubleRowBoxBlur_NEON) when kernelsize is 1. Or, uint16x8_t resultPixels will be overflow. BUG=skia:2845 R=senorblanco@chromium.org Review URL: https://codereview.chromium.org/587543003
* Use SkXfermode as public facing enum for GrPorterDuffXPGravatar egdaniel2015-02-17
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/926593005
* Upstream support for sfntly in Google3Gravatar mtklein2015-02-17
| | | | | | | | See cr/86110021. BUG=skia: Review URL: https://codereview.chromium.org/913143007
* Minor refactoringGravatar robertphillips2015-02-16
| | | | | | This CL just de-dups some code in GrInOrderDrawBuffer. Review URL: https://codereview.chromium.org/930953003
* fix for ASAN heap overflowGravatar joshualitt2015-02-16
| | | | | | | TBR=bsalomon@google.com BUG=skia: Review URL: https://codereview.chromium.org/930123002
* remove unneeded prerollGravatar reed2015-02-15
| | | | | | | BUG=skia: TBR=robertphilips Review URL: https://codereview.chromium.org/927443006
* Update SKP versionGravatar skia.buildbots2015-02-15
| | | | | | | | Automatic commit by the RecreateSKPs bot. TBR= Review URL: https://codereview.chromium.org/928823002
* Negative numbers are not flags.Gravatar mtklein2015-02-14
| | | | | | | | tested: d dm --matrix 1 -0.3 0 0.7 1 0 0 0 1 --src gm --config matrix-8888 -w bad BUG=skia: Review URL: https://codereview.chromium.org/924343002
* git: add skps to .gitignoreGravatar halcanary2015-02-13
| | | | | | NOTRY=true Review URL: https://codereview.chromium.org/924163003
* Labels when dumping fonts in Android parser test.Gravatar bungeman2015-02-13
| | | | | | | | | | When the FontConfigParserAndroid test runs, it optionally dumps the configurations it creates for manual inspection. This is useful, but since three different configurations exist, it can be difficult to tell them apart. This simply prints a short header before each dump so that the dump can be identified. Review URL: https://codereview.chromium.org/927533004
* Multi-string shadersGravatar joshualitt2015-02-13
| | | | | | | | BUG=skia: Committed: https://skia.googlesource.com/skia/+/1c3c2d83364ee228e0751df0e1b9c161c0ba8c1e Review URL: https://codereview.chromium.org/929503002
* Revert of Multi-string shaders (patchset #4 id:60001 of ↵Gravatar joshualitt2015-02-13
| | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/929503002/) Reason for revert: windows warnings Original issue's description: > Multi-string shaders > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/1c3c2d83364ee228e0751df0e1b9c161c0ba8c1e TBR=bsalomon@google.com,joshualitt@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/924973002
* Multi-string shadersGravatar joshualitt2015-02-13
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/929503002
* C++11 Unit Test for RValue semantics.Gravatar halcanary2015-02-13
| | | | | | | | | | Motivation: test that these works on all possible bots and for all possible clients (clients do run these unit tests, right?) Dear clients: if this unit test fails, let us know! BUG=skia:3427 Review URL: https://codereview.chromium.org/922043004
* DM: don't leak the null canvas in NullSink.Gravatar mtklein2015-02-13
| | | | | | | | Also make NullSink a real boy: declared among the other Sinks, impl in .cpp. BUG=skia: Review URL: https://codereview.chromium.org/922293003
* fix more tricky-float cases in SkRRect validateGravatar reed2015-02-13
| | | | | | BUG=458524,458522 Review URL: https://codereview.chromium.org/921163003
* AA stroke rects batchGravatar joshualitt2015-02-13
| | | | | | | | | | BUG=skia: Committed: https://skia.googlesource.com/skia/+/68c7b6aa98816badbbb69ef65a84a932b8dd58f3 Committed: https://skia.googlesource.com/skia/+/b0fb935bd544b0c6f68c692c7e1ee40876184a1a Review URL: https://codereview.chromium.org/912483004
* Split out methods in GrGpuResource::CacheAccess that can be called outside ↵Gravatar bsalomon2015-02-13
| | | | | | of the cache. Review URL: https://codereview.chromium.org/923143002
* YUV scale fixGravatar sugoi2015-02-13
| | | | | | | | | | There was a scaling mistake visible in some JPEG images because the ratio between Y, U and V planes were assumed to be the same ratios as the ratio between texture sizes, which was wrong because texture have a minimum size of 16 and are rounded up to the next POT. Since the ratios between Y and UV planes are generally 1, 2 or 4, rounding up to the next POT would generally preserve this ratio, so that this bug was not very visible, apart from very small jpeg images of 8 or less pixels in either width or height. BUG=457954 Committed: https://skia.googlesource.com/skia/+/e6eddf7dd85add7da41f22f2643bdd573ad1f1cf Review URL: https://codereview.chromium.org/922273002
* Stop lieing about GrBatch color for RectsGravatar joshualitt2015-02-13
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/920423003
* Fix conversion from enum to bool error.Gravatar scroggo2015-02-13
| | | | | | | | | | | | Needed for the DEPS roll. TBR=jvanverth@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/921293002
* Determine whether we can tweakAlphaForCoverage during Pipeline/XP creation.Gravatar egdaniel2015-02-13
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/927623002
* Revert of YUV scale fix (patchset #2 id:20001 of ↵Gravatar jvanverth2015-02-13
| | | | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/922273002/) Reason for revert: Turning Windows compile bots red. Original issue's description: > YUV scale fix > > There was a scaling mistake visible in some JPEG images because the ratio between Y, U and V planes were assumed to be the same ratios as the ratio between texture sizes, which was wrong because texture have a minimum size of 16 and are rounded up to the next POT. Since the ratios between Y and UV planes are generally 1, 2 or 4, rounding up to the next POT would generally preserve this ratio, so that this bug was not very visible, apart from very small jpeg images of 8 or less pixels in either width or height. > > BUG=457954 > > Committed: https://skia.googlesource.com/skia/+/e6eddf7dd85add7da41f22f2643bdd573ad1f1cf TBR=bsalomon@google.com,sugoi@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=457954 Review URL: https://codereview.chromium.org/926123002
* FontFamily SkTArrays can be moved.Gravatar bungeman2015-02-13
| | | | | | | | | The FontFamily type in SkFontConfigParser_android.h contains two fields of type SkTArray. Both of these contain types which can be moved instead of calling constructors and destructors on resize. This change avoids a number of unneeded SkString refs and unrefs. Review URL: https://codereview.chromium.org/929473002
* YUV scale fixGravatar sugoi2015-02-13
| | | | | | | | There was a scaling mistake visible in some JPEG images because the ratio between Y, U and V planes were assumed to be the same ratios as the ratio between texture sizes, which was wrong because texture have a minimum size of 16 and are rounded up to the next POT. Since the ratios between Y and UV planes are generally 1, 2 or 4, rounding up to the next POT would generally preserve this ratio, so that this bug was not very visible, apart from very small jpeg images of 8 or less pixels in either width or height. BUG=457954 Review URL: https://codereview.chromium.org/922273002
* Make SkImageGenerator::getPixels() return an enum.Gravatar scroggo2015-02-13
| | | | | | | | | | | | | | | | | | | | | | The new enum describes the nature of the failure. This is in preparation for writing a replacement for SkImageDecoder, which will use this interface. Update the comments for getPixels() to specify what it means to pass an SkImageInfo with a different size. Make SkImageGenerator Noncopyable. Leave onGetYUV8Planes alone, since we have separate discussions regarding modifying that API. Make callers of SkImageDecoder consistently handle kPartialSuccess. Previously, some callers considered it a failure, and others considered it a success. BUG=skia:3257 Review URL: https://codereview.chromium.org/919693002
* Remove SkPictureFlat.h include from SkDrawCommands.hGravatar robertphillips2015-02-13
| | | | | | | | | | This cannot land until https://codereview.chromium.org/909353004/ (Prepare SkiaBenchmarkingExtensionTest for upcoming Skia changes) lands in Chromium Committed: https://skia.googlesource.com/skia/+/5a4c233a3657d12d836de388b41e30405b4ab976 Committed: https://skia.googlesource.com/skia/+/fa3a83d0fe53907e618346d3f1c17a95da1a9d22 Review URL: https://codereview.chromium.org/912403004
* Add a templated priority queue class.Gravatar bsalomon2015-02-13
| | | | Review URL: https://codereview.chromium.org/914003004
* Revert of Use dst copies in porter duffer XP to correctly render certain ↵Gravatar egdaniel2015-02-13
| | | | | | | | | | | | | | | | | | | | | | blends. (patchset #4 id:60001 of https://codereview.chromium.org/914003003/) Reason for revert: Failing GLProgramTest passing in stupid coeffs Original issue's description: > Use dst copies in porter duffer XP to correctly render certain blends. > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/997c6358d94e188b1a7b89a4f86e24cbe0f5a164 TBR=bsalomon@google.com,joshualitt@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/923153003
* Use dst copies in porter duffer XP to correctly render certain blends.Gravatar egdaniel2015-02-13
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/914003003
* Fix leak in ComposedImageFilterOffset testGravatar ajuma2015-02-13
| | | | | | | | | This fixes a leak in the ComposedImageFilterOffset test that was introduced in https://codereview.chromium.org/908273002/. BUG=chromium:453924 Review URL: https://codereview.chromium.org/922063005
* Fix SkComposeImageFilter's bounds computation and offset handlingGravatar ajuma2015-02-13
| | | | | | | | | | | | | | This makes SkComposeImageFilter::computeFastBounds compose its filters' bounds (rather than falling back to SkImageFilter::computeFastBounds, which takes the union of the bounds). This also makes SkComposeImageFilter::onFilterImage correctly handle an offset produced when applying the inner filter; such offsets were previously ignored. BUG=chromium:453924 Review URL: https://codereview.chromium.org/908273002
* Adding new benchmark to test image decoding performance.Gravatar msarett2015-02-13
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/918673002
* Limit clear as draw workaround to Adreno 3xx.Gravatar bsalomon2015-02-13
| | | | | | This causes more problems than it fixes on the Nexus 6. Review URL: https://codereview.chromium.org/917313005
* Pass in ProcOptInfos into willNeedDstCopy on XPsGravatar egdaniel2015-02-13
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/912413002
* [SkSVGDevice] Full font family supportGravatar fmalita2015-02-13
| | | | | | | | | Use a family name iterator and list all names instead of just one returned by SkTypeface::getFamilyName(). R=reed@google.com,mtklein@google.com Review URL: https://codereview.chromium.org/923583002
* Verify all parsed test font files start with cap.Gravatar bungeman2015-02-13
| | | | | | | | | All of the Android test font configuration files have file names which start with a capital latin letter. Verify this is true of the parsed file names. This would have caught previous issues with slicing, and will hopefully prevent such issues going unnoticed in the future. Review URL: https://codereview.chromium.org/925933003
* interpret conic tolerance correctly -- big rebaselineGravatar reed2015-02-12
| | | | | | | | | need this to land in chrome https://codereview.chromium.org/918233002/ NOTREECHECKS=True Review URL: https://codereview.chromium.org/915103002
* Minimize SkTHash object copyingGravatar fmalita2015-02-12
| | | | | | | | AKA ref-constify the world. R=mtklein@google.com Review URL: https://codereview.chromium.org/919193002
* Revert of I cannot remember what the race here actually was. (patchset #1 ↵Gravatar mtklein2015-02-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | id:1 of https://codereview.chromium.org/922873002/) Reason for revert: http://build.chromium.org/p/client.skia/builders/Test-Ubuntu13.10-GCE-NoGPU-x86_64-Release-TSAN/builds/1557/steps/dm/logs/stdio Original issue's description: > I cannot remember what the race here actually was. > > Removing SkTRacy will show us, or perhaps show us we already fixed the race. > > BUG=chromium:437511 > > No public API changes. > TBR=reed@google.com > > Committed: https://skia.googlesource.com/skia/+/71409c83e69b387b3c5d567197d0421c36baf68d TBR=bungeman@google.com,mtklein@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=chromium:437511 Review URL: https://codereview.chromium.org/923543003
* I cannot remember what the race here actually was.Gravatar mtklein2015-02-12
| | | | | | | | | | | Removing SkTRacy will show us, or perhaps show us we already fixed the race. BUG=chromium:437511 No public API changes. TBR=reed@google.com Review URL: https://codereview.chromium.org/922873002
* Revert of AA stroke rects batch (patchset #6 id:100001 of ↵Gravatar joshualitt2015-02-12
| | | | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/912483004/) Reason for revert: breaks in debug nanobench Original issue's description: > AA stroke rects batch > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/68c7b6aa98816badbbb69ef65a84a932b8dd58f3 > > Committed: https://skia.googlesource.com/skia/+/b0fb935bd544b0c6f68c692c7e1ee40876184a1a TBR=bsalomon@google.com,robertphillips@google.com,joshualitt@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/924533004
generated by cgit v1.2.3 (git 2.39.1) at 2024-09-08 23:41:27 +0000