aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
* pdf-mac clear to black rebaselineGravatar halcanary2014-12-04
| | | | | | | | NOTRY=true TBR= NOTREECHECKS=true Review URL: https://codereview.chromium.org/783543002
* move program descriptor generation to flushGravatar joshualitt2014-12-04
| | | | | | | | BUG=skia: Committed: https://skia.googlesource.com/skia/+/829e1b80b1020b17f2078020c990e079b70c077c Review URL: https://codereview.chromium.org/777673003
* Upload picture byte size and op count metrics for SKP recording.Gravatar mtklein2014-12-04
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | Look okay? { "results" : { "desk_amazon.skp_1264_3999" : { "nonrendering" : { "bytes" : 75656, "max_ms" : 1.150187, "mean_ms" : 1.150187, "median_ms" : 1.150187, "min_ms" : 1.150187, "ops" : 659, "options" : { "bench_type" : "recording", "clip" : "0 0 1000 1000", "name" : "desk_amazon.skp", "scale" : "1", "source_type" : "skp" } } }, ... BUG=skia: Review URL: https://codereview.chromium.org/773323002
* Rebaseline multipicturedraw GMsGravatar robertphillips2014-12-04
| | | | | | | | This is clean up after https://codereview.chromium.org/753253002/ (Use variable length key (rather than accumulated matrix) as save layer hoisting key) TBR=halcanary@google.com Review URL: https://codereview.chromium.org/783523002
* quick fix to land clear() changeGravatar reed2014-12-04
| | | | | | | | BUG=skia: TBR= NOTRY=True Review URL: https://codereview.chromium.org/774863003
* Fix SKPBench tiling so MPD and non-MPD matchGravatar robertphillips2014-12-04
| | | | | | | | | | Two issues with the SKPBench tile computation were causing the MPD path to do more work: The clip from the parent canvas wasn't being used to trim content off the edges of the MPD tiles The non-MPD path was not taking the scale into account in its tile placement (resulting in it having fewer, larger active tiles when scaling). Review URL: https://codereview.chromium.org/776273002
* check (thread-tricky) colortable after we've copied the bitmap into a ↵Gravatar reed2014-12-04
| | | | | | | | per-thread context BUG=skia: Review URL: https://codereview.chromium.org/766283004
* Fix up RecordReplaceDrawTest memory leak.Gravatar mtklein2014-12-04
| | | | | | | | http://build.chromium.org/p/client.skia/builders/Test-Ubuntu12-ShuttleA-GTX550Ti-x86_64-Release-Valgrind/builds/151/steps/dm/logs/stdio BUG=skia: Review URL: https://codereview.chromium.org/779973002
* Change clear() to respect the clipGravatar reed2014-12-04
| | | | | | | | | This reverts commit af641a1c10f176cb9617026d3cc93c117a85d13d. BUG=skia: TBR= Review URL: https://codereview.chromium.org/783493002
* Add SSSE3 acceleration for S32_D16_filter_DXDYGravatar qiankun.miao2014-12-04
| | | | | | | | | | | | | With this CL, related nanobench can be improved for 565 config. bitmap_BGRA_8888_scale_rotate_bilerp 115us -> 70.5us 0.61x bitmap_BGRA_8888_update_volatile_scale_rotate_bilerp 115us -> 70.5us 0.61x bitmap_BGRA_8888_update_scale_rotate_bilerp 112us -> 68us 0.6x BUG=skia: Committed: https://skia.googlesource.com/skia/+/45a05780867a06b9f8a8d5240cf6c5d5a2c15a35 Review URL: https://codereview.chromium.org/773753002
* Revert of move program descriptor generation to flush (patchset #7 id:120001 ↵Gravatar joshualitt2014-12-04
| | | | | | | | | | | | | | | | | | | | | of https://codereview.chromium.org/777673003/) Reason for revert: breaking linux build Original issue's description: > move program descriptor generation to flush > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/829e1b80b1020b17f2078020c990e079b70c077c TBR=egdaniel@google.com,bsalomon@google.com,joshualitt@chromium.org NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/776243005
* Revert of create and thread batch tracker object (patchset #9 id:160001 of ↵Gravatar joshualitt2014-12-04
| | | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/772513002/) Reason for revert: Breaking linux layout test Original issue's description: > create and thread batch tracker object > > I remove the factory on the next CL > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/c3a6eb23483e5d28073b509a5f637f41660de294 TBR=bsalomon@google.com,joshualitt@chromium.org NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/774133003
* make clear private (before we delete it)Gravatar reed2014-12-04
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/781593002
* Update SKP versionGravatar skia.buildbots2014-12-03
| | | | | | | | Automatic commit by the RecreateSKPs bot. TBR= Review URL: https://codereview.chromium.org/776873006
* Revert of Add SSSE3 acceleration for S32_D16_filter_DXDY (patchset #3 ↵Gravatar jam2014-12-03
| | | | | | | | | | | | | | | | | | | | | | | | | | | id:40001 of https://codereview.chromium.org/773753002/) Reason for revert: breaks build when not using SSE3, since the two method definitions differ in parameter types (typo) Original issue's description: > Add SSSE3 acceleration for S32_D16_filter_DXDY > > With this CL, related nanobench can be improved for 565 config. > bitmap_BGRA_8888_scale_rotate_bilerp 115us -> 70.5us 0.61x > bitmap_BGRA_8888_update_volatile_scale_rotate_bilerp 115us -> 70.5us 0.61x > bitmap_BGRA_8888_update_scale_rotate_bilerp 112us -> 68us 0.6x > > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/45a05780867a06b9f8a8d5240cf6c5d5a2c15a35 TBR=mtklein@google.com,qkmiao@gmail.com,qiankun.miao@intel.com NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/761103003
* create and thread batch tracker objectGravatar joshualitt2014-12-03
| | | | | | | | I remove the factory on the next CL BUG=skia: Review URL: https://codereview.chromium.org/772513002
* move program descriptor generation to flushGravatar joshualitt2014-12-03
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/777673003
* Revert of Revert of remove unused device::eraseColor (patchset #1 id:1 of ↵Gravatar reed2014-12-03
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/777003002/) Reason for revert: fix landed in chrome to allow this (I hope) Original issue's description: > Revert of remove unused device::eraseColor (patchset #1 id:1 of https://codereview.chromium.org/765643004/) > > Reason for revert: > ../../skia/ext/bitmap_platform_device_cairo.cc:131:13: error: no member named 'eraseColor' in 'skia::BitmapPlatformDevice' > > > Original issue's description: > > remove unused device::eraseColor > > > > BUG=skia: > > TBR= > > > > Committed: https://skia.googlesource.com/skia/+/b1ab276c3719d6690e58f347c25126d2b8929c5d > > TBR=reed@google.com > NOTREECHECKS=true > NOTRY=true > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/8b0197ef2e32be9d62e79d58f8b50811975c79c6 TBR=halcanary@google.com NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/779543004
* Add SK_SUPPORT_LEGACY_PORTER_DUFF macro.Gravatar tfarina2014-12-03
| | | | | | | | | | This trick will be used to remove deprecated SkPorterDuff API. But first we need to add it so Android can be updated later. BUG=skia:3178 R=scroggo@google.com Review URL: https://codereview.chromium.org/758133004
* Turn on MPD threading in nanobench.Gravatar mtklein2014-12-03
| | | | | | | | | | Seems okay after this small patch to skip lockPixels() / unlockPixels(). BUG=skia:3149 CQ_EXTRA_TRYBOTS=client.skia:Test-Ubuntu13.10-GCE-NoGPU-x86_64-Release-TSAN-Trybot Review URL: https://codereview.chromium.org/773203003
* Remove guard on and enable vertexAttrib function pts on GrGLInterface.Gravatar egdaniel2014-12-03
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/774303003
* Use static XPF for porter duff xp factories.Gravatar egdaniel2014-12-03
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/776843004
* Copy src in SkDrawLooper::computeFastBounds().Gravatar mtklein2014-12-03
| | | | | | | | | | | | | We do alias src and dst when building the BBH. This meant we calculated the wrong bounds in LayoutTests/ietestcenter/css3/text/textshadow-010.htm. Not sure how best to test this inside Skia. It may become moot after I work through the API changes we were talking about today, but if not I'll make sure it's tested somehow before I'm done with SkDrawLoopers. BUG=skia:3186 Review URL: https://codereview.chromium.org/760023004
* Fix PdfViewer BGR byte orderGravatar halcanary2014-12-03
| | | | | | BUG=skia:3184 Review URL: https://codereview.chromium.org/775763003
* Add glGetShaderPrecisionFormatGravatar bsalomon2014-12-03
| | | | Review URL: https://codereview.chromium.org/778703003
* Revert of remove unused device::eraseColor (patchset #1 id:1 of ↵Gravatar halcanary2014-12-03
| | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/765643004/) Reason for revert: ../../skia/ext/bitmap_platform_device_cairo.cc:131:13: error: no member named 'eraseColor' in 'skia::BitmapPlatformDevice' Original issue's description: > remove unused device::eraseColor > > BUG=skia: > TBR= > > Committed: https://skia.googlesource.com/skia/+/b1ab276c3719d6690e58f347c25126d2b8929c5d TBR=reed@google.com NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/777003002
* Add XferProcessor factory in GrPaint and GrDrawState.Gravatar egdaniel2014-12-03
| | | | | | | | In this CL the XP should have zero effect on the actual rendering pipeline. BUG=skia: Review URL: https://codereview.chromium.org/751283002
* rebaseline imagesGravatar Brian Salomon2014-12-03
| | | | | | TBR= Review URL: https://codereview.chromium.org/776983003
* Update color emoji GMs after https://codereview.chromium.org/759613005/.Gravatar jvanverth2014-12-03
| | | | | | TBR=halcanary@google.com Review URL: https://codereview.chromium.org/775163002
* remove unused device::eraseColorGravatar reed2014-12-03
| | | | | | | BUG=skia: TBR= Review URL: https://codereview.chromium.org/765643004
* bug fix for cubic low precisionGravatar joshualitt2014-12-03
| | | | | | | TBR= BUG=skia: Review URL: https://codereview.chromium.org/776953002
* update tileimagefilter gm expectationsGravatar halcanary2014-12-03
| | | | | | | | TBR=reed@google.com NOTRY=true Review URL: https://codereview.chromium.org/777683002
* Decompose a weird assert.Gravatar mtklein2014-12-03
| | | | | | | | | | | I got my hands on a list of new warnings coming out of MSVC when run with /analyze. I scanned over them and didn't see any bugs, but I did notice this is just a weird way to write this expression. (MSVC /analyze was wary of '=' in a boolean expression.) BUG=skia: Review URL: https://codereview.chromium.org/776033002
* Rebaseline verylargebitmapGravatar Mike Klein2014-12-03
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/778693002
* Remove unused constants.Gravatar mtklein2014-12-03
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/773193004
* (re)baseline several of my gmsGravatar halcanary2014-12-03
| | | | | | | TBR= NOTRY=true Review URL: https://codereview.chromium.org/780593002
* Fuse GrReplacements and GrLayerCacheGravatar robertphillips2014-12-03
| | | | | | The conversion step from GrCachedLayer to ReplacementInfo isn't necessary. Review URL: https://codereview.chromium.org/769533004
* First step to moving vertex attributes to the geometryProcessorGravatar joshualitt2014-12-03
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/761563002
* Make debugger not fail with webpages_playback script and multiple skp filesGravatar kkinnunen2014-12-03
| | | | | | | | The debugger accepts either a single skp or a directory as a parameter. The webpages_playback script used to pass multiple files using a '*' glob. Review URL: https://codereview.chromium.org/775073002
* Update webpages_playback documentationGravatar kkinnunen2014-12-03
| | | | | | | Update the documentation on how to run webpages_playback. It was out of date. Review URL: https://codereview.chromium.org/778633002
* Remove Google Storage scheme prefix correctly from gs base name in ↵Gravatar kkinnunen2014-12-03
| | | | | | | | | | | | webpages_playback In python, str.lstrip([chars]) means "lstrip any of the chars from str", not "remove prefix". Thus "gs://skia".lstrip("gs://") yields "kia". Replace with custom function stripping the prefix from lstrip()ed gs base name. Review URL: https://codereview.chromium.org/780493002
* Make render_pdfs not crash webpages_playback scriptGravatar kkinnunen2014-12-03
| | | | | | | | | | | The render_pdfs nowadays seems to input the skps from the dir passed as '-r' parameter. The app will not output anything, but it will not crash the webpages_playback script either. Affects only runs that have --skia_tools=<dir> argument. Review URL: https://codereview.chromium.org/776873004
* Update SKP versionGravatar skia.buildbots2014-12-02
| | | | | | | | Automatic commit by the RecreateSKPs bot. TBR= Review URL: https://codereview.chromium.org/773113004
* Revert of rebaseline after clear change -- pdf was not treating clear as ↵Gravatar reed2014-12-02
| | | | | | | | | | | | | | | | | | | | | | | SRC_MODE (patchset #1 id:1 of https://codereview.chromium.org/759303004/) Reason for revert: reverted clear() change Original issue's description: > rebaseline after clear change -- pdf was not treating clear as SRC_MODE > > BUG=skia: > NOTRY=True > TBR= > > Committed: https://skia.googlesource.com/skia/+/2fb352abebc342a322d4a66c054bbff8bd565767 TBR=reed@google.com NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/774073002
* Revert "Change clear() to respect the clip"Gravatar reed2014-12-02
| | | | | | | | | This reverts commit 3729469d6a12266037b697c2192768545e097ab0. BUG=skia: TBR= Review URL: https://codereview.chromium.org/778563002
* Revert of implement device::clear until we can remove callers in chrome ↵Gravatar reed2014-12-02
| | | | | | | | | | | | | | | | | | | | | | skia/ext (patchset #1 id:1 of https://codereview.chromium.org/773903002/) Reason for revert: failing win unittests Original issue's description: > implement device::clear until we can remove callers in chrome skia/ext > > BUG=skia: > TBR= > > Committed: https://skia.googlesource.com/skia/+/252fccfb546efe83fb1d2c7a0b3c18a96ff46ea4 TBR=reed@google.com NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/774063002
* Add SSSE3 acceleration for S32_D16_filter_DXDYGravatar qiankun.miao2014-12-02
| | | | | | | | | | | With this CL, related nanobench can be improved for 565 config. bitmap_BGRA_8888_scale_rotate_bilerp 115us -> 70.5us 0.61x bitmap_BGRA_8888_update_volatile_scale_rotate_bilerp 115us -> 70.5us 0.61x bitmap_BGRA_8888_update_scale_rotate_bilerp 112us -> 68us 0.6x BUG=skia: Review URL: https://codereview.chromium.org/773753002
* Add matrix operations to C canvas APIGravatar abarth2014-12-02
| | | | | | | R=reed@google.com BUG=skia: Review URL: https://codereview.chromium.org/769123002
* work in progressGravatar halcanary2014-12-02
| | | | Review URL: https://codereview.chromium.org/769423002
* check matrix does not have rotation or perspective transformationGravatar derekf2014-12-02
| | | | | | | | | | | When checking whether we need apply AA to a rect for non-msaa target, simply checking that the mapped rect is integer is not enough. We need to check whether the transformation matrix has transformations other than simple translation. Original-Author: Henry Song <henrysong@samsung.com> Review URL: https://codereview.chromium.org/772953002
generated by cgit v1.2.3 (git 2.39.1) at 2024-09-13 04:26:02 +0000