platform/upstream/libSkiaSharp.git
10 years agoSpeculative Mac 10.6 fix: disable medium & high filter quality tests.
senorblanco@chromium.org [Tue, 25 Mar 2014 21:12:18 +0000 (21:12 +0000)]
Speculative Mac 10.6 fix: disable medium & high filter quality tests.

BUG=skia:
TBR=mtklein@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13947 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoFix bug on android with GL_EXT_debug_marker
commit-bot@chromium.org [Tue, 25 Mar 2014 21:01:13 +0000 (21:01 +0000)]
Fix bug on android with GL_EXT_debug_marker

A device was finding the extension string, but was returning NULL
when getting the proc addresses

BUG=skia:
R=bsalomon@google.com

Author: egdaniel@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13946 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoAdd a minimal --undefok to SkCommandLineFlags.
commit-bot@chromium.org [Tue, 25 Mar 2014 21:00:02 +0000 (21:00 +0000)]
Add a minimal --undefok to SkCommandLineFlags.

Similar in spirit to gflags' undefok, I'd like to be able to ignore
specific unknown flags.  This lets me run the same command line on, say,
a branch that's got a new flag and on a clean branch tracking
origin/master.  This is handy for performance comparison, etc.

It's not essential, and if you hate this I can find another way.

As an example, I want to compare the runtime of SKP recording with my new code.  I've added a flag --skr to bench_record to help this.  So I want to compare

origin/master: out/Release/bench_record
my patch:      out/Release/bench_record --skr

This lets me run both as out/Release/bench_record --undefok skr --skr, which is handy for scripts and things.

BUG=skia:
R=scroggo@google.com, mtklein@google.com

Author: mtklein@chromium.org

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13945 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoSpeculative Mac 10.6 fix: remove useless code from matriximagefilter GM.
senorblanco@chromium.org [Tue, 25 Mar 2014 20:47:29 +0000 (20:47 +0000)]
Speculative Mac 10.6 fix: remove useless code from matriximagefilter GM.

TBR=mtklein@google.com
BUG=skia:

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13944 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoSuppress failing GMs for rebaseline.
fmalita@google.com [Tue, 25 Mar 2014 19:36:20 +0000 (19:36 +0000)]
Suppress failing GMs for rebaseline.

https://codereview.chromium.org/203203004 caused a couple of minor GM
diffs:

fontscaler
bleed

TBR=mtklein@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13943 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoFurther tweak compare_codereview.py.
commit-bot@chromium.org [Tue, 25 Mar 2014 18:02:17 +0000 (18:02 +0000)]
Further tweak compare_codereview.py.

- only show details for bots where at least one side failed
- dedup identical failures

BUG=skia:
R=halcanary@google.com, mtklein@google.com

Author: mtklein@chromium.org

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13942 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoImplement a generic matrix transform image filter.
senorblanco@chromium.org [Tue, 25 Mar 2014 17:35:10 +0000 (17:35 +0000)]
Implement a generic matrix transform image filter.

This will be used in Blink to accommodate matrices that contain
rotation or shearing. This is a generalization of SkResizeImageFilter,
so I've replaced all uses of SkResizeImageFilter in Skia. (It might be
easier to review by diffing it with SkResizeImageFilter, too.)

R=reed@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13941 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoConsolidate SkCanvas matrix virtuals.
commit-bot@chromium.org [Tue, 25 Mar 2014 17:32:26 +0000 (17:32 +0000)]
Consolidate SkCanvas matrix virtuals.

Remove didTranslate, didScale, didRotate & didSkew, and rely on
didConcat instead. Subclasses can sniff the matrix type if they want to
differentiate.

(work in progress)

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

Author: fmalita@chromium.org

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13940 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoRevert "Revert of implement readPixels and writePixels natively, w/o using the (depre...
reed@google.com [Tue, 25 Mar 2014 16:20:24 +0000 (16:20 +0000)]
Revert "Revert of implement readPixels and writePixels natively, w/o using the (deprecated) (https://codereview.chromium.org/199733016/)"

This reverts commit 9a90bd16dc6756395c422adf0f24560d033ed9ea.

BUG=skia:
R=bsalomon@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13939 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoFix for int to bool in GrInOrderDrawBuffer
commit-bot@chromium.org [Tue, 25 Mar 2014 15:49:59 +0000 (15:49 +0000)]
Fix for int to bool in GrInOrderDrawBuffer
NOTREECHECK=True
NOTRY=True
R=bsalomon@google.com
TBR=bsalomon@google.com
BUG=skia:

Author: egdaniel@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13938 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoplatform_tools/barelinux/bin/arm64_make script
commit-bot@chromium.org [Tue, 25 Mar 2014 15:37:36 +0000 (15:37 +0000)]
platform_tools/barelinux/bin/arm64_make script

Motivation: an upcoming CL will change the Gyp variables.
That CL can change this script and the deps roll bot will
pick up those changes.
NOTRY=true
R=borenet@google.com, djsollen@google.com

Author: halcanary@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13937 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoAdd Gpu Tracing to Ganesh
commit-bot@chromium.org [Tue, 25 Mar 2014 15:17:32 +0000 (15:17 +0000)]
Add Gpu Tracing to Ganesh

BUG=skia:2316
R=bsalomon@google.com, tomhudson@google.com

Author: egdaniel@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13936 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoTweak compare_codereview.py.
commit-bot@chromium.org [Tue, 25 Mar 2014 15:16:27 +0000 (15:16 +0000)]
Tweak compare_codereview.py.

  - summary at end so I don't have to scroll back up
  - tweak the diff markers a bit
  - strip the test runtimes

BUG=skia:
R=halcanary@google.com, mtklein@google.com

Author: mtklein@chromium.org

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13935 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoRemove more flags not needed by android framework.
commit-bot@chromium.org [Tue, 25 Mar 2014 15:14:18 +0000 (15:14 +0000)]
Remove more flags not needed by android framework.

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

Author: scroggo@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13934 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoAdd GM that tests GrTextureDomain and fix bug where kDecal_Mode gets incorrectly...
commit-bot@chromium.org [Tue, 25 Mar 2014 15:13:18 +0000 (15:13 +0000)]
Add GM that tests GrTextureDomain and fix bug where kDecal_Mode gets incorrectly ignored.

R=robertphillips@google.com

Author: bsalomon@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13933 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoRevert of implement readPixels and writePixels natively, w/o using the (deprecated...
commit-bot@chromium.org [Tue, 25 Mar 2014 13:38:44 +0000 (13:38 +0000)]
Revert of implement readPixels and writePixels natively, w/o using the (deprecated) (https://codereview.chromium.org/199733016/)

Reason for revert:
Android bots segfaulting in tests.

Original issue's description:
> implement readPixels and writePixels natively, w/o using the (deprecated)
> SkCanvas::Config8888 enum.
>
> Revert "Revert "hide Config8888 entirely". Broke a bunch of builds."
>
> This reverts commit 763277ba157fef0f651004bb98a189e9f1ac730b.
>
> Needs chrome to remove the READPIXELS guard from skia's .gyp
>
> Committed: https://code.google.com/p/skia/source/detail?r=13931

R=reed@google.com
TBR=reed@google.com
NOTREECHECKS=true
NOTRY=true

Author: mtklein@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13932 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoimplement readPixels and writePixels natively, w/o using the (deprecated)
reed@google.com [Tue, 25 Mar 2014 12:00:30 +0000 (12:00 +0000)]
implement readPixels and writePixels natively, w/o using the (deprecated)
SkCanvas::Config8888 enum.

Revert "Revert "hide Config8888 entirely". Broke a bunch of builds."

This reverts commit 763277ba157fef0f651004bb98a189e9f1ac730b.

Needs chrome to remove the READPIXELS guard from skia's .gyp

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13931 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoMake it possible to draw multiple paths at once to a draw target
commit-bot@chromium.org [Tue, 25 Mar 2014 11:59:40 +0000 (11:59 +0000)]
Make it possible to draw multiple paths at once to a draw target

Add interface to draw multiple paths in a single "command" to a draw
target. Implement this interface in GrGpuGL with NVPR "instanced"
calls.

The instanced calls accept list of paths and list of transformations as
their parameters. The transformations are at this moment expected to be
2d affine transformations, as the functions are called only for text
rendering.

This will be used when drawing fonts. Later it can be maybe be used in
GrInOrderDrawBuffer to aggregate many draw calls into one instanced draw
call, similar to drawing rects.

R=jvanverth@google.com, bsalomon@google.com

Author: kkinnunen@nvidia.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13930 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoWhitespace change to trigger builds for new GCE compile bots
commit-bot@chromium.org [Tue, 25 Mar 2014 11:45:49 +0000 (11:45 +0000)]
Whitespace change to trigger builds for new GCE compile bots

NOTREECHECKS=true
BUG=skia:2323
TBR=

Author: rmistry@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13929 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoSanitizing source files in Housekeeper-Nightly
skia.committer@gmail.com [Tue, 25 Mar 2014 03:02:32 +0000 (03:02 +0000)]
Sanitizing source files in Housekeeper-Nightly

git-svn-id: http://skia.googlecode.com/svn/trunk@13928 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoSkPaint: eliminate some dead bytes in 64-bit build.
commit-bot@chromium.org [Mon, 24 Mar 2014 23:03:11 +0000 (23:03 +0000)]
SkPaint: eliminate some dead bytes in 64-bit build.

+ memcpy-based copy constructor was hiding this gap -> manual copy constructor.
+ Split tests for finer-grained failures.

BUG=skia:

Committed: http://code.google.com/p/skia/source/detail?r=13856

Committed: http://code.google.com/p/skia/source/detail?r=13887

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

Author: mtklein@chromium.org

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13927 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoSkFontHost_linux - dup stream instead of ref
commit-bot@chromium.org [Mon, 24 Mar 2014 21:42:01 +0000 (21:42 +0000)]
SkFontHost_linux - dup stream instead of ref

R=bungeman@google.com

Author: halcanary@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13926 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoFix Mac Debug clang build.
senorblanco@chromium.org [Mon, 24 Mar 2014 21:32:28 +0000 (21:32 +0000)]
Fix Mac Debug clang build.

TBR=bsalomon@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13925 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoMark dashing_gpu and complexclip2_rect_aa_gpu as ignore-failure on Win7 perspective
commit-bot@chromium.org [Mon, 24 Mar 2014 20:58:21 +0000 (20:58 +0000)]
Mark dashing_gpu and complexclip2_rect_aa_gpu as ignore-failure on Win7 perspective

BUG=skia:2245
R=epoger@google.com

Author: borenet@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13924 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoFix for crash on large image blur sigma values.
senorblanco@chromium.org [Mon, 24 Mar 2014 20:50:59 +0000 (20:50 +0000)]
Fix for crash on large image blur sigma values.

This was crashing on the GPU path, due to a failed texture allocation.
The belt-and-suspenders fix is to:

1) Limit the GPU path to only allocate up to maxTextureSize.
2) Limit both the raster and GPU paths to reasonable blur sizes (box blur
   kernel size of 1000, resulting in a sigma limit of 532).

R=bsalomon@google.com
BUG=skia:

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13923 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agogather GM tests which are disabled on Android
commit-bot@chromium.org [Mon, 24 Mar 2014 20:07:44 +0000 (20:07 +0000)]
gather GM tests which are disabled on Android

BUG=skia:2326
R=borenet@google.com, djsollen@google.com

Author: epoger@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13922 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agofix compare_to_expectations command line launch
commit-bot@chromium.org [Mon, 24 Mar 2014 19:51:57 +0000 (19:51 +0000)]
fix compare_to_expectations command line launch

(SkipBuildbotRuns)
NOTREECHECKS=True
NOTRY=True
R=rmistry@google.com
TBR=rmistry

Author: epoger@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13921 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoAdd SkASSERT's to gl/Interface validate
commit-bot@chromium.org [Mon, 24 Mar 2014 19:43:02 +0000 (19:43 +0000)]
Add SkASSERT's to gl/Interface validate

BUG=skia:
R=bsalomon@google.com

Author: egdaniel@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13920 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoAdd new experimental API to SkPicture to get "id" of current op
commit-bot@chromium.org [Mon, 24 Mar 2014 19:40:49 +0000 (19:40 +0000)]
Add new experimental API to SkPicture to get "id" of current op

When gathering information about a picture (in the new SkDevice::EXPERIMENTAL_optimize entry point) it is necessary to be able to correlate the gathered information with the command in the SkPicture (so the information can later be combined with the similarly indexed information from the BBH). This entry point exposes that information to friend classes.

R=reed@google.com

Author: robertphillips@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13919 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoWhen clipping treat rrect corners where either the x or y radius is < 0.5 as square.
commit-bot@chromium.org [Mon, 24 Mar 2014 19:24:59 +0000 (19:24 +0000)]
When clipping treat rrect corners where either the x or y radius is < 0.5 as square.

BUG=skia:2181
R=robertphillips@google.com, jvanverth@google.com

Author: bsalomon@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13918 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoClean up SkOnce usage in SkFontMgr_Indirect.
bungeman@google.com [Mon, 24 Mar 2014 18:38:25 +0000 (18:38 +0000)]
Clean up SkOnce usage in SkFontMgr_Indirect.

R=mtklein@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13917 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoMore distance field improvements:
commit-bot@chromium.org [Mon, 24 Mar 2014 18:34:04 +0000 (18:34 +0000)]
More distance field improvements:
- Make sure we only use the 0,0 subpixel glyph
- Copy image data into outsized buffer to ensure we capture outer edges
- Change edge dump to blend better with original glyph
- Edges are detected across >128 and <128 boundaries, or when
  there are two non-zero <128 pixels next to each other

BUG=skia:
R=robertphillips@google.com

Author: jvanverth@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13916 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoAdd tools to generated Android.mk
commit-bot@chromium.org [Mon, 24 Mar 2014 18:26:36 +0000 (18:26 +0000)]
Add tools to generated Android.mk

Include the skia tools (bench, gm, tests) in the generated Android
make file for the framework. They are commented out, so that a tester
can include them if desired.

BUG=skia:1975
R=halcanary@google.com, djsollen@google.com

Author: scroggo@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13915 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoremove SK_SUPPORT_LEGACY_READPIXELSCONFIG
reed@google.com [Mon, 24 Mar 2014 18:07:24 +0000 (18:07 +0000)]
remove SK_SUPPORT_LEGACY_READPIXELSCONFIG

BUG=skia:

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13914 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agorebaseline Nexus10 results obscured by skia:2313
commit-bot@chromium.org [Mon, 24 Mar 2014 17:52:28 +0000 (17:52 +0000)]
rebaseline Nexus10 results obscured by skia:2313

BUG=skia:2313
NOTREECHECKS=True
NOTRY=True
R=bsalomon@google.com
TBR=bsalomon

Author: epoger@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13913 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoAdd skimage expectations for Arm8_64
commit-bot@chromium.org [Mon, 24 Mar 2014 17:44:00 +0000 (17:44 +0000)]
Add skimage expectations for Arm8_64
NOTRY=true
R=djsollen@google.com, scroggo@google.com, borenet@google.com

Author: halcanary@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13912 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoRemove unnecessary isrect code in clip mask manager
commit-bot@chromium.org [Mon, 24 Mar 2014 17:30:48 +0000 (17:30 +0000)]
Remove unnecessary isrect code in clip mask manager

R=robertphillips@google.com

Author: bsalomon@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13911 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoUpdate expectations for Ubuntu12-GTX660
commit-bot@chromium.org [Mon, 24 Mar 2014 16:59:09 +0000 (16:59 +0000)]
Update expectations for Ubuntu12-GTX660

Move ATI5770 expectations to GTX660

BUG=skia:2325
NOTRY=true
R=epoger@google.com

Author: borenet@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13910 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoHave SkTArray state that it default-initializes.
bungeman@google.com [Mon, 24 Mar 2014 15:55:01 +0000 (15:55 +0000)]
Have SkTArray state that it default-initializes.

The current comments in SkTArray lead one to believe that it is default
constructing all types. However, it is not calling the default
constructor for primitive types, it is actually default-initializing.

R=bsalomon@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13909 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoManual memset to work around bogus compiler warning bug.
commit-bot@chromium.org [Mon, 24 Mar 2014 15:52:33 +0000 (15:52 +0000)]
Manual memset to work around bogus compiler warning bug.

BUG=skia:2215
R=bsalomon@google.com, mtklein@google.com

Author: mtklein@chromium.org

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13908 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoAdd skimage baselines for Ubuntu12-GTX660 bots
commit-bot@chromium.org [Mon, 24 Mar 2014 15:40:42 +0000 (15:40 +0000)]
Add skimage baselines for Ubuntu12-GTX660 bots

BUG=skia:2325
R=scroggo@google.com

Author: borenet@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13907 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoFix 10.6 build: double -> float.
commit-bot@chromium.org [Mon, 24 Mar 2014 14:54:04 +0000 (14:54 +0000)]
Fix 10.6 build: double -> float.

BUG=skia:
R=robertphillips@google.com, mtklein@google.com, reed@google.com

Author: mtklein@chromium.org

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13906 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoAdd test cases for small radii to rrect GMs.
commit-bot@chromium.org [Mon, 24 Mar 2014 14:53:09 +0000 (14:53 +0000)]
Add test cases for small radii to rrect GMs.

BUG=skia:2181
R=robertphillips@google.com

Author: bsalomon@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13905 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoupdate bitmapshader to avoid potential rendering error for kA8_Config
commit-bot@chromium.org [Mon, 24 Mar 2014 02:49:48 +0000 (02:49 +0000)]
update bitmapshader to avoid potential rendering error for kA8_Config

A local matrix is set to the shader, which will show the potential error for SkCanvas::drawBitmap when the bitmap is kA8_Config, as well as the potential error for drawing a geometry with a bitmapshader who's bitmap is kA8_Config.

A simple case is also added for SkCanvas::drawBitmap when the bitmap is kA8_Config, but there is not shader. Then the drawing area will be colorized by the solid color set in SkPaint.

BUG=skia:2278
BUG=skia:2293
R=reed@google.com, bsalomon@google.com

Author: yunchao.he@intel.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13904 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agodisable imagefilterscropexpand GM test on Android
commit-bot@chromium.org [Sun, 23 Mar 2014 23:08:18 +0000 (23:08 +0000)]
disable imagefilterscropexpand GM test on Android

BUG=skia:2313
NOTREECHECKS=True
NOTRY=True
R=borenet@google.com, senorblanco@google.com, senorblanco@chromium.org

Author: epoger@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13903 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoSanitizing source files in Housekeeper-Nightly
skia.committer@gmail.com [Sat, 22 Mar 2014 03:02:27 +0000 (03:02 +0000)]
Sanitizing source files in Housekeeper-Nightly

git-svn-id: http://skia.googlecode.com/svn/trunk@13902 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoFix r13897.
bungeman@google.com [Sat, 22 Mar 2014 00:25:34 +0000 (00:25 +0000)]
Fix r13897.

git-svn-id: http://skia.googlecode.com/svn/trunk@13901 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoFix r13897.
bungeman@google.com [Fri, 21 Mar 2014 23:19:10 +0000 (23:19 +0000)]
Fix r13897.

git-svn-id: http://skia.googlecode.com/svn/trunk@13900 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoFix r13897.
bungeman@google.com [Fri, 21 Mar 2014 23:07:24 +0000 (23:07 +0000)]
Fix r13897.

git-svn-id: http://skia.googlecode.com/svn/trunk@13899 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoFix r13897.
bungeman@google.com [Fri, 21 Mar 2014 23:00:41 +0000 (23:00 +0000)]
Fix r13897.

git-svn-id: http://skia.googlecode.com/svn/trunk@13898 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoA remotable font management interface and DirectWrite implementation.
bungeman@google.com [Fri, 21 Mar 2014 22:48:32 +0000 (22:48 +0000)]
A remotable font management interface and DirectWrite implementation.

The introduced SkRemotableFontMgr is a font management interface designed for simple and fast proxy support. SkFontMgr_Indirect bridges a SkRemotableFontMgr and a local SkFontMgr to present a SkFontMgr interface.

This change is to be followed by https://codereview.chromium.org/132113015/ and  https://codereview.chromium.org/206693003 .

R=reed@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13897 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoFix incorrect assert in r13895.
bungeman@google.com [Fri, 21 Mar 2014 20:14:05 +0000 (20:14 +0000)]
Fix incorrect assert in r13895.

git-svn-id: http://skia.googlecode.com/svn/trunk@13896 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoAdd removeShuffle to SkTArray and add SkTArray tests.
bungeman@google.com [Fri, 21 Mar 2014 19:39:02 +0000 (19:39 +0000)]
Add removeShuffle to SkTArray and add SkTArray tests.

R=reed@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13895 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agorebaseline_server: rename results.py to compare_to_expectations.py
commit-bot@chromium.org [Fri, 21 Mar 2014 17:54:14 +0000 (17:54 +0000)]
rebaseline_server: rename results.py to compare_to_expectations.py

BUG=skia:1919
NOTREECHECKS=True
NOTRY=True
R=borenet@google.com

Author: epoger@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13894 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agorebaseline_server: fix --editable mode
commit-bot@chromium.org [Fri, 21 Mar 2014 17:30:12 +0000 (17:30 +0000)]
rebaseline_server: fix --editable mode

complete revert of https://skia.googlesource.com/skia/+/736be3527214ddbd3dd7e42a89a374c62d98f370

partial revert of https://skia.googlesource.com/skia/+/579942387bed9259b03419c593503022e1399931

and glue made necessary by the above.

(SkipBuildbotRuns)
BUG=skia:2319
NOTREECHECKS=True
NOTRY=True
R=senorblanco@google.com, senorblanco@chromium.org

Author: epoger@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13893 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoFixed double offset in resize filter
commit-bot@chromium.org [Fri, 21 Mar 2014 16:19:28 +0000 (16:19 +0000)]
Fixed double offset in resize filter

The fix is trivial, simply remove the extra offset

I added another case to the resizeimagefilter gm and made it so that it looks exactly like the one next to it, so that failure is easy to detect visually.

BUG=skia:
R=senorblanco@google.com, senorblanco@chromium.org

Author: sugoi@chromium.org

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13892 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoSome stragglerbot rebaselines for lighting, imagefiltersscaled GMs.
senorblanco@chromium.org [Fri, 21 Mar 2014 15:04:12 +0000 (15:04 +0000)]
Some stragglerbot rebaselines for lighting, imagefiltersscaled GMs.

TBR=jcgregorio@google.com
BUG=skia:

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13891 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoFix broken Android builds due to missing includes for NEON defines
djsollen@google.com [Fri, 21 Mar 2014 13:13:22 +0000 (13:13 +0000)]
Fix broken Android builds due to missing includes for NEON defines

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13890 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoSanitizing source files in Housekeeper-Nightly
skia.committer@gmail.com [Fri, 21 Mar 2014 03:02:42 +0000 (03:02 +0000)]
Sanitizing source files in Housekeeper-Nightly

git-svn-id: http://skia.googlecode.com/svn/trunk@13889 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoRevert of SkPaint: eliminate some dead bytes in 64-bit build. (https://codereview...
commit-bot@chromium.org [Fri, 21 Mar 2014 01:40:07 +0000 (01:40 +0000)]
Revert of SkPaint: eliminate some dead bytes in 64-bit build. (https://codereview.chromium.org/203203003/)

Reason for revert:
Huh, some Android tests are still failing despite the fix.  IntelRhB, Xoom... that's weird.

Original issue's description:
> SkPaint: eliminate some dead bytes in 64-bit build.
>
> + memcpy-based copy constructor was hiding this gap -> manual copy constructor.
> + Split tests for finer-grained failures.
>
> BUG=skia:
>
> Committed: http://code.google.com/p/skia/source/detail?r=13856
>
> Committed: http://code.google.com/p/skia/source/detail?r=13887

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

Author: mtklein@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13888 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoSkPaint: eliminate some dead bytes in 64-bit build.
commit-bot@chromium.org [Thu, 20 Mar 2014 23:02:35 +0000 (23:02 +0000)]
SkPaint: eliminate some dead bytes in 64-bit build.

+ memcpy-based copy constructor was hiding this gap -> manual copy constructor.
+ Split tests for finer-grained failures.

BUG=skia:

Committed: http://code.google.com/p/skia/source/detail?r=13856

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

Author: mtklein@chromium.org

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13887 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoRemove the now-unnecessary enum in the elliptical clipping effect.
commit-bot@chromium.org [Thu, 20 Mar 2014 21:39:03 +0000 (21:39 +0000)]
Remove the now-unnecessary enum in the elliptical clipping effect.

BUG=skia:2181
R=robertphillips@google.com

Author: bsalomon@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13886 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoFix cull nesting assertion.
commit-bot@chromium.org [Thu, 20 Mar 2014 20:25:14 +0000 (20:25 +0000)]
Fix cull nesting assertion.

Cull rects are in local coordinates and cannot be compared directly.

No wonder it was so hard enforcing this in Blink :o

This moves the validation logic into SkCanvas, using a device-space
cull stack (debug build only).

There are still some Blink bugs causing violations, so for now I'd like
to keep this as an error message only.

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

Author: fmalita@chromium.org

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13885 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoreplace old SK_TRACE_EVENT macros with new TRACE_EVENT ones, and then remove them...
commit-bot@chromium.org [Thu, 20 Mar 2014 20:00:24 +0000 (20:00 +0000)]
replace old SK_TRACE_EVENT macros with new TRACE_EVENT ones, and then remove them entirely!

BUG=skia:353
R=tomhudson@google.com, epoger@google.com, reed@google.com, egdaniel@google.com, bsalomon@google.com

Author: humper@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13884 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agogyp_skia: make default GYP_GENERATORS for each platform explicit
commit-bot@chromium.org [Thu, 20 Mar 2014 19:59:09 +0000 (19:59 +0000)]
gyp_skia: make default GYP_GENERATORS for each platform explicit

BUG=skia:2317
R=bungeman@google.com

Author: epoger@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13883 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agofix error in linux makefile for some shells
commit-bot@chromium.org [Thu, 20 Mar 2014 19:56:55 +0000 (19:56 +0000)]
fix error in linux makefile for some shells

R=borenet@google.com
TBR=borenet@google.com
NOTRY=true

Author: halcanary@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13882 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoandroid_run_skia: fix error check
commit-bot@chromium.org [Thu, 20 Mar 2014 19:48:16 +0000 (19:48 +0000)]
android_run_skia: fix error check

(SkipBuildbotRuns)
BUG=skia:2313
NOTREECHECKS=True
NOTRY=True
R=borenet@google.com

Author: epoger@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13881 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoDon't use __ARM_HAVE_NEON to detect NEON's presence in SkBlitter_RGB16
commit-bot@chromium.org [Thu, 20 Mar 2014 18:37:06 +0000 (18:37 +0000)]
Don't use __ARM_HAVE_NEON to detect NEON's presence in SkBlitter_RGB16

Replace __ARM_HAVE_NEON by SK_ARM_NEON_IS_ALWAYS. It was the only
use of __ARM_HAVE_NEON apart from defining SK_ARM_NEON_IS_ALWAYS.

Signed-off-by: Kévin PETIT <kevin.petit@arm.com>
BUG=skia:
R=djsollen@google.com, mtklein@google.com, halcanary@google.com, reed@google.com

Author: kevin.petit@arm.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13880 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agorebaseline_server: when first started, tell client results are still being generated
commit-bot@chromium.org [Thu, 20 Mar 2014 18:31:00 +0000 (18:31 +0000)]
rebaseline_server: when first started, tell client results are still being generated

(SkipBuildbotRuns)
NOTREECHECKS=True
NOTRY=True
R=senorblanco@google.com, senorblanco@chromium.org

Author: epoger@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13879 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoNew baselines for imagefiltersscaled and lighting GMs.
senorblanco@chromium.org [Thu, 20 Mar 2014 17:41:32 +0000 (17:41 +0000)]
New baselines for imagefiltersscaled and lighting GMs.

TBR=jcgregorio@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13878 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoadd some debugging for bug skia:2313
commit-bot@chromium.org [Thu, 20 Mar 2014 17:32:27 +0000 (17:32 +0000)]
add some debugging for bug skia:2313

(SkipBuildbotRuns)
BUG=skia:2313
NOTREECHECKS=True
NOTRY=True
R=borenet@google.com

Author: epoger@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13877 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agorebaseline_server: serve JSON from static file, rather than generating it live
commit-bot@chromium.org [Thu, 20 Mar 2014 17:27:46 +0000 (17:27 +0000)]
rebaseline_server: serve JSON from static file, rather than generating it live

BUG=skia:1455, skia:2230
NOTREECHECKS=True
NOTRY=True
R=borenet@google.com

Author: epoger@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13876 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoMore gpu rebaselines
bsalomon@google.com [Thu, 20 Mar 2014 15:35:33 +0000 (15:35 +0000)]
More gpu rebaselines

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13875 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agorebaseline_server: image URLs are relative to JSON file's location
commit-bot@chromium.org [Thu, 20 Mar 2014 15:27:34 +0000 (15:27 +0000)]
rebaseline_server: image URLs are relative to JSON file's location

BUG=skia:1455, skia:2230
NOTREECHECKS=True
NOTRY=True
R=borenet@google.com

Author: epoger@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13874 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoRebaseline minor pixel diffs from recent GPU changes.
senorblanco@chromium.org [Thu, 20 Mar 2014 15:24:39 +0000 (15:24 +0000)]
Rebaseline minor pixel diffs from recent GPU changes.

TBR=bsalomon

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13873 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoRevert "hide Config8888 entirely". Broke a bunch of builds.
bsalomon@google.com [Thu, 20 Mar 2014 13:26:45 +0000 (13:26 +0000)]
Revert "hide Config8888 entirely". Broke a bunch of builds.

This reverts commit fa11c49cc11a6c9ebafbf9c59e118917f9b3cc56.

Revert "Sanitizing source files in Housekeeper-Nightly" to make the above revert clean.

This reverts commit b5787422c8eb2a27a9576777597fd9e06784acdb.

TBR=reed@google.com
TBR=jcgregorio@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13872 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoSanitizing source files in Housekeeper-Nightly
skia.committer@gmail.com [Thu, 20 Mar 2014 03:02:31 +0000 (03:02 +0000)]
Sanitizing source files in Housekeeper-Nightly

git-svn-id: http://skia.googlecode.com/svn/trunk@13871 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoFix Windows build break introduced in r13864.
bungeman@google.com [Wed, 19 Mar 2014 23:26:50 +0000 (23:26 +0000)]
Fix Windows build break introduced in r13864.

Need SkToBool(...).

git-svn-id: http://skia.googlecode.com/svn/trunk@13870 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoSilence warning after f5e315ccf1a.
commit-bot@chromium.org [Wed, 19 Mar 2014 22:58:52 +0000 (22:58 +0000)]
Silence warning after f5e315ccf1a.

(clang 3.3):

../tools/PictureRenderer.cpp:350:13: error: variable 'hash' is used uninitialized whenever 'if' condition is false
      [-Werror,-Wsometimes-uninitialized]
        if (!generatedHash) {
            ^~~~~~~~~~~~~~
../tools/PictureRenderer.cpp:354:53: note: uninitialized use occurs here
        jsonSummaryPtr->add(outputFilename.c_str(), hash);
                                                    ^~~~
../tools/PictureRenderer.cpp:350:9: note: remove the 'if' if its condition is always true
        if (!generatedHash) {
        ^~~~~~~~~~~~~~~~~~~~
../tools/PictureRenderer.cpp:334:18: note: initialize the variable 'hash' to silence this warning
    uint64_t hash;
                 ^
                  = 0

The warning is wrong, but the compiler does have a point: generatedHash is always false at that point.

R=epoger@google.com

Author: fmalita@chromium.org

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13869 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agofix non-gpu build
commit-bot@chromium.org [Wed, 19 Mar 2014 22:54:40 +0000 (22:54 +0000)]
fix non-gpu build

BUG=
R=bungeman@google.com
TBR=bungeman@google.com

Author: humper@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13868 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoOnly use analytic effect-based clipping when AA is required by at least one element.
commit-bot@chromium.org [Wed, 19 Mar 2014 21:26:35 +0000 (21:26 +0000)]
Only use analytic effect-based clipping when AA is required by at least one element.

BUG=352718
R=robertphillips@google.com

Author: bsalomon@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13867 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agostub for ganesh veto
commit-bot@chromium.org [Wed, 19 Mar 2014 21:24:25 +0000 (21:24 +0000)]
stub for ganesh veto

BUG=skia:
R=bsalomon@google.com, nduca@chromium.org

Author: humper@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13866 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agohide Config8888 entirely
commit-bot@chromium.org [Wed, 19 Mar 2014 21:20:24 +0000 (21:20 +0000)]
hide Config8888 entirely

BUG=skia:
R=bsalomon@google.com

Author: reed@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13865 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoAdding a new SkSurface factory for generating surfaces from the scratch texture pool.
commit-bot@chromium.org [Wed, 19 Mar 2014 21:19:16 +0000 (21:19 +0000)]
Adding a new SkSurface factory for generating surfaces from the scratch texture pool.

TEST=Surface unit test
BUG=crbug.com/351798
R=bsalomon@google.com, robertphillips@google.com, reed@google.com

Author: junov@chromium.org

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13864 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoUse rounding to 8888 in lighting filters, raster path.
senorblanco@chromium.org [Wed, 19 Mar 2014 19:44:41 +0000 (19:44 +0000)]
Use rounding to 8888 in lighting filters, raster path.

when converting float -> 8-bit BGRA in lighting filters, use rounding
rather than floor(). This makes the GPU and raster paths match more
closely (as tested by Blink test
feDiffuseLighting-linearrgb-lighting-color.svg).

Will affect the LSB of lighting, imagefiltersscaled GMs.

R=junov@chromium.org
BUG=skia:

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13863 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoFixed cross compilation with mingw.
commit-bot@chromium.org [Wed, 19 Mar 2014 19:28:00 +0000 (19:28 +0000)]
Fixed cross compilation with mingw.

Tested in Mozilla source tree. I tried to use skia build system, but it's up to the task for cross compilation.

SkHRESULT.cpp - Use proper file name (that matters on case sensitive OSes)

SkAtomics_win.h - Don't use pragma intrinsic on GCC (this causes massive warnings)

SkOSFile_win.cpp - This one is tricky. GCC doesn't allow (void*) casts in template argument constants and INVALID_HANDLE_VALUE looks like this:
((HANDLE)(LONG_PTR)-1)
where HANDLE is typedefed to void*. Changed the code to use LONG_PTR as template argument and cast it when needed.

BUG=skia:
R=bungeman@google.com, reed@google.com

Author: cjacek@gmail.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13862 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoAdd quiet mode to lua_pictures.
commit-bot@chromium.org [Wed, 19 Mar 2014 19:23:17 +0000 (19:23 +0000)]
Add quiet mode to lua_pictures.

This allows:

for i in {0..5}; do
    out/Debug/lua_pictures -q --skpPath $SKP_PATH -l $SCRIPT --modulo $i 6
done

to generate output similar to what telemetry will generate.

R=reed@google.com

Author: robertphillips@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13861 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoFix accessBitmap() with deferred clear, GPU path.
senorblanco@chromium.org [Wed, 19 Mar 2014 18:24:04 +0000 (18:24 +0000)]
Fix accessBitmap() with deferred clear, GPU path.

When accessBitmap() is called on a GPU device which has deferred clear,
and no drawing has been done, force the clear before returning the
bitmap.

This is covered by the top right test case in the imagefilterscropexpand
GM, where an offset image filter is drawn but the resulting bitmap is
cropped out, resulting in no drawing to the offscreen. It should be
empty, but without this fix it results in garbage (previous scratch
texture contents).

BUG=skia:
R=bsalomon@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13860 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoadd --writeChecksumBasedFilenames flag to render_pictures
commit-bot@chromium.org [Wed, 19 Mar 2014 17:26:07 +0000 (17:26 +0000)]
add --writeChecksumBasedFilenames flag to render_pictures

BUG=skia:1455,skia:2230
R=robertphillips@google.com, robertphillips@chromium.org

Author: epoger@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13859 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoRevert of SkPaint: eliminate some dead bytes in 64-bit build. (https://codereview...
commit-bot@chromium.org [Wed, 19 Mar 2014 14:36:29 +0000 (14:36 +0000)]
Revert of SkPaint: eliminate some dead bytes in 64-bit build. (https://codereview.chromium.org/203203003/)

Reason for revert:
Causing RunTest failures on Android.

Original issue's description:
> SkPaint: eliminate some dead bytes in 64-bit build.
>
> + memcpy-based copy constructor was hiding this gap -> manual copy constructor.
> + Split tests for finer-grained failures.
>
> BUG=skia:
>
> Committed: http://code.google.com/p/skia/source/detail?r=13856

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

Author: jcgregorio@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13858 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoWhitespace change to verify that buildbot scripts are fixed.
commit-bot@chromium.org [Wed, 19 Mar 2014 14:22:18 +0000 (14:22 +0000)]
Whitespace change to verify that buildbot scripts are fixed.

BUG=skia:2312
NOTRY=true
NOTREECHECKS=true
R=rmistry@google.com

Author: borenet@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13857 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoSkPaint: eliminate some dead bytes in 64-bit build.
commit-bot@chromium.org [Wed, 19 Mar 2014 11:42:34 +0000 (11:42 +0000)]
SkPaint: eliminate some dead bytes in 64-bit build.

+ memcpy-based copy constructor was hiding this gap -> manual copy constructor.
+ Split tests for finer-grained failures.

BUG=skia:
R=reed@google.com, mtklein@google.com

Author: mtklein@chromium.org

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13856 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoSanitizing source files in Housekeeper-Nightly
skia.committer@gmail.com [Wed, 19 Mar 2014 03:02:22 +0000 (03:02 +0000)]
Sanitizing source files in Housekeeper-Nightly

git-svn-id: http://skia.googlecode.com/svn/trunk@13855 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoRebaseline imagefiltersscaled GM.
senorblanco@chromium.org [Tue, 18 Mar 2014 17:59:47 +0000 (17:59 +0000)]
Rebaseline imagefiltersscaled GM.

Rebaseline imagefiltersscaled GM for new test cases added in
https://code.google.com/p/skia/source/detail?r=13798.

TBR=jcgregorio@google.com
BUG=skia:

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13854 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoAdd a means of extracting active operations from SkPicture
commit-bot@chromium.org [Tue, 18 Mar 2014 17:45:32 +0000 (17:45 +0000)]
Add a means of extracting active operations from SkPicture

For the "pull forward" task I will be comparing the two cases:
  analyze the whole skp and use the BBH information
  analyze only the active portion of the skp

In the first case we need a way to get the BBH information out of the picture in order to extract the relevant portions of the whole-skp analysis. This adds caching of the active ops so that work isn't duplicated between when the optimization path queries for that information and when the usual draw path queries for it.

Committed: http://code.google.com/p/skia/source/detail?r=13836

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

Author: robertphillips@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13853 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoAdd reg test for http://crbug.com/348821
commit-bot@chromium.org [Tue, 18 Mar 2014 17:38:34 +0000 (17:38 +0000)]
Add reg test for http://crbug.com/348821

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

Author: mtklein@chromium.org

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13852 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoupdate comment on setShader to clarify alpha-bitmap behavior in bitmapshaders
commit-bot@chromium.org [Tue, 18 Mar 2014 16:28:12 +0000 (16:28 +0000)]
update comment on setShader to clarify alpha-bitmap behavior in bitmapshaders

BUG=skia:2293
R=bsalomon@google.com, yunchao.he@intel.com

Author: reed@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13851 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoRevert of r13836 due to Chromium cc_unittests failures
robertphillips@google.com [Tue, 18 Mar 2014 13:56:09 +0000 (13:56 +0000)]
Revert of r13836 due to Chromium cc_unittests failures

https://codereview.chromium.org/203333005/

git-svn-id: http://skia.googlecode.com/svn/trunk@13850 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoRemove SkCanvas matrix ops return value.
commit-bot@chromium.org [Tue, 18 Mar 2014 12:51:48 +0000 (12:51 +0000)]
Remove SkCanvas matrix ops return value.

The internal SkMatrix ops can no longer fail -> we can remove the bool
return value.

R=bsalomon@google.com, reed@google.com, robertphillips@google.com, scroggo@google.com, fmalita@google.com

Author: fmalita@chromium.org

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13849 2bbb7eff-a529-9590-31e7-b0007b416f81

10 years agoPut SimpleOffsetFilter on heap.
commit-bot@chromium.org [Tue, 18 Mar 2014 10:28:27 +0000 (10:28 +0000)]
Put SimpleOffsetFilter on heap.

Effects want to live on the heap!

BUG=skia:
R=senorblanco@chromium.org, dominikg@chromium.org, scroggo@google.com, mtklein@google.com

Author: mtklein@chromium.org

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

git-svn-id: http://skia.googlecode.com/svn/trunk@13848 2bbb7eff-a529-9590-31e7-b0007b416f81