aboutsummaryrefslogtreecommitdiffhomepage
path: root/src
Commit message (Collapse)AuthorAge
* Revert of Revert of SkDrawCommand scrubbing (patchset #1 id:1 of ↵Gravatar fmalita2014-11-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/712683002/) Reason for revert: Chromium clients updated, this should be canary-proof now. Original issue's description: > Revert of SkDrawCommand scrubbing (patchset #2 id:20001 of https://codereview.chromium.org/706363002/) > > Reason for revert: > Canary borkage. > > Original issue's description: > > SkDrawCommand scrubbing > > > > Remove unused ctor, constify, etc. > > > > R=robertphillips@google.com > > > > Committed: https://skia.googlesource.com/skia/+/1931ec5b5dac68f1e452af0c65161bdce35b2dec > > TBR=robertphillips@google.com > NOTREECHECKS=true > NOTRY=true > > Committed: https://skia.googlesource.com/skia/+/9063ddd511f8e0d8c7454950cbc56e273678bf60 TBR=robertphillips@google.com NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/708333002
* Default geometry processorGravatar joshualitt2014-11-08
| | | | | | | | BUG=skia: Committed: https://skia.googlesource.com/skia/+/ff343074b2a3fdaa5f120600e28717e366bceadd Review URL: https://codereview.chromium.org/678953002
* simple cl to cleanup friends in GrGpuGravatar joshualitt2014-11-07
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/709003006
* cleaning up geometry handling in gpuGravatar joshualitt2014-11-07
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/707953004
* Update GrTRecorder to preserve memory allocationsGravatar cdalton2014-11-07
| | | | | | | | | | | | Rather than freeing almost all of its memory on calls to reset(), this change updates GrTRecorder so it keeps around enough to satisfy the storage requirements from last time, plus up to ~50% growth. This is based on the assumption that subsequent draw calls require roughly the same amount of memory. BUG=skia: Review URL: https://codereview.chromium.org/684203003
* Revert of SkDrawCommand scrubbing (patchset #2 id:20001 of ↵Gravatar fmalita2014-11-07
| | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/706363002/) Reason for revert: Canary borkage. Original issue's description: > SkDrawCommand scrubbing > > Remove unused ctor, constify, etc. > > R=robertphillips@google.com > > Committed: https://skia.googlesource.com/skia/+/1931ec5b5dac68f1e452af0c65161bdce35b2dec TBR=robertphillips@google.com NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/712683002
* SkDrawCommand scrubbingGravatar fmalita2014-11-07
| | | | | | | | Remove unused ctor, constify, etc. R=robertphillips@google.com Review URL: https://codereview.chromium.org/706363002
* Revert of Default geometry processor (patchset #9 id:160001 of ↵Gravatar joshualitt2014-11-07
| | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/678953002/) Reason for revert: breaks nexus 5 Original issue's description: > Default geometry processor > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/ff343074b2a3fdaa5f120600e28717e366bceadd TBR=bsalomon@google.com,joshualitt@chromium.org NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/691313003
* Default geometry processorGravatar joshualitt2014-11-07
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/678953002
* add patch and clicktracking to luaGravatar reed2014-11-07
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/712613002
* Mesa ES 3.0 requires sized internal formatsGravatar derekf2014-11-07
| | | | | | | Mesa's ES 3.0 implementation requires GL_R8 as an internal format instead of GL_RED Review URL: https://codereview.chromium.org/705183002
* Refactor DrawTarget and GPU to be independentGravatar joshualitt2014-11-07
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/705593002
* Get gpudft support working in dm, gm, nanobench and bench_picturesGravatar jvanverth2014-11-07
| | | | | | | | | | | | Adds a new config to test distance field text. Clean up some flags and #defines to read "distance field text", not "distance field fonts" to be consistent with Chromium NOTREECHECKS=true Committed: https://skia.googlesource.com/skia/+/06ba179838ba4fe187cf290750aeeb4a02a2960b Review URL: https://codereview.chromium.org/699453005
* for X in {Dec,BigDec,Scalar}, SkWStream::write"X"AsText no longer mallocsGravatar halcanary2014-11-07
| | | | Review URL: https://codereview.chromium.org/706063002
* Remove 1d glyph positions from nvpr textGravatar cdalton2014-11-07
| | | | | | | | | | | | | Removes the case for x-only glyph positions from nvpr text, opting to always send 2d glyph positions instead. The 1d glyph positions saved a bit on memory bandwidth, but ended up a net loss because they required more updates to the view matrix. Now we can draw an entire paragraph without touching the GL state, whereas before we would have to update the view matrix at every new line. BUG=skia: Review URL: https://codereview.chromium.org/700283002
* Revert of Get gpudft support working in dm, gm, nanobench and bench_pictures ↵Gravatar jvanverth2014-11-06
| | | | | | | | | | | | | | | | | | | | | | | | (patchset #2 id:20001 of https://codereview.chromium.org/699453005/) Reason for revert: Not compiling in ANGLE build Original issue's description: > Get gpudft support working in dm, gm, nanobench and bench_pictures > > Adds a new config to test distance field text. > Clean up some flags and #defines to read "distance field text", > not "distance field fonts" to be consistent with Chromium > > NOTREECHECKS=true > > Committed: https://skia.googlesource.com/skia/+/06ba179838ba4fe187cf290750aeeb4a02a2960b TBR=bsalomon@google.com,mtklein@google.com,reed@google.com NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/707723005
* Get gpudft support working in dm, gm, nanobench and bench_picturesGravatar jvanverth2014-11-06
| | | | | | | | | | Adds a new config to test distance field text. Clean up some flags and #defines to read "distance field text", not "distance field fonts" to be consistent with Chromium NOTREECHECKS=true Review URL: https://codereview.chromium.org/699453005
* Clip in grdrawtargetGravatar joshualitt2014-11-06
| | | | | | | NOTREECHECKS=true BUG=skia: Review URL: https://codereview.chromium.org/685883003
* Send coverage VA as floats instead of bytesGravatar egdaniel2014-11-06
| | | | | | | | NOTREECHECKS=True BUG=skia: Review URL: https://codereview.chromium.org/700943005
* Add mock context and use in ResourceCacheTest.Gravatar bsalomon2014-11-05
| | | | | | | | BUG=skia:2889 Committed: https://skia.googlesource.com/skia/+/820dd6c335411aad889c1d7e8a857642ecd87e30 Review URL: https://codereview.chromium.org/702083003
* Revert of Add mock context and use in ResourceCacheTest. (patchset #4 ↵Gravatar bsalomon2014-11-05
| | | | | | | | | | | | | | | | | | | | | id:60001 of https://codereview.chromium.org/702083003/) Reason for revert: Breaking tests Original issue's description: > Add mock context and use in ResourceCacheTest. > > BUG=skia:2889 > > Committed: https://skia.googlesource.com/skia/+/820dd6c335411aad889c1d7e8a857642ecd87e30 TBR=robertphillips@google.com NOTREECHECKS=true NOTRY=true BUG=skia:2889 Review URL: https://codereview.chromium.org/704563004
* Add mock context and use in ResourceCacheTest.Gravatar bsalomon2014-11-05
| | | | | | BUG=skia:2889 Review URL: https://codereview.chromium.org/702083003
* Avoid dec = -dec overflow when appending most negative signed integers.Gravatar mtklein2014-11-05
| | | | | | BUG=skia:3096 Review URL: https://codereview.chromium.org/700953003
* Whitelist intentionally racy TRACE_EVENT reads and writes.Gravatar mtklein2014-11-05
| | | | | | | | | | | | | | | | | | | | Chrome's tracing framework appears to be intentionally racy on its quick-reject checks, trading some data loss for better performance when disabled. People will never notice the data loss, but TSAN does. Let's assuage TSAN with some annotations. The 'volatile' val in SK_ANNOTATE_UNPROTECTED_WRITE was making this not compile, but that volatile doesn't really make sense there: the value we're writing is not what we care about, it's the destination. CQ_EXTRA_TRYBOTS=client.skia:Test-Ubuntu13.10-GCE-NoGPU-x86_64-Release-TSAN-Trybot No API changes. TBR=reed BUG=skia: Review URL: https://codereview.chromium.org/702883002
* Revert of Bisect Mac performance regression. (patchset #6 id:100001 of ↵Gravatar egdaniel2014-11-05
| | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/693213002/) Reason for revert: Block deps roll by breaking webkit test hit-test-counts on mac. Original issue's description: > With https://chromium.googlesource.com/skia/+/43b8b36b20ae00e2d78421c4cda1f3f922983a20 blink_perf.layout regressed. It appears that the typeface cache is being missed. This is a partial revert of the Skia change which reverts the smallest amount of code to restore performance. > > BUG=chromium:425566 > > Committed: https://skia.googlesource.com/skia/+/47eda52e8fb88f0725b401762a3e0bddc98e9fb3 TBR=reed@google.com,bungeman@google.com NOTREECHECKS=true NOTRY=true BUG=chromium:425566 Review URL: https://codereview.chromium.org/683753003
* Workaround for PowerVR clear issue.Gravatar bsalomon2014-11-05
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/701573002
* Bug fix for cull_line in SkDashPathEffect.Gravatar egdaniel2014-11-05
| | | | | | | | | | | In cull_line we must also check if both points are the same. Otherwise we fail the assert in the else "SkASSERT(dy && !dx)". This is currently blocking the roll as it fails a webkit test. BUG=skia: Review URL: https://codereview.chromium.org/703783002
* Crop the fast path dashed lines to the cull rectGravatar robertphillips2014-11-04
| | | | | | | | | | | | | | | | | | | | | | | Without: maxrss loops min median mean max stddev samples config bench 56M 1 13.3ms 13.6ms 13.6ms 14.2ms 2% Ooooo..... 8888 GM_dashing5_bw 56M 13 390us 417us 416us 459us 5% ooooO..o.o gpu GM_dashing5_bw 56M 1 13.4ms 13.9ms 14.1ms 15ms 3% Oooo..ooOo 8888 GM_dashing5_aa 56M 13 402us 421us 416us 425us 2% Ooo.ooOOOO gpu GM_dashing5_aa With: 40M 1 1.53ms 1.54ms 1.54ms 1.55ms 0% oo.O...o.. 8888 GM_dashing5_bw 40M 12 407us 412us 415us 445us 3% ...Oo..... gpu GM_dashing5_bw 40M 1 1.7ms 1.7ms 1.7ms 1.72ms 0% o.O....... 8888 GM_dashing5_aa 43M 13 405us 409us 409us 415us 1% ooo.Ooo..o gpu GM_dashing5_aa The GM images (including the new one) are the same with and without this CL. BUG=428296 Review URL: https://codereview.chromium.org/699623003
* DFText: fix placement of glyphs when we fall back to pathsGravatar jvanverth2014-11-04
| | | | | | | | Rendering a glyph with a path wants to place it at the (sx, sy) we get as input to the method, but we add (dx, dy) for the clipRect check. Hence, we need to subtract that out before we render the path. Review URL: https://codereview.chromium.org/699283003
* update slidesGravatar reed2014-11-04
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/686853005
* Revert of Override SkCanvas::drawImage() in SkDeferredCanvas and SkGPipe ↵Gravatar piotaixr2014-11-04
| | | | | | | | | | | | | | | | | | | | | | (patchset #8 id:140001 of https://codereview.chromium.org/613673005/) Reason for revert: The patch is breaking the telemetry blink build on some windows configurations Original issue's description: > Override SkCanvas::drawImage() in SkDeferredCanvas and SkGPipe > > Depend on https://codereview.chromium.org/663233002 > > BUG=skia:2947 > > Committed: https://skia.googlesource.com/skia/+/687732fe046c9cfec940f2d7f661cd97651d10a8 > > Committed: https://skia.googlesource.com/skia/+/9bb7539a59ab15749fe26fecfec05330cffae684 BUG=skia:2947, 426708 Review URL: https://codereview.chromium.org/705633002
* Add tracing to the SkCanvas drawFoo() methods to find long draw ops.Gravatar danakj2014-11-04
| | | | | | BUG=skia:3088 Review URL: https://codereview.chromium.org/702473004
* add more typeface methods to luaGravatar reed2014-11-04
| | | | | | | | BUG=skia: NOTRY=True TBR= Review URL: https://codereview.chromium.org/697053004
* Allow all font weights on Android and respect overrides.Gravatar bungeman2014-11-04
| | | | | | | This removes the code which forces 400 and 700 weights only, and also overrides the font weight with the configured weight. Review URL: https://codereview.chromium.org/694533006
* With ↵Gravatar bungeman2014-11-04
| | | | | | | | https://chromium.googlesource.com/skia/+/43b8b36b20ae00e2d78421c4cda1f3f922983a20 blink_perf.layout regressed. It appears that the typeface cache is being missed. This is a partial revert of the Skia change which reverts the smallest amount of code to restore performance. BUG=chromium:425566 Review URL: https://codereview.chromium.org/693213002
* Include SkTypes so that SK_SUPPORT_GPU is meaningful.Gravatar scroggo2014-11-04
| | | | | | | | | | For the Android framework build, we get our defines from SkUserConfig, rather than from the makefile, so we need to include it (via SkTypes) before we can use our defines. Fixes Android framework build. Review URL: https://codereview.chromium.org/700893002
* Enable distance field path rendering in Chrome.Gravatar jvanverth2014-11-04
| | | | | | | | | | - Add stat tracking for df-renderable paths - Modify GPU rasterization veto to account for df-renderable paths - Remove #define hiding df paths from Chromium and Android BUG=skia:2935 Review URL: https://codereview.chromium.org/685113003
* removing setVertexArraySource from drawtargetGravatar joshualitt2014-11-04
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/699733002
* Fix layer cache memory leakGravatar robertphillips2014-11-04
| | | | | | The issue was that, with caching disabled, the layer cache code was removing layers outside of a purge (i.e., in unlock) but not correctly cleaning up the GrPictureInfo objects (as is done in purgePlot). Review URL: https://codereview.chromium.org/703643002
* remove dead SK_DISABLE_DASHING_OPTIMIZATION flagGravatar reed2014-11-04
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/698003004
* update slide contentGravatar reed2014-11-03
| | | | | | | BUG=skia: TBR= Review URL: https://codereview.chromium.org/698563004
* add textblobs to luaGravatar reed2014-11-03
| | | | | | | BUG=skia: TBR= Review URL: https://codereview.chromium.org/688363003
* Turn SkTaskGroups back on.Gravatar mtklein2014-11-03
| | | | | | | | | | | | | Revert "Disable SkTaskGroup in SkMultiPictureDraw temporarily." Revert "Revert harder, removing SkTaskGroup.cpp from core temporarily." NOTREECHECKS=true BUG=skia: Committed: https://skia.googlesource.com/skia/+/2100c5ed7a5e5470a04e7af7309d8bd3fc4249f7 Review URL: https://codereview.chromium.org/687263007
* Manually load CONDITION_VARIABLE methods on Windows, checking for failure (XP).Gravatar mtklein2014-11-03
| | | | | | | | | | | | Tested by running DM on XP. Before this patch, it fails at startup (even just out/Debug/dm --help). Now it asserts for other reasons later on in user code, which is just fine by me. The net effect is that SkTaskGroups will always be synchronous on XP. That's not ideal, but a step up from crashing. CQ_EXTRA_TRYBOTS=client.skia:Test-Win7-ShuttleA-HD2000-x86-Release-Trybot,Test-Win7-ShuttleA-HD2000-x86_64-Release-Trybot BUG=skia: Review URL: https://codereview.chromium.org/700683002
* move SkTextBox into utilsGravatar reed2014-11-03
| | | | | | | BUG=skia: TBR= Review URL: https://codereview.chromium.org/696293005
* Rename and clarify semantics of GrContext::resolveRenderTarget.Gravatar bsalomon2014-11-03
| | | | Review URL: https://codereview.chromium.org/696293004
* Temporary fix to remove drawrect call from GpuGLGravatar joshualitt2014-11-03
| | | | | | | | BUG=skia: Committed: https://skia.googlesource.com/skia/+/d4a5c2028117c100ccf44263c0118a0b4745f627 Review URL: https://codereview.chromium.org/694933002
* Improve quality of distance field renderingGravatar jvanverth2014-11-03
| | | | | | | | | | - Update spacing of LODs to get better results scaling up to 256 - Remove unnecessary "bolding" from dftext - Add debug colors for dftext LODs BUG=skia:2933,skia:2935 Review URL: https://codereview.chromium.org/703463002
* Detect Mesa OpenGL ESGravatar derekf2014-11-03
| | | | | | Original-Author: Henry Song <henrysong@samsung.com> Review URL: https://codereview.chromium.org/700633002
* Revert changes to GrContext::resolveRenderTargetGravatar bsalomon2014-11-03
| | | | | | TBR=joshualitt@google.com,egdaniel@google.com Review URL: https://codereview.chromium.org/700483003