aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
...
* Massive N10 Rebaseline Part 2Gravatar Brian Salomon2014-12-08
| | | | | | BUG=skia:3205 Review URL: https://codereview.chromium.org/790533002
* Add new GM (filterfastbounds)Gravatar robertphillips2014-12-08
| | | | | | | | This new GM visualizes the fast bounds computed by various image-filter-based SkPaints. This is lead up to fixing some issues in fast bound computation. BUG=418417 Review URL: https://codereview.chromium.org/788613003
* Constify CreatePictureShader's picture arg.Gravatar fmalita2014-12-08
| | | | | | R=robertphillips@google.com Review URL: https://codereview.chromium.org/768113004
* Update dftext expectations after https://codereview.chromium.org/772633003/.Gravatar jvanverth2014-12-08
| | | | | | | BUG=skia:3195, skia:3188 TBR=scroggo@google.com Review URL: https://codereview.chromium.org/778043003
* remove device::clear -- no longer calledGravatar reed2014-12-08
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/778373003
* Massive N10 RebaselineGravatar Brian Salomon2014-12-08
| | | | | | BUG=skia:3193 Review URL: https://codereview.chromium.org/783173002
* make SkColorShader private, use public factoryGravatar reed2014-12-08
| | | | | | | | BUG=skia: Need https://codereview.chromium.org/785823002/ to land first Review URL: https://codereview.chromium.org/788533002
* add convexity logic and tests for scalar max, Inf, and NaNGravatar caryclark2014-12-08
| | | | | | | | | | PathOps relies on isConvex() only returning true for trivially convex paths. The old logic also returns true if the paths that contain NaNs and Infinities. Return kUnknown_Convexity instead in those cases and in cases where the convexity logic computes intermediaries that overflow. Review URL: https://codereview.chromium.org/784593002
* clear() is now an alias for drawPaint, so remove its virtualGravatar reed2014-12-05
| | | | | | | | | | Needs these to land first: https://codereview.chromium.org/733343005 https://codereview.chromium.org/777413004 BUG=skia: Review URL: https://codereview.chromium.org/780393002
* Add support for half float alpha textures.Gravatar jvanverth2014-12-05
| | | | | | | | | This allows us to create distance field textures with better precision, which may help text quality. BUG=skia:3103 Review URL: https://codereview.chromium.org/762923003
* Check XpFactory equality in DrawStateGravatar egdaniel2014-12-05
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/767873006
* rebaseline after GM changeGravatar reed2014-12-05
| | | | | | | | BUG=skia: TBR= NOTRY=True Review URL: https://codereview.chromium.org/784603002
* Fix include for GrPortderDuffXferProcessor.cppGravatar egdaniel2014-12-05
| | | | | | | | TBR=bsalomon@google.com BUG=skia: Review URL: https://codereview.chromium.org/771353005
* Factor text size device mapping in SkScalerContext.Gravatar bungeman2014-12-05
| | | | | | | | | | | | | All of our font back-ends use the text size in some way other than simple concatentation with the current matrix. The code here finds the full device matrix and then decomposes it to extract the text size. FreeType, GDI, and DirectWrite use the text size as the pre-hint scale. CoreText will not scale color emoji above the requested text size, and certain features like 'trak' are performed on the text size. Review URL: https://codereview.chromium.org/748883005
* get rid of static gpsGravatar joshualitt2014-12-05
| | | | | | | | | This may cause a perf regression TBR= BUG=skia: Review URL: https://codereview.chromium.org/749413003
* Move GrPorterDuffXferProcessor to include/gpu/effects.Gravatar egdaniel2014-12-05
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/780263003
* Use size of SkRect as the key for blur mask cacheGravatar qiankun.miao2014-12-05
| | | | | | BUG=429409 Review URL: https://codereview.chromium.org/708073002
* rebaseline after change to imagefilterscropped gm changeGravatar reed2014-12-05
| | | | | | | | BUG=skia: NOTRY=True TBR= Review URL: https://codereview.chromium.org/779313002
* Fix overoptimization in distance field code on N7 and N10.Gravatar jvanverth2014-12-05
| | | | | | BUG=skia:3188 Review URL: https://codereview.chromium.org/772633003
* remove unnecessary save/clip/restoreGravatar reed2014-12-05
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/785493003
* Don't upload metrics we don't want to track.Gravatar mtklein2014-12-05
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/758853004
* Tolerate unstarted builds in rebaseline server.Gravatar bungeman2014-12-05
| | | | | | | | Currently server.py --rietveld-issue will crash if there are any unstarted trybots on the latest patch. Instead of crashing, warn that the bot hasn't started yet. Review URL: https://codereview.chromium.org/758793003
* Update SKP versionGravatar skia.buildbots2014-12-04
| | | | | | | | Automatic commit by the RecreateSKPs bot. TBR= Review URL: https://codereview.chromium.org/781043002
* Ignore failures for n7 and n10 dftext GM.Gravatar jvanverth2014-12-04
| | | | | | | BUG=skia:3188 TBR=halcanary@google.com Review URL: https://codereview.chromium.org/785433002
* clean up default precision handlingGravatar bsalomon2014-12-04
| | | | | | R=joshualitt@google.com Review URL: https://codereview.chromium.org/766753006
* Remove backend factoriesGravatar joshualitt2014-12-04
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/778453002
* nanobench: upload peak memory usage as its own trace.Gravatar mtklein2014-12-04
| | | | | | | | | | | | | We'll end up with a result like this: "memory_usage" : { "meta" : { "max_rss_mb" : 57 } } BUG=skia: Review URL: https://codereview.chromium.org/780013002
* Ganesh text rendering cleanup.Gravatar jvanverth2014-12-04
| | | | | | | | | Rename GrTextStrike.{cpp.h} to GrFontCache.{cpp,h} Move contents of GrTextStrike_impl.h to GrFontCache.h Move glyph uploading to a separate function and remove harmful gotos Add assert on glyph upload failure (shouldn't happen) Review URL: https://codereview.chromium.org/780923002
* rebaseline pdf-nativeGravatar halcanary2014-12-04
| | | | | | | TBR= NOTRY=true Review URL: https://codereview.chromium.org/779073002
* create and thread batch tracker objectGravatar joshualitt2014-12-04
| | | | | | | | | | I remove the factory on the next CL BUG=skia: Committed: https://skia.googlesource.com/skia/+/c3a6eb23483e5d28073b509a5f637f41660de294 Review URL: https://codereview.chromium.org/772513002
* 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