| Commit message (Collapse) | Author | Age |
|
|
|
|
|
|
|
| |
BUG=None
TEST=./run_server.sh --help and ./run_server.sh -public
R=borenet@google.com
Review URL: https://codereview.chromium.org/649303003
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
#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
|
|
|
|
|
|
|
|
| |
A few references to "profile" still existed in SkColorCubeFilter, so I removed them.
BUG=skia:
Review URL: https://codereview.chromium.org/651303002
|
|
|
|
|
|
| |
BUG=skia:2443
Review URL: https://codereview.chromium.org/653153003
|
|
|
|
|
|
| |
TBR=
Review URL: https://codereview.chromium.org/657603005
|
|
|
|
|
|
| |
BUG=skia:2922
Review URL: https://codereview.chromium.org/638293004
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
| |
Rename GrContext::lockAndRefScratchTexture to refScratchTexture.
GrSurface::writePixels returns bool instead of void.
BUG=skia:2889
Review URL: https://codereview.chromium.org/638403003
|
|
|
|
|
|
| |
BUG=416289
Review URL: https://codereview.chromium.org/636233008
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/651823004
|
|
|
|
|
|
|
|
|
| |
Preliminary work for getting color emoji working with
distance field text.
BUG=skia:2887
Review URL: https://codereview.chromium.org/650273003
|
|
|
|
|
|
|
|
| |
of FP creation in SkImageFilter
BUG=skia:2922
Review URL: https://codereview.chromium.org/656853002
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/652273002
|
|
|
|
|
|
| |
BUG=skia:3012
Review URL: https://codereview.chromium.org/650223004
|
|
|
|
|
|
| |
I'm tired of readding this capability whenever I need to debug.
Review URL: https://codereview.chromium.org/654653006
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/647863002
|
|
|
|
|
|
| |
BUG=skia:761
Review URL: https://codereview.chromium.org/655643002
|
|
|
|
|
|
|
| |
BUG=skia:
TBR=
Review URL: https://codereview.chromium.org/643113005
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
| |
BUG=skia:
R=mtklein@google.com, tfarina@chromium.org
Review URL: https://codereview.chromium.org/649313002
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
#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
|
|
|
|
|
|
|
|
| |
BUG=None
TEST=None
R=borenet@google.com, rmistry@google.com
Review URL: https://codereview.chromium.org/646853002
|
|
|
|
|
|
|
| |
BUG=skia:
TBR=
Review URL: https://codereview.chromium.org/651023002
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/643743003
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
| |
BUG=skia:2887
Review URL: https://codereview.chromium.org/656433002
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/650263002
|
|
|
|
|
|
|
|
|
| |
BUG=None
TEST=NOne
R=humper@google.com
TBR=humper@google.com
Review URL: https://codereview.chromium.org/648563003
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
| |
BUG=skia:
R=jcgregorio@google.com
Review URL: https://codereview.chromium.org/656463002
|
|
|
|
|
|
|
|
| |
BUG=skia:761
TBR=rmistry
NOTREECHECKS=true
Review URL: https://codereview.chromium.org/621353002
|
|
|
|
|
|
|
|
| |
BUG=skia:2973
TBR=
NOTREECHECKS=true
Review URL: https://codereview.chromium.org/654463003
|
|
|
|
|
|
|
| |
TBR=
BUG=skia:
Review URL: https://codereview.chromium.org/646613004
|
|
|
|
|
|
|
| |
BUG=skia:
TBR=
Review URL: https://codereview.chromium.org/648083002
|
|
|
|
|
|
|
| |
BUG=skia:
TBR=
Review URL: https://codereview.chromium.org/649973002
|
|
|
|
|
|
| |
TBR=joshualitt@google.com
Review URL: https://codereview.chromium.org/648063002
|
|
|
|
|
|
|
| |
BUG=skia:
TBR=
Review URL: https://codereview.chromium.org/652473002
|
|
|
|
|
|
|
| |
BUG=skia:
TBR=
Review URL: https://codereview.chromium.org/649013002
|
|
|
|
|
|
|
| |
BUG=skia:
TBR=
Review URL: https://codereview.chromium.org/645283002
|