halcanary@google.com [Thu, 5 Dec 2013 14:18:07 +0000 (14:18 +0000)]
Fix MacOS build error from
36d08c5c90c7
BUG=
Review URL: https://codereview.chromium.org/
105453005
git-svn-id: http://skia.googlecode.com/svn/trunk@12506
2bbb7eff-a529-9590-31e7-
b0007b416f81
halcanary@google.com [Thu, 5 Dec 2013 14:00:03 +0000 (14:00 +0000)]
SkCachingPixelRef to use SkImageGenerator
- Remove SkLazyCachingPixelRef class.
- Refactor unit tests.
BUG=
R=reed@google.com, scroggo@google.com
Review URL: https://codereview.chromium.org/
84083002
git-svn-id: http://skia.googlecode.com/svn/trunk@12505
2bbb7eff-a529-9590-31e7-
b0007b416f81
djsollen@google.com [Thu, 5 Dec 2013 13:57:54 +0000 (13:57 +0000)]
update android build to use SDK API level 19.
BUG=1817
Review URL: https://codereview.chromium.org/
88223002
git-svn-id: http://skia.googlecode.com/svn/trunk@12504
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Thu, 5 Dec 2013 13:45:19 +0000 (13:45 +0000)]
V8 and Skia
R=robertphillips@google.com
Author: jcgregorio@google.com
Review URL: https://codereview.chromium.org/
93933005
git-svn-id: http://skia.googlecode.com/svn/trunk@12503
2bbb7eff-a529-9590-31e7-
b0007b416f81
robertphillips@google.com [Thu, 5 Dec 2013 13:39:47 +0000 (13:39 +0000)]
Image rebaselines
https://codereview.chromium.org/
101313004/
git-svn-id: http://skia.googlecode.com/svn/trunk@12502
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Thu, 5 Dec 2013 12:08:12 +0000 (12:08 +0000)]
Fix compilation with SK_ENABLE_INST_COUNT=1
Add INHERITED declarations to class declarations that prevent
compilation with the flag.
Remove SK_DEFINE_INST_COUNT from all class implementations. Instead,
use function-local static variables in the reference count helper
classes to create the global instances to store the needed info. The
accessor functions are defined inline in the helper classes, so
definitions are not needed. The initialization point of the variables
should be as well defined as previously.
Remove SK_DECLARE_INST_COUNT_TEMPLATE and use SK_DECLARE_INST_COUNT
instead. This avoids possible future compilation errors further.
For SK_ENABLE_INST_COUNT=0 compilation, add an empty static member
function to all classes that use SK_DECLARE_INST_COUNT and
SK_DECLARE_INST_COUNT_ROOT macros. The function ensures that classes
contain public INHERITED typedef. This member function seems to be
compiled away. This shouĺd ensure that part of the compilation errors
are caught earlier.
Also adds DSK_DECLARE_INST_COUNT to few SkPDFDict subclasses.
R=robertphillips@google.com, richardlin@chromium.org, bsalomon@google.com
Author: kkinnunen@nvidia.com
Review URL: https://codereview.chromium.org/
98703002
git-svn-id: http://skia.googlecode.com/svn/trunk@12501
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Thu, 5 Dec 2013 12:07:44 +0000 (12:07 +0000)]
Move non-trivial work in RectBench() to onPreDraw.
Saw this on a profile where rects was filtered out. ಠ_ಠ
BUG=
R=reed@google.com
Author: mtklein@google.com
Review URL: https://codereview.chromium.org/
93703003
git-svn-id: http://skia.googlecode.com/svn/trunk@12500
2bbb7eff-a529-9590-31e7-
b0007b416f81
skia.committer@gmail.com [Thu, 5 Dec 2013 07:02:16 +0000 (07:02 +0000)]
Sanitizing source files in Housekeeper-Nightly
git-svn-id: http://skia.googlecode.com/svn/trunk@12499
2bbb7eff-a529-9590-31e7-
b0007b416f81
bensong@google.com [Thu, 5 Dec 2013 01:31:56 +0000 (01:31 +0000)]
Fixes bench_util's CONFIG_RE to cope with space flexibility.
The real "fix" down the road is Joe's ongoing work of switching to JSON.
Review URL: https://codereview.chromium.org/
103943004
git-svn-id: http://skia.googlecode.com/svn/trunk@12498
2bbb7eff-a529-9590-31e7-
b0007b416f81
jvanverth@google.com [Thu, 5 Dec 2013 00:28:04 +0000 (00:28 +0000)]
More rebaselines for https://codereview.chromium.org/
102683002
Review URL: https://codereview.chromium.org/
105963002
git-svn-id: http://skia.googlecode.com/svn/trunk@12497
2bbb7eff-a529-9590-31e7-
b0007b416f81
jvanverth@google.com [Wed, 4 Dec 2013 23:46:30 +0000 (23:46 +0000)]
Rebaseline ANGLE for https://codereview.chromium.org/
102683002.
git-svn-id: http://skia.googlecode.com/svn/trunk@12496
2bbb7eff-a529-9590-31e7-
b0007b416f81
jvanverth@google.com [Wed, 4 Dec 2013 23:16:40 +0000 (23:16 +0000)]
Update ignored tests.
BUG=
Review URL: https://codereview.chromium.org/
105793002
git-svn-id: http://skia.googlecode.com/svn/trunk@12495
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Wed, 4 Dec 2013 21:33:08 +0000 (21:33 +0000)]
Simplify matrix generation for convex path renderer
Addresses precision issues which were leading to asserts in Debug
R=bsalomon@google.com, mtklein@google.com
Author: jvanverth@google.com
Review URL: https://codereview.chromium.org/
102683002
git-svn-id: http://skia.googlecode.com/svn/trunk@12494
2bbb7eff-a529-9590-31e7-
b0007b416f81
senorblanco@chromium.org [Wed, 4 Dec 2013 20:55:24 +0000 (20:55 +0000)]
Nit to self: NULL is not false.
R=mtklein@google.com, mtklein
BUG=
Review URL: https://codereview.chromium.org/
105423002
git-svn-id: http://skia.googlecode.com/svn/trunk@12493
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Wed, 4 Dec 2013 20:07:47 +0000 (20:07 +0000)]
Changed maxInputCount for exact inputCount
Processing still had issue when the number of inputs was 0, so I changed my previous fix from a maximum input count to an exact input count. -1 is used when the input count isn't fixed (but still has to be a non-negative number).
BUG=
R=senorblanco@chromium.org, reed@google.com, sugoi@google.com, bsalomon@google.com
Author: sugoi@chromium.org
Review URL: https://codereview.chromium.org/
100803004
git-svn-id: http://skia.googlecode.com/svn/trunk@12492
2bbb7eff-a529-9590-31e7-
b0007b416f81
scroggo@google.com [Wed, 4 Dec 2013 19:56:31 +0000 (19:56 +0000)]
Remove deprecated setPrefConfigTable.
Safe once its only caller (in Android) has been updated.
BUG=skia:1874
R=reed@google.com
Review URL: https://codereview.chromium.org/
99473004
git-svn-id: http://skia.googlecode.com/svn/trunk@12491
2bbb7eff-a529-9590-31e7-
b0007b416f81
senorblanco@chromium.org [Wed, 4 Dec 2013 19:53:41 +0000 (19:53 +0000)]
Do proper NEON checking for SkBoxBlur procs.
TBR=mtklein
BUG=
Review URL: https://codereview.chromium.org/
98373003
git-svn-id: http://skia.googlecode.com/svn/trunk@12490
2bbb7eff-a529-9590-31e7-
b0007b416f81
scroggo@google.com [Wed, 4 Dec 2013 18:34:58 +0000 (18:34 +0000)]
Force linking for skhello.
Otherwise encoders are stripped and the program produces nothing.
git-svn-id: http://skia.googlecode.com/svn/trunk@12487
2bbb7eff-a529-9590-31e7-
b0007b416f81
senorblanco@chromium.org [Wed, 4 Dec 2013 18:19:45 +0000 (18:19 +0000)]
Implement a NEON version of the RGBA gaussian blur. This shows a 9-15% speedup on Nexus-10.
R=mtklein@google.com, mtklein
before:
running bench [640 480] blur_image_filter_large_10.00_10.00 8888: cmsecs = 33063.23
running bench [640 480] blur_image_filter_small_10.00_10.00 8888: cmsecs = 32800.25
running bench [640 480] blur_image_filter_large_1.00_1.00 8888: cmsecs = 33017.88
running bench [640 480] blur_image_filter_small_1.00_1.00 8888: cmsecs = 32743.35
running bench [640 480] blur_image_filter_large_0.00_1.00 8888: cmsecs = 21024.04
running bench [640 480] blur_image_filter_large_0.00_10.00 8888: cmsecs = 22904.15
running bench [640 480] blur_image_filter_large_1.00_0.00 8888: cmsecs = 18738.08
running bench [640 480] blur_image_filter_large_10.00_0.00 8888: cmsecs = 18798.98
after:
running bench [640 480] blur_image_filter_large_10.00_10.00 8888: cmsecs = 30180.96
running bench [640 480] blur_image_filter_small_10.00_10.00 8888: cmsecs = 29861.90
running bench [640 480] blur_image_filter_large_1.00_1.00 8888: cmsecs = 30178.98
running bench [640 480] blur_image_filter_small_1.00_1.00 8888: cmsecs = 29911.25
running bench [640 480] blur_image_filter_large_0.00_1.00 8888: cmsecs = 19344.35
running bench [640 480] blur_image_filter_large_0.00_10.00 8888: cmsecs = 19957.07
running bench [640 480] blur_image_filter_large_1.00_0.00 8888: cmsecs = 17158.84
running bench [640 480] blur_image_filter_large_10.00_0.00 8888: cmsecs = 17330.73
Review URL: https://codereview.chromium.org/
99933004
git-svn-id: http://skia.googlecode.com/svn/trunk@12486
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Wed, 4 Dec 2013 17:55:15 +0000 (17:55 +0000)]
Remove problematic GrContext thread local instance counting
R=robertphillips@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/
104893002
git-svn-id: http://skia.googlecode.com/svn/trunk@12485
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Wed, 4 Dec 2013 17:06:49 +0000 (17:06 +0000)]
Fixed bad bitmap size crashes
There were 2 issues :
1 ) If the size of an SkBitmap's underlying SkPixelRef's alocated memory is too small to fit the bitmap, then the deserialization will now check this and set an error appropriately.
2 ) If a device fails to allocate its pixels, the device will be deleted and NULL will be returned to avoid attempting to draw on a bad device.
BUG=
R=senorblanco@chromium.org, reed@google.com, sugoi@google.com, halcanary@google.com, mtklein@google.com
Author: sugoi@chromium.org
Review URL: https://codereview.chromium.org/
92793002
git-svn-id: http://skia.googlecode.com/svn/trunk@12484
2bbb7eff-a529-9590-31e7-
b0007b416f81
vandebo@chromium.org [Wed, 4 Dec 2013 16:52:15 +0000 (16:52 +0000)]
Rebaseline after r12473
Review URL: https://codereview.chromium.org/
104783003
git-svn-id: http://skia.googlecode.com/svn/trunk@12483
2bbb7eff-a529-9590-31e7-
b0007b416f81
bungeman@google.com [Wed, 4 Dec 2013 16:40:42 +0000 (16:40 +0000)]
Put all of SkPostConfig.h inside top level ifdef.
Due to inconsistent indentation, the last third of this header is outside
of the top level ifdef header guard. This CL fixes that, as well as
makes the indents more consistent to avoid this issue in the future.
R=mtklein@google.com, reed@google.com
Review URL: https://codereview.chromium.org/
102153002
git-svn-id: http://skia.googlecode.com/svn/trunk@12482
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Wed, 4 Dec 2013 16:30:03 +0000 (16:30 +0000)]
Remove cityhash.gyp
According to https://code.google.com/p/skia/issues/detail?id=1286 it should be fine to remove this empty gyp file now.
BUG=None
TEST=None
R=epoger@google.com
Author: tfarina@chromium.org
Review URL: https://codereview.chromium.org/
102603004
git-svn-id: http://skia.googlecode.com/svn/trunk@12481
2bbb7eff-a529-9590-31e7-
b0007b416f81
jvanverth@google.com [Wed, 4 Dec 2013 15:04:11 +0000 (15:04 +0000)]
Last set of rebaselines for https://codereview.chromium.org/
99993002
BUG=
Review URL: https://codereview.chromium.org/
104733002
git-svn-id: http://skia.googlecode.com/svn/trunk@12480
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Wed, 4 Dec 2013 14:51:31 +0000 (14:51 +0000)]
Make SkTInternalLList::validate validate all nodes
Make SkTInternalLList::validate validate all nodes, including the
first one.
R=mtklein@google.com, bsalomon@google.com
Author: kkinnunen@nvidia.com
Review URL: https://codereview.chromium.org/
104553002
git-svn-id: http://skia.googlecode.com/svn/trunk@12479
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Wed, 4 Dec 2013 13:42:46 +0000 (13:42 +0000)]
Fix incremental visibility filter handling.
When drawing incrementally, SkDebugCanvas can end up applying the
visibility overlay multiple times (resulting in a fade-to-white effect).
The CL also includes a minor/unrelated SkDebugCanvas cleanup.
R=robertphillips@google.com
Author: fmalita@chromium.org
Review URL: https://codereview.chromium.org/
103083004
git-svn-id: http://skia.googlecode.com/svn/trunk@12478
2bbb7eff-a529-9590-31e7-
b0007b416f81
skia.committer@gmail.com [Wed, 4 Dec 2013 07:02:26 +0000 (07:02 +0000)]
Sanitizing source files in Housekeeper-Nightly
git-svn-id: http://skia.googlecode.com/svn/trunk@12477
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Wed, 4 Dec 2013 00:42:07 +0000 (00:42 +0000)]
SkSplay and SkUnsplay work on any uint32_t, not just SkPMColor.
BUG=
R=reed@google.com, tfarina@chromium.org
Author: mtklein@google.com
Review URL: https://codereview.chromium.org/
102813002
git-svn-id: http://skia.googlecode.com/svn/trunk@12476
2bbb7eff-a529-9590-31e7-
b0007b416f81
jvanverth@google.com [Tue, 3 Dec 2013 21:45:59 +0000 (21:45 +0000)]
Rebaseline GMs for https://codereview.chromium.org/
99993002/
Review URL: https://codereview.chromium.org/
102953002
git-svn-id: http://skia.googlecode.com/svn/trunk@12475
2bbb7eff-a529-9590-31e7-
b0007b416f81
jvanverth@google.com [Tue, 3 Dec 2013 21:30:16 +0000 (21:30 +0000)]
Ignore colortype_msaa4 (for https://codereview.chromium.org/
99993002)
Review URL: https://codereview.chromium.org/
102713003
git-svn-id: http://skia.googlecode.com/svn/trunk@12474
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Tue, 3 Dec 2013 21:08:46 +0000 (21:08 +0000)]
[PDF] Fixes the cases where SkPDFDevice::finishContentEntry is called with empty content.
There are some cases (like drawing a completely transparent image) where the code tries to finish a content entry with no content and an xfermode other than clear. This makes those cases work correctly.
This is likely what was happening in chromium:316546, but it wasn't clear what the core problem was. desk_techcrunch.skp tripped a debug only assert that exposed the core issue.
BUG=skia:1868,chromium:316546
R=reed@google.com, bungeman@google.com
Author: vandebo@chromium.org
Review URL: https://codereview.chromium.org/
92453002
git-svn-id: http://skia.googlecode.com/svn/trunk@12473
2bbb7eff-a529-9590-31e7-
b0007b416f81
robertphillips@google.com [Tue, 3 Dec 2013 19:46:58 +0000 (19:46 +0000)]
Code cleanup following recapture of skps
https://codereview.chromium.org/
88233003/
git-svn-id: http://skia.googlecode.com/svn/trunk@12472
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Tue, 3 Dec 2013 19:45:22 +0000 (19:45 +0000)]
Add GPU support for color bitmap fonts
BUG=skia:1869
R=bungeman@google.com, robertphillips@google.com, bsalomon@google.com
Author: jvanverth@google.com
Review URL: https://codereview.chromium.org/
99993002
git-svn-id: http://skia.googlecode.com/svn/trunk@12471
2bbb7eff-a529-9590-31e7-
b0007b416f81
mtklein@google.com [Tue, 3 Dec 2013 19:27:41 +0000 (19:27 +0000)]
Fix 32/64 bit warnings on g++-4.2.
BUG=
Review URL: https://codereview.chromium.org/
98343006
git-svn-id: http://skia.googlecode.com/svn/trunk@12470
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Tue, 3 Dec 2013 18:53:30 +0000 (18:53 +0000)]
Refactor FourByteInterps. Add 64-bit Fast version. Add tests.
Effect on benches (only _fast_ should be affected, and only on 64-bit):
Desktop (64-bit)
four_byte_interp_slow_255 NONRENDERING c 7.80 7.84 -0.04 -0.5%
four_byte_interp_slow_256 NONRENDERING c 7.38 7.36 +0.02 +0.3%
four_byte_interp_fast_256 NONRENDERING c 4.86 4.38 +0.48 +9.9%
four_byte_interp_fast_255 NONRENDERING c 5.80 5.16 +0.64 +11.0%
N5 (32-bit)
four_byte_interp_slow_256 NONRENDERING c 22.22 22.66 -0.44 -2.0%
four_byte_interp_fast_255 NONRENDERING c 22.22 22.22 +0.00 +0.0%
four_byte_interp_fast_256 NONRENDERING c 18.81 18.81 +0.00 +0.0%
four_byte_interp_slow_255 NONRENDERING c 22.42 22.42 +0.00 +0.0%
BUG=
R=reed@google.com
Author: mtklein@google.com
Review URL: https://codereview.chromium.org/
100923003
git-svn-id: http://skia.googlecode.com/svn/trunk@12468
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Tue, 3 Dec 2013 18:17:16 +0000 (18:17 +0000)]
Simplify benchmark internal API.
I'm not quite sure why I wrote such a convoluted API with setLoops()/getLoops().
This replaces it with a loops argument passed to onDraw().
This CL is largely mechanical translation from the old API to the new one.
MathBench used this->getLoops() outside onDraw(), which seems incorrect. I
fixed it.
BUG=
R=djsollen@google.com
Author: mtklein@google.com
Review URL: https://codereview.chromium.org/
99893003
git-svn-id: http://skia.googlecode.com/svn/trunk@12466
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Tue, 3 Dec 2013 18:16:48 +0000 (18:16 +0000)]
Add JSON output option to bench.
A new command-line flag --outResultsFile takes the filename to write the JSON into.
The human readable output is the same as before with one exception. Previously DEBUG would be printed if in debug mode, or nothing would be printed if in release mode. Now this is reported as a named option:
build=DEBUG
or
build=RELEASE
R=djsollen@google.com, bsalomon@google.com
Author: jcgregorio@google.com
Review URL: https://codereview.chromium.org/
83863002
git-svn-id: http://skia.googlecode.com/svn/trunk@12465
2bbb7eff-a529-9590-31e7-
b0007b416f81
robertphillips@google.com [Tue, 3 Dec 2013 17:15:36 +0000 (17:15 +0000)]
Fix for SkPathRef::CreateTransformedCopy bug
https://codereview.chromium.org/
99423004/
git-svn-id: http://skia.googlecode.com/svn/trunk@12464
2bbb7eff-a529-9590-31e7-
b0007b416f81
robertphillips@google.com [Tue, 3 Dec 2013 16:43:54 +0000 (16:43 +0000)]
Move fIsOval from SkPath to SkPathRef
https://codereview.chromium.org/
89123002/
git-svn-id: http://skia.googlecode.com/svn/trunk@12463
2bbb7eff-a529-9590-31e7-
b0007b416f81
scroggo@google.com [Tue, 3 Dec 2013 16:15:31 +0000 (16:15 +0000)]
Remove SkPdfNativeDoc::tokenizerOfStream.
Instead, initialize an SkPdfNativeTokenizer directly on the stack.
Fixes a memory leak in SkPdfFont.
R=mtklein@google.com
Review URL: https://codereview.chromium.org/
100323003
git-svn-id: http://skia.googlecode.com/svn/trunk@12462
2bbb7eff-a529-9590-31e7-
b0007b416f81
bungeman@google.com [Tue, 3 Dec 2013 15:23:37 +0000 (15:23 +0000)]
Use lowercase windows.h in includes to fix Windows cross compilation using mingw.
While it doesn't matter on Windows, mingw on case-sensitive OSes uses all lower case filenames for platform include files. I found the problem in SkCondVar.h from Mozilla checkout of skia sources, but the patch contains a fix for the whole skia tree.
R=bungeman@google.com
Review URL: https://codereview.chromium.org/
99173003
git-svn-id: http://skia.googlecode.com/svn/trunk@12461
2bbb7eff-a529-9590-31e7-
b0007b416f81
rmistry@google.com [Tue, 3 Dec 2013 14:51:00 +0000 (14:51 +0000)]
Refreshed mozilla,googleplus and wikipedia SKPs. Triggering rebuilds.
git-svn-id: http://skia.googlecode.com/svn/trunk@12459
2bbb7eff-a529-9590-31e7-
b0007b416f81
robertphillips@google.com [Tue, 3 Dec 2013 12:49:32 +0000 (12:49 +0000)]
Rebaseline GMs for r12440 (HQ filtering for tiled/bleed drawBitmap)
https://codereview.chromium.org/
101793002/
git-svn-id: http://skia.googlecode.com/svn/trunk@12455
2bbb7eff-a529-9590-31e7-
b0007b416f81
skia.committer@gmail.com [Tue, 3 Dec 2013 07:02:11 +0000 (07:02 +0000)]
Sanitizing source files in Housekeeper-Nightly
git-svn-id: http://skia.googlecode.com/svn/trunk@12453
2bbb7eff-a529-9590-31e7-
b0007b416f81
robertphillips@google.com [Tue, 3 Dec 2013 00:23:39 +0000 (00:23 +0000)]
Revert of r12450 (Move fIsOval from SkPath to SkPathRef)
git-svn-id: http://skia.googlecode.com/svn/trunk@12452
2bbb7eff-a529-9590-31e7-
b0007b416f81
mtklein@google.com [Tue, 3 Dec 2013 00:01:03 +0000 (00:01 +0000)]
Fix off-by-one in four_byte_interp benches.
BUG=
Review URL: https://codereview.chromium.org/
100563002
git-svn-id: http://skia.googlecode.com/svn/trunk@12451
2bbb7eff-a529-9590-31e7-
b0007b416f81
robertphillips@google.com [Mon, 2 Dec 2013 23:55:11 +0000 (23:55 +0000)]
Move fIsOval from SkPath to SkPathRef
https://codereview.chromium.org/
89123002/
git-svn-id: http://skia.googlecode.com/svn/trunk@12450
2bbb7eff-a529-9590-31e7-
b0007b416f81
bsalomon@google.com [Mon, 2 Dec 2013 22:54:50 +0000 (22:54 +0000)]
Fix ignored-tests.txt
Review URL: https://codereview.chromium.org/
100383002
git-svn-id: http://skia.googlecode.com/svn/trunk@12449
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Mon, 2 Dec 2013 22:40:56 +0000 (22:40 +0000)]
ARM Skia NEON patches - 29 - Xfermode: SkFourByteInterp
Xfermode: add a NEON version of SkFourByteInterp
Brings a modest performance improvement on its own in
ProcXfermodes when aa is neither zero nor FF. Combined
with 1-pixel NEON modeprocs, it brings up to 35% speed
improvement on the aa case.
Signed-off-by: Kévin PETIT <kevin.petit@arm.com>
BUG=
R=djsollen@google.com, mtklein@google.com, reed@google.com
Author: kevin.petit.arm@gmail.com
Review URL: https://codereview.chromium.org/
23724013
git-svn-id: http://skia.googlecode.com/svn/trunk@12448
2bbb7eff-a529-9590-31e7-
b0007b416f81
scroggo@google.com [Mon, 2 Dec 2013 22:34:45 +0000 (22:34 +0000)]
Simplify SkPdfTokenLooper behavior.
Instead of passing a pointer to a pointer to a NULL looper, which can
then be set to point to a new looper (which then must be looped and
deleted), pass a pointer to the current looper. Each function can then
create a looper based on the parent (on the stack) and call loop()
itself.
Remove setUp(). Now that there is a pointer to the parent at creation
time, there is no need for this function.
Modify the constructors to only provide ones that are needed.
Add documentation.
Remove PdfInlineImageLooper::done(), which is never used.
R=mtklein@google.com
Review URL: https://codereview.chromium.org/
83183004
git-svn-id: http://skia.googlecode.com/svn/trunk@12447
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Mon, 2 Dec 2013 22:32:58 +0000 (22:32 +0000)]
Delay setting initial memory block until it officially exists.
BUG=323017
R=robertphillips@google.com
Author: mtklein@google.com
Review URL: https://codereview.chromium.org/
99833002
git-svn-id: http://skia.googlecode.com/svn/trunk@12446
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Mon, 2 Dec 2013 22:32:54 +0000 (22:32 +0000)]
SkDiscardablePixelRef objects are now marked immutable.
I also specifically mentioned in the SkImageGenerator documentation
that it should always return the same information on subsequent calls
to getInfo() and getPixels().
Motivation: When a bitmap is recorded into a SkPicture, it will choose
whether to shallow- or deep-copy itself into the SkBitmapHeap based on
the immutable flag. Failing to set it would make picture recording slow.
BUG=
R=fmalita@google.com, reed@google.com, scroggo@google.com
Author: halcanary@google.com
Review URL: https://codereview.chromium.org/
99303003
git-svn-id: http://skia.googlecode.com/svn/trunk@12445
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Mon, 2 Dec 2013 22:32:47 +0000 (22:32 +0000)]
Add four_byte_interp benches.
Desktop:
running bench [640 480] four_byte_interp_slow_256 NONRENDERING: cmsecs = 6.18
running bench [640 480] four_byte_interp_slow_255 NONRENDERING: cmsecs = 6.83
running bench [640 480] four_byte_interp_fast_256 NONRENDERING: cmsecs = 5.02
running bench [640 480] four_byte_interp_fast_255 NONRENDERING: cmsecs = 5.88
N5:
running bench [640 480] four_byte_interp_slow_256 NONRENDERING: cmsecs = 22.84
running bench [640 480] four_byte_interp_slow_255 NONRENDERING: cmsecs = 25.11
running bench [640 480] four_byte_interp_fast_256 NONRENDERING: cmsecs = 18.89
running bench [640 480] four_byte_interp_fast_255 NONRENDERING: cmsecs = 22.32
BUG=
R=reed@google.com
Author: mtklein@google.com
Review URL: https://codereview.chromium.org/
93933003
git-svn-id: http://skia.googlecode.com/svn/trunk@12444
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Mon, 2 Dec 2013 22:23:03 +0000 (22:23 +0000)]
Remove the comments settings for vim tab width and expansion variables.
These add unnecessary bloat for everyone to carry around, so we just
remove them now.
The same change was made in chromium by Tony in
http://codereview.chromium.org/7310019 - crrev.com/92046
BUG=None
TEST=./gyp_skia
R=mtklein@google.com
Author: tfarina@chromium.org
Review URL: https://codereview.chromium.org/
92673003
git-svn-id: http://skia.googlecode.com/svn/trunk@12443
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Mon, 2 Dec 2013 22:22:45 +0000 (22:22 +0000)]
Remove GR_IMPLEMENTATION define.
skgputest is built as a static library and thus this define is not
needed. If the library was built as shared, then this define would
be necessary and used. But it isn't, so we remove it now.
BUG=None
TEST=None
R=bsalomon@google.com
Author: tfarina@chromium.org
Review URL: https://codereview.chromium.org/
90923005
git-svn-id: http://skia.googlecode.com/svn/trunk@12442
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Mon, 2 Dec 2013 22:22:40 +0000 (22:22 +0000)]
Missed this flag. imagemagnifier asserts without it.
BUG=skia:781
R=epoger@google.com
Author: mtklein@google.com
Review URL: https://codereview.chromium.org/
89533004
git-svn-id: http://skia.googlecode.com/svn/trunk@12441
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Mon, 2 Dec 2013 22:22:35 +0000 (22:22 +0000)]
HQ filtering for tiled/bleed drawBitmap
sample at pix center in bicubic
R=senorblanco@chromium.org, jvanverth@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/
83153006
git-svn-id: http://skia.googlecode.com/svn/trunk@12440
2bbb7eff-a529-9590-31e7-
b0007b416f81
bungeman@google.com [Mon, 2 Dec 2013 21:34:36 +0000 (21:34 +0000)]
Fix 'i-i' to 'i-1' in SkCanvasStack.
Pointed out in http://www.viva64.com/en/b/0225/ .
R=djsollen@google.com
Review URL: https://codereview.chromium.org/
99753002
git-svn-id: http://skia.googlecode.com/svn/trunk@12439
2bbb7eff-a529-9590-31e7-
b0007b416f81
scroggo@google.com [Mon, 2 Dec 2013 20:58:12 +0000 (20:58 +0000)]
Remove some PdfViewer warnings.
Fix build.
git-svn-id: http://skia.googlecode.com/svn/trunk@12438
2bbb7eff-a529-9590-31e7-
b0007b416f81
borenet@google.com [Mon, 2 Dec 2013 20:27:16 +0000 (20:27 +0000)]
Add some debugging to buildbot_globals, sort the trybot list
R=epoger@google.com, rmistry@google.com
Review URL: https://codereview.chromium.org/
99223002
git-svn-id: http://skia.googlecode.com/svn/trunk@12437
2bbb7eff-a529-9590-31e7-
b0007b416f81
scroggo@google.com [Mon, 2 Dec 2013 20:22:31 +0000 (20:22 +0000)]
Separate and update PDF_DIFF_TRACE_IN_PNG
Move its functionality out of readToken() and into its own class.
Callers of the previous readToken() now call
SkPdfNativeTokenizer::readToken(), which in turn calls a function
for writing the diff to a file, if the caller requests it and
PDF_TRACE_DIFF_IN_PNG is defined.
Do not attempt to draw a diff for compatibility sections, which we
do not draw.
Use SkString to handle string manipulation.
Hide globals only used by PDF_TRACE_DIFF_IN_PNG behind that flag.
Remove hasVisualEffects, which always returns true.
Rename gLastOpKeyword to gOpCounter for clarity.
In SkPdfNativeTokenizer, set fEmpty to true when the entire stream
has been read.
Use SkBitmap::copyTo instead of manually copying an SkBitmap.
Builds on https://codereview.chromium.org/
79933003/
R=mtklein@google.com
Review URL: https://codereview.chromium.org/
80463005
git-svn-id: http://skia.googlecode.com/svn/trunk@12436
2bbb7eff-a529-9590-31e7-
b0007b416f81
scroggo@google.com [Mon, 2 Dec 2013 20:18:09 +0000 (20:18 +0000)]
Restructuring of PdfViewer code.
The only change in behavior is that SkPdfAllocator on SkPdfContext is
no longer allocated on the heap.
In general, I have just moved code.
SkPdfContext:
Inherit from SkNoncopyable.
Make SkPdfContext directly own fTmpPageAllocator. fTmpPageAllocator is
created when SkPdfContext is, and destroyed at the same time as well,
so there is no reason for the extra allocation.
Add the function parseStream. This eliminates code duplication, and
allows making fTmpPageAllocator private.
Move PdfMainLooper into the implementation file, since it is now only
used by parseStream.
Move SkTDictWithDefaultConstructor and render stats info here, in
support of PdfMainLooper.
SkPdfTokenLooper:
Rename PdfTokenLooper to SkPdfTokenLooper.
Move readToken here, unchanged.
Remove tokenizer(), which is unused.
SkPdfNativeDoc:
Remove tokenizerOfPage and tokenizerOfBuffer, which are unused.
SkPdfOps:
Move gPdfOps and PdfOperatorRenderer into a header file (hidden for
now), so they can be accessed by both SkPdfRenderer.cpp and
SkPdfContext.cpp.
SkPdfRenderer:
Harvest things into other files:
PdfMainLooper (and the code that calls it) -> SkPdfContext.
readToken -> SkPdfTokenLooper.
R=mtklein@google.com
Review URL: https://codereview.chromium.org/
79933003
git-svn-id: http://skia.googlecode.com/svn/trunk@12435
2bbb7eff-a529-9590-31e7-
b0007b416f81
bungeman@google.com [Mon, 2 Dec 2013 19:54:33 +0000 (19:54 +0000)]
Libraries without '-l' treated as relative paths.
The following have the same effect on Windows:
'msvs_settings': { 'VCLinkerTool': { 'AdditionalDependencies': [ 'windowscodecs.lib', ],},},
'link_settings': {'libraries': ['-lwindowscodecs.lib',],},
But this one is different:
'link_settings': {'libraries': ['windowscodecs.lib',],},
since this last one will attempt to find the library at
third_party\skia\gyp\windowscodecs.lib
or a place like this, instead of looking in the library paths.
This also fixes capitalization of the affected libraries.
R=bsalomon@google.com
Review URL: https://codereview.chromium.org/
99463002
git-svn-id: http://skia.googlecode.com/svn/trunk@12434
2bbb7eff-a529-9590-31e7-
b0007b416f81
reed@google.com [Mon, 2 Dec 2013 19:50:16 +0000 (19:50 +0000)]
restore [6] for setPrefConfigTable, accidentally changed with A1 CL
BUG=
Review URL: https://codereview.chromium.org/
99963002
git-svn-id: http://skia.googlecode.com/svn/trunk@12433
2bbb7eff-a529-9590-31e7-
b0007b416f81
bungeman@google.com [Mon, 2 Dec 2013 19:12:04 +0000 (19:12 +0000)]
Fix spelling errors in SkWeakRefCnt.
git-svn-id: http://skia.googlecode.com/svn/trunk@12432
2bbb7eff-a529-9590-31e7-
b0007b416f81
bungeman@google.com [Mon, 2 Dec 2013 19:10:02 +0000 (19:10 +0000)]
Actually mark other types of fonts as 'kOther' in FreeType port.
Without this, the fType is garbage and will most likely cause an assert
or unwanted behavior later.
Review URL: https://codereview.chromium.org/
99453005
git-svn-id: http://skia.googlecode.com/svn/trunk@12431
2bbb7eff-a529-9590-31e7-
b0007b416f81
robertphillips@google.com [Mon, 2 Dec 2013 15:43:39 +0000 (15:43 +0000)]
Fix truncation of glyphs in SkScalerContext.cpp
https://codereview.chromium.org/
99323002/
git-svn-id: http://skia.googlecode.com/svn/trunk@12430
2bbb7eff-a529-9590-31e7-
b0007b416f81
rmistry@google.com [Mon, 2 Dec 2013 13:50:38 +0000 (13:50 +0000)]
Reverting r12427
git-svn-id: http://skia.googlecode.com/svn/trunk@12428
2bbb7eff-a529-9590-31e7-
b0007b416f81
skia.committer@gmail.com [Mon, 2 Dec 2013 13:36:22 +0000 (13:36 +0000)]
Sanitizing source files in Housekeeper-Nightly
git-svn-id: http://skia.googlecode.com/svn/trunk@12427
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Thu, 28 Nov 2013 08:24:29 +0000 (08:24 +0000)]
Speed up GrResourceCache lookup by inlining GrBinHashKey comparisons
The GCC compilers for Android and Ubuntu do not seem to be able to
inline the memcmp operations on GrBinHashKey data. Write the comparisons
manually. Also shortcut GrBinHashKey::EQ to skip comparison when hashes
do not match.
Speeds up grresourcecache_find test on ARM and x86_64. Speeds up
grresourcecache_add on x86_64.
In order to test the change, moves ad hoc Gr unit tests from
src/gr_unittest.cpp to tests/GrUnitTests to be consistent with other
tests and enables GrUnitTests.
Fixes a regression from r2863 with where re-setting GrBinHashKey data
would not set the hash correctly. This should also improve the hash
function itself. The regression caused many of the hash operations be
no-ops. This is caught by the unit test.
Renames the comparison functions that GrHashTable needs from EQ, LT to
Equals, LessThan.
Renames GrTBinHashKey to GrBinHashKey. The GrTBinHashKey used to
forward comparison functions to an ENTRY template class, which would
extract the key and call back to the GrTBinHashKey. This would save
the user from writing one comparison function when comparison was done
with int ENTRY::compare(). There's no real benefit in this now. Also
this was used only for one class (GrTextureStripAtlas). The other use
in GrResourceKey was not actually using the provided "shortcut". The
new GrBinHashKey is not templated with the entry, rather just provides
== and < functions. The users of GrTHashTable provide the needed
functions now.
Adds explicit documentation of functions that are actually needed
GrTHashTable for the Key template. Adds SK_DEBUG guards according to
the contract.
R=bsalomon@google.com, mtklein@google.com
Author: kkinnunen@nvidia.com
Review URL: https://codereview.chromium.org/
88113002
git-svn-id: http://skia.googlecode.com/svn/trunk@12426
2bbb7eff-a529-9590-31e7-
b0007b416f81
skia.committer@gmail.com [Thu, 28 Nov 2013 07:02:25 +0000 (07:02 +0000)]
Sanitizing source files in Housekeeper-Nightly
git-svn-id: http://skia.googlecode.com/svn/trunk@12425
2bbb7eff-a529-9590-31e7-
b0007b416f81
bungeman@google.com [Wed, 27 Nov 2013 22:05:22 +0000 (22:05 +0000)]
Remove completed TODO in SkFontMgr.h.
git-svn-id: http://skia.googlecode.com/svn/trunk@12424
2bbb7eff-a529-9590-31e7-
b0007b416f81
bungeman@google.com [Wed, 27 Nov 2013 20:53:45 +0000 (20:53 +0000)]
Document the SkScalerContext pure virtuals.
R=reed@google.com
Review URL: https://codereview.chromium.org/
92623002
git-svn-id: http://skia.googlecode.com/svn/trunk@12423
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Wed, 27 Nov 2013 20:22:23 +0000 (20:22 +0000)]
Trying to add the same scaled image twice shouldn't assert.
This unbreaks bench_pictures --multi foo for me.
BUG=skia:1868
R=reed@google.com
Author: mtklein@google.com
Review URL: https://codereview.chromium.org/
89293002
git-svn-id: http://skia.googlecode.com/svn/trunk@12422
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Wed, 27 Nov 2013 18:13:17 +0000 (18:13 +0000)]
Don't check the framebuffer status every time we flush when we're on Chromium;
this reduces the number of unnecessary command buffer flushes by about 50%
for page loads when doing Ganesh rendering, since we do a flush on clear.
BUG=
R=bsalomon@google.com
Author: humper@google.com
Review URL: https://codereview.chromium.org/
92103002
git-svn-id: http://skia.googlecode.com/svn/trunk@12421
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Wed, 27 Nov 2013 17:08:36 +0000 (17:08 +0000)]
ARM Skia NEON patches - 16/17 - Blitmask
Blitmask: NEON optimised version of the D32_A8 functions
Here are the microbenchmark results I got for the D32_A8
functions:
Cortex-A9:
==========
+-------+--------+--------+--------+
| count | Black | Opaque | Color |
+-------+--------+--------+--------+
| 1 | -14% | -39,5% | -37,5% |
+-------+--------+--------+--------+
| 2 | -3% | -29,9% | -25% |
+-------+--------+--------+--------+
| 4 | -11,3% | -22% | -14,5% |
+-------+--------+--------+--------+
| 8 | +128% | +66,6% | +105% |
+-------+--------+--------+--------+
| 16 | +159% | +102% | +149% |
+-------+--------+--------+--------+
| 64 | +189% | +136% | +189% |
+-------+--------+--------+--------+
| 256 | +126% | +102% | +149% |
+-------+--------+--------+--------+
| 1024 | +67,5% | +81,4% | +123% |
+-------+--------+--------+--------+
Cortex-A15:
===========
+-------+--------+--------+--------+
| count | Black | Opaque | Color |
+-------+--------+--------+--------+
| 1 | -24% | -46,5% | -37,5% |
+-------+--------+--------+--------+
| 2 | -18,5% | -35,5% | -28% |
+-------+--------+--------+--------+
| 4 | -5,2% | -17,5% | -15,5% |
+-------+--------+--------+--------+
| 8 | +72% | +65,8% | +84,7% |
+-------+--------+--------+--------+
| 16 | +168% | +117% | +149% |
+-------+--------+--------+--------+
| 64 | +165% | +110% | +145% |
+-------+--------+--------+--------+
| 256 | +106% | +99,6% | +141% |
+-------+--------+--------+--------+
| 1024 | +93,7% | +94,7% | +130% |
+-------+--------+--------+--------+
Blitmask: add NEON optimised PlatformBlitRowProcs16
Here are the microbenchmark results (speedup vs. C code):
+-------+-----------------+-----------------+
| | Cortex-A9 | Cortex-A15 |
| count +--------+--------+--------+--------+
| | Blend | Opaque | Blend | Opaque |
+-------+--------+--------+--------+--------+
| 1 | -19,2% | -36,7% | -33,6% | -44,7% |
+-------+--------+--------+--------+--------+
| 2 | -12,6% | -27,8% | -39% | -48% |
+-------+--------+--------+--------+--------+
| 4 | -11,5% | -21,6% | -37,7% | -44,3% |
+-------+--------+--------+--------+--------+
| 8 | +141% | +59,7% | +123% | +48,7% |
+-------+--------+--------+--------+--------+
| 16 | +213% | +119% | +214% | +121% |
+-------+--------+--------+--------+--------+
| 64 | +212% | +105% | +242% | +167% |
+-------+--------+--------+--------+--------+
| 256 | +289% | +167% | +249% | +207% |
+-------+--------+--------+--------+--------+
| 1024 | +273% | +169% | +146% | +220% |
+-------+--------+--------+--------+--------+
Signed-off-by: Kévin PETIT <kevin.petit@arm.com>
BUG=
R=djsollen@google.com, mtklein@google.com, reed@google.com
Author: kevin.petit.arm@gmail.com
Review URL: https://codereview.chromium.org/
23719002
git-svn-id: http://skia.googlecode.com/svn/trunk@12420
2bbb7eff-a529-9590-31e7-
b0007b416f81
bungeman@google.com [Wed, 27 Nov 2013 17:00:12 +0000 (17:00 +0000)]
Fix Valgrind reports of test branching on uninitialized data.
Fixes reports like:
[07:10:01.235147] [ 71/188] (9) Serialization
[07:10:01.235186] ==20052== Thread 3:
[07:10:01.235223] ==20052== Conditional jump or move depends on uninitialised value(s)
[07:10:01.235259] ==20052== at 0x546632: SkRRect::setRectRadii(SkRect const&, SkPoint const*) (SkRect.h:426)
[07:10:01.735876] ==20052== by 0x546FB7: SkRRect::readFromMemory(void const*, unsigned long) (SkRRect.cpp:362)
[07:10:01.735965] ==20052== by 0x47F87F: Tests(skiatest::Reporter*) (SerializationTest.cpp:20)
[07:10:01.736007] ==20052== by 0x480367: skiatest::SerializationClass::onRun(skiatest::Reporter*) (SerializationTest.cpp:212)
[07:10:01.736048] ==20052== by 0x4881DA: skiatest::Test::run() (Test.cpp:109)
[07:10:01.736086] ==20052== by 0x482516: SkTestRunnable::run() (skia_test.cpp:155)
[07:10:01.736122] ==20052== by 0x6236E5: SkThreadPool::Loop(void*) (SkThreadPool.cpp:97)
[07:10:01.736158] ==20052== by 0x62DDB2: thread_start(void*) (SkThreadUtils_pthread.cpp:66)
[07:10:01.736192] ==20052== by 0x4E39E99: start_thread (pthread_create.c:308)
[07:10:01.736228] ==20052== by 0x6A5BCCC: clone (clone.S:112)
[07:10:01.736262] ==20052== Uninitialised value was created by a stack allocation
[07:10:01.736296] ==20052== at 0x47EC6A: Tests(skiatest::Reporter*) (SerializationTest.cpp:155)
git-svn-id: http://skia.googlecode.com/svn/trunk@12419
2bbb7eff-a529-9590-31e7-
b0007b416f81
bungeman@google.com [Wed, 27 Nov 2013 16:08:37 +0000 (16:08 +0000)]
Fix warning as error on Mac for implicit narrowing conversion from r12413.
git-svn-id: http://skia.googlecode.com/svn/trunk@12418
2bbb7eff-a529-9590-31e7-
b0007b416f81
bungeman@google.com [Wed, 27 Nov 2013 15:55:57 +0000 (15:55 +0000)]
Ever yet even more remaining rebaslines for r12400.
git-svn-id: http://skia.googlecode.com/svn/trunk@12417
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Wed, 27 Nov 2013 15:43:22 +0000 (15:43 +0000)]
move to newer angle
R=robertphillips@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/
86423002
git-svn-id: http://skia.googlecode.com/svn/trunk@12416
2bbb7eff-a529-9590-31e7-
b0007b416f81
reed@google.com [Wed, 27 Nov 2013 14:47:30 +0000 (14:47 +0000)]
fix 8->1 bugs shown by removal of A1 config
BUG=
Review URL: https://codereview.chromium.org/
91563003
git-svn-id: http://skia.googlecode.com/svn/trunk@12415
2bbb7eff-a529-9590-31e7-
b0007b416f81
skia.committer@gmail.com [Wed, 27 Nov 2013 07:02:04 +0000 (07:02 +0000)]
Sanitizing source files in Housekeeper-Nightly
git-svn-id: http://skia.googlecode.com/svn/trunk@12414
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Wed, 27 Nov 2013 01:24:53 +0000 (01:24 +0000)]
Non-rect polygons are not covered in GM cases, such as triangle, trapezoid, diamond, polygons with lots of edges, concave polygons, etc, especially for stroke-style and stroke-and-fill style painters. So add a GM case to avoid potential rendering errors.
R=bsalomon@google.com, robertphillips@google.com
Author: yunchao.he@intel.com
Review URL: https://codereview.chromium.org/
80483002
git-svn-id: http://skia.googlecode.com/svn/trunk@12413
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Wed, 27 Nov 2013 00:44:03 +0000 (00:44 +0000)]
Fold bench source file list into bench.gyp.
It is one less file and doesn't make things much worse.
BUG=None
TEST=./gyp_skia
R=mtklein@google.com
Author: tfarina@chromium.org
Review URL: https://codereview.chromium.org/
83703004
git-svn-id: http://skia.googlecode.com/svn/trunk@12412
2bbb7eff-a529-9590-31e7-
b0007b416f81
mtklein@google.com [Wed, 27 Nov 2013 00:04:15 +0000 (00:04 +0000)]
Add missing SkIntToScalars.
BUG=
Review URL: https://codereview.chromium.org/
89663003
git-svn-id: http://skia.googlecode.com/svn/trunk@12411
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Tue, 26 Nov 2013 23:52:20 +0000 (23:52 +0000)]
Fixed bad memory access
BUG=323595
R=senorblanco@google.com, reed@google.com, sugoi@google.com, senorblanco@chromium.org
Author: sugoi@chromium.org
Review URL: https://codereview.chromium.org/
88643004
git-svn-id: http://skia.googlecode.com/svn/trunk@12410
2bbb7eff-a529-9590-31e7-
b0007b416f81
bungeman@google.com [Tue, 26 Nov 2013 23:37:01 +0000 (23:37 +0000)]
Ever yet even more rebaslines for r12400.
git-svn-id: http://skia.googlecode.com/svn/trunk@12409
2bbb7eff-a529-9590-31e7-
b0007b416f81
mtklein@google.com [Tue, 26 Nov 2013 23:36:51 +0000 (23:36 +0000)]
DM: add --tileGrid
BUG=
R=epoger@google.com
Review URL: https://codereview.chromium.org/
88563003
git-svn-id: http://skia.googlecode.com/svn/trunk@12408
2bbb7eff-a529-9590-31e7-
b0007b416f81
bungeman@google.com [Tue, 26 Nov 2013 23:17:46 +0000 (23:17 +0000)]
Remove unused fields from SkGlyphCache.
Review URL: https://codereview.chromium.org/
89423003
git-svn-id: http://skia.googlecode.com/svn/trunk@12407
2bbb7eff-a529-9590-31e7-
b0007b416f81
bungeman@google.com [Tue, 26 Nov 2013 23:15:51 +0000 (23:15 +0000)]
Yet even more rebaslines for r12400.
git-svn-id: http://skia.googlecode.com/svn/trunk@12406
2bbb7eff-a529-9590-31e7-
b0007b416f81
bungeman@google.com [Tue, 26 Nov 2013 22:58:30 +0000 (22:58 +0000)]
Even more rebaslines for r12400.
git-svn-id: http://skia.googlecode.com/svn/trunk@12405
2bbb7eff-a529-9590-31e7-
b0007b416f81
bungeman@google.com [Tue, 26 Nov 2013 22:26:51 +0000 (22:26 +0000)]
More rebaslines for r12400.
git-svn-id: http://skia.googlecode.com/svn/trunk@12404
2bbb7eff-a529-9590-31e7-
b0007b416f81
bungeman@google.com [Tue, 26 Nov 2013 22:15:49 +0000 (22:15 +0000)]
Rebasline for r12400.
git-svn-id: http://skia.googlecode.com/svn/trunk@12403
2bbb7eff-a529-9590-31e7-
b0007b416f81
mtklein@google.com [Tue, 26 Nov 2013 22:06:12 +0000 (22:06 +0000)]
DM: write failed comparison mode .pngs one more level deep in the tree.
E.g. instead of having to compare
/tmp/dm/565/optimizations.png
vs.
/tmp/dm/replay/optimizations_565.png
it's now
/tmp/dm/565/optimizations.png
vs.
/tmp/dm/replay/565/optimizations.png
This lets working with skdiff go a lot more smoothly.
BUG=
R=bsalomon@google.com
Review URL: https://codereview.chromium.org/
88773002
git-svn-id: http://skia.googlecode.com/svn/trunk@12402
2bbb7eff-a529-9590-31e7-
b0007b416f81
senorblanco@chromium.org [Tue, 26 Nov 2013 21:52:38 +0000 (21:52 +0000)]
Enhance SkDropShadowImageFilter to support separate X & Y sigmas, and crop rect. This required a new flavour of the constructor.
This is driven by the feDropShadow filter effect in SVG, which unlike the shorthand version accepts separate X & Y sigmas.
Covered by new GM dropshadowimagefilter.
R=reed@google.com
Review URL: https://codereview.chromium.org/
88243004
git-svn-id: http://skia.googlecode.com/svn/trunk@12401
2bbb7eff-a529-9590-31e7-
b0007b416f81
bungeman@google.com [Tue, 26 Nov 2013 20:43:27 +0000 (20:43 +0000)]
Minimal changes to not crash GPU with color fonts.
Review URL: https://codereview.chromium.org/
88993003
git-svn-id: http://skia.googlecode.com/svn/trunk@12400
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Tue, 26 Nov 2013 20:39:11 +0000 (20:39 +0000)]
Support linking skia statically on Android
Support linking skia statically on Android. Skia symbols should be
exported by the app library, to the extent that skia_launcher needs
them.
Syncs libskia_android.so to the device only if it exists.
R=djsollen@google.com
Author: kkinnunen@nvidia.com
Review URL: https://codereview.chromium.org/
61893008
git-svn-id: http://skia.googlecode.com/svn/trunk@12399
2bbb7eff-a529-9590-31e7-
b0007b416f81
mtklein@google.com [Tue, 26 Nov 2013 18:52:31 +0000 (18:52 +0000)]
DM: some refactoring
- rename ComparisonTask to ChecksumTask
- have ChecksumTask handle all the checksum-checking
- turn on all extra modes by default
- simplify progress output to a countdown
BUG=
R=bsalomon@google.com
Review URL: https://codereview.chromium.org/
88543002
git-svn-id: http://skia.googlecode.com/svn/trunk@12398
2bbb7eff-a529-9590-31e7-
b0007b416f81
epoger@google.com [Tue, 26 Nov 2013 17:59:28 +0000 (17:59 +0000)]
rebaseline_server: make --reload work in git checkout
(SkipBuildbotRuns)
R=rmistry@google.com
Review URL: https://codereview.chromium.org/
86343002
git-svn-id: http://skia.googlecode.com/svn/trunk@12397
2bbb7eff-a529-9590-31e7-
b0007b416f81