bsalomon [Thu, 16 Oct 2014 20:41:43 +0000 (13:41 -0700)]
Remove AutoScratchTexture
Remove AST from GrClipMaskCache
Remove AST from GrClipMaskManager
BUG=skia:2889
R=robertphillips@google.com
Review URL: https://codereview.chromium.org/
663583002
sugoi [Thu, 16 Oct 2014 20:10:57 +0000 (13:10 -0700)]
JPEG YUV Decoding
Enabling JPEG YUV Decoding in Skia
BUG=skia:3005, skia:1674, skia:3029
Committed: https://skia.googlesource.com/skia/+/
8e6c3b93a39e19111662a760ede97df55e51d39f
Review URL: https://codereview.chromium.org/
399683007
joshualitt [Thu, 16 Oct 2014 19:25:11 +0000 (12:25 -0700)]
Support GLSL es 3.00
BUG=skia:
Review URL: https://codereview.chromium.org/
659443007
piotaixr [Thu, 16 Oct 2014 18:58:39 +0000 (11:58 -0700)]
Override SkCanvas::drawImage() in SkRecorder.
BUG=skia:2947
Review URL: https://codereview.chromium.org/
610003002
piotaixr [Thu, 16 Oct 2014 18:55:35 +0000 (11:55 -0700)]
Override SkCanvas::drawImage() in SkDeferredCanvas and SkGPipe
BUG=skia:2947
Review URL: https://codereview.chromium.org/
613673005
fmalita [Thu, 16 Oct 2014 18:53:30 +0000 (11:53 -0700)]
Implicit SkTextBlob bounds
Compute blob bounds when the client doesn't pass explicit run rects to
the builder. This allows us to remove a couple of internal workarounds.
R=reed@google.com,mtklein@google.com,bungeman@google.com
Review URL: https://codereview.chromium.org/
654873003
Greg Humphreys [Thu, 16 Oct 2014 18:23:02 +0000 (14:23 -0400)]
fix display of fiddles that DON'T have compile errors
BUG=skia:
Review URL: https://codereview.chromium.org/
644253003
halcanary [Thu, 16 Oct 2014 17:32:52 +0000 (10:32 -0700)]
render_pdfs cull rect output formated better
Review URL: https://codereview.chromium.org/
657353004
halcanary [Thu, 16 Oct 2014 16:53:20 +0000 (09:53 -0700)]
Const-correctness in PDF drawText() call
If SkPDFDevice::drawText is called with glyph encoding and a font with
cubic outlines (CFF or Type1) and a glyph code > 255 then it was
mutating the in(const) glyph codes in place. Instead, a copy is made.
BUG=skia:3019
Review URL: https://codereview.chromium.org/
656143002
mtklein [Thu, 16 Oct 2014 16:23:21 +0000 (09:23 -0700)]
Tweak out SkTileGrid::insert() loop.
BUG=skia:
Review URL: https://codereview.chromium.org/
658913002
mtklein [Thu, 16 Oct 2014 16:23:17 +0000 (09:23 -0700)]
Pull nanomsg from our mirror instead of from github.
https://skia.googlesource.com/third_party/nanomsg/
BUG=skia:3027
Review URL: https://codereview.chromium.org/
657113004
humper [Thu, 16 Oct 2014 16:18:16 +0000 (09:18 -0700)]
Properly parse compiler errors for fiddles and mark them in the editor.
You can also click on the errors in the parsed output to jump directly to the offending location in the editor.
BUG=skia:
NOTRY=true
Review URL: https://codereview.chromium.org/
660573003
bsalomon [Thu, 16 Oct 2014 16:18:09 +0000 (09:18 -0700)]
Auto-compare GrProcessors' texture accesses in isEqual().
R=joshualitt@google.com
Review URL: https://codereview.chromium.org/
654313002
rmistry [Thu, 16 Oct 2014 15:37:31 +0000 (08:37 -0700)]
Whitespace change to verify lab machines can now sync
TBR=
BUG=skia:3031
NOTRY=true
NOTREECHECKS=true
Review URL: https://codereview.chromium.org/
664533002
jvanverth [Thu, 16 Oct 2014 15:04:39 +0000 (08:04 -0700)]
Revert of Change drawText() to generate positions and send to drawPosText() (patchset #4 id:60001 of https://codereview.chromium.org/
653133004/)
Reason for revert:
A large number of GMs on Ubuntu12 are failing. The text layout on GPU is visibly different than that for 8888.
Original issue's description:
> Change drawText() to generate positions and send to drawPosText()
>
> The idea here is to have a central place that does layout for drawText(), and
> then always feed text through drawPosText(). This both makes all of the
> GrTextContexts consistent in drawText() output, and does a better job of
> stressing drawPosText().
>
> Because of the effect of matrices on hinting and approximation error, the
> generated text is not 100% identical to that produced by the raster pipeline.
>
> BUG=skia:2778
>
> Committed: https://skia.googlesource.com/skia/+/
7851a56895c9c076f73a835a7dd51d3c6180c16f
TBR=cdalton.nvidia@gmail.com,bungeman@google.com,reed@google.com
NOTREECHECKS=true
NOTRY=true
BUG=skia:2778
Review URL: https://codereview.chromium.org/
659993003
jvanverth [Thu, 16 Oct 2014 13:20:35 +0000 (06:20 -0700)]
Change drawText() to generate positions and send to drawPosText()
The idea here is to have a central place that does layout for drawText(), and
then always feed text through drawPosText(). This both makes all of the
GrTextContexts consistent in drawText() output, and does a better job of
stressing drawPosText().
Because of the effect of matrices on hinting and approximation error, the
generated text is not 100% identical to that produced by the raster pipeline.
BUG=skia:2778
Review URL: https://codereview.chromium.org/
653133004
tfarina [Thu, 16 Oct 2014 12:57:39 +0000 (05:57 -0700)]
webtry: Do not forget the close the rows.
This should prevent further enumeration according to the docs.
http://golang.org/pkg/database/sql/#Rows.Close
BUG=None
TEST=None
R=humper@google.com
Review URL: https://codereview.chromium.org/
638373003
sugoi [Thu, 16 Oct 2014 12:19:31 +0000 (05:19 -0700)]
Disabled YUV decoding for subsets
Image subsets were always showing the same tile and were causing an image to be fully YUV decoded once per tile, which was both slow and wrong.
BUG=413001, 419718
Review URL: https://codereview.chromium.org/
661483002
kkinnunen [Thu, 16 Oct 2014 06:03:54 +0000 (23:03 -0700)]
Refactor SkGLContext to be actually extendable
Refactor SkGLContext to be actually extendable. Before, non-trivial subclass
would need to destroy the GL connection upon running the destructor. However,
the base class would run GL commands in its own destructor (with destroyed GL
connection)
Refactor so that SkGLContext subclass object creation is completely done by
the factory function. If the factory function returns a non-NULL ptr, it means the context
is usable.
The destruction is done with the destructor instead of virtual function called
upon destruction. Make the destructors not to call virtual functions, for
clarity.
Remove custom 1x1 FBO setup code from the base class. It appears not to be used
anymore.
BUG=skia:2992
Review URL: https://codereview.chromium.org/
640283004
bsalomon [Thu, 16 Oct 2014 02:06:21 +0000 (19:06 -0700)]
Make GrFragmentProcessor auto-compare coord xforms.
Review URL: https://codereview.chromium.org/
657923002
bsalomon [Thu, 16 Oct 2014 02:04:14 +0000 (19:04 -0700)]
Prefer to recycle non-RT scratch textures that don't have pending IO
BUG=skia:2889
Review URL: https://codereview.chromium.org/
650283002
bsalomon [Thu, 16 Oct 2014 01:34:46 +0000 (18:34 -0700)]
Last round of effect->processor (for now)
R=joshualitt@google.com
Review URL: https://codereview.chromium.org/
659803005
Thiago Farina [Wed, 15 Oct 2014 22:20:27 +0000 (19:20 -0300)]
webtry: Forgot onr log.Fatal.
Update it to use glog variant.
BUG=skia:2980
TEST=go build webtry.go
TBR=jcgregorio@google.com
Review URL: https://codereview.chromium.org/
659813003
tfarina [Wed, 15 Oct 2014 21:59:33 +0000 (14:59 -0700)]
webtry: Switch from Go's log package to golang/glog.
BUG=skia:2980
TEST=go get -d && go build webtry.go
R=jcgregorio@google.com
Review URL: https://codereview.chromium.org/
655323002
mtklein [Wed, 15 Oct 2014 21:20:00 +0000 (14:20 -0700)]
Treat (private, internal) grid bounds as doubly-inclusive in SkTileGrid.
The net effect is that two "+1" instructions are removed from insert().
search() nets no change: two +1 removed, two +1 added.
When vectorized, this removes the need to add in userToGrid() at all and
so the need to read an awkward {0, 0, 1, 1} constant from memory. Mostly
the benefit is less vector code to look at and think about.
BUG=skia:
Review URL: https://codereview.chromium.org/
659823004
bungeman [Wed, 15 Oct 2014 20:53:54 +0000 (13:53 -0700)]
Don't use 'defined' in macro expansion.
A careful reading of the preprocessor specification indicates that
any use of the 'defined' operator outside the form of 'defined X' or
'defined ( X )' directly in the constant expression of a '#if' or
'#elif' may cause undefined behavior.
In particular, msvc is very unpredictable. The 'defined X' and
'defined ( X )' forms behave differently when created from marco
expansion, with 'defined ( X )' generally evaluating to '0L'. The
'defined X' form generally behaves more the way one would expect,
but still has a number of quirks which should simply be considered
undefined behavior.
BUG=chromium:419245
Review URL: https://codereview.chromium.org/
657183002
egdaniel [Wed, 15 Oct 2014 20:49:02 +0000 (13:49 -0700)]
Move willUseInputColor check to computeInvariantOutput
BUG=skia:
Review URL: https://codereview.chromium.org/
656503002
tfarina [Wed, 15 Oct 2014 20:48:47 +0000 (13:48 -0700)]
webtry: Check the result of os.Chdir().
As mentioned in http://golang.org/doc/effective_go.html#blank_assign,
errors are provided for a reason, so always check them.
The documentation for this function is available at
http://golang.org/pkg/os/#Chdir.
BUG=None
TEST=None
R=jcgregorio@google.com
Review URL: https://codereview.chromium.org/
651343002
cdalton [Wed, 15 Oct 2014 20:43:48 +0000 (13:43 -0700)]
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
Review URL: https://codereview.chromium.org/
628453002
scroggo [Wed, 15 Oct 2014 20:43:36 +0000 (13:43 -0700)]
Remove unused NeedsDeepCopy.
It was used to check to see if an SkPaint had mutable fields on it.
Now that all the fields are immutable, this function is no longer
used. Remove it.
BUG=skia:2097
Review URL: https://codereview.chromium.org/
661543002
rmistry [Wed, 15 Oct 2014 20:15:58 +0000 (13:15 -0700)]
Revert of JPEG YUV Decoding (patchset #5 id:180001 of https://codereview.chromium.org/
399683007/)
Reason for revert:
Caused many test bots to go red:
http://build.chromium.org/p/client.skia/builders/Test-ChromeOS-Alex-GMA3150-x86-Release/builds/22/steps/dm/logs/stdio
http://build.chromium.org/p/client.skia/builders/Test-ChromeOS-Link-HD4000-x86_64-Debug/builds/168/steps/dm/logs/stdio
http://build.chromium.org/p/client.skia/builders/Test-Ubuntu12-ShuttleA-GTX660-x86-Release/builds/38/steps/dm/logs/stdio
Original issue's description:
> JPEG YUV Decoding
>
> Enabling JPEG YUV Decoding in Skia
>
> BUG=skia:3005, skia:1674
>
> Committed: https://skia.googlesource.com/skia/+/
8e6c3b93a39e19111662a760ede97df55e51d39f
TBR=reed@google.com,scroggo@google.com,sugoi@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=skia:3005, skia:1674
Review URL: https://codereview.chromium.org/
656163002
rmistry [Wed, 15 Oct 2014 18:58:48 +0000 (11:58 -0700)]
Whitespace change to confirm Google Storage uploads are working
BUG=skia:
TBR=
NOTRY=true
NOTREECHECKS=true
Review URL: https://codereview.chromium.org/
657083004
bsalomon [Wed, 15 Oct 2014 18:25:21 +0000 (11:25 -0700)]
More effect->processor cleanup
R=joshualitt@google.com
Review URL: https://codereview.chromium.org/
660563003
bsalomon [Wed, 15 Oct 2014 18:05:26 +0000 (11:05 -0700)]
Some cleanup in processor header files.
R=joshualitt@google.com
Review URL: https://codereview.chromium.org/
659803004
sugoi [Wed, 15 Oct 2014 18:04:17 +0000 (11:04 -0700)]
JPEG YUV Decoding
Enabling JPEG YUV Decoding in Skia
BUG=skia:3005, skia:1674
Review URL: https://codereview.chromium.org/
399683007
bsalomon [Wed, 15 Oct 2014 17:50:36 +0000 (10:50 -0700)]
Split GrFragmentProcessor into its own header
Review URL: https://codereview.chromium.org/
660573002
reed [Wed, 15 Oct 2014 16:00:27 +0000 (09:00 -0700)]
update dox for bounds in supersampler
BUG=skia:
TBR=caryclark@google.com
NOTRY=True
Review URL: https://codereview.chromium.org/
657793003
reed [Wed, 15 Oct 2014 15:52:00 +0000 (08:52 -0700)]
interesct path bounds with clip bounds before initializing supersampler
BUG=skia:
Review URL: https://codereview.chromium.org/
656473004
bsalomon [Wed, 15 Oct 2014 15:19:04 +0000 (08:19 -0700)]
Push isEqual/onIsEqual down from GrProcessor to subclasses.
R=joshualitt@google.com
Review URL: https://codereview.chromium.org/
654273002
reed [Wed, 15 Oct 2014 14:05:09 +0000 (07:05 -0700)]
add tests for large cordinates bounds when building aaclip
BUG=skia:
TBR=
Review URL: https://codereview.chromium.org/
658753002
tfarina [Wed, 15 Oct 2014 03:14:03 +0000 (20:14 -0700)]
bug_chomper: Consolidate public flag in a single place.
BUG=None
TEST=./run_server.sh --help and ./run_server.sh -public
R=borenet@google.com
Review URL: https://codereview.chromium.org/
649303003
mtklein [Tue, 14 Oct 2014 21:29:30 +0000 (14:29 -0700)]
Revert of Create a single command buffer for GrInOrderDrawBuffer (patchset #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
sugoi [Tue, 14 Oct 2014 20:15:58 +0000 (13:15 -0700)]
Changed "Profile" to "Cube" in ColorCube
A few references to "profile" still existed in SkColorCubeFilter, so I removed them.
BUG=skia:
Review URL: https://codereview.chromium.org/
651303002
Brian Salomon [Tue, 14 Oct 2014 20:12:23 +0000 (16:12 -0400)]
Ignore ubuntu nvprmsaa4/mixed_xfermodes GM result.
BUG=skia:2443
Review URL: https://codereview.chromium.org/
653153003
Brian Salomon [Tue, 14 Oct 2014 19:52:46 +0000 (15:52 -0400)]
Rebaseline mixedxfermodes on ubuntu/nvprmsaa4
TBR=
Review URL: https://codereview.chromium.org/
657603005
bsalomon [Tue, 14 Oct 2014 19:29:17 +0000 (12:29 -0700)]
Only use sized format on IMG/ES3 for single channel.
BUG=skia:2922
Review URL: https://codereview.chromium.org/
638293004
cdalton [Tue, 14 Oct 2014 18:53:04 +0000 (11:53 -0700)]
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().
Review URL: https://codereview.chromium.org/
628453002
bsalomon [Tue, 14 Oct 2014 18:47:22 +0000 (11:47 -0700)]
Remove uses of GrAutoScratchTexture.
Rename GrContext::lockAndRefScratchTexture to refScratchTexture.
GrSurface::writePixels returns bool instead of void.
BUG=skia:2889
Review URL: https://codereview.chromium.org/
638403003
bsalomon [Tue, 14 Oct 2014 17:52:00 +0000 (10:52 -0700)]
Flush text contexts before drawing text as path
BUG=416289
Review URL: https://codereview.chromium.org/
636233008
tfarina [Tue, 14 Oct 2014 16:39:02 +0000 (09:39 -0700)]
webtry: Simplify template construction.
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
reed [Tue, 14 Oct 2014 16:34:52 +0000 (09:34 -0700)]
create and modify matrices in lua
BUG=skia:
Review URL: https://codereview.chromium.org/
651823004
jvanverth [Tue, 14 Oct 2014 15:45:50 +0000 (08:45 -0700)]
Change GrTextContext fallbacks to be a linked list chain.
Preliminary work for getting color emoji working with
distance field text.
BUG=skia:2887
Review URL: https://codereview.chromium.org/
650273003
bsalomon [Tue, 14 Oct 2014 15:40:57 +0000 (08:40 -0700)]
Use GL_R8 rather than GL_RED for glTexImage2D on IMG/ES3. Check for failure of FP creation in SkImageFilter
BUG=skia:2922
Review URL: https://codereview.chromium.org/
656853002
mtklein [Tue, 14 Oct 2014 15:40:43 +0000 (08:40 -0700)]
nanobench: flush after recording every Nth data point.
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
reed [Tue, 14 Oct 2014 15:26:33 +0000 (08:26 -0700)]
add gamma value to deviceproperties
BUG=skia:
Review URL: https://codereview.chromium.org/
652273002
borenet [Tue, 14 Oct 2014 15:05:59 +0000 (08:05 -0700)]
Adjust skimage expectations for Android reference device
BUG=skia:3012
Review URL: https://codereview.chromium.org/
650223004
robertphillips [Tue, 14 Oct 2014 14:07:59 +0000 (07:07 -0700)]
Add GrLayerCache::writeLayersToDisk
I'm tired of readding this capability whenever I need to debug.
Review URL: https://codereview.chromium.org/
654653006
egdaniel [Tue, 14 Oct 2014 13:48:46 +0000 (06:48 -0700)]
Track if all stages modulate inputColor in computeInvariantOutput
BUG=skia:
Review URL: https://codereview.chromium.org/
647863002
borenet [Tue, 14 Oct 2014 11:44:44 +0000 (04:44 -0700)]
Fix links to skia-buildbot code in preparation for deletion
BUG=skia:761
Review URL: https://codereview.chromium.org/
655643002
reed [Tue, 14 Oct 2014 02:43:17 +0000 (19:43 -0700)]
parse outline to generate slides
BUG=skia:
TBR=
Review URL: https://codereview.chromium.org/
643113005
Greg Humphreys [Tue, 14 Oct 2014 01:52:05 +0000 (21:52 -0400)]
Re-land ganesh checkbox. I needed to increase the size of the address
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
humper [Mon, 13 Oct 2014 23:05:05 +0000 (16:05 -0700)]
Revert of add ganesh checkbox to skfiddle interface (patchset #6 id:270001 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
Greg Humphreys [Mon, 13 Oct 2014 22:12:33 +0000 (18:12 -0400)]
add ganesh checkbox to skfiddle interface
BUG=skia:
R=mtklein@google.com, tfarina@chromium.org
Review URL: https://codereview.chromium.org/
649313002
mtklein [Mon, 13 Oct 2014 21:50:41 +0000 (14:50 -0700)]
Revert of No threadsafe statics. (patchset #1 id:1 of 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
mtklein [Mon, 13 Oct 2014 21:00:42 +0000 (14:00 -0700)]
Revert of Create a single command buffer for GrInOrderDrawBuffer (patchset #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
Thiago Farina [Mon, 13 Oct 2014 20:56:30 +0000 (17:56 -0300)]
Link to skiadocs site, since that is the canonical location for documentation.
BUG=None
TEST=None
R=borenet@google.com, rmistry@google.com
Review URL: https://codereview.chromium.org/
646853002
reed [Mon, 13 Oct 2014 20:53:48 +0000 (13:53 -0700)]
experimental parser
BUG=skia:
TBR=
Review URL: https://codereview.chromium.org/
651023002
Thiago Farina [Mon, 13 Oct 2014 20:51:57 +0000 (17:51 -0300)]
Fix reference to non-existant 'tests' target.
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
mtklein [Mon, 13 Oct 2014 20:17:56 +0000 (13:17 -0700)]
Require SK_DECLARE_STATIC_LAZY_PTR is used in global scope.
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
tfarina [Mon, 13 Oct 2014 19:58:33 +0000 (12:58 -0700)]
Archive svndiff script.
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
egdaniel [Mon, 13 Oct 2014 19:53:46 +0000 (12:53 -0700)]
Create helper functions to use in computeInvariantOutput calls
BUG=skia:
Review URL: https://codereview.chromium.org/
643743003
mtklein [Mon, 13 Oct 2014 19:53:27 +0000 (12:53 -0700)]
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:
Review URL: https://codereview.chromium.org/
654663002
mtklein [Mon, 13 Oct 2014 19:48:16 +0000 (12:48 -0700)]
Fix race in supports_simd().
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
jvanverth [Mon, 13 Oct 2014 19:48:10 +0000 (12:48 -0700)]
Enable color emoji GM on Android
BUG=skia:2887
Review URL: https://codereview.chromium.org/
656433002
cdalton [Mon, 13 Oct 2014 19:43:10 +0000 (12:43 -0700)]
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().
Review URL: https://codereview.chromium.org/
628453002
tfarina [Mon, 13 Oct 2014 19:43:01 +0000 (12:43 -0700)]
bug_chomper: Make server's port configurable.
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
reed [Mon, 13 Oct 2014 19:38:04 +0000 (12:38 -0700)]
add pictures to lua
BUG=skia:
Review URL: https://codereview.chromium.org/
650263002
tfarina [Mon, 13 Oct 2014 19:37:52 +0000 (12:37 -0700)]
Fix webtry setup once instructions in README doc.
BUG=None
TEST=NOne
R=humper@google.com
TBR=humper@google.com
Review URL: https://codereview.chromium.org/
648563003
bsalomon [Mon, 13 Oct 2014 19:32:55 +0000 (12:32 -0700)]
Devirtualize read/write pixels on surface.
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
mtklein [Mon, 13 Oct 2014 19:32:34 +0000 (12:32 -0700)]
Allow GCC to build Skia on Mac.
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
Greg Humphreys [Mon, 13 Oct 2014 17:58:09 +0000 (13:58 -0400)]
add support for skfiddle width/height options
BUG=skia:
R=jcgregorio@google.com
Review URL: https://codereview.chromium.org/
656463002
borenet [Mon, 13 Oct 2014 16:52:20 +0000 (09:52 -0700)]
Whitespace change to trigger builds
BUG=skia:761
TBR=rmistry
NOTREECHECKS=true
Review URL: https://codereview.chromium.org/
621353002
rmistry [Mon, 13 Oct 2014 13:02:28 +0000 (06:02 -0700)]
Whitespace change to test the CQ's new trybots
BUG=skia:2973
TBR=
NOTREECHECKS=true
Review URL: https://codereview.chromium.org/
654463003
reed [Mon, 13 Oct 2014 02:05:52 +0000 (19:05 -0700)]
add fade_slide transition to lua slides
TBR=
BUG=skia:
Review URL: https://codereview.chromium.org/
646613004
reed [Sun, 12 Oct 2014 19:18:40 +0000 (12:18 -0700)]
add canvas:clear to lua
BUG=skia:
TBR=
Review URL: https://codereview.chromium.org/
648083002
reed [Sun, 12 Oct 2014 17:36:47 +0000 (10:36 -0700)]
add surfaces to lua
BUG=skia:
TBR=
Review URL: https://codereview.chromium.org/
649973002
bsalomon [Sun, 12 Oct 2014 12:40:00 +0000 (05:40 -0700)]
Fix leak in GLPrograms test.
TBR=joshualitt@google.com
Review URL: https://codereview.chromium.org/
648063002
reed [Sat, 11 Oct 2014 20:13:11 +0000 (13:13 -0700)]
add key handlers to lua
BUG=skia:
TBR=
Review URL: https://codereview.chromium.org/
652473002
reed [Sat, 11 Oct 2014 18:28:07 +0000 (11:28 -0700)]
allow for lua click handlers
BUG=skia:
TBR=
Review URL: https://codereview.chromium.org/
649013002
reed [Sat, 11 Oct 2014 03:31:24 +0000 (20:31 -0700)]
move test for lua into separate resource file
BUG=skia:
TBR=
Review URL: https://codereview.chromium.org/
645283002
tfarina [Sat, 11 Oct 2014 01:40:45 +0000 (18:40 -0700)]
bug_chomper: Use parenthesized declaration style for const and vars.
It seems cleaner that way and nicer to read, also avoids repeating
'const' and 'var' everytime you have to add more items.
BUG=None
TEST=./run_server.sh
R=borenet@google.com
Review URL: https://codereview.chromium.org/
645803003
joshualitt [Sat, 11 Oct 2014 00:47:00 +0000 (17:47 -0700)]
Opt state takes a GP instead of a GeometryStage
BUG=skia:
Committed: https://skia.googlesource.com/skia/+/
71856d520461ae025a0332aa0ce9735a096d9baf
Review URL: https://codereview.chromium.org/
637003003
reed [Fri, 10 Oct 2014 22:28:19 +0000 (15:28 -0700)]
add drawImageRect to lua, update sample
BUG=skia:
TBR=
Review URL: https://codereview.chromium.org/
642353005
joshualitt [Fri, 10 Oct 2014 21:11:59 +0000 (14:11 -0700)]
Revert of Opt state takes a GP instead of a GeometryStage (patchset #18 id:1500001 of https://codereview.chromium.org/
637003003/)
Reason for revert:
Revert because this may be causing a break in a chrome gpu test
Original issue's description:
> Opt state takes a GP instead of a GeometryStage
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/
71856d520461ae025a0332aa0ce9735a096d9baf
TBR=bsalomon@google.com,egdaniel@google.com,danakj@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/
647183002
tfarina [Fri, 10 Oct 2014 20:34:25 +0000 (13:34 -0700)]
Remove empty recordy.gypi file.
Fixes the TODO in that file.
BUG=None
TEST=None
R=mtklein@google.com
Review URL: https://codereview.chromium.org/
643543002
jvanverth [Fri, 10 Oct 2014 19:58:46 +0000 (12:58 -0700)]
Update coloremoji expectations
TBR=reed@google.com
BUG=skia:2887
Review URL: https://codereview.chromium.org/
649753003
robertphillips [Fri, 10 Oct 2014 18:38:29 +0000 (11:38 -0700)]
Fix bug in GrCachedLayer reuse
In the new MultiPictureDraw tests a single hoisted layer is reused multiple times. The previous plot locking scheme allowed GrCachedLayer objects to be aggressively deleted prematurely leaving the reusing GrHoistedLayer objects with dangling pointers.
This CL changes adds a new pseudo-ref to GrCachedLayer. (It can't be a real ref since the cached layers aren't deleted when it goes to 0).
NOTRY=true
Committed: https://skia.googlesource.com/skia/+/
5c481666c9678f43e039ad605457be3854cf8de3
Review URL: https://codereview.chromium.org/
640323002
jvanverth [Fri, 10 Oct 2014 18:36:12 +0000 (11:36 -0700)]
Fix color emoji.
Removes the GrMaskFormat and single atlas in GrTextStrike.
Replaces it by storing the GrMaskFormat in each GrGlyph, and
doing a lookup for the correct atlas based on that.
Disables color glyph rendering in GrDistanceFieldTextContext
for now.
BUG=skia:2887
Committed: https://skia.googlesource.com/skia/+/
bc92163ddfe957ad6ffbb02ac40e0ba75ff82216
Review URL: https://codereview.chromium.org/
636183005
joshualitt [Fri, 10 Oct 2014 16:56:55 +0000 (09:56 -0700)]
Opt state takes a GP instead of a GeometryStage
BUG=skia:
Review URL: https://codereview.chromium.org/
637003003
reed [Fri, 10 Oct 2014 16:09:52 +0000 (09:09 -0700)]
detect --loops is < 0 and interpret that as running forever (mostly)
BUG=skia:
Review URL: https://codereview.chromium.org/
643143002