platform/upstream/libSkiaSharp.git
9 years agofaster edge re-sort, drop trailing edges
reed [Fri, 6 Feb 2015 22:18:46 +0000 (14:18 -0800)]
faster edge re-sort, drop trailing edges

1. drop edges that are wholly on the right (in the non-convex walker)
2. scan and swap once, instead of swapping as we go during re-sort

BUG=skia:

Review URL: https://codereview.chromium.org/891613003

9 years agoadd bench for very big paths
reed [Fri, 6 Feb 2015 21:04:16 +0000 (13:04 -0800)]
add bench for very big paths

BUG= 455429
TBR=

Review URL: https://codereview.chromium.org/909563002

9 years agoAdd SkSVGCanvas
fmalita [Fri, 6 Feb 2015 20:51:10 +0000 (12:51 -0800)]
Add SkSVGCanvas

Add a public SVG canvas factory + move the SVG files from
experimental -> src/svg/.

Update current clients to the new API.

R=reed@google.com, mtklein@google.com, halcanary@google.com

Review URL: https://codereview.chromium.org/902583006

9 years agoReimplement gpu message bus for invalidated bitmap gen IDs
bsalomon [Fri, 6 Feb 2015 19:54:28 +0000 (11:54 -0800)]
Reimplement gpu message bus for invalidated bitmap gen IDs

Review URL: https://codereview.chromium.org/902873002

9 years agoOne createTexture function, attempt to recycle scratch in createTexture.
bsalomon [Fri, 6 Feb 2015 16:49:24 +0000 (08:49 -0800)]
One createTexture function, attempt to recycle scratch in createTexture.

Review URL: https://codereview.chromium.org/864383003

9 years agoMove SVG parser files under {include,src}/svg/parser/
fmalita [Fri, 6 Feb 2015 16:43:51 +0000 (08:43 -0800)]
Move SVG parser files under {include,src}/svg/parser/

R=reed@google.com

Review URL: https://codereview.chromium.org/884713007

9 years agoRename SkCanvasDrawable to SkDrawable, and make public
reed [Fri, 6 Feb 2015 16:36:15 +0000 (08:36 -0800)]
Rename SkCanvasDrawable to SkDrawable, and make public

 (patchset #2 id:20001 of https://codereview.chromium.org/903993002/)"

This reverts commit c4e87724920222a218f31b22612efc5b1ec0ed6c.

BUG=skia:
TBR=
NOTREECHECKS=True

Review URL: https://codereview.chromium.org/898343004

9 years agoRevert of rename SkCanvasDrawable to SkDrawable, and make public (patchset #2 id...
reed [Fri, 6 Feb 2015 16:20:07 +0000 (08:20 -0800)]
Revert of rename SkCanvasDrawable to SkDrawable, and make public (patchset #2 id:20001 of https://codereview.chromium.org/903993002/)

Reason for revert:
bug in gyp

Original issue's description:
> rename SkCanvasDrawable to SkDrawable, and make public
>
> BUG=skia:
> NOTRY=True
> ... winbuilder flake
>
> Committed: https://skia.googlesource.com/skia/+/4ae9eb7463cf2160723407359608f221c0d5e2a6

TBR=robertphillips@google.com,djsollen@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:

Review URL: https://codereview.chromium.org/882853006

9 years agorename SkCanvasDrawable to SkDrawable, and make public
reed [Fri, 6 Feb 2015 16:02:57 +0000 (08:02 -0800)]
rename SkCanvasDrawable to SkDrawable, and make public

BUG=skia:
NOTRY=True
... winbuilder flake

Review URL: https://codereview.chromium.org/903993002

9 years agoAdd factory for Android font manager.
bungeman [Fri, 6 Feb 2015 15:59:19 +0000 (07:59 -0800)]
Add factory for Android font manager.

This allows users to create an Android font manager with their own set
of fonts, or augment the system set. This will allow for removal of
the current globals which are used for a similar, but more constained,
purpose.

BUG=skia:2817,skia:3314,chromium:407340

Review URL: https://codereview.chromium.org/887113002

9 years agoRevert of Revert of Move DstCopy on gpu into the GrXferProcessor. (patchset #1 id...
bsalomon [Fri, 6 Feb 2015 15:02:37 +0000 (07:02 -0800)]
Revert of Revert of Move DstCopy on gpu into the GrXferProcessor. (patchset #1 id:1 of https://codereview.chromium.org/901663007/)

Reason for revert:
The revert didn't help the 10.9 bot. Unreverting by reverting the revert (which is basically relanding the original patch which itself was a revert of a revert). Revert.

Original issue's description:
> Revert of Move DstCopy on gpu into the GrXferProcessor. (patchset #11 id:200001 of https://codereview.chromium.org/885923002/)
>
> Reason for revert:
> Testing to see if reverting fixes 10.9 bots.
>
> Original issue's description:
> > Move DstCopy on gpu into the GrXferProcessor.
> >
> > BUG=skia:
> >
> > Committed: https://skia.googlesource.com/skia/+/74a11753604768bf461b80cabb66060e8564d82c
> >
> > Committed: https://skia.googlesource.com/skia/+/5e1378d0e075a323144ba14e0a4cbcca35eccc69
>
> TBR=joshualitt@google.com,egdaniel@google.com
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/21b2c53218ab25f4268e3992e51d916076a2a7ee

TBR=joshualitt@google.com,egdaniel@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:

Review URL: https://codereview.chromium.org/873723009

9 years agoUpdate SKP version
skia.buildbots [Fri, 6 Feb 2015 07:00:16 +0000 (23:00 -0800)]
Update SKP version

Automatic commit by the RecreateSKPs bot.

TBR=

Review URL: https://codereview.chromium.org/901403002

9 years agomigrate more samples over to SkAnimTImer
reed [Fri, 6 Feb 2015 06:02:37 +0000 (22:02 -0800)]
migrate more samples over to SkAnimTImer

BUG=skia:
TBR=

Review URL: https://codereview.chromium.org/901933004

9 years agoConvex batch
joshualitt [Thu, 5 Feb 2015 22:39:01 +0000 (14:39 -0800)]
Convex batch

BUG=skia:

Review URL: https://codereview.chromium.org/880643002

9 years agoRevert of Move DstCopy on gpu into the GrXferProcessor. (patchset #11 id:200001 of...
bsalomon [Thu, 5 Feb 2015 22:18:05 +0000 (14:18 -0800)]
Revert of Move DstCopy on gpu into the GrXferProcessor. (patchset #11 id:200001 of https://codereview.chromium.org/885923002/)

Reason for revert:
Testing to see if reverting fixes 10.9 bots.

Original issue's description:
> Move DstCopy on gpu into the GrXferProcessor.
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/74a11753604768bf461b80cabb66060e8564d82c
>
> Committed: https://skia.googlesource.com/skia/+/5e1378d0e075a323144ba14e0a4cbcca35eccc69

TBR=joshualitt@google.com,egdaniel@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:

Review URL: https://codereview.chromium.org/901663007

9 years agowhitespace
joshualitt [Thu, 5 Feb 2015 21:55:34 +0000 (13:55 -0800)]
whitespace

NOTRY=True
TBR=
BUG=skia:

Review URL: https://codereview.chromium.org/881413006

9 years agoMove DstCopy on gpu into the GrXferProcessor.
egdaniel [Thu, 5 Feb 2015 19:11:13 +0000 (11:11 -0800)]
Move DstCopy on gpu into the GrXferProcessor.

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/74a11753604768bf461b80cabb66060e8564d82c

Review URL: https://codereview.chromium.org/885923002

9 years agoDocument my current understanding of C++11 in Skia.
mtklein [Thu, 5 Feb 2015 18:58:48 +0000 (10:58 -0800)]
Document my current understanding of C++11 in Skia.

BUG=skia:

Review URL: https://codereview.chromium.org/901053007

9 years agoGM for http://skbug.com/3398
halcanary [Thu, 5 Feb 2015 18:26:27 +0000 (10:26 -0800)]
GM for http://skbug.com/3398

BUG=skia:3398

Review URL: https://codereview.chromium.org/896093005

9 years agoTry again to use a C++11 feature in DM as a canary.
mtklein [Thu, 5 Feb 2015 17:53:44 +0000 (09:53 -0800)]
Try again to use a C++11 feature in DM as a canary.

CQ_EXTRA_TRYBOTS=client.skia.compile:Build-Ubuntu13.10-GCC4.8-NaCl-Release-Trybot,Build-Mac10.7-Clang-Arm7-Debug-iOS-Trybot

BUG=skia:

Review URL: https://codereview.chromium.org/898863003

9 years agoBuild in C++11 mode on Unix-like bots.
mtklein [Thu, 5 Feb 2015 17:23:56 +0000 (09:23 -0800)]
Build in C++11 mode on Unix-like bots.

Mac and Windows bots are already building in C++11 mode.
This turns on the rest, mostly to see what work remains.

This will probably break a few bots.  It'd be nice if we could let those
all come in as red before reverting this so I can see the full list to fix.

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/779e49602a9c8f4d2799504822e01bcafbcaa534

CQ_EXTRA_TRYBOTS=client.skia.compile:Build-Ubuntu13.10-GCC4.8-NaCl-Release-Trybot,Build-Mac10.7-Clang-Arm7-Debug-iOS-Trybot

Review URL: https://codereview.chromium.org/868233008

9 years agoSuppress another FontConfig 'leak'.
bungeman [Thu, 5 Feb 2015 17:11:46 +0000 (09:11 -0800)]
Suppress another FontConfig 'leak'.

FontConfig uses offsets for elements instead of pointers, so any value
held by an element goes missing from memory checkers. Previous
suppressions took care of rule copies and adds to elements, but value
lists may be copied as well. When they are, values are copied, including
the content of strings and matricies. Since value lists are effectively
like Fc*Add functions (even calling the same underlying helpers), treat
them as such and suppress any 'leaks' they may cause.

Review URL: https://codereview.chromium.org/894863003

9 years agoBUG=skia:
joshualitt [Thu, 5 Feb 2015 16:19:35 +0000 (08:19 -0800)]
BUG=skia:

Review URL: https://codereview.chromium.org/894693003

9 years agoUpdate SKP version
skia.buildbots [Thu, 5 Feb 2015 06:55:44 +0000 (22:55 -0800)]
Update SKP version

Automatic commit by the RecreateSKPs bot.

TBR=

Review URL: https://codereview.chromium.org/901153002

9 years agoFix GIF transparency.
jei.mayol [Thu, 5 Feb 2015 05:31:23 +0000 (21:31 -0800)]
Fix GIF transparency.

The GIF decoder, in the onDecode() function, tries to obtain the
transparency info from the temporary SavedImage object, temp_save,
but the decoder had been changed to save extension block data to
the GifFileType object instead. As a result, find_transpIndex() can't
find the correct transaprency index and returns -1.

This had been reported in Android 4.4. AOSP pulled skia changes
related to giflib 5.x changes after Android 4.3, which is why
the problem is not reproducible in versions before KitKat.

AOSP Issue 62016:
https://code.google.com/p/android/issues/detail?id=62016

BUG=skia:

Review URL: https://codereview.chromium.org/429053004

9 years agoforce a build
mtklein [Thu, 5 Feb 2015 03:52:16 +0000 (19:52 -0800)]
force a build

BUG=skia:
NOTRY=true
TBR=

Review URL: https://codereview.chromium.org/899503002

9 years agoadd length parameter to addText, to match attr values
reed [Thu, 5 Feb 2015 02:29:27 +0000 (18:29 -0800)]
add length parameter to addText, to match attr values

BUG=skia:3392

Review URL: https://codereview.chromium.org/896363002

9 years ago[SkSVGDevice] drawTextOnPath() support
fmalita [Thu, 5 Feb 2015 01:54:46 +0000 (17:54 -0800)]
[SkSVGDevice] drawTextOnPath() support

R=reed@google.com,mtklein@google.com

Review URL: https://codereview.chromium.org/874853003

9 years ago[SkSVGDevice] Add support for more stroke params
fmalita [Wed, 4 Feb 2015 22:56:35 +0000 (14:56 -0800)]
[SkSVGDevice] Add support for more stroke params

stroke-linecap, stroke-linejoin, stroke-miterlimit, non-scaling-stroke.

R=mtklein@google.com

Review URL: https://codereview.chromium.org/897953004

9 years agoMore cleanup around GrContext, textures, and SkGr.cpp
bsalomon [Wed, 4 Feb 2015 18:55:54 +0000 (10:55 -0800)]
More cleanup around GrContext, textures, and SkGr.cpp

Review URL: https://codereview.chromium.org/880983008

9 years agoNo more caching volatile bitmaps
sugoi [Wed, 4 Feb 2015 18:53:03 +0000 (10:53 -0800)]
No more caching volatile bitmaps

VideoFrames are always marked as volatile and load_yuv_texture() was caching everything, even volatile bitmaps. To solve this issue, we no longer cache volatile bitmaps' YUV planes in load_yuv_texture().

There is another issue which cause this to happen, related to how VideoImageGenerator::onGetYUV8Planes() is implemented, which is logged here: crbug.com/455235

BUG=450706, 450699

Review URL: https://codereview.chromium.org/900943002

9 years agoFix casting bug in GrArithmeticXP
egdaniel [Wed, 4 Feb 2015 17:22:33 +0000 (09:22 -0800)]
Fix casting bug in GrArithmeticXP

BUG=skia:

Review URL: https://codereview.chromium.org/858373006

9 years agoMake npot resizing work properly for bmps that are explicitly texture backed.
bsalomon [Wed, 4 Feb 2015 17:12:46 +0000 (09:12 -0800)]
Make npot resizing work properly for bmps that are explicitly texture backed.

Review URL: https://codereview.chromium.org/890223003

9 years agoreturn reference to cache instead of copying the mask
reed [Wed, 4 Feb 2015 17:07:17 +0000 (09:07 -0800)]
return reference to cache instead of copying the mask

BUG=437128

Review URL: https://codereview.chromium.org/889303005

9 years ago[SkSVGDevice] Initial clipping support
fmalita [Wed, 4 Feb 2015 15:39:34 +0000 (07:39 -0800)]
[SkSVGDevice] Initial clipping support

Implement SVG clips based on clip stack flattening -
which is now exposed in SkClipStack::asPath() and shared
with SkCanvas's simplify-clip code.

R=reed@google.com,mtklein@google.com

Review URL: https://codereview.chromium.org/876923003

9 years agoRevert of Build in C++11 mode on Unix-like bots. (patchset #4 id:60001 of https:...
mtklein [Wed, 4 Feb 2015 15:06:26 +0000 (07:06 -0800)]
Revert of Build in C++11 mode on Unix-like bots. (patchset #4 id:60001 of https://codereview.chromium.org/868233008/)

Reason for revert:
Perf-Ubuntu12, and Test-Ubuntu12, Build-Nacl  all too old.

Android and Chrome OS builders look ok.
Android testers look ok.  Chrome OS testers haven't run yet.

Original issue's description:
> Build in C++11 mode on Unix-like bots.
>
> Mac and Windows bots are already building in C++11 mode.
> This turns on the rest, mostly to see what work remains.
>
> This will probably break a few bots.  It'd be nice if we could let those
> all come in as red before reverting this so I can see the full list to fix.
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/779e49602a9c8f4d2799504822e01bcafbcaa534

TBR=stephana@google.com,mtklein@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:

Review URL: https://codereview.chromium.org/879803003

9 years agoBuild in C++11 mode on Unix-like bots.
mtklein [Wed, 4 Feb 2015 14:55:36 +0000 (06:55 -0800)]
Build in C++11 mode on Unix-like bots.

Mac and Windows bots are already building in C++11 mode.
This turns on the rest, mostly to see what work remains.

This will probably break a few bots.  It'd be nice if we could let those
all come in as red before reverting this so I can see the full list to fix.

BUG=skia:

Review URL: https://codereview.chromium.org/868233008

9 years agoRevert "Move DstCopy on gpu into the GrXferProcessor."
egdaniel [Wed, 4 Feb 2015 14:25:28 +0000 (06:25 -0800)]
Revert "Move DstCopy on gpu into the GrXferProcessor."

This reverts commit 74a11753604768bf461b80cabb66060e8564d82c.

TBR=joshualitt@google.com,bsalomon@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:

Committed: https://skia.googlesource.com/skia/+/3e9dfdb3784c0cbfecf7589a74aa9aff7ef40abd

Review URL: https://codereview.chromium.org/896163003

9 years agoRevert of Revert "Move DstCopy on gpu into the GrXferProcessor." (patchset #1 id...
egdaniel [Wed, 4 Feb 2015 14:20:25 +0000 (06:20 -0800)]
Revert of Revert "Move DstCopy on gpu into the GrXferProcessor." (patchset #1 id:1 of https://codereview.chromium.org/896163003/)

Reason for revert:
failed on my manual revert

Original issue's description:
> Revert "Move DstCopy on gpu into the GrXferProcessor."
>
> This reverts commit 74a11753604768bf461b80cabb66060e8564d82c.
>
> TBR=joshualitt@google.com,bsalomon@google.com
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/3e9dfdb3784c0cbfecf7589a74aa9aff7ef40abd

TBR=
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:

Review URL: https://codereview.chromium.org/900913002

9 years agoRevert "Move DstCopy on gpu into the GrXferProcessor."
egdaniel [Wed, 4 Feb 2015 14:14:22 +0000 (06:14 -0800)]
Revert "Move DstCopy on gpu into the GrXferProcessor."

This reverts commit 74a11753604768bf461b80cabb66060e8564d82c.

TBR=joshualitt@google.com,bsalomon@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:

Review URL: https://codereview.chromium.org/896163003

9 years agoFix the URL for previewing Markdown changes.
tfarina [Wed, 4 Feb 2015 14:05:58 +0000 (06:05 -0800)]
Fix the URL for previewing Markdown changes.

BUG=None
R=jcgregorio@google.com
NOTRY=true

Review URL: https://codereview.chromium.org/883003004

9 years agoUpdate SKP version
skia.buildbots [Wed, 4 Feb 2015 07:27:08 +0000 (23:27 -0800)]
Update SKP version

Automatic commit by the RecreateSKPs bot.

TBR=

Review URL: https://codereview.chromium.org/876313006

9 years agoRemove unused and unimplemented abstract functions from GrRenderTarget
kkinnunen [Wed, 4 Feb 2015 06:20:19 +0000 (22:20 -0800)]
Remove unused and unimplemented abstract functions from GrRenderTarget

Remove unused and unimplemented abstract functions from GrRenderTarget.
The functions might cause confusion later.

BUG=skia:3388

Review URL: https://codereview.chromium.org/865603007

9 years ago[SkSVGDevice] Initial text support
fmalita [Wed, 4 Feb 2015 01:47:12 +0000 (17:47 -0800)]
[SkSVGDevice] Initial text support

R=reed@google.com,mtklein@google.com

Review URL: https://codereview.chromium.org/899683002

9 years agofixing GrBatch leak
joshualitt [Wed, 4 Feb 2015 01:40:16 +0000 (17:40 -0800)]
fixing GrBatch leak

TBR=
BUG=skia:

Review URL: https://codereview.chromium.org/886393008

9 years agoHairline batch
joshualitt [Wed, 4 Feb 2015 00:41:41 +0000 (16:41 -0800)]
Hairline batch

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/6eff8701f027016fbb3147412ec2292dcec2b7f5

Committed: https://skia.googlesource.com/skia/+/658d55cd6121c67488aaf5d0832c9712737f26a5

Review URL: https://codereview.chromium.org/876673002

9 years agoDash using vertex attributes for interval and stroke width
joshualitt [Tue, 3 Feb 2015 23:07:51 +0000 (15:07 -0800)]
Dash using vertex attributes for interval and stroke width

BUG=skia:

Review URL: https://codereview.chromium.org/892203003

9 years agoCleanup the android scripts.
djsollen [Tue, 3 Feb 2015 23:07:30 +0000 (15:07 -0800)]
Cleanup the android scripts.

Rename a few files to make their function clearer.
Update other files to remove dead code or improve function.

Review URL: https://codereview.chromium.org/865943007

9 years agoMove DstCopy on gpu into the GrXferProcessor.
egdaniel [Tue, 3 Feb 2015 23:02:43 +0000 (15:02 -0800)]
Move DstCopy on gpu into the GrXferProcessor.

BUG=skia:

Review URL: https://codereview.chromium.org/885923002

9 years agoadd textual overview
reed [Tue, 3 Feb 2015 23:02:24 +0000 (15:02 -0800)]
add textual overview

BUG=skia:

Review URL: https://codereview.chromium.org/898573002

9 years agoPort SkRefCnt.h to new SkAtomics.h
mtklein [Tue, 3 Feb 2015 21:38:58 +0000 (13:38 -0800)]
Port SkRefCnt.h to new SkAtomics.h

This adds sk_memory_barrier(), implemented using sk_atomic_fetch_add() on an uninitialized variable.  If that becomes a problem we can drop this to the porting layer, using std::atomic_thread_fence() / __atomic_thread_fence() / __sync_synchronize().

The big win is that ref() doesn't generate a memory barrier any more on ARM.
This is an instance of SkSafeRef() in SkPaint(const SkPaint&) after this CL:
     4d0:       684a            ldr     r2, [r1, #4]
     4d2:       6018            str     r0, [r3, #0]
     4d4:       b13a            cbz     r2, 4e6 <_ZN7SkPaintC1ERKS_+0x2e>
     4d6:       1d10            adds    r0, r2, #4
     4d8:       e850 4f00       ldrex   r4, [r0]
     4dc:       3401            adds    r4, #1
     4de:       e840 4500       strex   r5, r4, [r0]
     4e2:       2d00            cmp     r5, #0
     4e4:       d1f8            bne.n   4d8 <_ZN7SkPaintC1ERKS_+0x20>

Here's the before,  pretty much the same with two memory barriers surrounding the ref():
     4d8:       684a            ldr     r2, [r1, #4]
     4da:       6018            str     r0, [r3, #0]
     4dc:       b15a            cbz     r2, 4f6 <_ZN7SkPaintC1ERKS_+0x3e>
     4de:       1d10            adds    r0, r2, #4
     4e0:       f3bf 8f5f       dmb     sy
     4e4:       e850 4f00       ldrex   r4, [r0]
     4e8:       3401            adds    r4, #1
     4ea:       e840 4500       strex   r5, r4, [r0]
     4ee:       2d00            cmp     r5, #0
     4f0:       d1f8            bne.n   4e4 <_ZN7SkPaintC1ERKS_+0x2c>
     4f2:       f3bf 8f5f       dmb     sy

The miscellaneous files in here are just fixups to explicitly include SkMutex.h,
instead of leeching it off SkRefCnt.h.

No public API changes.
TBR=reed@google.com

Build trybots seem hosed.
NOTRY=true

BUG=skia:

Review URL: https://codereview.chromium.org/896803002

9 years agoBuild in C++11 mode on Macs.
mtklein [Tue, 3 Feb 2015 21:24:36 +0000 (13:24 -0800)]
Build in C++11 mode on Macs.

Build trybots are not triggering.
NOTRY=true

BUG=skia:

Review URL: https://codereview.chromium.org/894773002

9 years agoRemove link to Push application.
tfarina [Tue, 3 Feb 2015 20:07:20 +0000 (12:07 -0800)]
Remove link to Push application.

BUG=None
R=jcgregorio@google.com
NOTRY=true

Review URL: https://codereview.chromium.org/884203002

9 years agoremove remaining parts of SkExample
caryclark [Tue, 3 Feb 2015 19:16:05 +0000 (11:16 -0800)]
remove remaining parts of SkExample

R=reed@google.com

Review URL: https://codereview.chromium.org/895103002

9 years agoadd new Skia team member to committers list
hcm [Tue, 3 Feb 2015 19:10:56 +0000 (11:10 -0800)]
add new Skia team member to committers list

BUG=skia:

NOTRY=true
TBR=rmistry@google.com

Review URL: https://codereview.chromium.org/885493008

9 years agoSave and temp dst and src FBO on GrGLGpu and use the temp FBOs for copySurface if...
egdaniel [Tue, 3 Feb 2015 19:10:51 +0000 (11:10 -0800)]
Save and temp dst and src FBO on GrGLGpu and use the temp FBOs for copySurface if needed.

BUG=skia:

Review URL: https://codereview.chromium.org/893703003

9 years agoRevert of Hairline batch (patchset #17 id:360001 of https://codereview.chromium.org...
joshualitt [Tue, 3 Feb 2015 16:40:22 +0000 (08:40 -0800)]
Revert of Hairline batch (patchset #17 id:360001 of https://codereview.chromium.org/876673002/)

Reason for revert:
still a performance regression

Original issue's description:
> Hairline batch
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/6eff8701f027016fbb3147412ec2292dcec2b7f5
>
> Committed: https://skia.googlesource.com/skia/+/658d55cd6121c67488aaf5d0832c9712737f26a5

TBR=bsalomon@google.com,joshualitt@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:

Review URL: https://codereview.chromium.org/882883003

9 years agoLand Ben's 32-bit symlink script.
Mike Klein [Tue, 3 Feb 2015 15:43:28 +0000 (10:43 -0500)]
Land Ben's 32-bit symlink script.

BUG=skia:

Review URL: https://codereview.chromium.org/890613005

9 years agoFix references to https://sites.google.com/site/skiadocs/.
Thiago Farina [Tue, 3 Feb 2015 15:12:54 +0000 (13:12 -0200)]
Fix references to https://sites.google.com/site/skiadocs/.

Point them to their equivalent in https://skia.org.

BUG=None
R=hcm@google.com, jcgregorio@google.com
NOTRY=true

Review URL: https://codereview.chromium.org/856353004

9 years agoHairline batch
joshualitt [Tue, 3 Feb 2015 14:56:44 +0000 (06:56 -0800)]
Hairline batch

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/6eff8701f027016fbb3147412ec2292dcec2b7f5

Review URL: https://codereview.chromium.org/876673002

9 years ago[SkSVGDevice] Initial shader/gradient support
fmalita [Tue, 3 Feb 2015 13:44:40 +0000 (05:44 -0800)]
[SkSVGDevice] Initial shader/gradient support

* linear gradient support (based on shawcroft@google.com's CL)
* generic paint resources reorg
* opacity support

R=reed@google.com,mtklein@google.com,halcanary@google.com

Review URL: https://codereview.chromium.org/892973002

9 years agoWhitespace change to test new compile bots
Ravi Mistry [Tue, 3 Feb 2015 13:41:46 +0000 (08:41 -0500)]
Whitespace change to test new compile bots

9 years agoUpdate SKP version
skia.buildbots [Tue, 3 Feb 2015 06:56:50 +0000 (22:56 -0800)]
Update SKP version

Automatic commit by the RecreateSKPs bot.

TBR=

Review URL: https://codereview.chromium.org/897603002

9 years agoAdd texture create/upload stats and make nanobench have explicit gpu stats flag
bsalomon [Tue, 3 Feb 2015 05:19:50 +0000 (21:19 -0800)]
Add texture create/upload stats and make nanobench have explicit gpu stats flag

Review URL: https://codereview.chromium.org/891973002

9 years agoRevert of Make the glyph array entries inline. (patchset #10 id:170001 of https:...
bsalomon [Tue, 3 Feb 2015 05:06:23 +0000 (21:06 -0800)]
Revert of Make the glyph array entries inline. (patchset #10 id:170001 of https://codereview.chromium.org/885903002/)

Reason for revert:
I suspect this is causing the off-by-one character issues that show up in gold.skia.org as of now. All the errors I've seen are on a Win7 bot. Example:

good:
https://gold.skia.org/img/images/35396bb0d299b81c0031dc0632a019d4.png

bad:
https://gold.skia.org/img/images/484e511f9e696d95031cd25aeae59da0.png

Original issue's description:
> Make the glyph array entries inline.
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/4c08f16b252a55e438a61f26e5581394ed177da1

TBR=mtklein@google.com,reed@google.com,herb@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:

Review URL: https://codereview.chromium.org/897463004

9 years agopatch from issue 886233004 at patchset 40001 (http://crrev.com/886233004#ps40001)
mtklein [Tue, 3 Feb 2015 02:26:03 +0000 (18:26 -0800)]
patch from issue 886233004 at patchset 40001 (crrev.com/886233004#ps40001)

... with changes proposed in the review.

BUG=skia:

Review URL: https://codereview.chromium.org/894013002

9 years agoRevert of Hairline batch (patchset #16 id:300001 of https://codereview.chromium.org...
joshualitt [Tue, 3 Feb 2015 02:12:39 +0000 (18:12 -0800)]
Revert of Hairline batch (patchset #16 id:300001 of https://codereview.chromium.org/876673002/)

Reason for revert:
Regression

Original issue's description:
> Hairline batch
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/6eff8701f027016fbb3147412ec2292dcec2b7f5

TBR=bsalomon@google.com,joshualitt@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:

Review URL: https://codereview.chromium.org/894273002

9 years agoMake the glyph array entries inline.
herb [Tue, 3 Feb 2015 01:47:31 +0000 (17:47 -0800)]
Make the glyph array entries inline.
BUG=skia:

Review URL: https://codereview.chromium.org/885903002

9 years agomake getContentKey() available in GrGpuResource public interface
bsalomon [Tue, 3 Feb 2015 01:25:26 +0000 (17:25 -0800)]
make getContentKey() available in GrGpuResource public interface

TBR=robertphillips@google.com

Committed: https://skia.googlesource.com/skia/+/85ed2db092e75db41b711b9116a8d5b465fc2b0c

Review URL: https://codereview.chromium.org/886313005

9 years agoHairline batch
joshualitt [Tue, 3 Feb 2015 01:19:40 +0000 (17:19 -0800)]
Hairline batch

BUG=skia:

Review URL: https://codereview.chromium.org/876673002

9 years agoWhitespace change to test new GCE compile bot
Ravi Mistry [Tue, 3 Feb 2015 00:40:55 +0000 (19:40 -0500)]
Whitespace change to test new GCE compile bot

9 years agoWhitespace change to test new GCE compile bot
Ravi Mistry [Tue, 3 Feb 2015 00:05:32 +0000 (19:05 -0500)]
Whitespace change to test new GCE compile bot

9 years agoWhitespace change to test new GCE compile bot
Ravi Mistry [Mon, 2 Feb 2015 23:16:22 +0000 (18:16 -0500)]
Whitespace change to test new GCE compile bot

9 years agoRevert of make getContentKey() available in GrGpuResource public interface (patchset...
stephana [Mon, 2 Feb 2015 22:52:11 +0000 (14:52 -0800)]
Revert of make getContentKey() available in GrGpuResource public interface (patchset #2 id:20001 of https://codereview.chromium.org/886313005/)

Reason for revert:
This causes in one case the linker and in the other the compiler to segfault. On

Test-Ubuntu12-ShuttleA-GTX660-x86_64-Debug/Release

Original issue's description:
> make getContentKey() available in GrGpuResource public interface
>
> TBR=robertphillips@google.com
>
> Committed: https://skia.googlesource.com/skia/+/85ed2db092e75db41b711b9116a8d5b465fc2b0c

TBR=robertphillips@google.com,bsalomon@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

Review URL: https://codereview.chromium.org/864343005

9 years agomake getContentKey() available in GrGpuResource public interface
bsalomon [Mon, 2 Feb 2015 22:08:09 +0000 (14:08 -0800)]
make getContentKey() available in GrGpuResource public interface

TBR=robertphillips@google.com

Review URL: https://codereview.chromium.org/886313005

9 years agoDM: wire up --leaks / -l again
mtklein [Mon, 2 Feb 2015 21:24:37 +0000 (13:24 -0800)]
DM: wire up --leaks / -l again

No public API changes.
TBR=reed@google.com

BUG=skia:

Review URL: https://codereview.chromium.org/893043002

9 years agoMove npot resizing out of GrContext and simplify GrContext texture functions.
bsalomon [Mon, 2 Feb 2015 21:00:10 +0000 (13:00 -0800)]
Move npot resizing out of GrContext and simplify GrContext texture functions.

Committed: https://skia.googlesource.com/skia/+/8a8100349105c8c6de39fcb34e47679da7a67f54

Committed: https://skia.googlesource.com/skia/+/6c96672491b04cb782bce8fee778124df66524a0

Review URL: https://codereview.chromium.org/882223003

9 years agomove HelloWorld to be a peer of SampleApp
caryclark [Mon, 2 Feb 2015 20:55:14 +0000 (12:55 -0800)]
move HelloWorld to be a peer of SampleApp

This is working towards making a simple example part of the buildbot compile step and removing SkExamples from the experimental directory.

This works on Mac, Windows, and Linux but isn't complete for Android, ChromeOS and iOS.

Review URL: https://codereview.chromium.org/886413004

9 years agoadd SkAnimTimer, SPACE = pause/resume, ESP = stop
reed [Mon, 2 Feb 2015 20:55:02 +0000 (12:55 -0800)]
add SkAnimTimer, SPACE = pause/resume, ESP = stop

BUG=skia:

Review URL: https://codereview.chromium.org/894083003

9 years agoAtomics overhaul.
mtklein [Mon, 2 Feb 2015 20:22:07 +0000 (12:22 -0800)]
Atomics overhaul.

This merges and refactors SkAtomics.h and SkBarriers.h into SkAtomics.h and
some ports/ implementations. The major new feature is that we can express
memory orders explicitly rather than only through comments.

The porting layer is reduced to four template functions:
  - sk_atomic_load
  - sk_atomic_store
  - sk_atomic_fetch_add
  - sk_atomic_compare_exchange
From those four we can reconstruct all our previous sk_atomic_foo.

There are three ports:
  - SkAtomics_std:    uses C++11 <atomic>,             used with MSVC
  - SkAtomics_atomic: uses newer GCC/Clang intrinsics, used on not-MSVC where possible
  - SkAtomics_sync:   uses older GCC/Clang intrinsics, used where SkAtomics_atomic not supported

No public API changes.
TBR=reed@google.com

BUG=skia:

Review URL: https://codereview.chromium.org/896553002

9 years agoAdd missing SK_OVERRIDE
rmistry [Mon, 2 Feb 2015 20:08:18 +0000 (12:08 -0800)]
Add missing SK_OVERRIDE

Tested by running on clang head + ubuntu 14.04:
GYP_DEFINES=”skia_gpu=0 skia_warnings_as_errors=1" tools/xsan_build thread dm BUILDTYPE=Release
out/Release/dm -v

BUG=skia:3386

Review URL: https://codereview.chromium.org/894833002

9 years agoadd missing SK_OVERRIDE
mtklein [Mon, 2 Feb 2015 19:49:05 +0000 (11:49 -0800)]
add missing SK_OVERRIDE

BUG=skia:

Review URL: https://codereview.chromium.org/894153002

9 years agoDon't read unecessary font files on Android.
bungeman [Mon, 2 Feb 2015 19:19:44 +0000 (11:19 -0800)]
Don't read unecessary font files on Android.

With a new style fonts.xml (version >= 21) fallback fonts are specified
as font families without a name. The current code reads these, but then
also reads in the old fallback and vendor xml files. With this change,
when a new style xml file is encoutered, no further files will be read.
This both lowers memory use and speeds font lookup. Locally, this
change reduces the number of font families loaded on my 'L' device from
148 to 80. All of the families removed are duplicates.

Review URL: https://codereview.chromium.org/888923003

9 years agoRevert of Revert of SSE4 opaque blend using intrinsics instead of assembly. (patchset...
stephana [Mon, 2 Feb 2015 18:02:48 +0000 (10:02 -0800)]
Revert of Revert of SSE4 opaque blend using intrinsics instead of assembly. (patchset #1 id:1 of https://codereview.chromium.org/873553003/)

Reason for revert:
Reverted the wrong CL.

Original issue's description:
> Revert of SSE4 opaque blend using intrinsics instead of assembly. (patchset #16 id:300001 of https://codereview.chromium.org/874863002/)
>
> Reason for revert:
> This causes a bug on the 'hittestpath' GM on MacMini 4,1
>
> See:
>
> https://gold.skia.org/#/triage/hittestpath?head=0
>
> for details.
>
> Original issue's description:
> > SSE4 opaque blend using intrinsics instead of assembly.
> >
> > Since we had such a hard time with the assembly versions of this blit (to the
> > point that we have them completely disabled everywhere), I thought I'd take
> > a shot at writing a version of the blit using intrinsics.
> >
> > The key feature of SSE4 we're exploiting is that we can use ptest (_mm_test*)
> > to skip the blend when the 16 src pixels we consider each loop are all opaque
> > or all transparent.  _mm_shuffle_epi8 from SSSE3 also lends a hand to extract
> > all those alphas.
> >
> > It's worth looking to see if we can backport this type of logic to SSE2 using
> > _mm_movemask_epi8, or up to 32 pixels at a time using AVX.
> >
> > My local performance testing doesn't show this to be an unambiguous win
> > (there are probably microbenchmarks and SKPs where we'd be better off just
> > powering through the blend rather than looking at alphas), but the potential
> > does seem tantalizing enough to let skiaperf vet it on the bots.  (< 1.0x is a win.)
> >
> > DM says it draws pixel perfect compare to the old code.
> >
> > Microbenchmarks:
> >                bitmap_RGBA_8888_A_source_stripes_two   14us -> 14.4us 1.03x
> >              bitmap_RGBA_8888_A_source_stripes_three 14.3us -> 14.5us 1.01x
> >                        bitmap_RGBA_8888_scale_bilerp 61.9us -> 62.2us 1.01x
> > bitmap_RGBA_8888_update_volatile_scale_rotate_bilerp  102us ->  101us 0.99x
> >                 bitmap_RGBA_8888_scale_rotate_bilerp  103us ->  101us 0.99x
> >                               bitmap_RGBA_8888_scale 18.4us -> 18.2us 0.99x
> >              bitmap_RGBA_8888_A_scale_rotate_bicubic   71us ->   70us 0.99x
> >          bitmap_RGBA_8888_update_scale_rotate_bilerp  103us ->  101us 0.99x
> >               bitmap_RGBA_8888_A_scale_rotate_bilerp  112us ->  109us 0.98x
> >                     bitmap_RGBA_8888_update_volatile 5.72us -> 5.58us 0.98x
> >                                     bitmap_RGBA_8888 5.73us -> 5.58us 0.97x
> >                              bitmap_RGBA_8888_update 5.78us ->  5.6us 0.97x
> >                      bitmap_RGBA_8888_A_scale_bilerp 70.7us ->   68us 0.96x
> >                     bitmap_RGBA_8888_A_scale_bicubic 23.7us -> 21.8us 0.92x
> >                                   bitmap_RGBA_8888_A 13.9us -> 10.9us 0.78x
> >                     bitmap_RGBA_8888_A_source_opaque   14us -> 6.29us 0.45x
> >                bitmap_RGBA_8888_A_source_transparent   14us -> 3.65us 0.26x
> >
> > Running over our ~70 SKP web page captures, this looks like we spend 0.7x
> > the time in S32A_Opaque_BlitRow compared to the SSE2 version, which should
> > be a decent predictor of real-world impact.
> >
> > BUG=chromium:399842
> >
> > Committed: https://skia.googlesource.com/skia/+/04bc91b972417038fecfa87c484771eac2b9b785
> >
> > CQ_EXTRA_TRYBOTS=client.skia:Test-Mac10.6-MacMini4.1-GeForce320M-x86_64-Release-Trybot
> >
> > Committed: https://skia.googlesource.com/skia/+/6dbfb21a6c88af6d94e8c823c3ad559f1a41b493
>
> TBR=henrik.smiding@intel.com,mtklein@google.com,herb@google.com,reed@google.com,thakis@chromium.org,mtklein@chromium.org
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=chromium:399842
>
> Committed: https://skia.googlesource.com/skia/+/4988891a1173cd405bf1c1dd3a3668c451f45e4c

TBR=henrik.smiding@intel.com,mtklein@google.com,herb@google.com,reed@google.com,thakis@chromium.org,mtklein@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=chromium:399842

Review URL: https://codereview.chromium.org/894083002

9 years agoRevert of SSE4 opaque blend using intrinsics instead of assembly. (patchset #16 id...
stephana [Mon, 2 Feb 2015 17:52:43 +0000 (09:52 -0800)]
Revert of SSE4 opaque blend using intrinsics instead of assembly. (patchset #16 id:300001 of https://codereview.chromium.org/874863002/)

Reason for revert:
This causes a bug on the 'hittestpath' GM on MacMini 4,1

See:

https://gold.skia.org/#/triage/hittestpath?head=0

for details.

Original issue's description:
> SSE4 opaque blend using intrinsics instead of assembly.
>
> Since we had such a hard time with the assembly versions of this blit (to the
> point that we have them completely disabled everywhere), I thought I'd take
> a shot at writing a version of the blit using intrinsics.
>
> The key feature of SSE4 we're exploiting is that we can use ptest (_mm_test*)
> to skip the blend when the 16 src pixels we consider each loop are all opaque
> or all transparent.  _mm_shuffle_epi8 from SSSE3 also lends a hand to extract
> all those alphas.
>
> It's worth looking to see if we can backport this type of logic to SSE2 using
> _mm_movemask_epi8, or up to 32 pixels at a time using AVX.
>
> My local performance testing doesn't show this to be an unambiguous win
> (there are probably microbenchmarks and SKPs where we'd be better off just
> powering through the blend rather than looking at alphas), but the potential
> does seem tantalizing enough to let skiaperf vet it on the bots.  (< 1.0x is a win.)
>
> DM says it draws pixel perfect compare to the old code.
>
> Microbenchmarks:
>                bitmap_RGBA_8888_A_source_stripes_two   14us -> 14.4us 1.03x
>              bitmap_RGBA_8888_A_source_stripes_three 14.3us -> 14.5us 1.01x
>                        bitmap_RGBA_8888_scale_bilerp 61.9us -> 62.2us 1.01x
> bitmap_RGBA_8888_update_volatile_scale_rotate_bilerp  102us ->  101us 0.99x
>                 bitmap_RGBA_8888_scale_rotate_bilerp  103us ->  101us 0.99x
>                               bitmap_RGBA_8888_scale 18.4us -> 18.2us 0.99x
>              bitmap_RGBA_8888_A_scale_rotate_bicubic   71us ->   70us 0.99x
>          bitmap_RGBA_8888_update_scale_rotate_bilerp  103us ->  101us 0.99x
>               bitmap_RGBA_8888_A_scale_rotate_bilerp  112us ->  109us 0.98x
>                     bitmap_RGBA_8888_update_volatile 5.72us -> 5.58us 0.98x
>                                     bitmap_RGBA_8888 5.73us -> 5.58us 0.97x
>                              bitmap_RGBA_8888_update 5.78us ->  5.6us 0.97x
>                      bitmap_RGBA_8888_A_scale_bilerp 70.7us ->   68us 0.96x
>                     bitmap_RGBA_8888_A_scale_bicubic 23.7us -> 21.8us 0.92x
>                                   bitmap_RGBA_8888_A 13.9us -> 10.9us 0.78x
>                     bitmap_RGBA_8888_A_source_opaque   14us -> 6.29us 0.45x
>                bitmap_RGBA_8888_A_source_transparent   14us -> 3.65us 0.26x
>
> Running over our ~70 SKP web page captures, this looks like we spend 0.7x
> the time in S32A_Opaque_BlitRow compared to the SSE2 version, which should
> be a decent predictor of real-world impact.
>
> BUG=chromium:399842
>
> Committed: https://skia.googlesource.com/skia/+/04bc91b972417038fecfa87c484771eac2b9b785
>
> CQ_EXTRA_TRYBOTS=client.skia:Test-Mac10.6-MacMini4.1-GeForce320M-x86_64-Release-Trybot
>
> Committed: https://skia.googlesource.com/skia/+/6dbfb21a6c88af6d94e8c823c3ad559f1a41b493

TBR=henrik.smiding@intel.com,mtklein@google.com,herb@google.com,reed@google.com,thakis@chromium.org,mtklein@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=chromium:399842

Review URL: https://codereview.chromium.org/873553003

9 years agoAdd resourcePath to running dm on Android doc.
bungeman [Mon, 2 Feb 2015 17:27:14 +0000 (09:27 -0800)]
Add resourcePath to running dm on Android doc.

Without specifying the resource path a number of tests and gms do not
work properly. The example given in the docs should specify the
resource path so that the example run is exemplary.

Review URL: https://codereview.chromium.org/898453003

9 years agoOptimize SSE2 opaque blend
qiankun.miao [Mon, 2 Feb 2015 14:28:53 +0000 (06:28 -0800)]
Optimize SSE2 opaque blend

Backport optimization from https://codereview.chromium.org/874863002/.
Microbenchmarks data compared to previous SSE2 implementation:
             bitmap_BGRA_8888_A_source_stripes_three    7.52us -> 8.67us        1.15x
               bitmap_BGRA_8888_A_source_stripes_two    7.48us -> 8.56us        1.15x
         bitmap_BGRA_8888_update_scale_rotate_bilerp    63.4us ->   64us        1.01x
                    bitmap_BGRA_8888_update_volatile    3.31us -> 3.33us        1.01x
                              bitmap_BGRA_8888_scale    11.1us -> 11.2us        1x
                       bitmap_BGRA_8888_scale_bilerp    35.8us -> 35.9us        1x
                                    bitmap_BGRA_8888    3.33us -> 3.33us        1x
             bitmap_BGRA_8888_A_scale_rotate_bicubic    66.7us -> 66.5us        1x
bitmap_BGRA_8888_update_volatile_scale_rotate_bilerp    65.1us ->   64us        0.98x
                bitmap_BGRA_8888_scale_rotate_bilerp    65.1us ->   64us        0.98x
                    bitmap_BGRA_8888_A_scale_bicubic    30.6us -> 29.9us        0.98x
                     bitmap_BGRA_8888_A_scale_bilerp    42.7us -> 41.4us        0.97x
              bitmap_BGRA_8888_A_scale_rotate_bilerp      71us -> 67.7us        0.95x
                                  bitmap_BGRA_8888_A    7.44us ->  5.7us        0.77x
                    bitmap_BGRA_8888_A_source_opaque    7.46us -> 3.72us        0.5x
               bitmap_BGRA_8888_A_source_transparent    7.46us -> 1.96us        0.26x

BUG=skia:

Review URL: https://codereview.chromium.org/886403002

9 years agoDisable LCD text when rasterizing SkPictureShader tiles.
fmalita [Mon, 2 Feb 2015 13:25:04 +0000 (05:25 -0800)]
Disable LCD text when rasterizing SkPictureShader tiles.

BUG=chromium:453299
R=reed@google.com

Review URL: https://codereview.chromium.org/884163003

9 years agoUpdate SKP version
skia.buildbots [Mon, 2 Feb 2015 06:36:03 +0000 (22:36 -0800)]
Update SKP version

Automatic commit by the RecreateSKPs bot.

TBR=

Review URL: https://codereview.chromium.org/890293003

9 years agoallow GMs to animate
reed [Mon, 2 Feb 2015 03:01:04 +0000 (19:01 -0800)]
allow GMs to animate

BUG=skia:

Review URL: https://codereview.chromium.org/888283002

9 years agoUpdate SKP version
skia.buildbots [Sun, 1 Feb 2015 06:53:02 +0000 (22:53 -0800)]
Update SKP version

Automatic commit by the RecreateSKPs bot.

TBR=

Review URL: https://codereview.chromium.org/888343002

9 years agoRevert of Move npot resizing out of GrContext and simplify GrContext texture function...
bsalomon [Sun, 1 Feb 2015 04:10:56 +0000 (20:10 -0800)]
Revert of Move npot resizing out of GrContext and simplify GrContext texture functions. (patchset #10 id:200001 of https://codereview.chromium.org/882223003/)

Reason for revert:
perf fix didn't fix the cr webgl conformance tests

Original issue's description:
> Move npot resizing out of GrContext and simplify GrContext texture functions.
>
> Committed: https://skia.googlesource.com/skia/+/8a8100349105c8c6de39fcb34e47679da7a67f54
>
> Committed: https://skia.googlesource.com/skia/+/6c96672491b04cb782bce8fee778124df66524a0

TBR=robertphillips@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

Review URL: https://codereview.chromium.org/887303002

9 years agoSVG backend in DM
mtklein [Sun, 1 Feb 2015 04:00:58 +0000 (20:00 -0800)]
SVG backend in DM

Not enabled by default, but this should get you SKPs, GMs etc for free to play with.

$ out/Debug/dm -w svgs --src gm skp --config svg

BUG=skia:

Review URL: https://codereview.chromium.org/892693002

9 years agoMove npot resizing out of GrContext and simplify GrContext texture functions.
bsalomon [Sun, 1 Feb 2015 03:27:53 +0000 (19:27 -0800)]
Move npot resizing out of GrContext and simplify GrContext texture functions.

Committed: https://skia.googlesource.com/skia/+/8a8100349105c8c6de39fcb34e47679da7a67f54

Review URL: https://codereview.chromium.org/882223003

9 years agoRemove extraneous newline.
Brian Salomon [Sat, 31 Jan 2015 19:29:16 +0000 (14:29 -0500)]
Remove extraneous newline.

TBR=mtklein@google.com

Review URL: https://codereview.chromium.org/878333004

9 years agoAdd standard way to indicate GM is GPU-only.
bsalomon [Sat, 31 Jan 2015 15:51:14 +0000 (07:51 -0800)]
Add standard way to indicate GM is GPU-only.

TBR=egdaniel@google.com,mtklein@google.com

Review URL: https://codereview.chromium.org/869393007

9 years agoRevert of Move npot resizing out of GrContext and simplify GrContext texture function...
fmalita [Sat, 31 Jan 2015 15:02:18 +0000 (07:02 -0800)]
Revert of Move npot resizing out of GrContext and simplify GrContext texture functions. (patchset #9 id:160001 of https://codereview.chromium.org/882223003/)

Reason for revert:
webGL conformance failures:

WebglConformance.conformance_textures_tex_image_and_sub_image_2d_with_video
WebglConformance.conformance_textures_texture_npot_video

https://codereview.chromium.org/892773003/
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_ng/builds/29272

Original issue's description:
> Move npot resizing out of GrContext and simplify GrContext texture functions.
>
> Committed: https://skia.googlesource.com/skia/+/8a8100349105c8c6de39fcb34e47679da7a67f54

TBR=robertphillips@google.com,bsalomon@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

Review URL: https://codereview.chromium.org/868233005

9 years agoUpdate SKP version
skia.buildbots [Sat, 31 Jan 2015 07:00:10 +0000 (23:00 -0800)]
Update SKP version

Automatic commit by the RecreateSKPs bot.

TBR=

Review URL: https://codereview.chromium.org/875243006

9 years agoUpdate dump code in FontConfigParser test.
bungeman [Sat, 31 Jan 2015 03:58:19 +0000 (19:58 -0800)]
Update dump code in FontConfigParser test.

This code is still quite useful for debugging, but has bit rotted.
Update to match current structure of code.

Review URL: https://codereview.chromium.org/890073002

9 years agoConvert FamilyData to Skia style field names.
bungeman [Fri, 30 Jan 2015 23:35:09 +0000 (15:35 -0800)]
Convert FamilyData to Skia style field names.

The Skia convention is to prefix field names with 'f'.
This change is mostly mechanical.

Review URL: https://codereview.chromium.org/892733002