jvanverth@google.com [Tue, 6 Aug 2013 18:28:25 +0000 (18:28 +0000)]
One last rebaseline to try to get Nexus-10 GMs green.
Unreviewed.
(SkipBuildbotRuns)
git-svn-id: http://skia.googlecode.com/svn/trunk@10574
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Tue, 6 Aug 2013 18:13:01 +0000 (18:13 +0000)]
Tweak -mssse3 on Mac a little.
Using OTHER_CPLUSPLUSFLAGS instead of OTHER_CFLAGS will append -mssse3 into the
argument list instead of overwriting as the old note warns about. (So it's
actually there twice now for the files in opts_ssse3, and we can still build if
we remove -mssse3 from common_conditions.gypi.)
We could also just delete this clause entirely given that
common_conditions.gypi sets it anyway. Which do you think is best? This code
won't compile unless _someone_ has set -mssse3. Seems to me the redundancy
helps communicate that and protect against changes in common_conditions.gypi.
BUG=
R=epoger@google.com, bungeman@google.com
Author: mtklein@google.com
Review URL: https://chromiumcodereview.appspot.com/
21279005
git-svn-id: http://skia.googlecode.com/svn/trunk@10573
2bbb7eff-a529-9590-31e7-
b0007b416f81
jvanverth@google.com [Tue, 6 Aug 2013 17:30:28 +0000 (17:30 +0000)]
Fix mime-type on expectations.
Rebaseline one DirectWrite GM.
(SkipBuildbotRuns)
git-svn-id: http://skia.googlecode.com/svn/trunk@10570
2bbb7eff-a529-9590-31e7-
b0007b416f81
jvanverth@google.com [Tue, 6 Aug 2013 16:34:38 +0000 (16:34 +0000)]
Rebaseline Nexus-10 GMs... again.
(SkipBuildbotRuns)
git-svn-id: http://skia.googlecode.com/svn/trunk@10568
2bbb7eff-a529-9590-31e7-
b0007b416f81
bsalomon@google.com [Tue, 6 Aug 2013 16:06:14 +0000 (16:06 +0000)]
Revert Accidental commit "patch from issue
20151002"
This reverts commit r10565.
git-svn-id: http://skia.googlecode.com/svn/trunk@10567
2bbb7eff-a529-9590-31e7-
b0007b416f81
bsalomon@google.com [Tue, 6 Aug 2013 16:04:42 +0000 (16:04 +0000)]
Rebaseline flaky non AA lines on intel/win/gpu
git-svn-id: http://skia.googlecode.com/svn/trunk@10566
2bbb7eff-a529-9590-31e7-
b0007b416f81
bsalomon@google.com [Tue, 6 Aug 2013 16:04:39 +0000 (16:04 +0000)]
patch from issue
20151002
git-svn-id: http://skia.googlecode.com/svn/trunk@10565
2bbb7eff-a529-9590-31e7-
b0007b416f81
borenet@google.com [Tue, 6 Aug 2013 15:42:15 +0000 (15:42 +0000)]
Move toolchain setup into a function in android_setup.sh
Addresses https://code.google.com/p/skia/issues/detail?id=1486
R=djsollen@google.com
Review URL: https://codereview.chromium.org/
22339008
git-svn-id: http://skia.googlecode.com/svn/trunk@10563
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Tue, 6 Aug 2013 15:07:23 +0000 (15:07 +0000)]
Add blend optimization helpers and use to convert rect draws to clears.
Committed: http://code.google.com/p/skia/source/detail?r=10537
R=robertphillips@google.com, jvanverth@google.com, reed@google.com
Author: bsalomon@google.com
Review URL: https://chromiumcodereview.appspot.com/
21877006
git-svn-id: http://skia.googlecode.com/svn/trunk@10562
2bbb7eff-a529-9590-31e7-
b0007b416f81
jvanverth@google.com [Tue, 6 Aug 2013 14:17:16 +0000 (14:17 +0000)]
Rebaseline Nexus 10 GMs
(SkipBuildbotRuns)
git-svn-id: http://skia.googlecode.com/svn/trunk@10561
2bbb7eff-a529-9590-31e7-
b0007b416f81
borenet@google.com [Tue, 6 Aug 2013 13:47:54 +0000 (13:47 +0000)]
Normalize indents in android_setup.sh
R=djsollen@google.com
Review URL: https://codereview.chromium.org/
22422002
git-svn-id: http://skia.googlecode.com/svn/trunk@10560
2bbb7eff-a529-9590-31e7-
b0007b416f81
robertphillips@google.com [Tue, 6 Aug 2013 12:20:38 +0000 (12:20 +0000)]
N7 Performance Trigger rebaselines
(SkipBuildbotRuns)
git-svn-id: http://skia.googlecode.com/svn/trunk@10557
2bbb7eff-a529-9590-31e7-
b0007b416f81
djsollen@google.com [Tue, 6 Aug 2013 12:19:38 +0000 (12:19 +0000)]
Fix race condition when locking/unlocking pixels.
R=reed@google.com
Review URL: https://codereview.chromium.org/
22235002
git-svn-id: http://skia.googlecode.com/svn/trunk@10556
2bbb7eff-a529-9590-31e7-
b0007b416f81
robertphillips@google.com [Tue, 6 Aug 2013 12:17:53 +0000 (12:17 +0000)]
Ubuntu Performance trigger rebaseline
(SkipBuildbotRuns)
git-svn-id: http://skia.googlecode.com/svn/trunk@10555
2bbb7eff-a529-9590-31e7-
b0007b416f81
skia.committer@gmail.com [Tue, 6 Aug 2013 07:01:27 +0000 (07:01 +0000)]
Sanitizing source files in Housekeeper-Nightly
git-svn-id: http://skia.googlecode.com/svn/trunk@10554
2bbb7eff-a529-9590-31e7-
b0007b416f81
mtklein@google.com [Mon, 5 Aug 2013 23:08:19 +0000 (23:08 +0000)]
old compiler is dumb
BUG=
Review URL: https://codereview.chromium.org/
22318002
git-svn-id: http://skia.googlecode.com/svn/trunk@10553
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Mon, 5 Aug 2013 22:31:20 +0000 (22:31 +0000)]
revise SkTDynamicHash and add unit tests
BUG=
R=reed@google.com
Author: mtklein@google.com
Review URL: https://chromiumcodereview.appspot.com/
22292004
git-svn-id: http://skia.googlecode.com/svn/trunk@10552
2bbb7eff-a529-9590-31e7-
b0007b416f81
edisonn@google.com [Mon, 5 Aug 2013 22:08:50 +0000 (22:08 +0000)]
pdfviewer: remove debug traces
Review URL: https://codereview.chromium.org/
22284004
git-svn-id: http://skia.googlecode.com/svn/trunk@10551
2bbb7eff-a529-9590-31e7-
b0007b416f81
jvanverth@google.com [Mon, 5 Aug 2013 21:47:24 +0000 (21:47 +0000)]
Rebaseline imagemagnifier GM for Nexus-4
git-svn-id: http://skia.googlecode.com/svn/trunk@10549
2bbb7eff-a529-9590-31e7-
b0007b416f81
edisonn@google.com [Mon, 5 Aug 2013 20:45:40 +0000 (20:45 +0000)]
pdfviewer: debug code for crash
Review URL: https://codereview.chromium.org/
22269002
git-svn-id: http://skia.googlecode.com/svn/trunk@10548
2bbb7eff-a529-9590-31e7-
b0007b416f81
egdaniel@google.com [Mon, 5 Aug 2013 20:43:12 +0000 (20:43 +0000)]
Separate quad UV calculation from bloat_quad
For GPU hairlines move UV calculation on vertices for quads to its
own function outside of bloat_quad. This is done since conics share
the bloat quad function and don't need to do this calcuation.
BUG=
R=bsalomon@google.com
Review URL: https://codereview.chromium.org/
22043006
git-svn-id: http://skia.googlecode.com/svn/trunk@10547
2bbb7eff-a529-9590-31e7-
b0007b416f81
senorblanco@chromium.org [Mon, 5 Aug 2013 20:29:59 +0000 (20:29 +0000)]
Remove warning comment from SkImageFilter.
R=reed@google.com
Review URL: https://codereview.chromium.org/
22236002
git-svn-id: http://skia.googlecode.com/svn/trunk@10546
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Mon, 5 Aug 2013 20:25:57 +0000 (20:25 +0000)]
Minor sk_memset{16|32}_SSE2 optimization.
Using explicitly indexed references allows some compilers to generate more efficient loops. For gcc 4.6.3:
613c18: 83 ea 10 sub $0x10,%edx
613c1b: 66 0f 7f 07 movdqa %xmm0,(%rdi)
613c1f: 66 0f 7f 47 10 movdqa %xmm0,0x10(%rdi)
613c24: 66 0f 7f 47 20 movdqa %xmm0,0x20(%rdi)
613c29: 66 0f 7f 47 30 movdqa %xmm0,0x30(%rdi)
613c2e: 48 83 c7 40 add $0x40,%rdi
613c32: 83 fa 0f cmp $0xf,%edx
613c35: 7f e1 jg 613c18 <_Z16sk_memset32_SSE2Pjji+0x38>
vs. previous:
613c18: 83 ea 10 sub $0x10,%edx
613c1b: 66 0f 7f 07 movdqa %xmm0,(%rdi)
613c1f: 66 0f 7f 47 10 movdqa %xmm0,0x10(%rdi)
613c24: 66 0f 7f 47 20 movdqa %xmm0,0x20(%rdi)
613c29: 48 83 c7 40 add $0x40,%rdi
613c2d: 83 fa 0f cmp $0xf,%edx
613c30: 66 0f 7f 47 f0 movdqa %xmm0,-0x10(%rdi)
613c35: 7f e1 jg 613c18 <_Z16sk_memset32_SSE2Pjji+0x38>
This yields a 0.2% - 1% improvement with the memset micro benchmarks, presumably due to avoiding a stall on the next store after the %rdi increment.
R=reed@google.com, senorblanco@chromium.org
Author: fmalita@chromium.org
Review URL: https://chromiumcodereview.appspot.com/
21703003
git-svn-id: http://skia.googlecode.com/svn/trunk@10545
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Mon, 5 Aug 2013 19:42:56 +0000 (19:42 +0000)]
GPU Font Cache improvements:
- If a strike has multiple atlases, check all for room for a new glyph
- Mark remaining atlases unused after a purge, then check for an unused
atlas before purging (reduces TextContext flushes and ghosting)
- Hide Atlas management a little better inside AtlasMgr
R=robertphillips@google.com, bsalomon@google.com
Author: jvanverth@google.com
Review URL: https://chromiumcodereview.appspot.com/
21594005
git-svn-id: http://skia.googlecode.com/svn/trunk@10544
2bbb7eff-a529-9590-31e7-
b0007b416f81
bsalomon@google.com [Mon, 5 Aug 2013 18:27:23 +0000 (18:27 +0000)]
Revert "Add blend optimization helpers and use to convert rect draws to clears."
This reverts commit r10537.
git-svn-id: http://skia.googlecode.com/svn/trunk@10542
2bbb7eff-a529-9590-31e7-
b0007b416f81
edisonn@google.com [Mon, 5 Aug 2013 18:18:06 +0000 (18:18 +0000)]
pdfviewer: add virtual destructor for SkPdfFont
Review URL: https://codereview.chromium.org/
22239002
git-svn-id: http://skia.googlecode.com/svn/trunk@10541
2bbb7eff-a529-9590-31e7-
b0007b416f81
edisonn@google.com [Mon, 5 Aug 2013 17:59:11 +0000 (17:59 +0000)]
pdfviewer: fix typo
Review URL: https://codereview.chromium.org/
22076007
git-svn-id: http://skia.googlecode.com/svn/trunk@10540
2bbb7eff-a529-9590-31e7-
b0007b416f81
edisonn@google.com [Mon, 5 Aug 2013 17:26:11 +0000 (17:26 +0000)]
pdfviewer: set backdround to transparent, and add an app to chop transparent content in png files.
Review URL: https://codereview.chromium.org/
21981008
git-svn-id: http://skia.googlecode.com/svn/trunk@10538
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Mon, 5 Aug 2013 16:55:55 +0000 (16:55 +0000)]
Add blend optimization helpers and use to convert rect draws to clears.
R=robertphillips@google.com, jvanverth@google.com, reed@google.com
Author: bsalomon@google.com
Review URL: https://chromiumcodereview.appspot.com/
21877006
git-svn-id: http://skia.googlecode.com/svn/trunk@10537
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Mon, 5 Aug 2013 16:53:50 +0000 (16:53 +0000)]
Added ctm matrix to GPU path
There should be no changes in behavior caused by this cl, it just adds the ctm matrix to filterImageGPU so that it may be used for scaling on all platforms when it is implemented on the blink side.
BUG=
R=senorblanco@google.com, senorblanco@chromium.org
Author: sugoi@chromium.org
Review URL: https://chromiumcodereview.appspot.com/
22209002
git-svn-id: http://skia.googlecode.com/svn/trunk@10536
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Mon, 5 Aug 2013 16:31:27 +0000 (16:31 +0000)]
Fix OverdrawFilter not set on first call to SkDebugCanvas::drawTo()
R=robertphillips@google.com
Author: egraether@chromium.org
Review URL: https://chromiumcodereview.appspot.com/
21789005
git-svn-id: http://skia.googlecode.com/svn/trunk@10535
2bbb7eff-a529-9590-31e7-
b0007b416f81
edisonn@google.com [Mon, 5 Aug 2013 16:23:23 +0000 (16:23 +0000)]
pdfviewer: add indexed rbg image support, enhanche caching(setData) for SkPdfObject
Review URL: https://codereview.chromium.org/
21738005
git-svn-id: http://skia.googlecode.com/svn/trunk@10534
2bbb7eff-a529-9590-31e7-
b0007b416f81
jvanverth@google.com [Mon, 5 Aug 2013 14:50:31 +0000 (14:50 +0000)]
Temporarily disable GpuTestPath.
git-svn-id: http://skia.googlecode.com/svn/trunk@10532
2bbb7eff-a529-9590-31e7-
b0007b416f81
djsollen@google.com [Mon, 5 Aug 2013 13:38:28 +0000 (13:38 +0000)]
Enable SkUtilsArm on all ARM platforms and always use NDK compliant NEON detection on Android.
R=scroggo@google.com
Review URL: https://codereview.chromium.org/
22193002
git-svn-id: http://skia.googlecode.com/svn/trunk@10530
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Mon, 5 Aug 2013 13:30:59 +0000 (13:30 +0000)]
Actually flush the path stencil transform when the matrix changes
Fixes many tests to match the master gm more closely. Examples of fixes:
mixed_xfermodes, strokes_poly, gradients_view_perspective, strokes_round,
... .
Total 58 differing results.
BUG=1479
R=bsalomon@google.com
Author: kkinnunen@nvidia.com
Review URL: https://chromiumcodereview.appspot.com/
22176002
git-svn-id: http://skia.googlecode.com/svn/trunk@10529
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Mon, 5 Aug 2013 13:28:55 +0000 (13:28 +0000)]
Fix a crash on stroking empty paths with nv_path_rendering enabled
Fix the crash by defining that GrPathRenderer::drawPath and
GrPathRenderer::stencilPath are called only with non-empty paths.
Adds a new test "GpuDrawPath" and tests the condition.
BUG=1477
R=bsalomon@google.com
Author: kkinnunen@nvidia.com
Review URL: https://chromiumcodereview.appspot.com/
22173002
git-svn-id: http://skia.googlecode.com/svn/trunk@10528
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Mon, 5 Aug 2013 13:23:13 +0000 (13:23 +0000)]
Avoid counting verbs twice in SkPath::isEmpty()
Remove redundant call to SkPathRef::countVerbs. The intention was
probably to count points. Instead, assert that all two-verb paths
begin with a 'move' and that if the second verb is a 'line', then the
point count is indeed two.
BUG=1478
R=bsalomon@google.com
Author: kkinnunen@nvidia.com
Review URL: https://chromiumcodereview.appspot.com/
22171002
git-svn-id: http://skia.googlecode.com/svn/trunk@10527
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Sun, 4 Aug 2013 15:09:23 +0000 (15:09 +0000)]
In preparation for make Chromium's skia.gyp use pdf.gypi.
R=edisonn@google.com, djsollen@google.com
Author: robertphillips@google.com
Review URL: https://chromiumcodereview.appspot.com/
21952002
git-svn-id: http://skia.googlecode.com/svn/trunk@10526
2bbb7eff-a529-9590-31e7-
b0007b416f81
robertphillips@google.com [Sun, 4 Aug 2013 15:00:58 +0000 (15:00 +0000)]
N7 Performance trigger rebaselines
(SkipBuildbotRuns)
git-svn-id: http://skia.googlecode.com/svn/trunk@10525
2bbb7eff-a529-9590-31e7-
b0007b416f81
epoger@google.com [Fri, 2 Aug 2013 20:54:46 +0000 (20:54 +0000)]
Delete image-based rebaselining tool; we have switched to checksums
R=borenet@google.com
Review URL: https://codereview.chromium.org/
21901004
git-svn-id: http://skia.googlecode.com/svn/trunk@10524
2bbb7eff-a529-9590-31e7-
b0007b416f81
edisonn@google.com [Fri, 2 Aug 2013 20:24:48 +0000 (20:24 +0000)]
pdfviewer: implementation of one type of pattern - simple tile patern, colored, with xstep and ystep positive.
Review URL: https://codereview.chromium.org/
21919003
git-svn-id: http://skia.googlecode.com/svn/trunk@10523
2bbb7eff-a529-9590-31e7-
b0007b416f81
bsalomon@google.com [Fri, 2 Aug 2013 19:54:28 +0000 (19:54 +0000)]
correctly copy constant coverage from GrPaint to GrDrawState
R=robertphillips@google.com
Review URL: https://codereview.chromium.org/
21924006
git-svn-id: http://skia.googlecode.com/svn/trunk@10522
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Fri, 2 Aug 2013 19:10:06 +0000 (19:10 +0000)]
Adds poppler and dependencies to DEPS. This is a pre-CL for
20220002, and getting DEPS committed first is necessary to get the trybots to run on the larger CL (since the trybots do 'gclient sync' before patching).
R=vandebo@chromium.org, edisonn@google.com, djsollen@chromium.org, bungeman@chromium.org, djsollen@google.com
Author: richardlin@chromium.org
Review URL: https://chromiumcodereview.appspot.com/
21945002
git-svn-id: http://skia.googlecode.com/svn/trunk@10521
2bbb7eff-a529-9590-31e7-
b0007b416f81
epoger@google.com [Fri, 2 Aug 2013 19:08:52 +0000 (19:08 +0000)]
Create AJAX live-viewer of expected-vs-actual GM results
R=zachr@google.com
Review URL: https://codereview.chromium.org/
20526007
git-svn-id: http://skia.googlecode.com/svn/trunk@10520
2bbb7eff-a529-9590-31e7-
b0007b416f81
scroggo@google.com [Fri, 2 Aug 2013 16:09:10 +0000 (16:09 +0000)]
Restore a default timer to bench_pictures.
https://codereview.chromium.org/
19862002 removed the default timer
to display from PictureBenchmark. Restore a default by putting it
in the flag description.
R=mtklein@google.com
Review URL: https://codereview.chromium.org/
21567005
git-svn-id: http://skia.googlecode.com/svn/trunk@10516
2bbb7eff-a529-9590-31e7-
b0007b416f81
zachr@google.com [Fri, 2 Aug 2013 15:54:30 +0000 (15:54 +0000)]
fix skpdiff viewer bug when using relative paths
BUG=skia:1463
R=djsollen@google.com
Review URL: https://codereview.chromium.org/
21601002
git-svn-id: http://skia.googlecode.com/svn/trunk@10515
2bbb7eff-a529-9590-31e7-
b0007b416f81
sugoi@google.com [Fri, 2 Aug 2013 15:45:41 +0000 (15:45 +0000)]
Removed duplicate file
BUG=
Review URL: https://codereview.chromium.org/
21890003
git-svn-id: http://skia.googlecode.com/svn/trunk@10514
2bbb7eff-a529-9590-31e7-
b0007b416f81
junov@chromium.org [Fri, 2 Aug 2013 15:36:02 +0000 (15:36 +0000)]
Fixing SkDeferredCanvas::writePixels to trigger appropriate change notifications to SkSurface
BUG=crbug.com/256269
TEST=DeferredCanvas unit test, TestDeferredCanvasWritePixelsToSurface
R=reed@google.com
Review URL: https://codereview.chromium.org/
20628005
git-svn-id: http://skia.googlecode.com/svn/trunk@10513
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Fri, 2 Aug 2013 15:19:40 +0000 (15:19 +0000)]
Add nv_path_rendering entry points to Unix GLInterface
Add nv_path_rendering entry points to Unix GLInterface.
Also add few other GL entry points that are used by nv_path_rendering
codepaths.
BUG=1468
R=bsalomon@google.com
Author: kkinnunen@nvidia.com
Review URL: https://chromiumcodereview.appspot.com/
21808003
git-svn-id: http://skia.googlecode.com/svn/trunk@10512
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Fri, 2 Aug 2013 15:14:24 +0000 (15:14 +0000)]
add remote debugging support for x86 phone
R=djsollen@google.com, borenet@google.com
Author: yunchao.he@intel.com
Review URL: https://chromiumcodereview.appspot.com/
19920003
git-svn-id: http://skia.googlecode.com/svn/trunk@10511
2bbb7eff-a529-9590-31e7-
b0007b416f81
bsalomon@google.com [Fri, 2 Aug 2013 14:39:25 +0000 (14:39 +0000)]
Add fullscreen rect bench
R=robertphillips@google.com
Review URL: https://codereview.chromium.org/
21592002
git-svn-id: http://skia.googlecode.com/svn/trunk@10510
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Fri, 2 Aug 2013 13:59:50 +0000 (13:59 +0000)]
Support MSAA in the picture debugger
Add radio buttons for setting the GL sample count to 0 ("off"), 4 or
16.
Change the default mode of the GL widget to MSAA4. Previous behavior
corresponded to "off".
BUG=1459
R=robertphillips@google.com
Author: kkinnunen@nvidia.com
Review URL: https://chromiumcodereview.appspot.com/
21752002
git-svn-id: http://skia.googlecode.com/svn/trunk@10509
2bbb7eff-a529-9590-31e7-
b0007b416f81
skia.committer@gmail.com [Fri, 2 Aug 2013 07:00:59 +0000 (07:00 +0000)]
Sanitizing source files in Housekeeper-Nightly
git-svn-id: http://skia.googlecode.com/svn/trunk@10503
2bbb7eff-a529-9590-31e7-
b0007b416f81
bungeman@google.com [Fri, 2 Aug 2013 00:20:34 +0000 (00:20 +0000)]
Add onRefMatchingStyle to SkFontHost_linux.
git-svn-id: http://skia.googlecode.com/svn/trunk@10502
2bbb7eff-a529-9590-31e7-
b0007b416f81
bungeman@google.com [Thu, 1 Aug 2013 22:48:21 +0000 (22:48 +0000)]
Fix SkTMin call introduced in 10500 which fails on some platforms.
git-svn-id: http://skia.googlecode.com/svn/trunk@10501
2bbb7eff-a529-9590-31e7-
b0007b416f81
bungeman@google.com [Thu, 1 Aug 2013 22:29:43 +0000 (22:29 +0000)]
Implement onGetTableTags and onGetTableData on Windows.
Implements these and removes default implementation, making the
declaration in SkTypeface pure virtual.
Review URL: https://codereview.chromium.org/
20672004
git-svn-id: http://skia.googlecode.com/svn/trunk@10500
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Thu, 1 Aug 2013 21:54:26 +0000 (21:54 +0000)]
Fix memory leak in SkPDFResourceDict
R=vandebo@chromium.org, edisonn@google.com
Author: richardlin@chromium.org
Review URL: https://chromiumcodereview.appspot.com/
20655004
git-svn-id: http://skia.googlecode.com/svn/trunk@10499
2bbb7eff-a529-9590-31e7-
b0007b416f81
edisonn@google.com [Thu, 1 Aug 2013 21:20:47 +0000 (21:20 +0000)]
pdfviewer: debug the parameters for snc, and for resolveReference
Review URL: https://codereview.chromium.org/
21604003
git-svn-id: http://skia.googlecode.com/svn/trunk@10498
2bbb7eff-a529-9590-31e7-
b0007b416f81
bungeman@google.com [Thu, 1 Aug 2013 21:01:28 +0000 (21:01 +0000)]
Reintroduce default impls removed in 10495.
Due to poor placement of overrides in the hierarchy, nacl is failing.
git-svn-id: http://skia.googlecode.com/svn/trunk@10497
2bbb7eff-a529-9590-31e7-
b0007b416f81
bungeman@google.com [Thu, 1 Aug 2013 20:18:41 +0000 (20:18 +0000)]
Implement onGetTableTags and onGetTableData on Windows.
Implements these and removes default implementation, making the
declaration in SkTypeface pure virtual.
Review URL: https://codereview.chromium.org/
20672004/
git-svn-id: http://skia.googlecode.com/svn/trunk@10495
2bbb7eff-a529-9590-31e7-
b0007b416f81
edisonn@google.com [Thu, 1 Aug 2013 20:06:42 +0000 (20:06 +0000)]
pdfviewer: finish implementation of SkPdfObject.toString(), and include in debugging traces we leave when rendering.
Review URL: https://codereview.chromium.org/
20628008
git-svn-id: http://skia.googlecode.com/svn/trunk@10494
2bbb7eff-a529-9590-31e7-
b0007b416f81
djsollen@google.com [Thu, 1 Aug 2013 18:36:08 +0000 (18:36 +0000)]
Revert "Don't call SkGpuDevice::drawVertices from drawPoints when AA is required."
This reverts commit r10489.
Review URL: https://codereview.chromium.org/
21496003
git-svn-id: http://skia.googlecode.com/svn/trunk@10492
2bbb7eff-a529-9590-31e7-
b0007b416f81
senorblanco@chromium.org [Thu, 1 Aug 2013 18:33:03 +0000 (18:33 +0000)]
New baselines for imagefiltersgraph GM after r10482.
TBR=djsollen
BUG=
Review URL: https://codereview.chromium.org/
20938004
git-svn-id: http://skia.googlecode.com/svn/trunk@10491
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Thu, 1 Aug 2013 17:32:56 +0000 (17:32 +0000)]
Added 3-color gradient scraper for detecting symmetrical gradients.
Made changes to lua to scrape info about 3 color gradients
BUG=
R=reed@google.com
Author: dierk@google.com
Review URL: https://chromiumcodereview.appspot.com/
21571002
git-svn-id: http://skia.googlecode.com/svn/trunk@10490
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Thu, 1 Aug 2013 17:30:32 +0000 (17:30 +0000)]
Don't call SkGpuDevice::drawVertices from drawPoints when AA is required.
R=jvanverth@google.com
Author: bsalomon@google.com
Review URL: https://chromiumcodereview.appspot.com/
12943007
git-svn-id: http://skia.googlecode.com/svn/trunk@10489
2bbb7eff-a529-9590-31e7-
b0007b416f81
egdaniel@google.com [Thu, 1 Aug 2013 17:09:11 +0000 (17:09 +0000)]
Add Loop Blinn hairline conics to GPU
BUG=
R=bsalomon@google.com
Review URL: https://codereview.chromium.org/
21036008
git-svn-id: http://skia.googlecode.com/svn/trunk@10487
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Thu, 1 Aug 2013 15:58:07 +0000 (15:58 +0000)]
When skia run bench cases to test performance, it will run constructors for all cases one by one, then getName to skip unnecessary cases according to command line parameters, so these constructors should be lightweight enough to avoid redundant computing. Unfortunately, some constructors contain intensive computing/rendering. They are very heavy, maybe much heavier than need-to-run bench case itself. And these redundant computation will be run every time you run bench, even you just test a single simple case. Moreover, it will mislead the real hotspot/bottleneck of the case itself.
For example, run a lightweight case, say, region_intersectsrgn_16, the hot spots are gles operation, SuperBlitter, SkRTree... introduced by irrelevant cases' constructors. These redundant computation will mislead performance tuning.
So we can move these intensive computation to onPreDraw() of these case. They will be executed only if this case should be run.
R=reed@google.com, robertphillips@google.com, humper@google.com, tomhudson@chromium.org
Author: yunchao.he@intel.com
Review URL: https://chromiumcodereview.appspot.com/
20997003
git-svn-id: http://skia.googlecode.com/svn/trunk@10486
2bbb7eff-a529-9590-31e7-
b0007b416f81
reed@google.com [Thu, 1 Aug 2013 15:53:39 +0000 (15:53 +0000)]
stub support for SK_FONTHOST_USES_FONTMGR
BUG=
Review URL: https://codereview.chromium.org/
21563002
git-svn-id: http://skia.googlecode.com/svn/trunk@10485
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Thu, 1 Aug 2013 15:35:06 +0000 (15:35 +0000)]
Make SkPath::conservativelyContainsRect not assert on paths that begin with repeated moveTos
R=reed@google.com
Author: bsalomon@google.com
Review URL: https://chromiumcodereview.appspot.com/
21565002
git-svn-id: http://skia.googlecode.com/svn/trunk@10484
2bbb7eff-a529-9590-31e7-
b0007b416f81
scroggo@google.com [Thu, 1 Aug 2013 15:03:42 +0000 (15:03 +0000)]
Fixes for JPEG subset decoding.
Ensure that the stream passed to JPEG for subset decoding is
neither deleted before it is no longer needed nor deleted an
extra time.
src/images/SkJpegUtility.h:
src/images/SkJpegUtility.cpp:
Always ref and unref the stream provided to skjpeg_source_mgr.
Add some comments explaining how skjpeg_source_mgr's members
handle ownership.
Fix a warning comparing signed and unsigned numbers, converting
to size_t which is more appropriate for measuring bytes.
Remove dead code referring to fMemoryBase and fMemoryBaseSize,
which are never used.
src/images/SkImageDecoder_libjpeg.cpp:
Call the new constructor for skjpeg_source_mgr, which no longer
takes a boolean to determine ownership.
include/images/SkBitmapRegionDecoder.h
src/images/SkBitmapRegionDecoder.cpp:
This small shim has been removed, since it is not needed to use
Skia's image decoders. Its functionality will be folded into
Android.
Required for the merge to Android.
R=djsollen@google.com
Review URL: https://codereview.chromium.org/
21561002
git-svn-id: http://skia.googlecode.com/svn/trunk@10483
2bbb7eff-a529-9590-31e7-
b0007b416f81
senorblanco@chromium.org [Thu, 1 Aug 2013 14:59:05 +0000 (14:59 +0000)]
Remove SkBlendImageFilter, and all tests. Its functionality has been subsumed by SkXfermodeImageFilter.
R=robertphillips@google.com
Review URL: https://codereview.chromium.org/
21404002
git-svn-id: http://skia.googlecode.com/svn/trunk@10482
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Thu, 1 Aug 2013 14:23:33 +0000 (14:23 +0000)]
Mark GrMemoryPool methods as static.
R=jvanverth@google.com
Author: bsalomon@google.com
Review URL: https://chromiumcodereview.appspot.com/
21553002
git-svn-id: http://skia.googlecode.com/svn/trunk@10481
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Thu, 1 Aug 2013 14:23:23 +0000 (14:23 +0000)]
ARM Skia NEON patches - 14 - S32A_Blend
Blitrow32: S32A_Blend new NEON version
Adding a NEON version of S32A_Blend_BlitRow32. Here are the
benchmark results:
+-------+--------------------------+--------------------------+
| | Speedup vs. C | Speedup vs. ARM asm |
| count +------------+-------------+------------+-------------+
| | Cortex A-9 | Cortex A-15 | Cortex A-9 | Cortex A-15 |
+-------+------------+-------------+------------+-------------+
| 1 | +8,5% | +18,5% | +0.9% | +2,9% |
+-------+------------+-------------+------------+-------------+
| 2 | +65,6% | +94% | +70,3% | +80% |
+-------+------------+-------------+------------+-------------+
| 4 | +42,4% | +87,8% | +56,8% | +84,4% |
+-------+------------+-------------+------------+-------------+
| 8 | +30% | +90% | +49,9% | +82,7% |
+-------+------------+-------------+------------+-------------+
| 16 | +23,1% | +95,4% | +46,6% | +87,6% |
+-------+------------+-------------+------------+-------------+
| 64 | +23,1% | +95,7% | +46,1% | +89,4% |
+-------+------------+-------------+------------+-------------+
| 256 | +35,5% | +122% | +53,6% | +99,2% |
+-------+------------+-------------+------------+-------------+
| 1024 | +61,8% | +101% | +64,2% | +91,2% |
+-------+------------+-------------+------------+-------------+
BUG=
R=djsollen@google.com
Author: kevin.petit.arm@gmail.com
Review URL: https://chromiumcodereview.appspot.com/
18614010
git-svn-id: http://skia.googlecode.com/svn/trunk@10480
2bbb7eff-a529-9590-31e7-
b0007b416f81
edisonn@google.com [Thu, 1 Aug 2013 13:24:00 +0000 (13:24 +0000)]
pdfviewer: remove simple implementation for non-isolated groups. will be implemented in skia most probably. also it is not generated by SkPDFDevice
Review URL: https://codereview.chromium.org/
21537003
git-svn-id: http://skia.googlecode.com/svn/trunk@10479
2bbb7eff-a529-9590-31e7-
b0007b416f81
robertphillips@google.com [Thu, 1 Aug 2013 12:08:58 +0000 (12:08 +0000)]
N7 performance trigger rebaselines
(SkipBuildbotRuns)
git-svn-id: http://skia.googlecode.com/svn/trunk@10478
2bbb7eff-a529-9590-31e7-
b0007b416f81
skia.committer@gmail.com [Thu, 1 Aug 2013 07:01:05 +0000 (07:01 +0000)]
Sanitizing source files in Housekeeper-Nightly
git-svn-id: http://skia.googlecode.com/svn/trunk@10477
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Wed, 31 Jul 2013 22:54:31 +0000 (22:54 +0000)]
Inverse fill support in PDF
BUG= https://code.google.com/p/skia/issues/detail?id=241 (partial fix)
R=edisonn@google.com, vandebo@chromium.org, reed@google.com
Author: richardlin@chromium.org
Review URL: https://chromiumcodereview.appspot.com/
19519017
git-svn-id: http://skia.googlecode.com/svn/trunk@10476
2bbb7eff-a529-9590-31e7-
b0007b416f81
senorblanco@chromium.org [Wed, 31 Jul 2013 20:21:50 +0000 (20:21 +0000)]
New baselines for imagefilterscropped.
TBR=robertphillips
BUG=
Review URL: https://codereview.chromium.org/
21340005
git-svn-id: http://skia.googlecode.com/svn/trunk@10475
2bbb7eff-a529-9590-31e7-
b0007b416f81
mtklein@google.com [Wed, 31 Jul 2013 20:09:25 +0000 (20:09 +0000)]
Enable runtime checks for SSSE3 on x86 on Android.
$ compare-android.sh bench --match bitmap_ --repeat 30
master -> ssse3
N=30 p=0.001000 (corrected to 0.000033)
sig? speedup bench
n -1.16% bitmap_scale_filter_256_64
y -0.72% bitmap_8888_A_scale_bicubic
y -0.21% bitmap_index8_A
n -0.00% bitmap_565
n -0.00% bitmap_scale_filter_90_80
n 0.03% bitmap_8888_A_source_transparent
y 0.06% bitmap_index8
y 0.30% bitmap_8888_A_source_stripes_two
n 0.34% bitmap_scale_filter_80_90
y 0.42% bitmap_8888_A
y 0.44% bitmap_8888_A_source_opaque
n 0.53% bitmap_scale_filter_90_10
y 0.71% bitmap_8888_A_source_stripes_three
y 0.91% bitmap_8888_A_scale_rotate_bicubic
y 1.04% bitmap_8888_update
n 1.19% bitmap_scale_filter_10_90
n 1.39% bitmap_scale_filter_90_90
y 1.77% bitmap_8888_update_volatile
y 1.89% bitmap_8888
y 2.37% bitmap_scale_filter_30_90
y 9.57% bitmap_scale_filter_64_256
n 17.86% bitmap_scale_filter_90_30
y 25.40% bitmap_8888_A_scale_rotate_bilerp
y 27.19% bitmap_8888_scale_rotate_bilerp
y 27.23% bitmap_8888_update_scale_rotate_bilerp
y 27.29% bitmap_8888_update_volatile_scale_rotate_bilerp
y 55.08% bitmap_8888_A_scale_bilerp
y 58.75% bitmap_8888_update_volatile_scale_bilerp
y 58.90% bitmap_8888_scale_bilerp
y 58.92% bitmap_8888_update_scale_bilerp
Overall speedup: 10.52%
BUG=skia:1111
R=djsollen@google.com
Review URL: https://codereview.chromium.org/
21203005
git-svn-id: http://skia.googlecode.com/svn/trunk@10474
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Wed, 31 Jul 2013 20:00:56 +0000 (20:00 +0000)]
Committed: code.google.com/p/skia/source/detail?r=10372
R=borenet@google.com, bungeman@google.com, robertphillips@google.com, scroggo@google.com, sglez@google.com
Author: bsalomon@google.com
Review URL: https://chromiumcodereview.appspot.com/
19862002
git-svn-id: http://skia.googlecode.com/svn/trunk@10473
2bbb7eff-a529-9590-31e7-
b0007b416f81
reed@google.com [Wed, 31 Jul 2013 19:36:07 +0000 (19:36 +0000)]
Revert "Revert "Revert "try enabling FONTHOST_USES_FONTMGR and see what breaks"""
This reverts commit
d0a1a568ac5d5c2ed94e826e91b01241de2ca0ce.
git-svn-id: http://skia.googlecode.com/svn/trunk@10472
2bbb7eff-a529-9590-31e7-
b0007b416f81
reed@google.com [Wed, 31 Jul 2013 19:00:30 +0000 (19:00 +0000)]
Revert "Revert "try enabling FONTHOST_USES_FONTMGR and see what breaks""
This reverts commit
a8946cf4180908d577681cb62d621db0ebcbe67e.
i.e. reapply the change to try enabling FONTHOST_USES_FONTMGR
git-svn-id: http://skia.googlecode.com/svn/trunk@10471
2bbb7eff-a529-9590-31e7-
b0007b416f81
bsalomon@google.com [Wed, 31 Jul 2013 18:52:31 +0000 (18:52 +0000)]
set nominmax on windows
R=scroggo@google.com
Review URL: https://codereview.chromium.org/
20990007
git-svn-id: http://skia.googlecode.com/svn/trunk@10470
2bbb7eff-a529-9590-31e7-
b0007b416f81
reed@google.com [Wed, 31 Jul 2013 18:49:33 +0000 (18:49 +0000)]
add impls for createfromstream for fontconfig fontmgr
git-svn-id: http://skia.googlecode.com/svn/trunk@10469
2bbb7eff-a529-9590-31e7-
b0007b416f81
reed@google.com [Wed, 31 Jul 2013 18:25:33 +0000 (18:25 +0000)]
Revert "try enabling FONTHOST_USES_FONTMGR and see what breaks"
This reverts commit
a289c38fc14664b5ba1269cfe08b3048cdbd1e97.
git-svn-id: http://skia.googlecode.com/svn/trunk@10468
2bbb7eff-a529-9590-31e7-
b0007b416f81
edisonn@google.com [Wed, 31 Jul 2013 18:22:36 +0000 (18:22 +0000)]
pdfviewer: transparency groups - implementation of non-isolated and non-knockout flags
Review URL: https://codereview.chromium.org/
21288002
git-svn-id: http://skia.googlecode.com/svn/trunk@10467
2bbb7eff-a529-9590-31e7-
b0007b416f81
reed@google.com [Wed, 31 Jul 2013 18:08:49 +0000 (18:08 +0000)]
try enabling FONTHOST_USES_FONTMGR and see what breaks
git-svn-id: http://skia.googlecode.com/svn/trunk@10466
2bbb7eff-a529-9590-31e7-
b0007b416f81
reed@google.com [Wed, 31 Jul 2013 18:02:55 +0000 (18:02 +0000)]
add dox to SkTDynamicHash, fix typo of Key instead of KEY
BUG=
R=mtklein@google.com
Review URL: https://codereview.chromium.org/
21028008
git-svn-id: http://skia.googlecode.com/svn/trunk@10465
2bbb7eff-a529-9590-31e7-
b0007b416f81
reed@google.com [Wed, 31 Jul 2013 17:37:31 +0000 (17:37 +0000)]
adapt FontConfig to use SK_FONTHOST_USES_FONTMGR
BUG=
Review URL: https://codereview.chromium.org/
21331003
git-svn-id: http://skia.googlecode.com/svn/trunk@10463
2bbb7eff-a529-9590-31e7-
b0007b416f81
mtklein@google.com [Wed, 31 Jul 2013 16:01:25 +0000 (16:01 +0000)]
Remove a stray double assign that g++ 4.8 thinks has ill-defined sequencing.
BUG=
R=bungeman@google.com
Review URL: https://codereview.chromium.org/
21352003
git-svn-id: http://skia.googlecode.com/svn/trunk@10462
2bbb7eff-a529-9590-31e7-
b0007b416f81
djsollen@google.com [Wed, 31 Jul 2013 15:48:10 +0000 (15:48 +0000)]
Upstream changes from Android
R=scroggo@google.com
Review URL: https://codereview.chromium.org/
21053009
git-svn-id: http://skia.googlecode.com/svn/trunk@10461
2bbb7eff-a529-9590-31e7-
b0007b416f81
robertphillips@google.com [Wed, 31 Jul 2013 14:57:53 +0000 (14:57 +0000)]
added drawVertices path to GameBench
https://codereview.chromium.org/
21168006/
git-svn-id: http://skia.googlecode.com/svn/trunk@10459
2bbb7eff-a529-9590-31e7-
b0007b416f81
djsollen@google.com [Wed, 31 Jul 2013 12:57:27 +0000 (12:57 +0000)]
update gyp for ARM builds to be more well defined.
R=mtklein@google.com
Review URL: https://codereview.chromium.org/
21096005
git-svn-id: http://skia.googlecode.com/svn/trunk@10454
2bbb7eff-a529-9590-31e7-
b0007b416f81
skia.committer@gmail.com [Wed, 31 Jul 2013 07:01:00 +0000 (07:01 +0000)]
Sanitizing source files in Housekeeper-Nightly
git-svn-id: http://skia.googlecode.com/svn/trunk@10449
2bbb7eff-a529-9590-31e7-
b0007b416f81
senorblanco@chromium.org [Wed, 31 Jul 2013 01:17:42 +0000 (01:17 +0000)]
Rebaseline blur tests for r10444. Max color mismatch 1.
TBR=
BUG=
Review URL: https://codereview.chromium.org/
21235011
git-svn-id: http://skia.googlecode.com/svn/trunk@10448
2bbb7eff-a529-9590-31e7-
b0007b416f81
robertphillips@google.com [Wed, 31 Jul 2013 00:00:56 +0000 (00:00 +0000)]
Alter assert that is causing Chromium compilation failure
../../third_party/skia/include/core/SkTypeface.h:303:19: error: implicit conversion turns string literal into bool: 'const char [14]' to 'bool' [-Werror,-Wstring-conversion]
SkASSERT(!"unimplemented");
~^~~~~~~~~~~~~~~
git-svn-id: http://skia.googlecode.com/svn/trunk@10447
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Tue, 30 Jul 2013 21:54:10 +0000 (21:54 +0000)]
+ ,fLastAllocated(NULL)
BUG=
R=scroggo@google.com
Author: mtklein@google.com
Review URL: https://chromiumcodereview.appspot.com/
21046011
git-svn-id: http://skia.googlecode.com/svn/trunk@10446
2bbb7eff-a529-9590-31e7-
b0007b416f81
reed@google.com [Tue, 30 Jul 2013 21:25:49 +0000 (21:25 +0000)]
support SK_FONTHOST_USES_FONTMGR on mac
BUG=
Review URL: https://codereview.chromium.org/
21142004
git-svn-id: http://skia.googlecode.com/svn/trunk@10445
2bbb7eff-a529-9590-31e7-
b0007b416f81
senorblanco@chromium.org [Tue, 30 Jul 2013 21:11:05 +0000 (21:11 +0000)]
Optimize the blur convolution by only applying domain clamping where necessary. Split the blur convolution into left and right margins which have clamping applied and a center portion which doesn't. Also reorder the convolve_gaussian() parameters to match GrConvolutionEffect params. Correctness is covered by existing tests; performance is covered by BlurImageFilter bench.
R=bsalomon@google.com
Review URL: https://codereview.chromium.org/
21224003
git-svn-id: http://skia.googlecode.com/svn/trunk@10444
2bbb7eff-a529-9590-31e7-
b0007b416f81
bungeman@google.com [Tue, 30 Jul 2013 19:32:16 +0000 (19:32 +0000)]
Remove old default implementation of SkTypeface::onGetUPEM.
In order to introduce onGetUPEM there was written a generic but slow
version in SkTypeface.cpp. All ports now implement this method and the
declaration is now marked as pure virtual.
git-svn-id: http://skia.googlecode.com/svn/trunk@10442
2bbb7eff-a529-9590-31e7-
b0007b416f81