aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
* bug_chomper: Consolidate public flag in a single place.Gravatar tfarina2014-10-14
| | | | | | | | BUG=None TEST=./run_server.sh --help and ./run_server.sh -public R=borenet@google.com Review URL: https://codereview.chromium.org/649303003
* Revert of Create a single command buffer for GrInOrderDrawBuffer (patchset ↵Gravatar mtklein2014-10-14
| | | | | | | | | | | | | | | | | | | | | | #17 id:1240001 of https://codereview.chromium.org/628453002/) Reason for revert: Leaking memory: http://build.chromium.org/p/client.skia/builders/Test-Ubuntu12-ShuttleA-GTX550Ti-x86_64-Release-Valgrind/builds/9/steps/gm/logs/stdio Original issue's description: > Adds a GrTRecorder class that GrInOrderDrawBuffer uses to allocate > all its commands interleaved in contiguous memory. GrTRecorder also > supports extra data associated with objects, so we can store arrays > inline without having to call malloc(). > > Committed: https://skia.googlesource.com/skia/+/360b6801cfd90485891d709e44cf395d527ba69e TBR=bsalomon@google.com,reed@google.com,cdalton@nvidia.com NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/654863003
* Changed "Profile" to "Cube" in ColorCubeGravatar sugoi2014-10-14
| | | | | | | | A few references to "profile" still existed in SkColorCubeFilter, so I removed them. BUG=skia: Review URL: https://codereview.chromium.org/651303002
* Ignore ubuntu nvprmsaa4/mixed_xfermodes GM result.Gravatar Brian Salomon2014-10-14
| | | | | | BUG=skia:2443 Review URL: https://codereview.chromium.org/653153003
* Rebaseline mixedxfermodes on ubuntu/nvprmsaa4Gravatar Brian Salomon2014-10-14
| | | | | | TBR= Review URL: https://codereview.chromium.org/657603005
* Only use sized format on IMG/ES3 for single channel.Gravatar bsalomon2014-10-14
| | | | | | BUG=skia:2922 Review URL: https://codereview.chromium.org/638293004
* Adds a GrTRecorder class that GrInOrderDrawBuffer uses to allocateGravatar cdalton2014-10-14
| | | | | | | | all its commands interleaved in contiguous memory. GrTRecorder also supports extra data associated with objects, so we can store arrays inline without having to call malloc(). Review URL: https://codereview.chromium.org/628453002
* Remove uses of GrAutoScratchTexture.Gravatar bsalomon2014-10-14
| | | | | | | | | | Rename GrContext::lockAndRefScratchTexture to refScratchTexture. GrSurface::writePixels returns bool instead of void. BUG=skia:2889 Review URL: https://codereview.chromium.org/638403003
* Flush text contexts before drawing text as pathGravatar bsalomon2014-10-14
| | | | | | BUG=416289 Review URL: https://codereview.chromium.org/636233008
* webtry: Simplify template construction.Gravatar tfarina2014-10-14
| | | | | | | | | | | | | | | | text/template and html/template both have a helper function called "Must" that according to the docs in http://golang.org/pkg/text/template/#Must and http://golang.org/pkg/html/template/#Must, wraps a call to a function returning (*Template, error) and panics if the error is non-nil. So we don't need to that ourselves. Which is a good thing and makes code cleaner. BUG=None TEST=None R=jcgregorio@google.com Review URL: https://codereview.chromium.org/642243004
* create and modify matrices in luaGravatar reed2014-10-14
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/651823004
* Change GrTextContext fallbacks to be a linked list chain.Gravatar jvanverth2014-10-14
| | | | | | | | | Preliminary work for getting color emoji working with distance field text. BUG=skia:2887 Review URL: https://codereview.chromium.org/650273003
* Use GL_R8 rather than GL_RED for glTexImage2D on IMG/ES3. Check for failure ↵Gravatar bsalomon2014-10-14
| | | | | | | | of FP creation in SkImageFilter BUG=skia:2922 Review URL: https://codereview.chromium.org/656853002
* nanobench: flush after recording every Nth data point.Gravatar mtklein2014-10-14
| | | | | | | | | | Got to keep our precious data in event of a crash. With --flushEvery 10 I'm not seeing this cost any wall time. BUG=skia: Review URL: https://codereview.chromium.org/653083003
* add gamma value to devicepropertiesGravatar reed2014-10-14
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/652273002
* Adjust skimage expectations for Android reference deviceGravatar borenet2014-10-14
| | | | | | BUG=skia:3012 Review URL: https://codereview.chromium.org/650223004
* Add GrLayerCache::writeLayersToDiskGravatar robertphillips2014-10-14
| | | | | | I'm tired of readding this capability whenever I need to debug. Review URL: https://codereview.chromium.org/654653006
* Track if all stages modulate inputColor in computeInvariantOutputGravatar egdaniel2014-10-14
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/647863002
* Fix links to skia-buildbot code in preparation for deletionGravatar borenet2014-10-14
| | | | | | BUG=skia:761 Review URL: https://codereview.chromium.org/655643002
* parse outline to generate slidesGravatar reed2014-10-13
| | | | | | | BUG=skia: TBR= Review URL: https://codereview.chromium.org/643113005
* Re-land ganesh checkbox. I needed to increase the size of the addressGravatar Greg Humphreys2014-10-13
| | | | | | | | | space available to a fiddle because linking against libGL causes the amount of memory used by a process to bloat to over 100M. Yikes. BUG=skia: Review URL: https://codereview.chromium.org/638003008
* Revert of add ganesh checkbox to skfiddle interface (patchset #6 id:270001 ↵Gravatar humper2014-10-13
| | | | | | | | | | | | | | | | | | | | | | of https://codereview.chromium.org/649313002/) Reason for revert: adding -Dskia_mesa=1 broke production; don't yet know why and don't want to leave things in a broken state. Original issue's description: > add ganesh checkbox to skfiddle interface > > BUG=skia: > R=mtklein@google.com, tfarina@chromium.org > > Committed: https://skia.googlesource.com/skia/+/ccb54895c614632fd01d22b64ab614921f980eb1 TBR=tfarina@chromium.org,jcgregorio@google.com,mtklein@google.com NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/655633003
* add ganesh checkbox to skfiddle interfaceGravatar Greg Humphreys2014-10-13
| | | | | | | BUG=skia: R=mtklein@google.com, tfarina@chromium.org Review URL: https://codereview.chromium.org/649313002
* Revert of No threadsafe statics. (patchset #1 id:1 of ↵Gravatar mtklein2014-10-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/654663002/) Reason for revert: Want to think about how to do this only to skialib and not to tools (gms, tests, etc.) Original issue's description: > No threadsafe statics. > > Chrome disables these for speed and code size, so we need > to disable them to make sure our code is safe when used > this way. > > int foo() { > static int32_t atomic_thing; > return sk_atomic_inc(&atomic_thing); > } > > is not safe in Chrome. Making the static global is: > > static int32_t atomic_thing; > int foo() { > return sk_atomic_inc(&atomic_thing); > } > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/cad5d3e264535c919b80e1e2a85407307961f221 TBR=bungeman@google.com,mtklein@chromium.org NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/649343003
* Revert of Create a single command buffer for GrInOrderDrawBuffer (patchset ↵Gravatar mtklein2014-10-13
| | | | | | | | | | | | | | | | | | | | | #14 id:1050001 of https://codereview.chromium.org/628453002/) Reason for revert: New test failing on Android: http://build.chromium.org/p/client.skia.android/builders/Test-Android-Nexus7-Tegra3-Arm7-Release/builds/89/steps/dm/logs/stdio Original issue's description: > Adds a GrTBaseList class that GrInOrderDrawBuffer uses to allocate > all its commands interleaved in contiguous memory. GrTBaseList also > supports extra data associated with objects, so we can store arrays > inline without having to call malloc(). > > Committed: https://skia.googlesource.com/skia/+/47c844aaba81e5a29c773b660e1d6062c766d253 TBR=bsalomon@google.com,reed@google.com,cdalton@nvidia.com NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/652843002
* Link to skiadocs site, since that is the canonical location for documentation.Gravatar Thiago Farina2014-10-13
| | | | | | | | BUG=None TEST=None R=borenet@google.com, rmistry@google.com Review URL: https://codereview.chromium.org/646853002
* experimental parserGravatar reed2014-10-13
| | | | | | | BUG=skia: TBR= Review URL: https://codereview.chromium.org/651023002
* Fix reference to non-existant 'tests' target.Gravatar Thiago Farina2014-10-13
| | | | | | | | | | This target was previously folded into dm. Do not advertise it anymore. BUG=None TEST=None R=borenet@google.com Review URL: https://codereview.chromium.org/646843002
* Require SK_DECLARE_STATIC_LAZY_PTR is used in global scope.Gravatar mtklein2014-10-13
| | | | | | | | | | | | | Function- or method- local scope isn't threadsafe; the pointer is generally zero-initialized on first use in function scope (i.e. lazily... we have to go deeper), but for globals we can be pretty sure the linker will do that for us. BUG=skia: No public API changes. TBR=reed@google.com Review URL: https://codereview.chromium.org/651723003
* Archive svndiff script.Gravatar tfarina2014-10-13
| | | | | | | | | | | Nobody is using this script anymore, so to prevent more changes to it, let's remove it now. BUG=None TEST=None R=borenet@google.com Review URL: https://codereview.chromium.org/646043002
* Create helper functions to use in computeInvariantOutput callsGravatar egdaniel2014-10-13
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/643743003
* No threadsafe statics.Gravatar mtklein2014-10-13
| | | | | | | | | | | | | | | | | | | | | | Chrome disables these for speed and code size, so we need to disable them to make sure our code is safe when used this way. int foo() { static int32_t atomic_thing; return sk_atomic_inc(&atomic_thing); } is not safe in Chrome. Making the static global is: static int32_t atomic_thing; int foo() { return sk_atomic_inc(&atomic_thing); } BUG=skia: Review URL: https://codereview.chromium.org/654663002
* Fix race in supports_simd().Gravatar mtklein2014-10-13
| | | | | | | | | | Local statics are not thread safe in Chrome. Use an SkLazyPtr instead. See https://code.google.com/p/chromium/issues/detail?id=418041 BUG=418041 Review URL: https://codereview.chromium.org/655573002
* Enable color emoji GM on AndroidGravatar jvanverth2014-10-13
| | | | | | BUG=skia:2887 Review URL: https://codereview.chromium.org/656433002
* Adds a GrTBaseList class that GrInOrderDrawBuffer uses to allocateGravatar cdalton2014-10-13
| | | | | | | | all its commands interleaved in contiguous memory. GrTBaseList also supports extra data associated with objects, so we can store arrays inline without having to call malloc(). Review URL: https://codereview.chromium.org/628453002
* bug_chomper: Make server's port configurable.Gravatar tfarina2014-10-13
| | | | | | | | | | | | | | Make 'port' a flag so you can change it from the command line, making the server more flexible and allowing us to change in which port it listen to requests. $ ./run_server.sh --port :8002 BUG=None TEST=see above R=borenet@google.com Review URL: https://codereview.chromium.org/649663003
* add pictures to luaGravatar reed2014-10-13
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/650263002
* Fix webtry setup once instructions in README doc.Gravatar tfarina2014-10-13
| | | | | | | | | BUG=None TEST=NOne R=humper@google.com TBR=humper@google.com Review URL: https://codereview.chromium.org/648563003
* Devirtualize read/write pixels on surface.Gravatar bsalomon2014-10-13
| | | | | | | | Consolidate read/write funcs in context. Remove support for reading pixels from a surface that's not a target. It's currently broken and neither used nor tested. Review URL: https://codereview.chromium.org/648863002
* Allow GCC to build Skia on Mac.Gravatar mtklein2014-10-13
| | | | | | | | | | | | | GCC doesn't understand -fasm-blocks or -mpascal-strings, but we don't care about them. While looking around in Gyp, I noticed a better way to disable warnings about offsetof so that it doesn't tell us "disabling this warning makes no sense in C" for every C source file we compile. BUG=skia: Review URL: https://codereview.chromium.org/650553002
* add support for skfiddle width/height optionsGravatar Greg Humphreys2014-10-13
| | | | | | | BUG=skia: R=jcgregorio@google.com Review URL: https://codereview.chromium.org/656463002
* Whitespace change to trigger buildsGravatar borenet2014-10-13
| | | | | | | | BUG=skia:761 TBR=rmistry NOTREECHECKS=true Review URL: https://codereview.chromium.org/621353002
* Whitespace change to test the CQ's new trybotsGravatar rmistry2014-10-13
| | | | | | | | BUG=skia:2973 TBR= NOTREECHECKS=true Review URL: https://codereview.chromium.org/654463003
* add fade_slide transition to lua slidesGravatar reed2014-10-12
| | | | | | | TBR= BUG=skia: Review URL: https://codereview.chromium.org/646613004
* add canvas:clear to luaGravatar reed2014-10-12
| | | | | | | BUG=skia: TBR= Review URL: https://codereview.chromium.org/648083002
* add surfaces to luaGravatar reed2014-10-12
| | | | | | | BUG=skia: TBR= Review URL: https://codereview.chromium.org/649973002
* Fix leak in GLPrograms test.Gravatar bsalomon2014-10-12
| | | | | | TBR=joshualitt@google.com Review URL: https://codereview.chromium.org/648063002
* add key handlers to luaGravatar reed2014-10-11
| | | | | | | BUG=skia: TBR= Review URL: https://codereview.chromium.org/652473002
* allow for lua click handlersGravatar reed2014-10-11
| | | | | | | BUG=skia: TBR= Review URL: https://codereview.chromium.org/649013002
* move test for lua into separate resource fileGravatar reed2014-10-10
| | | | | | | BUG=skia: TBR= Review URL: https://codereview.chromium.org/645283002