aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
* make imagefilters* imagemagnifier imageresizetiled portableGravatar caryclark2015-07-16
| | | | | | TBR=reed@google.com Review URL: https://codereview.chromium.org/1238083004
* Revert of Remove rt flags that are not necessary (patchset #1 id:1 of ↵Gravatar bsalomon2015-07-16
| | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/1226123012/) Reason for revert: breaks tests, e.g. https://build.chromium.org/p/client.skia.android/builders/Test-Android-GCC-Nexus9-GPU-TegraK1-Arm64-Release/builds/752/steps/dm/logs/stdio Original issue's description: > Remove rt flags that are not necessary > > Committed: https://skia.googlesource.com/skia/+/b6b0a6ef179f4143c94ba7e0e1be6e7ce4b0a23a TBR=joshualitt@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/1234503004
* Remove rt flags that are not necessaryGravatar bsalomon2015-07-16
| | | | Review URL: https://codereview.chromium.org/1226123012
* All batches do their own pipeline compareGravatar joshualitt2015-07-16
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1242033002
* Make readpixels work on GrTexturesGravatar bsalomon2015-07-16
| | | | Review URL: https://codereview.chromium.org/1234313002
* Give GrBatch a pointer to GrPipelineGravatar joshualitt2015-07-16
| | | | | | | | BUG=skia: Committed: https://skia.googlesource.com/skia/+/1170a12839218f7a23c93487bf95fd83aae0201f Review URL: https://codereview.chromium.org/1237283007
* Various GL testing fixesGravatar jvanverth2015-07-16
| | | | | | | Check for GL context when printing NVPR error string. Fix some indenting. Review URL: https://codereview.chromium.org/1235283004
* have canvas send discard instead of retain if the draw would overwrite ↵Gravatar reed2015-07-16
| | | | | | | | | | | | everything Motivation: - perf win for clients that overwrite the surface after a snapshot. - may allow us to eliminate SkDeferredCanvas, as this was its primary advantage. BUG=skia: Review URL: https://codereview.chromium.org/1236023004
* Revert of Give GrBatch a pointer to GrPipeline (patchset #1 id:1 of ↵Gravatar joshualitt2015-07-16
| | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/1237283007/) Reason for revert: breaking nanobench Original issue's description: > Give GrBatch a pointer to GrPipeline > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/1170a12839218f7a23c93487bf95fd83aae0201f TBR=bsalomon@google.com,joshualitt@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/1239073002
* Give GrBatch a pointer to GrPipelineGravatar joshualitt2015-07-16
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1237283007
* increase gm offsetimagefilter stroke widthGravatar caryclark2015-07-16
| | | | | | | | | | With a hairline width, offsetimagefilter fails to draw the rectangle on some platforms. Make the width 2 so that it shows up everywhere. TBR=reed@google.com Review URL: https://codereview.chromium.org/1236133003
* Coverage script: fix missing dataGravatar borenet2015-07-16
| | | | | | BUG=skia:2430 Review URL: https://codereview.chromium.org/1242023002
* Replace buggy_blend_modes GM with an exhaustive test.Gravatar mtklein2015-07-16
| | | | | | | | | | | | | | The new test is disabled by default, as it's quite slow. We can run it if we suspect problems by passing -x to DM. This test would have been failing before the bug fix, and now is passing. Assuming the Priv on the end means it's not considered public API... TBR=reed@google.com BUG=skia:4052 Review URL: https://codereview.chromium.org/1228333003
* guard to remove DrawBitmapRectFlagsGravatar reed2015-07-16
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1235393003
* make *imagefilter morphology iamge bitmapsource gm portableGravatar caryclark2015-07-15
| | | | | | TBR=reed@google.com Review URL: https://codereview.chromium.org/1236403002
* make strokefill gm portableGravatar caryclark2015-07-15
| | | | | | | | | | | | | Convert the Mac-specific font text statically into paths so that they draw the same on all platforms. Add drawing portable text using the fake bold paint setting so that the code path is tested everywhere and but draws the same thing. R=reed@google.com Review URL: https://codereview.chromium.org/1234463006
* rename utility to see if a paint will overwrite its pixelsGravatar reed2015-07-15
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1228853005
* Added GLVec4ScalarBench that doesn't use VAOsGravatar wangyix2015-07-15
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1235153008
* make gradtext quadpaths gm portableGravatar caryclark2015-07-15
| | | | | | TBR=reed@google.com Review URL: https://codereview.chromium.org/1234233003
* Fix leak in Android FontMgr test.Gravatar bungeman2015-07-15
| | | | | | FontMgrAndroidParserTest deos not free the font families it creates. This issue was discovered by asan when enabling this test on Linux platforms.
* Allow Android font manager on *nix.Gravatar bungeman2015-07-15
| | | | | | | | | | | | There is nothing left in the SkFontMgr_android.cpp which is specific to Android, so allow it to be built and used on *nix platforms. This allows for easier development and testing. The only reason to limit to *nix platforms are the dependencies on Expat and FreeType. This should be buildable and runnable on other platforms when these dependencies are also available. Review URL: https://codereview.chromium.org/1228833003
* Fix for android command line arguments to VisualBenchGravatar joshualitt2015-07-15
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1241923002
* Only run test_wrapped_resources() when GL is activeGravatar jvanverth2015-07-15
| | | | Review URL: https://codereview.chromium.org/1234233002
* Don't upload texture smaller than the min texture size when stretching a bmp ↵Gravatar bsalomon2015-07-15
| | | | | | | | up to a POT BUG=skia:3932 Review URL: https://codereview.chromium.org/1238913002
* make colorwheelnative displacement dropshadowimagefilter getposttextpath gm ↵Gravatar caryclark2015-07-15
| | | | | | | | portable TBR=reed@google.com Review URL: https://codereview.chromium.org/1237233004
* make path-reverse gm portableGravatar caryclark2015-07-15
| | | | | | | | | | | | | Put font glyph data in test as path instead of getting data from the Mac font. THe font may change, and is not available on most platforms. Also remove unreachable code. R=reed@google.com Review URL: https://codereview.chromium.org/1230703005
* Match works correctly on visualbenchGravatar joshualitt2015-07-15
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1234123004
* experimental: remove old PDF benchmarking toolsGravatar halcanary2015-07-15
| | | | | | TBR= Review URL: https://codereview.chromium.org/1233093004
* llvm-coverage script: respect SKIA_OUTGravatar borenet2015-07-15
| | | | | | | NOTRY=true BUG=skia:2430 Review URL: https://codereview.chromium.org/1239803002
* Fix undefined behavior.Gravatar herb2015-07-15
| | | | | | BUG=skia:4050 Review URL: https://codereview.chromium.org/1232003008
* Optimize RGB16 blitV functions with NEON for ARM platform.Gravatar yang.zhang2015-07-15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Here are some performance resultsi on Nexus 9: SkRGB16BlitterBlitV_neon: +--------+-----------+ |height | C/NEON | +--------+-----------+ |1 | 0.765230 | +--------+-----------+ |8 | 1.273330 | +--------+-----------+ |18 | 1.441462 | +--------+-----------+ |32 | 1.627798 | +--------+-----------+ |76 | 1.683131 | +--------+-----------+ |85 | 1.679456 | +--------+-----------+ |120 | 1.721311 | +--------+-----------+ |128 | 1.725482 | +--------+-----------+ |512 | 1.784117 | +--------+-----------+ BUG=skia: Review URL: https://codereview.chromium.org/1213723002
* Revert of Added a GLBench for testing performance of vec4 vs scalar for ↵Gravatar scroggo2015-07-15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | coverage in generated shaders. (patchset #6 id:90001 of https://codereview.chromium.org/1225383002/) Reason for revert: Still failing on Windows. e.g. https://uberchromegw.corp.google.com/i/client.skia/builders/Perf-Win8-MSVC-ShuttleB-GPU-HD4600-x86_64-Release-ANGLE/builds/231/steps/nanobench/logs/stdio skbug.com/4053 Original issue's description: > Added a glBench for testing performance of vec4 vs scalar coverage in generated shaders. > > Added bench for timing vec4 vs scalar type for coverage in shaders > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/6104ced165f17eb2f765ace354d5895c0bc890c5 > > Committed: https://skia.googlesource.com/skia/+/c734e69e8cf94bacaf68d3d8ee3310d1ad1fe8b8 TBR=joshualitt@google.com,tomhudson@google.com,wangyix@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/1239503003
* Fix VisualBench to hold onto a surfaceGravatar joshualitt2015-07-15
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1236723003
* make linepaths gm portableGravatar caryclark2015-07-15
| | | | | | TBR=reed@google.com Review URL: https://codereview.chromium.org/1225303010
* remove funkster from dcshader gmGravatar caryclark2015-07-15
| | | | | | | | | | | | | | The multicolored font that funster enables only draws and measures on some platforms. Removing it from the dcshader gm allows the layout to be uniform on all devices, making it easier to compare the output in gold. TBR=bsalomon@google.com Review URL: https://codereview.chromium.org/1231853004
* Document new CQ keywordGravatar rmistry2015-07-15
| | | | | | | | | BUG=skia: NOTRY=true DOCS_PREVIEW= https://skia.org/?cl=1237693005 TBR=jcgregorio Review URL: https://codereview.chromium.org/1237693005
* Fixed valgrind issues with libjpeg-turboGravatar msarett2015-07-14
| | | | | | | | | | | The fixes are in the updated libjpeg-turbo repository pulled in by DEPS. The fixes are detailed in the linked skia bug. To summarize briefly, we now use calloc() instead of malloc(). BUG=skia:4030 Review URL: https://codereview.chromium.org/1237213004
* Updated PDFViewer drawImageRect() overridesGravatar fmalita2015-07-14
| | | | | | R=reed@google.com Review URL: https://codereview.chromium.org/1240573008
* Let Android use old version of onDrawBitmapRect.Gravatar scroggo2015-07-14
| | | | | | | | | | | | Define SK_SUPPORT_LEGACY_ONDRAWIMAGERECT when building for the Android framework, since SkiaCanvasProxy overrides the old version of the method. Fixes master-skia build. NOTRY=True Review URL: https://codereview.chromium.org/1237903002
* Fix buggy blend modes.Gravatar mtklein2015-07-14
| | | | | | | | | | | | The problem turns out to be not over- or underflow like I thought, but that I used different math for x*y/255 for alphas and colors, sometimes resulting in colors where alpha was one less than the maximum color component, which is not a valid SkPMColor. To be safe, I've switched over all four of these similar complex modes to use exact math everywhere. They now match the byte-by-byte code in SkXfermode.cpp exactly. This will slow down Darken and Lighten by about 2x. I plan to follow up with a CL to see if I can eek out some speed there, and another CL to add asserts that Sk4px code always writes valid SkPMColors. BUG=skia:4052 Review URL: https://codereview.chromium.org/1241683003
* Preserve SKP cullrects on deserializationGravatar fmalita2015-07-14
| | | | | | | | Let's not ignore the offset. R=reed@google.com,mtklein@google.com Review URL: https://codereview.chromium.org/1235953004
* Very tiny paths are subsumed by the ConvexicatorGravatar caryclark2015-07-14
| | | | | | | | | | | | | | | and are treated as convex when they are not. Allow the SkPath::Iter to leave degenerate path segments unmolested by passing an additional exact bool to next(). Treat any non-zero length as significant in addPt(). R=reed@google.com,robertphillips@google.com BUG=493450 Review URL: https://codereview.chromium.org/1228383002
* Remove GrRedBlackTreeGravatar joshualitt2015-07-14
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/1226203013
* Rename backing texture testing routinesGravatar jvanverth2015-07-14
| | | | Review URL: https://codereview.chromium.org/1230193006
* Another trivial cleanupGravatar joshualitt2015-07-14
| | | | | | | | | TBR=bsalomon@google.com BUG=skia: Committed: https://skia.googlesource.com/skia/+/52e7657cd850f95e66eb23c6d138ee45149a1039 Review URL: https://codereview.chromium.org/1229303003
* Disable optimization as HWUI does not yet support color shaders.Gravatar djsollen2015-07-14
| | | | Review URL: https://codereview.chromium.org/1235913005
* Implement support for CHROMIUM_path_rendering pseudo extensionGravatar kkinnunen2015-07-14
| | | | | | | | | | | | | | | | Implement support for path rendering in Chromium through CHROMIUM_path_rendering pseudo extension. The extension defines a new pseudo-gl function, BindFragmentInputLocation. This behaves similarly to the BindUniformLocation pseudo-gl function. The idea is to assign fragment input location to a fragment input before linking the program. BUG=chromium:344330 Committed: https://skia.googlesource.com/skia/+/eeef46d181f9f8db388ecea81df699fc1b3c9280 Review URL: https://codereview.chromium.org/1192663002
* 3-15% speedup to HardLight / Overlay xfermodes.Gravatar mtklein2015-07-14
| | | | | | | | | | | | | | | While investigating my bug (skia:4052) I saw this TODO and figured it'd make me feel better about an otherwise unsuccessful investigation. This speeds up HardLight and Overlay (same code) by about 15% with SSE, mostly by rewriting the logic from 1 cheap comparison and 2 expensive div255() calls to 2 cheap comparisons and 1 expensive div255(). NEON speeds up by a more modest ~3%. BUG=skia: Review URL: https://codereview.chromium.org/1230663005
* add src-rect-constraint to drawImageRectGravatar reed2015-07-14
| | | | | | | | | | | Follow-on work - unify around SrcRectConstraint (i.e. drawBitmapRect) - remove silly drawBitmapRectToRect alias - clean-up (possibly remove) alias problems around drawBitmapRect + IRect parameter BUG=skia: Review URL: https://codereview.chromium.org/1228083004
* Add a GYP flag to define SKNX_NO_SIMD when set.Gravatar mtklein2015-07-14
| | | | | | | | Will use this with a new -SKNX_NO_SIMD bot. BUG=skia: Review URL: https://codereview.chromium.org/1227163016
generated by cgit v1.2.3 (git 2.39.1) at 2024-07-04 08:29:03 +0000