aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
* Stop using the NVPR glyph loading functionalityGravatar kkinnunen2015-06-23
| | | | | | Remove the NVPR glyph loading functionality. A bit risky feature. Review URL: https://codereview.chromium.org/1176113008
* Rename SkFontHost_none.cpp to SkFontMgr_empty_factory.Gravatar bungeman2015-06-23
| | | | | | | | | | When the current function was added in https://skia.googlesource.com/skia/+/070da5edcb1c6e9b9507f06a5e2c270e1c1fd796%5E%21/#F3 the SkFontHost was still alive and well. Now that this file no longer contains any SkFontHost related code, rename the file for clarity and uniformity. Review URL: https://codereview.chromium.org/1196313003
* add autotuning to visualbenchGravatar joshualitt2015-06-23
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1201003002
* Fix memory leak in SkPictureRecordGravatar robertphillips2015-06-23
| | | | Review URL: https://codereview.chromium.org/1197283007
* Fix indirect use of defined() directiveGravatar tomhudson2015-06-23
| | | | | | | BUG=432391 R=bsalomon,bungeman1 Review URL: https://codereview.chromium.org/876933004
* Propagate SkSurfaceProps to more call sitesGravatar robertphillips2015-06-23
| | | | | | | | | | Start moving to a world where everyone provides surface properties. Most notably this exposes a portion of SkSurfaceProps to the C API. BUG=skia:3934 Review URL: https://codereview.chromium.org/1195003003
* Tweak SkiaLab docsGravatar borenet2015-06-23
| | | | | | | | BUG=skia:3850 NOTRY=true DOCS_PREVIEW= https://skia.org/?cl=1203443003 Review URL: https://codereview.chromium.org/1203443003
* add simple test for offsetimagefilterGravatar reed2015-06-23
| | | | | | | BUG=skia: TBR= Review URL: https://codereview.chromium.org/1204723002
* add break statement -- lost during rewrite earlierGravatar reed2015-06-22
| | | | | | | BUG=skia: TBR= Review URL: https://codereview.chromium.org/1193333004
* change old picture serialization to really handle imagesGravatar reed2015-06-22
| | | | | | BUG=skia:3965 Review URL: https://codereview.chromium.org/1199473002
* fix for calling bindFragDataLocation on OpenGL 2.1Gravatar joshualitt2015-06-22
| | | | | | BUG=skia:3966 Review URL: https://codereview.chromium.org/1201623003
* Clean up usage of SkSurfacePropsGravatar robertphillips2015-06-22
| | | | | | | | | | | | This CL continues cleaning up Skia's usage of SkSurfaceProps. It: Removes the duplicate SkSurfaceProps object from SkImageFilter::Proxy. Removes a dispreferred ctor from SkCanvas Removes the initForRootLayer entry point from SkDevice (since the root device and the canvas should always have the same pixel geometry now). Review URL: https://codereview.chromium.org/1201983006
* Use vmulq_n_u32(..., 0x01010101) to distribute alphas.Gravatar mtklein2015-06-22
| | | | | | | | | | | | | | | | | | | | | | | This seems to make alphas() faster and Load[24]Alphas() no slower. The change is particularly noticeable on xfermodes that call alphas() twice (on src and dst), with a 10-12% speedup. Xfermode_Difference_aa 29ms -> 28.4ms 0.98x Xfermode_DstATop_aa 27.2ms -> 26.7ms 0.98x Xfermode_Xor_aa 27.2ms -> 26.5ms 0.98x Xfermode_DstOver 23.6ms -> 22.9ms 0.97x Xfermode_DstOver_aa 27.8ms -> 26.8ms 0.96x Xfermode_DstOut 22.6ms -> 21.7ms 0.96x Xfermode_Multiply_aa 30ms -> 28.5ms 0.95x Xfermode_DstOut_aa 26.1ms -> 24.8ms 0.95x Xfermode_DstIn_aa 25.4ms -> 24.1ms 0.95x Xfermode_DstATop 28.7ms -> 26ms 0.9x Xfermode_Multiply 35.5ms -> 31.3ms 0.88x Xfermode_Difference 31.8ms -> 27.7ms 0.87x Xfermode_Xor 30.1ms -> 26.1ms 0.87x BUG=skia: Review URL: https://codereview.chromium.org/1203513002
* fix spurious init on VisualBench startupGravatar joshualitt2015-06-22
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1202023002
* Implemented onGetScanlines and onSkipScanlines for interlaced pngsGravatar emmaleer2015-06-22
| | | | | | | | Modified DM tests to be faster when decoding interlaced pngs BUG=skia: Review URL: https://codereview.chromium.org/1194703002
* Update some Sk4px APIs.Gravatar mtklein2015-06-22
| | | | | | | | | | | | | | | Mostly this is about ergonomics, making it easier to do good operations and hard / impossible to do bad ones. - SkAlpha / SkPMColor constructors become static factories. - Remove div255TruncNarrow(), rename div255RoundNarrow() to div255(). In practice we always want to round, and the narrowing to 8-bit is contextually obvious. - Rename fastMulDiv255Round() approxMulDiv255() to stress it's approximate-ness over its speed. Drop Round for the same reason as above... we should always round. - Add operator overloads so we don't have to keep throwing in seemingly-random Sk4px() or Sk4px::Wide() casts. - use operator*() for 8-bit x 8-bit -> 16-bit math. It's always what we want, and there's generally no 8x8->8 alternative. - MapFoo can take a const Func&. Don't think it makes a big difference, but nice to do. BUG=skia: Review URL: https://codereview.chromium.org/1202013002
* Begin kLegacyFontHost_InitType cleanupGravatar robertphillips2015-06-22
| | | | | | | | | | | | | | | | | | This CL starts the process of pushing kLegacyFontHost_InitType-type SkSurfaceProps up the call stack and out of Skia. It: Gets rid of the default SkBaseDevice ctor. This means everyone has to always hand an explicit SkSurfaceProps to it. It makes public the SkBitmapDevice creation methods that require SkSurfaceProps. Removes (in Skia's code base) all SkBitmapDevice ctor calls w/o SkSurfaceProps. Makes the "recording" canvases (e.g., pdf, svg, xps) explicitly not use kLegacyFontHost_InitType. Replicates the creating canvas/device's flags on saveLayer devices BUG=skia:3934 Review URL: https://codereview.chromium.org/1204433002
* clean offsetimagefilter gmGravatar reed2015-06-22
| | | | | | | BUG=skia: TBR=robertphilips Review URL: https://codereview.chromium.org/1203473002
* Change large glyph check back to > threshold.Gravatar jvanverth2015-06-22
| | | | | | | | | Using >= misses some Chromium test cases that are still being rendered as paths. BUG=chromium:467569 Review URL: https://codereview.chromium.org/1202433002
* Remove distance field flag from SkPaintGravatar jvanverth2015-06-22
| | | | Review URL: https://codereview.chromium.org/1192413005
* Fixup android launch script to launch VisualBench as well as SampleAppGravatar joshualitt2015-06-22
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1185473003
* Make GrGLProgramDataManager not refcountedGravatar kkinnunen2015-06-21
| | | | | | | Make GrGLProgramDataManager not refcounted, refcounting is not used for the instances. Review URL: https://codereview.chromium.org/1195573002
* Update SKP versionGravatar skia.buildbots2015-06-21
| | | | | | | | | Automatic commit by the RecreateSKPs bot. TBR= NO_MERGE_BUILDS Review URL: https://codereview.chromium.org/1194233002
* remove SK_SUPPORT_LEGACY_OPTIONLESS_GET_PIXELS (now in SkUserConfig.h)Gravatar reed2015-06-20
| | | | | | | | BUG=skia: TBR= NOTRY=True Review URL: https://codereview.chromium.org/1200623003
* Rename all things "leaky" in SkDeviceGravatar robertphillips2015-06-19
| | | | Review URL: https://codereview.chromium.org/1198603002
* Fix for assert in VisualBenchGravatar joshualitt2015-06-19
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1198433004
* remove SkDevicePropertiesGravatar robertphillips2015-06-19
| | | | | | There is a lot more clean up to do here but this is probably a big enough bite. Review URL: https://codereview.chromium.org/1196683003
* Switch to glyphs as paths at a higher point size on AndroidGravatar jvanverth2015-06-19
| | | | | | | | | | | | | | | The regression occurred when we dropped the maximum DF size from 192 to 162, which meant that any glyph > 324 ended up being rendered as paths rather than the previous > 384. This pushes the threshold for rendering paths up to 384. Quality looks fine on high-res devices which is why this is restricted to Android-only (low-res Android devices should only rarely have text that large). BUG=chromium:467569 Committed: https://skia.googlesource.com/skia/+/932d413e69845989fadaecf5bcb8686ec8c05032 Review URL: https://codereview.chromium.org/1183053005
* Move rect_memcopy from helper to global static.Gravatar egdaniel2015-06-19
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1197713003
* Do not fail on images that are too small to subset decode.Gravatar msarett2015-06-19
| | | | | | | | | Specifically (0x0) images being produced by webp scaled decodes are causing problems. BUG=skia: Review URL: https://codereview.chromium.org/1192373003
* Prevent webp from producing 0 dimensional imagesGravatar msarett2015-06-19
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1196643002
* Revert of Disable failing GPU tests on Nexus 6. (patchset #1 id:1 of ↵Gravatar joshualitt2015-06-19
| | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/1185753002/) Reason for revert: this should hopefully be fixed now. Original issue's description: > Disable failing GPU tests on Nexus 6. > > BUG=skia:3942 > > Committed: https://skia.googlesource.com/skia/+/8ed08e54f4868d3ef88225bf873ce678e7716bd9 TBR=bsalomon@google.com,djsollen@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia:3942 Review URL: https://codereview.chromium.org/1196653002
* Fix precision error in https://codereview.chromium.org/1188433011/Gravatar robertphillips2015-06-19
| | | | | | TBR=reed@google.com Review URL: https://codereview.chromium.org/1192853002
* Added check for ill-conditioned invertGravatar robertphillips2015-06-19
| | | | | | | | sk_inv_determinant has a guard that the determinant can't get too big so this CL only checks if the determinant gets too small. BUG=492263 Review URL: https://codereview.chromium.org/1188433011
* Expose SkFaceRec less.Gravatar bungeman2015-06-19
| | | | | | | | | | | The SkFaceRec class is some data which needs to remain alive for the life of an FT_Face. ref_ft_face returns SkFaceRec, but confusingly unref_ft_face takes an FT_Face. Since no one was using the SkFaceRec for anything other than accessing the FT_Face, have ref_ft_face return FT_Face instead and remove the unused SkFaceRec pointer from the scaler context. Review URL: https://codereview.chromium.org/1180223005
* Remove gamma field from SkDevicePropertiesGravatar robertphillips2015-06-19
| | | | Review URL: https://codereview.chromium.org/1189223002
* remove deprecated NewRasterPMColorGravatar reed2015-06-19
| | | | | | | BUG=skia: TBR= Review URL: https://codereview.chromium.org/1192353002
* Revert of Switch to glyphs as paths at a higher point size on Android ↵Gravatar fmalita2015-06-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (patchset #1 id:1 of https://codereview.chromium.org/1183053005/) Reason for revert: GM:mixedtextblobs looks significantly worse on Android. Before: https://gold.skia.org/img/images/19d171dd72ba68df9501736dbbc2aa44.png After: https://gold.skia.org/img/images/d4c38072d2836356c00fdc1a7b537350.png Original issue's description: > Switch to glyphs as paths at a higher point size on Android > > The regression occurred when we dropped the maximum DF size from 192 > to 162, which meant that any glyph > 324 ended up being rendered as paths > rather than the previous > 384. This pushes the threshold for > rendering paths up to 384. Quality looks fine on high-res devices > which is why this is restricted to Android-only (low-res Android devices > should only rarely have text that large). > > BUG=chromium:467569 > > Committed: https://skia.googlesource.com/skia/+/932d413e69845989fadaecf5bcb8686ec8c05032 TBR=joshualitt@google.com,jvanverth@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=chromium:467569 Review URL: https://codereview.chromium.org/1192203002
* Work around for nexus 6 TexSubImage issueGravatar joshualitt2015-06-18
| | | | | | | | BUG=skia: Committed: https://skia.googlesource.com/skia/+/c69fe20ef35df0776b179cbd5ce4d1bdc0ba4090 Review URL: https://codereview.chromium.org/1173203005
* privatizeGravatar reed2015-06-18
| | | | | | | BUG=skia: TBR=bsalomon@google.com Review URL: https://codereview.chromium.org/1189173005
* disable floating point textures if GLSL version is < 330Gravatar joshualitt2015-06-18
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1178723007
* Switch to glyphs as paths at a higher point size on AndroidGravatar jvanverth2015-06-18
| | | | | | | | | | | | | The regression occurred when we dropped the maximum DF size from 192 to 162, which meant that any glyph > 324 ended up being rendered as paths rather than the previous > 384. This pushes the threshold for rendering paths up to 384. Quality looks fine on high-res devices which is why this is restricted to Android-only (low-res Android devices should only rarely have text that large). BUG=chromium:467569 Review URL: https://codereview.chromium.org/1183053005
* remove dead guarded code for legacy SkData procGravatar reed2015-06-18
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1173173007
* add callbacks to Images that wrap client-provided contentGravatar reed2015-06-18
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1169553003
* Initialize the sample config for wrapped RTs to kUnified.Gravatar senorblanco2015-06-18
| | | | | | BUG=skia:3958 Review URL: https://codereview.chromium.org/1177163005
* Modify android_install_app to install VisualBenchGravatar joshualitt2015-06-18
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1181973008
* Add SkWebpCodec, for decoding .webp images.Gravatar scroggo2015-06-18
| | | | | | | | | | | Based on SkImageDecoder_libwebp. TODO: Support YUV? (Longer term - may influence our API for SkImageGenerator) BUG=skia:3257 Review URL: https://codereview.chromium.org/1044433002
* remove SkData proc guardGravatar reed2015-06-18
| | | | | | | BUG=skia: NOTRY=True Review URL: https://codereview.chromium.org/1188773008
* Don't call bindTexture from GrGLProgramGravatar cdalton2015-06-18
| | | | | | | | | | Updates GrGLProgram to tell the gpu object which textures it wants bound, instead of calling bindTexture directly. This begins to break its dependence on the specific GrGLGpu object. BUG=skia: Review URL: https://codereview.chromium.org/1192463003
* Allows windowed apps to be built on the Mac just by writing a SkOSWindow ↵Gravatar bsalomon2015-06-18
| | | | | | | | subclass without needing nib/plist files. Makes visualbench work on the Mac. Review URL: https://codereview.chromium.org/1184143011