aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
...
* Revert of Fix to set correct output type when blending when we've read dst ↵Gravatar joshualitt2014-12-10
| | | | | | | | | | | | | | | | | | | | | (patchset #4 id:60001 of https://codereview.chromium.org/791143002/) Reason for revert: Breaks DM on windows Original issue's description: > Fix to set correct output type when blending when we've read dst > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/733fc2054044377e914559fa4f46db66ab7d9cea TBR=bsalomon@google.com,egdaniel@google.com NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/795783002
* fix for chrome bug with sub pixel hairlinesGravatar joshualitt2014-12-10
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/796573002
* Initial baselines for tall_stretched_bitmaps GMGravatar Brian Salomon2014-12-10
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/789293003
* Revert of Remove GP from drawstate, revision of invariant output for GP ↵Gravatar joshualitt2014-12-10
| | | | | | | | | | | | | | | | | | | | | (patchset #9 id:160001 of https://codereview.chromium.org/791743003/) Reason for revert: breaks mac Original issue's description: > Remove GP from drawstate, revision of invariant output for GP > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/c6bc58eded89b0c0a36b8e20e193c200f297a0da TBR=bsalomon@google.com,egdaniel@google.com,joshualitt@chromium.org NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/794843002
* Remove GP from drawstate, revision of invariant output for GPGravatar joshualitt2014-12-10
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/791743003
* Don't store an SkMatrix in BitmapShaderKey.Gravatar fmalita2014-12-10
| | | | | | | | | | SkMatrix makes for a poor key component due to mutable/cache fields. Use its canonical scalars instead. R=reed@google.com,mtklein@google.com Review URL: https://codereview.chromium.org/792123002
* Fix to set correct output type when blending when we've read dstGravatar egdaniel2014-12-10
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/791143002
* fix for valgrind uninit variablesGravatar joshualitt2014-12-10
| | | | | | | TBR=bsalomon@google.com BUG=skia: Review URL: https://codereview.chromium.org/793773002
* Rebaseline imagefiltersscaled images (and rm from ignored-tests file)Gravatar robertphillips2014-12-10
| | | | | | | | Rebaselines for https://codereview.chromium.org/785893004/ (Fix MatrixImageFilter computeFastBounds) TBR=scroggo@google.com Review URL: https://codereview.chromium.org/794673004
* More rebaselines.Gravatar scroggo2014-12-10
| | | | | | | TBR=bsalomon BUG=skia: Review URL: https://codereview.chromium.org/790263002
* Remove MULTILINE, ^, and $ from re finding actual_results.json.Gravatar bungeman2014-12-10
| | | | | | | | | | | | | | | MULTILINE, ^, and $ work based on the platform, but the files being parsed are from other platforms. As a result, the current code for extracting the actual_results.json version will not find it in logfiles produced on Windows when run on Mac. The code for extracting the exact actual_results.json file to use from a logfile is itself something of a hack, as this information should be provided ina more structured manner. This proposed method of finding the exact file is no worse than the old one, and in cases like above, better. Review URL: https://codereview.chromium.org/789253002
* Detect discarded SkPictureShader pixel refs.Gravatar fmalita2014-12-10
| | | | | | | | | | This doesn't address the more general problem, but reduces the race window significantly. BUG=440282 R=reed@google.com Review URL: https://codereview.chromium.org/787043006
* Rebaseline.Gravatar scroggo2014-12-10
| | | | | | | | | | All these images barely differ. TBR=bsalomon BUG=skia: Review URL: https://codereview.chromium.org/795673002
* Include SkTypes.h from SkDynamicAnnotations.hGravatar mtklein2014-12-10
| | | | | | | | | This allows us to control SK_DYNAMIC_ANNOTATIONS_ENABLED by Sk*Config.h files. This is a no-op today, because we control it from the compiler command line. BUG=430815 Review URL: https://codereview.chromium.org/787003003
* Gradient shaders: make fPtsToUnit const, pre-cache getType().Gravatar mtklein2014-12-10
| | | | | | | | | | | This prevents races when calling fPtsToUnit.getType() from multiple threads. This introduces a small amount of redundant code in SkTwoPointRadialGradient, but it's probably optimized together into no extra run-time work. BUG=437511 Review URL: https://codereview.chromium.org/793763003
* Change how SkDebugf is sent to stdout on Android.Gravatar scroggo2014-12-10
| | | | | | | | | | | | | Previously, a function was called using dlsym in skia_launcher. Add a static initializer that changes the setting, and include that for the tools we automate for testing. Also only do va_copy if we actually use it. BUG=skia:2454 Review URL: https://codereview.chromium.org/753543003
* dd readPixels to SkImageGravatar reed2014-12-10
| | | | | | | | patch from issue 789673007 at patchset 1 (http://crrev.com/789673007#ps1) BUG=skia: Review URL: https://codereview.chromium.org/793723002
* Suppress clipped-bitmap-shaders-clampGravatar Florin Malita2014-12-10
| | | | | | | | Pending PDF rebaseline after c54d8db4d169 TBR=scroggo@google.com Review URL: https://codereview.chromium.org/788143003
* Remove SkCanvas::drawBitmapMatrix()Gravatar Florin Malita2014-12-10
| | | | | | R=mtklein@google.com, reed@google.com, robertphillips@google.com Review URL: https://codereview.chromium.org/789033002
* more rebaselinesGravatar Brian Salomon2014-12-10
| | | | Review URL: https://codereview.chromium.org/789263002
* Add more ignore gm's for recent Xfer Processor CLGravatar egdaniel2014-12-10
| | | | | | | | | TBR=bsalomon@google.com NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/786083004
* Add baseline images for new filterfastbounds GMGravatar robertphillips2014-12-10
| | | | | | TBR=scroggo@google.com Review URL: https://codereview.chromium.org/787423002
* Create xfer processor backend.Gravatar egdaniel2014-12-10
| | | | | | | | | | | This includes: -Having an actual XP stage at the end of the gl pipeline. -All Blending work is handled by XP until actually setting GL blend states -GLPrograms test to test XP BUG=skia: Review URL: https://codereview.chromium.org/764643004
* Add GM for 2-rects blur which cannot break into nine-patchGravatar qiankun.miao2014-12-10
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/791943002
* Remove canvas::NewRaster, and rename surface::NewRasterPMColor to N32PremulGravatar reed2014-12-10
| | | | | | | | | | patch from issue 781403002 at patchset 20001 (http://crrev.com/781403002#ps20001) BUG=skia: TBR= re-landing after chrome fixes have landed Review URL: https://codereview.chromium.org/784223007
* Add SSSE3 acceleration for S32_D16_filter_DXGravatar qiankun.miao2014-12-10
| | | | | | | | | | | With this CL, related nanobench can be improved for 565 config. bitmap_BGRA_8888_update_scale_bilerp 76.1us -> 46.7us 0.61x bitmap_BGRA_8888_scale_bilerp 78.7us -> 47us 0.6x bitmap_BGRA_8888_update_volatile_scale_bilerp 82.7us -> 46.9us 0.57x BUG=skia: Review URL: https://codereview.chromium.org/788853002
* fix surface test for gpu and codecsGravatar reed2014-12-10
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/789173004
* Update SKP versionGravatar skia.buildbots2014-12-09
| | | | | | | | Automatic commit by the RecreateSKPs bot. TBR= Review URL: https://codereview.chromium.org/790073002
* Add bench to measure blur rects performanceGravatar qiankun.miao2014-12-09
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/787913002
* Update SKP versionGravatar skia.buildbots2014-12-09
| | | | | | | | Automatic commit by the RecreateSKPs bot. TBR= Review URL: https://codereview.chromium.org/760943003
* add readPixels to SkSurface (later try to remove it from SkCanvas)Gravatar reed2014-12-09
| | | | | | | BUG=skia: TBR=bsalomon, robertphilips Review URL: https://codereview.chromium.org/789993002
* Skia: Track the fIsWrapped separately so that we delete correctlyGravatar hendrikw2014-12-09
| | | | | | | | | | | | | | | GrGlTextureRenderTarget inherits virtually from both GrGlRenderTarget and GrGLTexture, which both have a 'wrap' flag. The passed in wrap setting could be different for the two base classes, but since it's virtually inherited, they share the same flag, so they're either both on, or both off. As a result, we fail to delete the frambuffer. To fix this, we now keep a separate isWrapped flag for GrGlRenderTarget. BUG=437998 Review URL: https://codereview.chromium.org/791493003
* rrrrrrrrrrrreeeeeeeeeeebaseline!Gravatar Brian Salomon2014-12-09
| | | | Review URL: https://codereview.chromium.org/790643004
* Revert of remove (dumb) canvas::NewRaster, and rename ↵Gravatar reed2014-12-09
| | | | | | | | | | | | | | | | | | | | | | | surface::NewRasterPMColor to N32Premul (patchset #3 id:40001 of https://codereview.chromium.org/790733003/) Reason for revert: need to update chrome first Original issue's description: > remove (dumb) canvas::NewRaster, and rename surface::NewRasterPMColor to N32Premul > > patch from issue 781403002 at patchset 20001 (http://crrev.com/781403002#ps20001) > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/2c1605a1fbaa2e35a27399a34254fb1200ec2ae6 TBR=fmalita@google.com,fmalita@chromium.org NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/791763002
* This cl moves color and coverage off of drawstate. In an effort to keep ↵Gravatar joshualitt2014-12-09
| | | | | | | | | | this CL manageable, I have left the compute invariant input / output in a bit of a strange state(fixing this will be complicated). In addition, NVPR makes this very complicated, and I haven't quite figured out a good way to handle it, so for now color and coverage DO live on optstate, but I will figure out some way to refactor that in future CLs. BUG=skia: Review URL: https://codereview.chromium.org/783763002
* Adding an option for pixelated rendering in SkPictureImageFilterGravatar junov2014-12-09
| | | | | | BUG=skia:3209 Review URL: https://codereview.chromium.org/787073003
* More N10 rebaselinesGravatar Brian Salomon2014-12-09
| | | | | | TBR= Review URL: https://codereview.chromium.org/784413002
* Remove unused static function skcolor_to_grcolor_nopremultiply.Gravatar egdaniel2014-12-09
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/762223003
* remove (dumb) canvas::NewRaster, and rename surface::NewRasterPMColor to ↵Gravatar reed2014-12-09
| | | | | | | | | | N32Premul patch from issue 781403002 at patchset 20001 (http://crrev.com/781403002#ps20001) BUG=skia: Review URL: https://codereview.chromium.org/790733003
* Rebaseline after texture coord precision changes.Gravatar Brian Salomon2014-12-09
| | | | | | TBR=scroggo@google.com Review URL: https://codereview.chromium.org/788903003
* change SkPoint::setLength to set itself to (0,0) if it starting length is ↵Gravatar reed2014-12-09
| | | | | | | | | degenerate. BUG=skia:3203 TBR=caryclark Review URL: https://codereview.chromium.org/785933003
* Fix for alpha textures on Nexus 6.Gravatar jvanverth2014-12-09
| | | | | | | | Nexus 6 appears to require a sized internal format for A8 textures, much like other newer mobile devices. Changed to use sized format for A8 textures in general with ES 3.0. Review URL: https://codereview.chromium.org/783523003
* Don't call GetShaderPrecisionFormat for geometry shaders.Gravatar bsalomon2014-12-09
| | | | | | TBR=egdaniel@google.com Review URL: https://codereview.chromium.org/791713002
* Make all blending up to GrOptDrawState be handled by the xp/xp factory.Gravatar egdaniel2014-12-09
| | | | | | | | | | | | In this cl the blending information is extracted for the xp and stored in the ODS which is then used as it currently is. In the follow up cl, an XP backend will be added and at that point all blending work will take place inside XP's. BUG=skia: Committed: https://skia.googlesource.com/skia/+/7c66342a399b529634bed0fabfaa562db2c0dbd4 Review URL: https://codereview.chromium.org/759713002
* Use threshold of 1 texture coord value per pixel w/ nearest neighbor.Gravatar bsalomon2014-12-09
| | | | Review URL: https://codereview.chromium.org/787873002
* Use GrCoordTransform precision to set uniform matrix precisionGravatar bsalomon2014-12-09
| | | | Review URL: https://codereview.chromium.org/784103003
* Rename CustomCoordTextureEffect to GrBitmapTextGeoProc.Gravatar egdaniel2014-12-09
| | | | | | | | Also add in explicit set for LCD text in invariantOutput. BUG=skia: Review URL: https://codereview.chromium.org/786293002
* Update nanobench so the non-MPD path doesn't permit layer hoistingGravatar robertphillips2014-12-09
| | | | Review URL: https://codereview.chromium.org/787923002
* Switch non-MPD nanobench path to use a separate canvas per tileGravatar robertphillips2014-12-09
| | | | | | | | | | | | | | | | | | | | | | | | It is desirable that, when layer hoisting is disabled, the MPD and non-MPD timings be roughly the same. Unfortunately, using a separate canvas for each tile (a requirement for MPD) introduces its own discrepancy into the timing. Using a separate canvas for each tile doesn't seem to make a difference for 8888 (see the non-MPD 8888 column below) but slows down GPU rendering (see the non-MPD GPU column below). Since this is how Chromium renders I propose switching to this regimen (even though it is "slowing down" GPU rendering). nanobench mean times (ms) with layer hoisting disabled (for desk_amazon.skp) 8888 MPD non-MPD 1 canvas (old-style) 0.628 1.71 separate (new-style) 0.795 1.63 GPU MPD non-MPD 1 canvas (old-style) 2.34 1.69 separate (new-style) 2.32 2.66 Review URL: https://codereview.chromium.org/779643002
* Increase timeouts for CSS3Gradients and GooglePlus pagesetsGravatar rmistry2014-12-09
| | | | | | | BUG=skia:3187 BUG=skia:3196 Review URL: https://codereview.chromium.org/786353002
generated by cgit v1.2.3 (git 2.39.1) at 2024-09-07 06:18:39 +0000