reed@google.com [Mon, 6 Jan 2014 18:49:32 +0000 (18:49 +0000)]
remove SK_SUPPORTED_DEPRECATED_FIXEDROUND, no longer needed
git-svn-id: http://skia.googlecode.com/svn/trunk@12918
2bbb7eff-a529-9590-31e7-
b0007b416f81
rmistry@google.com [Mon, 6 Jan 2014 18:37:24 +0000 (18:37 +0000)]
Support mesa config in render_pictures.
R=epoger@google.com, robertphillips@google.com
Review URL: https://codereview.chromium.org/
125083004
git-svn-id: http://skia.googlecode.com/svn/trunk@12916
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Mon, 6 Jan 2014 18:33:19 +0000 (18:33 +0000)]
create gm/rebaseline_server/test_all.py
Uses proper Python unittest framework to launch all unittests within the directory.
NOTRY=True
R=rmistry@google.com
Author: epoger@google.com
Review URL: https://codereview.chromium.org/
125253002
git-svn-id: http://skia.googlecode.com/svn/trunk@12915
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Mon, 6 Jan 2014 18:29:14 +0000 (18:29 +0000)]
These strings are unused.
R=reed@google.com, halcanary@google.com
BUG=
Author: mtklein@google.com
Review URL: https://codereview.chromium.org/
102393005
git-svn-id: http://skia.googlecode.com/svn/trunk@12914
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Mon, 6 Jan 2014 18:17:24 +0000 (18:17 +0000)]
Renamed JsCanvas to JsContext and moved into it's own file.
Fleshed out a lot of the Path interface.
BUG=
R=robertphillips@google.com, tfarina@chromium.org, reed@google.com
Author: jcgregorio@google.com
Review URL: https://codereview.chromium.org/
121303004
git-svn-id: http://skia.googlecode.com/svn/trunk@12913
2bbb7eff-a529-9590-31e7-
b0007b416f81
reed@google.com [Mon, 6 Jan 2014 17:08:27 +0000 (17:08 +0000)]
Revert "Revert "Revert "Revert of https://codereview.chromium.org/
110593003/"""
This reverts commit
aaa89649590323fe40f52439d9a9a3376bb3b8ae.
BUG=
Review URL: https://codereview.chromium.org/
123223007
git-svn-id: http://skia.googlecode.com/svn/trunk@12910
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Mon, 6 Jan 2014 16:54:20 +0000 (16:54 +0000)]
Avoid some work on SkClipStack equality when the topmost genids match
Two clip stacks are defined being equal if the topmost genids match.
R=bsalomon@google.com
Author: kkinnunen@nvidia.com
Review URL: https://codereview.chromium.org/
115573005
git-svn-id: http://skia.googlecode.com/svn/trunk@12909
2bbb7eff-a529-9590-31e7-
b0007b416f81
reed@google.com [Mon, 6 Jan 2014 15:01:48 +0000 (15:01 +0000)]
add rowBytes param to setPreLock, in prep for onNewLockPixels change
need _win.cc change from https://codereview.chromium.org/
124503002/ when this lands in chrome
R=robertphillips@google.com
Review URL: https://codereview.chromium.org/
125063002
git-svn-id: http://skia.googlecode.com/svn/trunk@12907
2bbb7eff-a529-9590-31e7-
b0007b416f81
reed@google.com [Mon, 6 Jan 2014 13:34:39 +0000 (13:34 +0000)]
Revert "Revert "Revert of https://codereview.chromium.org/
110593003/""
This reverts commit
0fef787f33aa38109a0c8427e0098d997efdd5ff.
failed in chrome: https://codereview.chromium.org/
124503002/
Review URL: https://codereview.chromium.org/
105523008
git-svn-id: http://skia.googlecode.com/svn/trunk@12906
2bbb7eff-a529-9590-31e7-
b0007b416f81
tfarina@chromium.org [Mon, 6 Jan 2014 12:43:53 +0000 (12:43 +0000)]
Convert two more tests to DEF_TEST() macro.
Out of three, two were converted.
These tests were found with the following command line:
$ g grep "public Test"
BUG=None
TEST=ran tests executable fine.
R=mtklein@google.com
Review URL: https://codereview.chromium.org/
122943004
git-svn-id: http://skia.googlecode.com/svn/trunk@12905
2bbb7eff-a529-9590-31e7-
b0007b416f81
mike@reedtribe.org [Mon, 6 Jan 2014 03:02:37 +0000 (03:02 +0000)]
SK_SUPPORTED_DEPRECATED_FIXEDROUND around deprecated fixed[round,ceil,floor]
git-svn-id: http://skia.googlecode.com/svn/trunk@12903
2bbb7eff-a529-9590-31e7-
b0007b416f81
senorblanco@chromium.org [Sat, 4 Jan 2014 17:22:04 +0000 (17:22 +0000)]
Rebaselines for more straggling bots.
TBR=
Review URL: https://codereview.chromium.org/
121943003
git-svn-id: http://skia.googlecode.com/svn/trunk@12902
2bbb7eff-a529-9590-31e7-
b0007b416f81
skia.committer@gmail.com [Sat, 4 Jan 2014 07:01:40 +0000 (07:01 +0000)]
Sanitizing source files in Housekeeper-Nightly
git-svn-id: http://skia.googlecode.com/svn/trunk@12901
2bbb7eff-a529-9590-31e7-
b0007b416f81
senorblanco@chromium.org [Sat, 4 Jan 2014 04:19:38 +0000 (04:19 +0000)]
Yet more rebaselines from straggling bots.
BUG=
TBR=
Review URL: https://codereview.chromium.org/
116463005
git-svn-id: http://skia.googlecode.com/svn/trunk@12900
2bbb7eff-a529-9590-31e7-
b0007b416f81
senorblanco@chromium.org [Sat, 4 Jan 2014 02:36:24 +0000 (02:36 +0000)]
Stragglers.
TBR=
Review URL: https://codereview.chromium.org/
124353004
git-svn-id: http://skia.googlecode.com/svn/trunk@12899
2bbb7eff-a529-9590-31e7-
b0007b416f81
senorblanco@chromium.org [Sat, 4 Jan 2014 00:14:54 +0000 (00:14 +0000)]
New baselines for tileimagefilter, imagefiltersgraph, and everything Nexus4.
TBR=
Review URL: https://codereview.chromium.org/
101693010
git-svn-id: http://skia.googlecode.com/svn/trunk@12898
2bbb7eff-a529-9590-31e7-
b0007b416f81
senorblanco@chromium.org [Fri, 3 Jan 2014 22:27:05 +0000 (22:27 +0000)]
Widen suppression to tileimagefilter-pdf.
TBR=
Review URL: https://codereview.chromium.org/
122683004
git-svn-id: http://skia.googlecode.com/svn/trunk@12897
2bbb7eff-a529-9590-31e7-
b0007b416f81
senorblanco@chromium.org [Fri, 3 Jan 2014 22:18:12 +0000 (22:18 +0000)]
Ignore failures on tileimagefilter-pdf_poppler.
TBR=
Review URL: https://codereview.chromium.org/
124263003
git-svn-id: http://skia.googlecode.com/svn/trunk@12896
2bbb7eff-a529-9590-31e7-
b0007b416f81
senorblanco@chromium.org [Fri, 3 Jan 2014 21:48:22 +0000 (21:48 +0000)]
Make SkImageFilter crop rects relative to the primitive origin, instead of relative to their parent's crop rect. This is required by SVG semantics, and is more sane anyway.
To do this, this patch changes the "offset/loc" parameter in filterImage() / onFilterImage() from an inout-param to an out-param only, so that the calling filter can know how much the input filter wants its result offset (and doesn't include the original primitive position). This offset can then be applied to the current filter's crop rect. (I've renamed the parameter "offset" in all cases to make this clear.) This makes the call sites in SkCanvas/SkGpuDevice responsible for applying the resulting offset to the primitive's position, which is actually a fairly small change.
This change also fixes SkTileImageFilter and SkOffsetImageFilter to correctly handle an input offset, which they weren't before. This required modifying the GM's, since they assumed the broken behaviour.
NOTE: this will require rebaselining the imagefiltersgraph test, since it has a new test case.
NOTE: this will "break" the Blink layout tests css3/filters/effect-reference-subregion-chained-hw.html and css3/filters/effect-reference-subregion-hw.html, but it actually makes them give correct results. It should be suppressed on the skia roll, and I'll rebaseline it.
R=reed@google.com
Review URL: https://codereview.chromium.org/
112803004
git-svn-id: http://skia.googlecode.com/svn/trunk@12895
2bbb7eff-a529-9590-31e7-
b0007b416f81
reed@google.com [Fri, 3 Jan 2014 17:58:57 +0000 (17:58 +0000)]
Revert "Revert of https://codereview.chromium.org/
110593003/"
This reverts commit
c7abb25b25ba8b97948371d2bf0a2e3e78468f73.
and fixes the ashmem break
BUG=
Review URL: https://codereview.chromium.org/
119753010
git-svn-id: http://skia.googlecode.com/svn/trunk@12887
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Fri, 3 Jan 2014 17:25:52 +0000 (17:25 +0000)]
Add SkPreConfig.h to core.gypi
BUG=
R=reed@google.com
Author: borenet@google.com
Review URL: https://codereview.chromium.org/
116903007
git-svn-id: http://skia.googlecode.com/svn/trunk@12886
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Fri, 3 Jan 2014 16:37:16 +0000 (16:37 +0000)]
rebaseline gm self-tests after https://code.google.com/p/skia/source/detail?r=12850
The GM self-test's fragility is already noted in https://code.google.com/p/skia/issues/detail?id=1068 ('make gm self-test more robust')
NOTRY=True
NOTREECHECKS=True
R=mtklein@google.com
TBR=mtklein
Author: epoger@google.com
Review URL: https://codereview.chromium.org/
112203006
git-svn-id: http://skia.googlecode.com/svn/trunk@12885
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Fri, 3 Jan 2014 16:32:45 +0000 (16:32 +0000)]
Revert of https://codereview.chromium.org/
110593003/
Reason for revert: SkImageRef_ashmem doesn't compile
R=halcanary@google.com, scroggo@google.com
TBR=halcanary@google.com, scroggo@google.com
NOTREECHECKS=true
NOTRY=true
BUG=
Author: reed@google.com
Review URL: https://codereview.chromium.org/
119753009
git-svn-id: http://skia.googlecode.com/svn/trunk@12884
2bbb7eff-a529-9590-31e7-
b0007b416f81
reed@google.com [Fri, 3 Jan 2014 16:22:48 +0000 (16:22 +0000)]
Add onNewLockPixels, that returns rowbytes and relies on info in pixelref
This reverts commit
890a6ec633c1f54891104a072a8964b4c2c81af9.
BUG=
R=scroggo@google.com
Review URL: https://codereview.chromium.org/
110593003
git-svn-id: http://skia.googlecode.com/svn/trunk@12883
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Fri, 3 Jan 2014 16:04:11 +0000 (16:04 +0000)]
make results_test.py work better within python unittest framework
BUG=
R=rmistry@google.com
Author: epoger@google.com
Review URL: https://codereview.chromium.org/
122443006
git-svn-id: http://skia.googlecode.com/svn/trunk@12882
2bbb7eff-a529-9590-31e7-
b0007b416f81
reed@google.com [Fri, 3 Jan 2014 13:43:01 +0000 (13:43 +0000)]
assert in setPixelRef that the pr matches the bitmap's config
BUG=
R=halcanary@google.com
Review URL: https://codereview.chromium.org/
120063003
git-svn-id: http://skia.googlecode.com/svn/trunk@12880
2bbb7eff-a529-9590-31e7-
b0007b416f81
skia.committer@gmail.com [Fri, 3 Jan 2014 07:01:45 +0000 (07:01 +0000)]
Sanitizing source files in Housekeeper-Nightly
git-svn-id: http://skia.googlecode.com/svn/trunk@12875
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Thu, 2 Jan 2014 22:21:47 +0000 (22:21 +0000)]
An SkOSWindow for Unix may experience stuttering, with missed events or long delays before events are processed when using SkEvent.postDelay().
The issue is with the use of a select loop to watch for events coming into the x11 file descriptor for the display. It turns out there may be XEvents queued in memory but not yet processed that need to be handled before entering the select loop. See: http://developerweb.net/viewtopic.php?id=3184
BUG=1960
R=reed@google.com
Author: jcgregorio@google.com
Review URL: https://codereview.chromium.org/
123303004
git-svn-id: http://skia.googlecode.com/svn/trunk@12874
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Thu, 2 Jan 2014 22:20:49 +0000 (22:20 +0000)]
Function pointers -> templates in SkPictureFlat.
These flatten/unflatten function pointers were driving me nuts when reading the
generated assembly for this code. We don't need the flexibility of function
pointers here, so let's use templates to make it more manageable. You'll
notice we get much better typing now on flatten/unflatten.
BUG=
R=reed@google.com
Author: mtklein@google.com
Review URL: https://codereview.chromium.org/
123213004
git-svn-id: http://skia.googlecode.com/svn/trunk@12873
2bbb7eff-a529-9590-31e7-
b0007b416f81
tfarina@chromium.org [Thu, 2 Jan 2014 21:45:03 +0000 (21:45 +0000)]
Get rid of DEFINE_TESTCLASS_SHORT() macro.
Instead tests should be written using DEF_TEST() macro, which is much
nicer and simplifies the process of setting up an unit test.
BUG=None
TEST=skpskgr_test, pathops_unittest
R=mtklein@google.com
Review URL: https://codereview.chromium.org/
117863005
git-svn-id: http://skia.googlecode.com/svn/trunk@12870
2bbb7eff-a529-9590-31e7-
b0007b416f81
halcanary@google.com [Thu, 2 Jan 2014 20:43:26 +0000 (20:43 +0000)]
Update 4 skimage expectations.
Grab expectations missed in codereview:
102213009, sicne the bots were slow:
- Test-ChromeOS-Daisy-MaliT604-Arm7-Debug
- Test-ChromeOS-Daisy-MaliT604-Arm7-Release
- Test-Ubuntu12-ShuttleA-ATI5770-x86_64-Release-Valgrind
- Test-Ubuntu13-ShuttleA-HD2000-x86_64-Debug-ASAN
BUG=
R=epoger@google.com
Review URL: https://codereview.chromium.org/
93713007
git-svn-id: http://skia.googlecode.com/svn/trunk@12869
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Thu, 2 Jan 2014 19:42:15 +0000 (19:42 +0000)]
take 2: create simple 'skpmaker' tool to generate SKP files for testing
(re-landing https://codereview.chromium.org/
108343006 without the build breakages)
BUG=skia:1943
R=rmistry@google.com
Author: epoger@google.com
Review URL: https://codereview.chromium.org/
112253004
git-svn-id: http://skia.googlecode.com/svn/trunk@12866
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Thu, 2 Jan 2014 18:51:38 +0000 (18:51 +0000)]
Revert of https://codereview.chromium.org/
108343006/
Reason for revert: broke some builds
R=rmistry@google.com, mtklein@google.com
TBR=mtklein@google.com, rmistry@google.com
NOTREECHECKS=true
NOTRY=true
BUG=skia:1943
Author: epoger@google.com
Review URL: https://codereview.chromium.org/
102243012
git-svn-id: http://skia.googlecode.com/svn/trunk@12865
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Thu, 2 Jan 2014 18:10:34 +0000 (18:10 +0000)]
create simple 'skpmaker' tool to generate SKP files for testing
(needed to create self-test for render_pictures)
BUG=skia:1943
R=rmistry@google.com, mtklein@google.com
Author: epoger@google.com
Review URL: https://codereview.chromium.org/
108343006
git-svn-id: http://skia.googlecode.com/svn/trunk@12864
2bbb7eff-a529-9590-31e7-
b0007b416f81
reed@google.com [Thu, 2 Jan 2014 18:07:46 +0000 (18:07 +0000)]
remove SK_SUPPORT_LEGACY_PIXELREF_CONSTRUCTOR and make fInfo const
BUG=
R=halcanary@google.com
Review URL: https://codereview.chromium.org/
107373004
git-svn-id: http://skia.googlecode.com/svn/trunk@12863
2bbb7eff-a529-9590-31e7-
b0007b416f81
halcanary@google.com [Thu, 2 Jan 2014 17:29:28 +0000 (17:29 +0000)]
Add a release procedure to SkMallocPixelRef; remove SkDataPixelRef
This works in a way that is similar to SkData.
SkMallocPixelRef::NewWithProc
Motivation: Chrome has a ETC1PixelRef which calls delete[] on the
pixles on destruction. There is no reason for them to almost
duplicate our class, when we can provide them a more flexible
class. Example use:
static void delete_uint8_proc(void* ptr, void*) {
delete[] static_cast<uint8_t>(ptr);
}
SkPixelRef* new_delete_pixref(const SkImageInfo& info,
SkColorTable* ctable) {
size_t rb = info.minRowBytes();
return SkMallocPixelRef::NewWithProc(
info, rb, ctable,
new uint8_t[info.getSafeSize(rb)],
delete_uint8_proc, NULL);
}
SkMallocPixelRef::NewWithData
Motivation: This allows up to eliminate SkDataPixelRef. We
modified SkImage_Raster to use MallocPixelRef rather than
SkDataPixlRef.
Also: Unit tests in tests/MallocPixelRefTest.
BUG=
R=reed@google.com
Review URL: https://codereview.chromium.org/
106883006
git-svn-id: http://skia.googlecode.com/svn/trunk@12861
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Thu, 2 Jan 2014 16:19:53 +0000 (16:19 +0000)]
Make tests output spin on the same line by default.
-v now gives a cleaned-up version of the existing output (every test timed, useless information removed)
Example output, default:
[ 36/193] PathOpsCubicIntersectionOneOffTest
then later when finished...
[193/193] BlurMaskFilter
Example output, -v: (note, codereview is messing up my pretty spacing)
Skia UnitTests: --resourcePath resources SK_RELEASE SK_SCALAR_IS_FLOAT skia_arch_width=32
[ 1/193] 0ms PathOpsSimplifyDontFailOneTest
[ 2/193] 0ms PathOpsSimplifyFailOneTest
[ 3/193] 30ms PathOpsSkpTest
[ 4/193] 21ms PathOpsSimplifyFailTest
....
[182/193] 1026ms BlitRow
[183/193] 808ms AAClip
[184/193] 4333ms Math
[185/193] 5068ms PackBits
[186/193] 2265ms DrawText_DrawPosText
[187/193] 9163ms PathOpsRectsThreadedTest
[188/193] 5540ms GLPrograms
[189/193] 0ms GLInterfaceValidation
[190/193] 2ms DeferredCanvas
[191/193] 1ms ClipCache
[192/193] 30ms BlurMaskFilter
[193/193] 10396ms PathOpsOpCubicsThreadedTest
Finished 193 tests, 0 failures, 0 skipped. (622610 internal tests)
BUG=
R=halcanary@google.com, mtklein@google.com, bungeman@google.com
Author: mtklein@google.com
Review URL: https://codereview.chromium.org/
109513002
git-svn-id: http://skia.googlecode.com/svn/trunk@12860
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Thu, 2 Jan 2014 16:15:32 +0000 (16:15 +0000)]
Remove unused function.
BUG=
R=reed@google.com
Author: mtklein@google.com
Review URL: https://codereview.chromium.org/
121113005
git-svn-id: http://skia.googlecode.com/svn/trunk@12859
2bbb7eff-a529-9590-31e7-
b0007b416f81
halcanary@google.com [Thu, 2 Jan 2014 13:15:13 +0000 (13:15 +0000)]
Add Options to SkDecodingImageGenerator, simplify API.
Motivation: We want to remove redundant classes from Skia. To
that end we want to remove SkImageRef and its subclasses and
replace their uses with SkDiscardablePixelRef +
SkDecodingImageGenerator. Since Android uses SkImageRef, we need
to make sure that SkDecodingImageGenerator allows all of the
settings that Android exposes in BitmapFactory.Options.
To that end, we have created an Options struct for the
SkDecodingImageGenerator which lets the client of the generator set
sample size, dithering, and bitmap config.
We have made the SkDecodingImageGenerator constructor private
and replaced the SkDecodingImageGenerator::Install functions
with a SkDecodingImageGenerator::Create functions (one for
SkData and one for SkStream) which now take a
SkDecodingImageGenerator::Options struct.
Also added a ImageDecoderOptions test which loops through a list
of sets of options and tries them on a set of 5 small encoded
images.
Also updated several users of SkDecodingImageGenerator::Install to
follow new call signature - gm/factory.cpp, LazyDecodeBitmap.cpp,
and PictureTest.cpp, CachedDecodingPixelRefTest.cpp.
We also added a new ImprovedBitmapFactory Test which simulates the
exact function that Android will need to modify to use this,
installPixelRef() in BitmapFactory.
R=reed@google.com, scroggo@google.com
Committed: https://code.google.com/p/skia/source/detail?r=12744
Review URL: https://codereview.chromium.org/
93703004
git-svn-id: http://skia.googlecode.com/svn/trunk@12855
2bbb7eff-a529-9590-31e7-
b0007b416f81
skia.committer@gmail.com [Thu, 2 Jan 2014 07:01:35 +0000 (07:01 +0000)]
Sanitizing source files in Housekeeper-Nightly
git-svn-id: http://skia.googlecode.com/svn/trunk@12853
2bbb7eff-a529-9590-31e7-
b0007b416f81
mike@reedtribe.org [Wed, 1 Jan 2014 20:32:45 +0000 (20:32 +0000)]
speedup SkRect::intersect
git-svn-id: http://skia.googlecode.com/svn/trunk@12851
2bbb7eff-a529-9590-31e7-
b0007b416f81
tfarina@chromium.org [Tue, 31 Dec 2013 14:29:52 +0000 (14:29 +0000)]
Add default resource path for skia test cases in gm and bench tools.
BUG=skia:1765
TEST=./out/Release/gm --match downsamplebitmap_image
R=mtklein@google.com, scroggo@google.com, yunchao.he@intel.com
Review URL: https://codereview.chromium.org/
120873002
git-svn-id: http://skia.googlecode.com/svn/trunk@12850
2bbb7eff-a529-9590-31e7-
b0007b416f81
skia.committer@gmail.com [Tue, 31 Dec 2013 07:01:36 +0000 (07:01 +0000)]
Sanitizing source files in Housekeeper-Nightly
git-svn-id: http://skia.googlecode.com/svn/trunk@12849
2bbb7eff-a529-9590-31e7-
b0007b416f81
reed@google.com [Mon, 30 Dec 2013 19:21:22 +0000 (19:21 +0000)]
ComputeRowBytes must not overflow width when it shifts
BUG=
R=halcanary@google.com
Review URL: https://codereview.chromium.org/
122473002
git-svn-id: http://skia.googlecode.com/svn/trunk@12848
2bbb7eff-a529-9590-31e7-
b0007b416f81
reed@google.com [Mon, 30 Dec 2013 16:52:00 +0000 (16:52 +0000)]
fix drawVertices when we have both colors and textures
fix: don't re-call setContext on the composshader (i.e. on the tricolor shader) for each triangle,
since the trishader handles that explicitly. Just call setContext on the original shader.
BUG=
Review URL: https://codereview.chromium.org/
102193006
git-svn-id: http://skia.googlecode.com/svn/trunk@12847
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Mon, 30 Dec 2013 16:21:06 +0000 (16:21 +0000)]
Possibly uninitialized SkRgnBuilder fStorage.
fStorage needs to be initialized regardless of SkRgnBuilder::init()'s
outcome - otherwise the destructor can end up freeing garbage.
BUG=330293
R=reed@google.com, fmalita@google.com
Author: fmalita@chromium.org
Review URL: https://codereview.chromium.org/
122313002
git-svn-id: http://skia.googlecode.com/svn/trunk@12846
2bbb7eff-a529-9590-31e7-
b0007b416f81
reed@google.com [Mon, 30 Dec 2013 15:51:25 +0000 (15:51 +0000)]
reenable vertices gm, adding picture support
BUG=
Review URL: https://codereview.chromium.org/
112913005
git-svn-id: http://skia.googlecode.com/svn/trunk@12845
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Mon, 30 Dec 2013 15:48:10 +0000 (15:48 +0000)]
Fixing crash found by fuzzer
A previous fix only partially fixed this issue by adding validation on some inputs of SkImageInfo. If anything invalid is detected in SkImageInfo, unfortunately, this can cause getSafeSize() to do an illegal memory access while calling bytesPerPixel(), which could have a bad color type at this point.
A possible fix is to simply make sure we are in a valid state before calling getSafeSize().
BUG=329254
R=reed@google.com, mtklein@google.com, bsalomon@google.com, sugoi@google.com
Author: sugoi@chromium.org
Review URL: https://codereview.chromium.org/
107003006
git-svn-id: http://skia.googlecode.com/svn/trunk@12844
2bbb7eff-a529-9590-31e7-
b0007b416f81
reed@google.com [Mon, 30 Dec 2013 14:40:38 +0000 (14:40 +0000)]
Revert "Revert of https://codereview.chromium.org/
113823003/"
This reverts commit
68b4b32066ea0ba9dbb5d326a836f8a54297b7aa.
BUG=
Review URL: https://codereview.chromium.org/
122293002
git-svn-id: http://skia.googlecode.com/svn/trunk@12842
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Mon, 30 Dec 2013 14:24:37 +0000 (14:24 +0000)]
Revert of https://codereview.chromium.org/
113823003/
Reason for revert: need to update callsites in linux codecs
R=robertphillips@google.com
TBR=robertphillips@google.com
NOTREECHECKS=true
NOTRY=true
BUG=
Author: reed@google.com
Review URL: https://codereview.chromium.org/
122283002
git-svn-id: http://skia.googlecode.com/svn/trunk@12841
2bbb7eff-a529-9590-31e7-
b0007b416f81
reed@google.com [Mon, 30 Dec 2013 14:14:42 +0000 (14:14 +0000)]
remove Sk64 from public API, and start to remove usage internally
BUG=
R=robertphillips@google.com
Review URL: https://codereview.chromium.org/
113823003
git-svn-id: http://skia.googlecode.com/svn/trunk@12840
2bbb7eff-a529-9590-31e7-
b0007b416f81
skia.committer@gmail.com [Mon, 30 Dec 2013 07:01:37 +0000 (07:01 +0000)]
Sanitizing source files in Housekeeper-Nightly
git-svn-id: http://skia.googlecode.com/svn/trunk@12838
2bbb7eff-a529-9590-31e7-
b0007b416f81
mike@reedtribe.org [Mon, 30 Dec 2013 04:34:44 +0000 (04:34 +0000)]
disable gm/vertices until I can find/fix all warnings
git-svn-id: http://skia.googlecode.com/svn/trunk@12837
2bbb7eff-a529-9590-31e7-
b0007b416f81
mike@reedtribe.org [Mon, 30 Dec 2013 04:27:22 +0000 (04:27 +0000)]
really disable replay
git-svn-id: http://skia.googlecode.com/svn/trunk@12836
2bbb7eff-a529-9590-31e7-
b0007b416f81
mike@reedtribe.org [Mon, 30 Dec 2013 04:20:38 +0000 (04:20 +0000)]
suppress pictures for vertices for now
git-svn-id: http://skia.googlecode.com/svn/trunk@12835
2bbb7eff-a529-9590-31e7-
b0007b416f81
mike@reedtribe.org [Mon, 30 Dec 2013 04:07:34 +0000 (04:07 +0000)]
add gm::vertices
git-svn-id: http://skia.googlecode.com/svn/trunk@12834
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Thu, 26 Dec 2013 15:50:29 +0000 (15:50 +0000)]
Revert of https://codereview.chromium.org/
119943002/
Reason for revert: False alert on zheng.xu's original CL - it didn't break anything.
R=bungeman@google.com, reed@google.com, zheng.xu@arm.com, robertphillips@google.com, rmistry@google.com
TBR=bungeman@google.com, reed@google.com, rmistry@google.com, robertphillips@google.com, zheng.xu@arm.com
NOTREECHECKS=true
NOTRY=true
BUG=
Author: bensong@google.com
Review URL: https://codereview.chromium.org/
117963003
git-svn-id: http://skia.googlecode.com/svn/trunk@12822
2bbb7eff-a529-9590-31e7-
b0007b416f81
skia.committer@gmail.com [Tue, 24 Dec 2013 07:01:39 +0000 (07:01 +0000)]
Sanitizing source files in Housekeeper-Nightly
git-svn-id: http://skia.googlecode.com/svn/trunk@12820
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Mon, 23 Dec 2013 22:47:15 +0000 (22:47 +0000)]
create unittest for gm/rebaseline_server/results.py
NOTRY=True
R=rmistry@google.com
Author: epoger@google.com
Review URL: https://codereview.chromium.org/
117413006
git-svn-id: http://skia.googlecode.com/svn/trunk@12819
2bbb7eff-a529-9590-31e7-
b0007b416f81
rmistry@google.com [Sat, 21 Dec 2013 19:07:40 +0000 (19:07 +0000)]
Adding additional optional parameters to imagediffdb to make calling from Cluster Telemetry possible.
R=epoger@google.com
Review URL: https://codereview.chromium.org/
117823008
git-svn-id: http://skia.googlecode.com/svn/trunk@12816
2bbb7eff-a529-9590-31e7-
b0007b416f81
skia.committer@gmail.com [Sat, 21 Dec 2013 07:01:35 +0000 (07:01 +0000)]
Sanitizing source files in Housekeeper-Nightly
git-svn-id: http://skia.googlecode.com/svn/trunk@12812
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Fri, 20 Dec 2013 21:49:33 +0000 (21:49 +0000)]
Adds non-1 scalar to config names.
BUG=
R=robertphillips@google.com, bungeman@google.com, halcanary@google.com, scroggo@google.com, borenet@google.com
Author: bensong@google.com
Review URL: https://codereview.chromium.org/
106823005
git-svn-id: http://skia.googlecode.com/svn/trunk@12811
2bbb7eff-a529-9590-31e7-
b0007b416f81
borenet@google.com [Fri, 20 Dec 2013 21:35:51 +0000 (21:35 +0000)]
Empty skimage expectations for Logan
R=scroggo@google.com
Review URL: https://codereview.chromium.org/
110393005
git-svn-id: http://skia.googlecode.com/svn/trunk@12809
2bbb7eff-a529-9590-31e7-
b0007b416f81
halcanary@google.com [Fri, 20 Dec 2013 19:22:35 +0000 (19:22 +0000)]
Rebaseline for WebP
What I did:
#!/bin/sh
cd expectations/skimage
for D in *; do
[ -d "$D" ] || continue
[ -f "${D}/expected-results.json" ] || continue
gsutil cp \
"gs://chromium-skia-gm/skimage/actuals/${D}/actual-results.json" \
"${D}/expected-results.json"
done
cd ../..
R=robertphillips@google.com
Review URL: https://codereview.chromium.org/
102213009
git-svn-id: http://skia.googlecode.com/svn/trunk@12805
2bbb7eff-a529-9590-31e7-
b0007b416f81
halcanary@google.com [Fri, 20 Dec 2013 16:35:22 +0000 (16:35 +0000)]
Fix Red/Blue decoding problem in WebP.
Added unit test with lossless webp data.
BUG=skia:1402
Will need to rebaseline webp expectations for some systems.
R=scroggo@google.com
Review URL: https://codereview.chromium.org/
105443005
git-svn-id: http://skia.googlecode.com/svn/trunk@12803
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Fri, 20 Dec 2013 15:56:52 +0000 (15:56 +0000)]
Allow multiple concurrent timers.
BUG=
R=robertphillips@google.com
Author: jcgregorio@google.com
Review URL: https://codereview.chromium.org/
118473006
git-svn-id: http://skia.googlecode.com/svn/trunk@12802
2bbb7eff-a529-9590-31e7-
b0007b416f81
borenet@google.com [Fri, 20 Dec 2013 14:48:38 +0000 (14:48 +0000)]
Change bench_pictures configs
Aiming at CPU vs GPU
R=epoger@google.com, robertphillips@google.com
Review URL: https://codereview.chromium.org/
111893004
git-svn-id: http://skia.googlecode.com/svn/trunk@12799
2bbb7eff-a529-9590-31e7-
b0007b416f81
borenet@google.com [Fri, 20 Dec 2013 14:35:24 +0000 (14:35 +0000)]
Skimage expectations for Valgrind builder
R=robertphillips@google.com
Review URL: https://codereview.chromium.org/
115693005
git-svn-id: http://skia.googlecode.com/svn/trunk@12797
2bbb7eff-a529-9590-31e7-
b0007b416f81
reed@google.com [Fri, 20 Dec 2013 14:24:21 +0000 (14:24 +0000)]
Revert "Revert "begin to remove SkLONGLONG and wean Skia off of Sk64""
This reverts commit
15b986baf026a3da5e2cac8106a1b753df242c39.
BUG=
Review URL: https://codereview.chromium.org/
119353003
git-svn-id: http://skia.googlecode.com/svn/trunk@12796
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Fri, 20 Dec 2013 13:52:54 +0000 (13:52 +0000)]
Revert of https://codereview.chromium.org/
101423004/
Reason for revert: broke some win7 bots
R=bungeman@google.com, reed@google.com, zheng.xu@arm.com, robertphillips@google.com, rmistry@google.com
TBR=bungeman@google.com, reed@google.com, zheng.xu@arm.com
NOTREECHECKS=true
NOTRY=true
BUG=
Author: bensong@google.com
Review URL: https://codereview.chromium.org/
119943002
git-svn-id: http://skia.googlecode.com/svn/trunk@12794
2bbb7eff-a529-9590-31e7-
b0007b416f81
skia.committer@gmail.com [Fri, 20 Dec 2013 07:01:36 +0000 (07:01 +0000)]
Sanitizing source files in Housekeeper-Nightly
git-svn-id: http://skia.googlecode.com/svn/trunk@12793
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Fri, 20 Dec 2013 04:39:12 +0000 (04:39 +0000)]
This patch makes SkScalerContext_FreeType to be the only one which embolden the glyphs.
Add bench cases for different font styles.
BUG=
R=bungeman@google.com, reed@google.com
Author: zheng.xu@arm.com
Review URL: https://codereview.chromium.org/
101423004
git-svn-id: http://skia.googlecode.com/svn/trunk@12792
2bbb7eff-a529-9590-31e7-
b0007b416f81
robertphillips@google.com [Thu, 19 Dec 2013 23:17:56 +0000 (23:17 +0000)]
Disable SkFloat unit test due to Chromium-side compilation errors
git-svn-id: http://skia.googlecode.com/svn/trunk@12791
2bbb7eff-a529-9590-31e7-
b0007b416f81
reed@google.com [Thu, 19 Dec 2013 22:28:48 +0000 (22:28 +0000)]
Revert "begin to remove SkLONGLONG and wean Skia off of Sk64"
This reverts commit
784890196fdab96289f9389db43aca01f35db0f9.
Revert "use LL suffix for 64bit literal"
This reverts commit
9634295aff9bffd7a3875a0ca4a9b1a27d0793fc.
BUG=
Review URL: https://codereview.chromium.org/
116543009
git-svn-id: http://skia.googlecode.com/svn/trunk@12790
2bbb7eff-a529-9590-31e7-
b0007b416f81
reed@google.com [Thu, 19 Dec 2013 22:18:51 +0000 (22:18 +0000)]
use LL suffix for 64bit literal
BUG=
Review URL: https://codereview.chromium.org/
108683009
git-svn-id: http://skia.googlecode.com/svn/trunk@12789
2bbb7eff-a529-9590-31e7-
b0007b416f81
reed@google.com [Thu, 19 Dec 2013 21:54:27 +0000 (21:54 +0000)]
begin to remove SkLONGLONG and wean Skia off of Sk64
BUG=
R=caryclark@google.com
Review URL: https://codereview.chromium.org/
99433009
git-svn-id: http://skia.googlecode.com/svn/trunk@12788
2bbb7eff-a529-9590-31e7-
b0007b416f81
bsalomon@google.com [Thu, 19 Dec 2013 21:36:35 +0000 (21:36 +0000)]
rebaselinedownsamplebitmap_image_high_mandrill_512.png
BUG=skia:1879
Review URL: https://codereview.chromium.org/
101413010
git-svn-id: http://skia.googlecode.com/svn/trunk@12787
2bbb7eff-a529-9590-31e7-
b0007b416f81
halcanary@google.com [Thu, 19 Dec 2013 20:57:45 +0000 (20:57 +0000)]
Sanitize bitmaps produced by libgif
Motivation: fix the bug linked to below.
Some rare GIFs in the wild have a color table of size smaller than 256
and have one or more pixels with values that index out side of the
colortable. Since it would be expensive to check the index every time
we do a color lookup, we sanitize it at the source, in
SkImageDecoder_libgif.cpp. The new function sanitize_indexed_bitmap
checks each pixel to see if the index is outside of its allowed mask,
and if so, sets that pixel to point at color 0.
The bit mask optimization in this function relies on the fact that GIF
requires the color table size to be a power of 2.
To test:
$ skia_images_gif_suppressDecoderWarnings=0 \
..../render_pictures \
--bbh grid 256 256 --clone 1 --config 8888 --mode tile 256 256 \
-r ..../http___www_cafe24_com_.skp -w /tmp
This should show verbose errors.
BUG=skia:1946
R=robertphillips@google.com
Review URL: https://codereview.chromium.org/
103343006
git-svn-id: http://skia.googlecode.com/svn/trunk@12786
2bbb7eff-a529-9590-31e7-
b0007b416f81
bsalomon@google.com [Thu, 19 Dec 2013 20:54:47 +0000 (20:54 +0000)]
Some more medium/MIP downsample gpu rebaslines and misc gpu innocuous rebaselines
Review URL: https://codereview.chromium.org/
106523004
git-svn-id: http://skia.googlecode.com/svn/trunk@12785
2bbb7eff-a529-9590-31e7-
b0007b416f81
bsalomon@google.com [Thu, 19 Dec 2013 20:44:04 +0000 (20:44 +0000)]
rebaseline clipped-bitmap-shaders GMs for gpu
Review URL: https://codereview.chromium.org/
119443002
git-svn-id: http://skia.googlecode.com/svn/trunk@12784
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Thu, 19 Dec 2013 20:01:34 +0000 (20:01 +0000)]
rebaseline_server: stop catching KeyboardInterrupt (ctrl-C)
(SkipBuildbotRuns)
NOTRY=True
NOTREECHECKS=True
R=rmistry@google.com
TBR=rmistry
Author: epoger@google.com
Review URL: https://codereview.chromium.org/
115863003
git-svn-id: http://skia.googlecode.com/svn/trunk@12783
2bbb7eff-a529-9590-31e7-
b0007b416f81
robertphillips@google.com [Thu, 19 Dec 2013 19:37:10 +0000 (19:37 +0000)]
Updated GLintptr and GLsizeiptr to be the appropriate types on 64 bit Windows (Take 2)
https://codereview.chromium.org/
118533003/
git-svn-id: http://skia.googlecode.com/svn/trunk@12781
2bbb7eff-a529-9590-31e7-
b0007b416f81
bsalomon@google.com [Thu, 19 Dec 2013 19:36:18 +0000 (19:36 +0000)]
rebaseline medium quality downscale images after mipmap change
Review URL: https://codereview.chromium.org/
113803004
git-svn-id: http://skia.googlecode.com/svn/trunk@12780
2bbb7eff-a529-9590-31e7-
b0007b416f81
bungeman@google.com [Thu, 19 Dec 2013 19:34:22 +0000 (19:34 +0000)]
Provide last resort for fCapHeight in SkFontHost_FreeType::onGetAdvancedTypefaceMetrics.
This was a pre-existing issue, but r12689 uncovered it in more cases.
Since there was no last resort, fonts without a cap height used
uninitialized data as their cap height here.
R=robertphillips@google.com
Review URL: https://codereview.chromium.org/
114773004
git-svn-id: http://skia.googlecode.com/svn/trunk@12779
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Thu, 19 Dec 2013 19:08:31 +0000 (19:08 +0000)]
rebaseline_server: add more timing and progress info to server-side log
(SkipBuildbotRuns)
BUG=skia:1941
NOTRY=True
NOTREECHECKS=True
R=bsalomon@google.com
Author: epoger@google.com
Review URL: https://codereview.chromium.org/
105213007
git-svn-id: http://skia.googlecode.com/svn/trunk@12777
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Thu, 19 Dec 2013 18:22:32 +0000 (18:22 +0000)]
create tools/tests/run_all.py to run all our Python self-tests within tools
Eventually, we want all of these self-tests to be written in Python;
this change makes it easier for us to add new Python self-tests, and discourages
the addition of bash self-tests.
BUG=skia:677,skia:1943
NOTRY=True
R=rmistry@google.com, scroggo@google.com
Author: epoger@google.com
Review URL: https://codereview.chromium.org/
112163004
git-svn-id: http://skia.googlecode.com/svn/trunk@12775
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Thu, 19 Dec 2013 17:00:46 +0000 (17:00 +0000)]
Accept displacement with no displacement input
BUG=
R=senorblanco@google.com, senorblanco@chromium.org, sugoi@google.com, bsalomon@google.com, bsalomon@chromium.org
Author: sugoi@chromium.org
Review URL: https://codereview.chromium.org/
104853005
git-svn-id: http://skia.googlecode.com/svn/trunk@12773
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Thu, 19 Dec 2013 16:20:04 +0000 (16:20 +0000)]
Rebaseline for Nexus7 tile multithreads config. Widens the range for benched calculated from average.
BUG=
R=robertphillips@google.com
Author: bensong@google.com
Review URL: https://codereview.chromium.org/
118593002
git-svn-id: http://skia.googlecode.com/svn/trunk@12772
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Thu, 19 Dec 2013 16:18:01 +0000 (16:18 +0000)]
Add detection of mip support
R=robertphillips@google.com, jvanverth@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/
102853005
git-svn-id: http://skia.googlecode.com/svn/trunk@12771
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Thu, 19 Dec 2013 16:12:25 +0000 (16:12 +0000)]
Move distance field font code into GrDistanceFieldTextContext.
This avoids the SkDraw path and renders the distance field glyphs directly from
GrDistanceFieldTextContext. It also disables LCD, subpixel and autohinting, and
removes the supporting code when rendering DF fonts.
R=reed@google.com, bsalomon@google.com, robertphillips@google.com
Author: jvanverth@google.com
Review URL: https://codereview.chromium.org/
85653004
git-svn-id: http://skia.googlecode.com/svn/trunk@12770
2bbb7eff-a529-9590-31e7-
b0007b416f81
senorblanco@chromium.org [Thu, 19 Dec 2013 15:41:12 +0000 (15:41 +0000)]
New baselines for xfermodeimagefilter post-r12652.
TBR=reed@google.com
BUG=
Review URL: https://codereview.chromium.org/
118803003
git-svn-id: http://skia.googlecode.com/svn/trunk@12769
2bbb7eff-a529-9590-31e7-
b0007b416f81
reed@google.com [Thu, 19 Dec 2013 15:00:18 +0000 (15:00 +0000)]
fix win build
git-svn-id: http://skia.googlecode.com/svn/trunk@12768
2bbb7eff-a529-9590-31e7-
b0007b416f81
reed@google.com [Thu, 19 Dec 2013 14:22:03 +0000 (14:22 +0000)]
remove unused SkFixed and SkFract functions
BUG=
R=caryclark@google.com
Review URL: https://codereview.chromium.org/
113873008
git-svn-id: http://skia.googlecode.com/svn/trunk@12767
2bbb7eff-a529-9590-31e7-
b0007b416f81
robertphillips@google.com [Thu, 19 Dec 2013 13:41:48 +0000 (13:41 +0000)]
Yet another round of valgrind suppressions
https://codereview.chromium.org/
116053004/
git-svn-id: http://skia.googlecode.com/svn/trunk@12766
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Thu, 19 Dec 2013 10:39:59 +0000 (10:39 +0000)]
Add cast to fix Win* compile complaint/failure
R=tomhudson@google.com, reed@google.com
TBR=tomhudson@google.com
Author: robertphillips@google.com
Review URL: https://codereview.chromium.org/
100313005
git-svn-id: http://skia.googlecode.com/svn/trunk@12764
2bbb7eff-a529-9590-31e7-
b0007b416f81
skia.committer@gmail.com [Thu, 19 Dec 2013 07:01:38 +0000 (07:01 +0000)]
Sanitizing source files in Housekeeper-Nightly
git-svn-id: http://skia.googlecode.com/svn/trunk@12763
2bbb7eff-a529-9590-31e7-
b0007b416f81
robertphillips@google.com [Wed, 18 Dec 2013 23:56:59 +0000 (23:56 +0000)]
Revert r12758 (Updated GLintptr and GLsizeiptr to be the appropriate types on 64 bit Windows (Take 2) - https://codereview.chromium.org/
118533003) due to Chromium-side compilation issues
git-svn-id: http://skia.googlecode.com/svn/trunk@12762
2bbb7eff-a529-9590-31e7-
b0007b416f81
vandebo@chromium.org [Wed, 18 Dec 2013 22:20:16 +0000 (22:20 +0000)]
Update expectations for r12729
"[PDF] Fix clipping in xfermode improvement."
TBR=bungeman@google.com
Review URL: https://codereview.chromium.org/
112343007
git-svn-id: http://skia.googlecode.com/svn/trunk@12761
2bbb7eff-a529-9590-31e7-
b0007b416f81
tfarina@chromium.org [Wed, 18 Dec 2013 22:15:12 +0000 (22:15 +0000)]
Get rid of DEFINE_TESTCLASS() macro.
Remaining tests were rewrite to use DEF_TEST() macro instead. This fixes
the FIXME in TestClassDef.h
BUG=None
TEST=tests
R=mtklein@google.com
Review URL: https://codereview.chromium.org/
114563003
git-svn-id: http://skia.googlecode.com/svn/trunk@12760
2bbb7eff-a529-9590-31e7-
b0007b416f81
commit-bot@chromium.org [Wed, 18 Dec 2013 21:58:53 +0000 (21:58 +0000)]
Remove SkBitmapAlphaThresholdShader
R=bsalomon@google.com
Author: zork@chromium.org
Review URL: https://codereview.chromium.org/
108653012
git-svn-id: http://skia.googlecode.com/svn/trunk@12759
2bbb7eff-a529-9590-31e7-
b0007b416f81