platform/upstream/libSkiaSharp.git
10 years agoManually revert "4x allocation in PipeController is probably overkill."
commit-bot@chromium.org [Wed, 14 May 2014 14:50:57 +0000 (14:50 +0000)]
Manually revert "4x allocation in PipeController is probably overkill."

This reverts commit 2d91efffdb57646a495de5bf859ff281ef86dd12.

Conflicts:
src/pipe/SkGPipeWrite.cpp

BUG=372671
R=mtklein@google.com

Author: mtklein@chromium.org

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

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

10 years agoAdd ownership dox for SkShader::asACompose.
commit-bot@chromium.org [Wed, 14 May 2014 14:28:34 +0000 (14:28 +0000)]
Add ownership dox for SkShader::asACompose.

Supplement the comment added in
http://code.google.com/p/skia/source/detail?r=14716

R=reed@google.com

Author: scroggo@google.com

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

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

10 years agoBy default, Chromium optimizes for size when compiling on Android.
commit-bot@chromium.org [Wed, 14 May 2014 13:34:42 +0000 (13:34 +0000)]
By default, Chromium optimizes for size when compiling on Android.
Forcing the SkAlphaMulQ() function inline can yield as much as a 5-10%
improvement in rasterization time for some of Chromium's telemetry
tests on the Nexus 10, since it's in the inner loop of complex blends.

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

Author: tomhudson@chromium.org

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

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

10 years agoInline noop willFoo/didFoo into SkCanvas.h.
commit-bot@chromium.org [Wed, 14 May 2014 13:13:44 +0000 (13:13 +0000)]
Inline noop willFoo/didFoo into SkCanvas.h.

We've got our canvas subclasses all calling back up to these via
INHERITED, all noops.  That's fine but currently a little sad as they
can't be optimized away without link-time optimization, which we and
Chrome only do on Windows.  We actually make a call for each of these
today on non-Windows.

So, move the empty implementations into the header so those chaining
calls really can be optimized away.

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

Author: mtklein@chromium.org

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

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

10 years agoUpdate layer-pre-rendering for Chrome flag
commit-bot@chromium.org [Wed, 14 May 2014 13:11:48 +0000 (13:11 +0000)]
Update layer-pre-rendering for Chrome flag

As currently plumbed into Chromium, EXPERIMENTAL_optimize can be called multiple times per picture. Additionally, Chromium generates a lot of pictures with no layers at all.

R=bsalomon@google.com

Author: robertphillips@google.com

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

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

10 years agoPing MySQL every minute in an attempt to keep the connection alive.
commit-bot@chromium.org [Wed, 14 May 2014 12:55:34 +0000 (12:55 +0000)]
Ping MySQL every minute in an attempt to keep the connection alive.

I've tried various MySQL server settings, and some TCP settings on the webtry server, none seem to keep the connection from hanging, so this is a brute-force solution.

BUG=skia:
R=mtklein@google.com

Author: jcgregorio@google.com

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

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

10 years agoComment out call broken call to pushMatrix in LuaCanvas
commit-bot@chromium.org [Tue, 13 May 2014 20:32:32 +0000 (20:32 +0000)]
Comment out call broken call to pushMatrix in LuaCanvas

BUG=skia:
R=robertphillips@google.com

Author: egdaniel@google.com

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

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

10 years agoBugChomper: Get the oauth_client_secret from GS
commit-bot@chromium.org [Tue, 13 May 2014 19:18:55 +0000 (19:18 +0000)]
BugChomper: Get the oauth_client_secret from GS

BUG=skia:
NOTRY=true
(SkipBuildbotRuns)

R=jcgregorio@google.com

Author: borenet@google.com

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

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

10 years agobench rebase after f71be96
commit-bot@chromium.org [Tue, 13 May 2014 19:13:36 +0000 (19:13 +0000)]
bench rebase after f71be96
R=robertphillips@google.com
TBR=robertphillips@google.com
NOTRY=true

Author: borenet@google.com

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

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

10 years agoAdd functions for inspecting SkShader.
commit-bot@chromium.org [Tue, 13 May 2014 18:14:45 +0000 (18:14 +0000)]
Add functions for inspecting SkShader.

Add a function to inspect an SkShader to determine if it is an
SkComposeShader.

Add a virtual function for determining if an SkShader is a custom
shader, which returns a custom set of information. The
implementation is in Android, and this function is only defined
if SK_BUILD_FOR_ANDROID_FRAMEWORK.

BUG=b/10650594
R=reed@google.com, scroggo@google.com

Author: djsollen@google.com

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

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

10 years agoBugChomper utility - rewrite in Go
commit-bot@chromium.org [Tue, 13 May 2014 18:03:45 +0000 (18:03 +0000)]
BugChomper utility - rewrite in Go

BUG=skia:
R=jcgregorio@google.com

Author: borenet@google.com

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

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

10 years agoFixed flaky test
commit-bot@chromium.org [Tue, 13 May 2014 17:47:08 +0000 (17:47 +0000)]
Fixed flaky test

BUG=skia:
R=bensong@google.com

Author: kelvinly@google.com

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

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

10 years agoAdds dashboard link for one-line plot on each alert item.
commit-bot@chromium.org [Tue, 13 May 2014 17:38:17 +0000 (17:38 +0000)]
Adds dashboard link for one-line plot on each alert item.

BUG=skia:2523
NOTRY=true
R=kelvinly@google.com
TBR=rmistry@google.com

Author: bensong@google.com

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

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

10 years agoAdds kelvinly@google.com to CQ_COMMITTERS
commit-bot@chromium.org [Tue, 13 May 2014 17:33:56 +0000 (17:33 +0000)]
Adds kelvinly@google.com to CQ_COMMITTERS

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

Author: bensong@google.com

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

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

10 years agoUpdate TSAN suppressions.
commit-bot@chromium.org [Tue, 13 May 2014 14:47:21 +0000 (14:47 +0000)]
Update TSAN suppressions.

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

Author: mtklein@chromium.org

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

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

10 years agoRemove legacy SkLayerRasterizer flag for Android.
commit-bot@chromium.org [Tue, 13 May 2014 14:47:17 +0000 (14:47 +0000)]
Remove legacy SkLayerRasterizer flag for Android.

Android's uses of SkLayerRasterizer have been cleaned up and no longer require
the legacy API. Remove the flag for Android.
We still have to keep the legacy API around until Mozilla have cleaned up their
uses.

BUG=skia:2187
R=reed@google.com, djsollen@google.com

Author: dominikg@chromium.org

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

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

10 years agoAlways inline Filter_32_*_neon functions
commit-bot@chromium.org [Tue, 13 May 2014 14:47:11 +0000 (14:47 +0000)]
Always inline Filter_32_*_neon functions

The functions are rather performance critical and already marked
'inline'. However, Chrome for Android will not have these functions
inlined due to it being compiled with -Os and a small -finline-limit.

This avoids one call in the filtering functions.

Does not increase the library size.

BUG=chromium:363073
R=mtklein@google.com

Author: kkinnunen@nvidia.com

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

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

10 years agoAdd check for bad dash interval in dash effect picture read
commit-bot@chromium.org [Tue, 13 May 2014 14:45:29 +0000 (14:45 +0000)]
Add check for bad dash interval in dash effect picture read

BUG=skia:2542
R=caryclark@google.com, reed@google.com

Author: egdaniel@google.com

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

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

10 years agobench rebase after e92c68f
commit-bot@chromium.org [Tue, 13 May 2014 14:16:42 +0000 (14:16 +0000)]
bench rebase after e92c68f
R=robertphillips@google.com
TBR=robertphillips@google.com
NOTRY=true

Author: bensong@google.com

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

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

10 years agoFix the android framework build as it still uses mutable local matrices.
djsollen@google.com [Tue, 13 May 2014 13:35:04 +0000 (13:35 +0000)]
Fix the android framework build as it still uses mutable local matrices.

R=reed@google.com

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

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

10 years agoSanitizing source files in Housekeeper-Nightly
skia.committer@gmail.com [Tue, 13 May 2014 03:06:05 +0000 (03:06 +0000)]
Sanitizing source files in Housekeeper-Nightly

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

10 years agoadd SK_SUPPORT_LEGACY_SHADER_LOCALMATRIX for chrome
commit-bot@chromium.org [Tue, 13 May 2014 00:43:44 +0000 (00:43 +0000)]
add SK_SUPPORT_LEGACY_SHADER_LOCALMATRIX for chrome

TBR=borenet@google.com
NOTRY=True

Author: reed@chromium.org

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

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

10 years agocomment-only fix to tools/update-doxygen.sh
commit-bot@chromium.org [Mon, 12 May 2014 20:47:02 +0000 (20:47 +0000)]
comment-only fix to tools/update-doxygen.sh

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

Author: epoger@google.com

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

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

10 years agore-land hide get/setLocalMatrix
commit-bot@chromium.org [Mon, 12 May 2014 20:42:21 +0000 (20:42 +0000)]
re-land hide get/setLocalMatrix

This reverts commit b1d702a43b07934f5b001b1b09db2c57ede909a1.

TBR=scroggo@google.com

Author: reed@google.com

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

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

10 years agorename rebaseline_server JSON constants to be more consistent
commit-bot@chromium.org [Mon, 12 May 2014 20:40:29 +0000 (20:40 +0000)]
rename rebaseline_server JSON constants to be more consistent
(no behavioral change, no change to JSON file format)

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

Author: epoger@google.com

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

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

10 years agobench rebase after 261d015
commit-bot@chromium.org [Mon, 12 May 2014 20:23:34 +0000 (20:23 +0000)]
bench rebase after 261d015
R=robertphillips@google.com
TBR=robertphillips@google.com
NOTRY=true

Author: bensong@google.com

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

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

10 years agoapply localmatrix in pre-order, to match other shaders
reed@google.com [Mon, 12 May 2014 18:12:24 +0000 (18:12 +0000)]
apply localmatrix in pre-order, to match other shaders

BUG=skia:

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

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

10 years agoadd new perlin noise gm to test localmatrix
reed@google.com [Mon, 12 May 2014 17:18:45 +0000 (17:18 +0000)]
add new perlin noise gm to test localmatrix

BUG=skia:

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

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

10 years agoClean up global gSkSuppressFontCachePurgeSpew.
commit-bot@chromium.org [Mon, 12 May 2014 15:48:38 +0000 (15:48 +0000)]
Clean up global gSkSuppressFontCachePurgeSpew.

No one is using it except to flick it to true to shut up the spew,
but the spew is already guarded by #ifdef SPEW_PURGE_STATUS.  Let's
leave it to SPEW_PURGE_STATUS.

Noticed this specificially when TSAN complained about FontScalerBench
modifying it in a thread-unsafe way.

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

Author: mtklein@chromium.org

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

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

10 years agoMake gMask_00FF00FF a constant
commit-bot@chromium.org [Mon, 12 May 2014 15:38:00 +0000 (15:38 +0000)]
Make gMask_00FF00FF a constant

This is to optimize SkAlphaMulQ() in PIC mode. With the visibility=default
symbol the constant is not known at compile time (and is not a constant), but
instead is fetched through a double indirection through GOT. The function is
quite hot on one of the chromium benchmarks:
rasterize_and_record_micro.key_silk_cases.

This change replaces the symbol with a compile-time constant. As a bonus the
variable is not exported from the dynamic library, i. e. a cleaner library
interface.

See specific performance improvements on Android here:
  http://goo.gl/iMuTDt

R=skyostil@chromium.org, tomhudson@chromium.org, mtklein@google.com, reed@google.com, tomhudson@google.com

Author: pasko@chromium.org

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

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

10 years agoadd --readJsonSummaryPath to render_pictures
commit-bot@chromium.org [Mon, 12 May 2014 15:37:20 +0000 (15:37 +0000)]
add --readJsonSummaryPath to render_pictures

BUG=skia:1942
R=borenet@google.com

Author: epoger@google.com

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

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

10 years agoAdds the ability to filter bench expectations in builder/config level.
commit-bot@chromium.org [Mon, 12 May 2014 14:36:43 +0000 (14:36 +0000)]
Adds the ability to filter bench expectations in builder/config level.

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

Author: bensong@google.com

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

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

10 years agoImproved x86 SSE build and run-time checks.
commit-bot@chromium.org [Mon, 12 May 2014 14:16:19 +0000 (14:16 +0000)]
Improved x86 SSE build and run-time checks.

Replaces the current build/run-time checks for SSE level in
opts_check_x86.cpp with a simpler and more future-proof version.
Also adds SSE versions 4.1 and 4.2 to the config file.

Author: henrik.smiding@intel.com

Signed-off-by: Henrik Smiding <henrik.smiding@intel.com>
Committed: http://code.google.com/p/skia/source/detail?r=14644

R=reed@google.com, mtklein@google.com, tomhudson@google.com, djsollen@google.com, joakim.landberg@intel.com

Author: henrik.smiding@intel.com

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

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

10 years agoRemove SK_SUPPORT_LEGACY_PUBLICEFFECTCONSTRUCTORS flag from chromium_defines.
commit-bot@chromium.org [Mon, 12 May 2014 14:12:38 +0000 (14:12 +0000)]
Remove SK_SUPPORT_LEGACY_PUBLICEFFECTCONSTRUCTORS flag from chromium_defines.

Chrome/Blink doesn't use any of the legacy methods behind this flag any more.

BUG=skia:2187

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

R=reed@google.com

Author: dominikg@chromium.org

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

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

10 years agoremove unused Kernel33MaskFilter
commit-bot@chromium.org [Mon, 12 May 2014 13:35:55 +0000 (13:35 +0000)]
remove unused Kernel33MaskFilter

R=robertphillips@google.com
TBR=robertphilips@google.com

Author: reed@google.com

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

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

10 years agobench rebase after 108eddb
commit-bot@chromium.org [Sat, 10 May 2014 12:57:48 +0000 (12:57 +0000)]
bench rebase after 108eddb
R=robertphillips@google.com
TBR=robertphillips@google.com
NOTRY=true

Author: bensong@google.com

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

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

10 years agoSanitizing source files in Housekeeper-Nightly
skia.committer@gmail.com [Sat, 10 May 2014 03:04:40 +0000 (03:04 +0000)]
Sanitizing source files in Housekeeper-Nightly

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

10 years agoadd localmatrix param to asNewEffect
reed@google.com [Fri, 9 May 2014 20:52:23 +0000 (20:52 +0000)]
add localmatrix param to asNewEffect

BUG=skia:

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

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

10 years agoFactor GrTexture into public GrTexture and private GrTextureImpl.
commit-bot@chromium.org [Fri, 9 May 2014 20:46:48 +0000 (20:46 +0000)]
Factor GrTexture into public GrTexture and private GrTextureImpl.

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

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

Author: bsalomon@google.com

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

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

10 years agoadd localmatrix parameter to shader's asNewEffect
commit-bot@chromium.org [Fri, 9 May 2014 20:28:11 +0000 (20:28 +0000)]
add localmatrix parameter to shader's asNewEffect

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

Author: reed@google.com

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

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

10 years agoRevert "remove SK_SUPPORT_LEGACY_BUILDMIPMAP"
reed@google.com [Fri, 9 May 2014 20:06:57 +0000 (20:06 +0000)]
Revert "remove SK_SUPPORT_LEGACY_BUILDMIPMAP"

This reverts commit 9acd04797101990af23f5c374b7dfcbaac12fd2b.

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

10 years agoremove SK_SUPPORT_LEGACY_BUILDMIPMAP
commit-bot@chromium.org [Fri, 9 May 2014 19:54:42 +0000 (19:54 +0000)]
remove SK_SUPPORT_LEGACY_BUILDMIPMAP

TBR=djsollen@google.com
NOTRY=True

Author: reed@google.com

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

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

10 years agoUn-swap underline position and thickness calcualtions in DirectWrite.
bungeman@google.com [Fri, 9 May 2014 19:25:43 +0000 (19:25 +0000)]
Un-swap underline position and thickness calcualtions in DirectWrite.

The change at r13635 got these backwards.

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

10 years agoFlatten resources directory
halcanary@google.com [Fri, 9 May 2014 18:08:24 +0000 (18:08 +0000)]
Flatten resources directory

On android trybots, the files in a subdirectory weren't being
loaded onto the device.  Consequently, the ImageDecoderOptions
test was not testing anything. I had to relax that unit test
to reflect the existing behavior of our libjpeg decoder on
Android.

R=djsollen@google.com

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

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

10 years agoRevert of Factor GrTexture into public GrTexture and private GrTextureImpl. (https...
commit-bot@chromium.org [Fri, 9 May 2014 18:02:51 +0000 (18:02 +0000)]
Revert of Factor GrTexture into public GrTexture and private GrTextureImpl. (https://codereview.chromium.org/275903002/)

Reason for revert:
Breaks chrome build.

Original issue's description:
> Factor GrTexture into public GrTexture and private GrTextureImpl.
>
> Committed: http://code.google.com/p/skia/source/detail?r=14680

R=jvanverth@google.com, robertphillips@google.com
TBR=jvanverth@google.com, robertphillips@google.com
NOTREECHECKS=true
NOTRY=true

Author: bsalomon@google.com

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

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

10 years agoFactor GrTexture into public GrTexture and private GrTextureImpl.
commit-bot@chromium.org [Fri, 9 May 2014 17:37:55 +0000 (17:37 +0000)]
Factor GrTexture into public GrTexture and private GrTextureImpl.

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

Author: bsalomon@google.com

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

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

10 years agoRevert "Flatten resources directory"
djsollen@google.com [Fri, 9 May 2014 17:24:31 +0000 (17:24 +0000)]
Revert "Flatten resources directory"

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

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

10 years agoFlatten resources directory
commit-bot@chromium.org [Fri, 9 May 2014 16:24:06 +0000 (16:24 +0000)]
Flatten resources directory

On android trybots, the files in a subdirectory weren't being
loaded onto the device.  Consequently, the ImageDecoderOptions
test was not testing anything. I had to relax that unit test
to reflect the existing behavior of our libjpeg decoder on
Android.

R=djsollen@google.com

Author: halcanary@google.com

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

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

10 years agoRevert of hide get/setLocalMatrix (https://codereview.chromium.org/279563002/)
commit-bot@chromium.org [Fri, 9 May 2014 15:42:07 +0000 (15:42 +0000)]
Revert of hide get/setLocalMatrix (https://codereview.chromium.org/279563002/)

Reason for revert:
broke gms

Original issue's description:
> hide get/setLocalMatrix
>
> BUG=skia:
>
> Committed: http://code.google.com/p/skia/source/detail?r=14675

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

Author: reed@google.com

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

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

10 years agoAllow for no pdf rasterizers in gm.
bungeman@google.com [Fri, 9 May 2014 15:22:41 +0000 (15:22 +0000)]
Allow for no pdf rasterizers in gm.

Our poppler does not build properly with VS2013, but using
gyp_skia -Dskia_enable_poppler_enabled=0
also does not build due to kPDFRasterizers being statically empty.
This adds a do nothing, not run by default 'pdf rasterizer'.

R=epoger@google.com

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

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

10 years agohide get/setLocalMatrix
commit-bot@chromium.org [Fri, 9 May 2014 15:19:48 +0000 (15:19 +0000)]
hide get/setLocalMatrix

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

Author: reed@google.com

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

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

10 years agoAdd a dry-run option to gyp_skia.
bungeman@google.com [Fri, 9 May 2014 15:04:18 +0000 (15:04 +0000)]
Add a dry-run option to gyp_skia.

There are times when I want to do some low-level fiddling with the gyp
command. It's nice to start with whatever the 'default' is, and it's also
nice to peek behind the curtain to see what's going on. To this end,
a '--dry-run' in the argument list will spit out the command to run,
rather than running the command.

R=epoger@google.com

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

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

10 years agoUse a tilegrid for bench_playback.
commit-bot@chromium.org [Fri, 9 May 2014 15:00:10 +0000 (15:00 +0000)]
Use a tilegrid for bench_playback.

Using --tile wasn't a fair comparison: SkPicture didn't get to optimize its
playback, but SkRecord did.  Numbers here are starting to agree with what I saw
in telemetry: SkRecord with culling is on average at 0.8-0.9 of the speed of
SkPicture (10-30% slower) than SkPicture SkPicture with a tile grid, but some
in some cases it's down at 0.3, taking almost 4x as long to draw a 256x256 tile.

0.4x tabl_mercurynews.skp 0.109889 0.252483
0.5x tabl_cnet.skp 0.250367 0.527789
1.0x desk_mapsvg.skp 1.48698 1.49939
1.2x desk_yahoogames.skp 0.07745 0.065343
0.9x tabl_mlb.skp 0.294509 0.312505
0.8x tabl_hsfi.skp 0.250678 0.316337
0.6x tabl_worldjournal.skp 0.510204 0.828798
0.7x desk_facebook.skp 0.318981 0.483744
0.9x desk_googlehome.skp 0.06983 0.078964
0.9x desk_tigersvg.skp 0.019232 0.022611
1.0x desk_youtubetvvideo.skp 0.401614 0.401862
1.1x desk_amazon.skp 0.415552 0.362111
0.7x desk_yahooanswers.skp 0.237995 0.358659
0.7x desk_twitter.skp 0.323466 0.439772
1.2x desk_fontwipe.skp 0.121577 0.097845
1.1x desk_css3gradients.skp 0.395335 0.362087
0.9x desk_googlespreadsheet.skp 0.362056 0.406688
1.0x tabl_frantzen.skp 15.2475 14.8521
0.6x desk_forecastio.skp 0.154236 0.276881
0.7x desk_pinterest.skp 0.077433 0.11324
0.5x tabl_cuteoverload.skp 0.173014 0.361815
0.5x tabl_androidpolice.skp 0.225903 0.465608
0.6x tabl_vnexpress.skp 0.151075 0.241104
0.6x tabl_nytimes.skp 0.06515 0.117229
0.8x desk_ebay.skp 0.353437 0.422586
0.8x tabl_deviantart.skp 0.223103 0.295496
1.1x tabl_culturalsolutions.skp 0.551348 0.506104
0.9x mobi_wikipedia.skp 0.302603 0.337292
0.6x desk_linkedin.skp 0.165102 0.276149
0.6x tabl_digg.skp 0.159074 0.281529
1.2x desk_oldinboxapp.skp 0.019815 0.017146
1.0x tabl_nofolo.skp 5.18914 5.42784
0.6x desk_jsfiddlehumperclip.skp 0.076869 0.133665
1.0x desk_blogger.skp 0.206702 0.19688
0.8x desk_espn.skp 0.346386 0.451871
1.0x desk_samoasvg.skp 3.60037 3.43147
0.8x desk_booking.skp 0.338818 0.438267
1.3x desk_yahoosports.skp 0.603811 0.470052
0.6x tabl_engadget.skp 0.248995 0.404372
0.7x desk_wowwiki.skp 0.325801 0.479559
0.8x tabl_ukwsj.skp 0.326985 0.405971
0.9x tabl_gmail.skp 0.2604 0.276744
0.9x desk_googleplus.skp 8.08874 8.71105
1.1x tabl_slashdot.skp 0.559332 0.524313
0.9x desk_gmailthread.skp 0.671569 0.74134
0.8x desk_weather.skp 0.194162 0.247681
1.3x desk_rectangletransition.skp 0.218372 0.173685
0.6x tabl_gamedeksiam.skp 0.243374 0.4134
0.9x desk_jsfiddlebigcar.skp 0.162282 0.186035
0.3x desk_pokemonwiki.skp 0.202878 0.778934
0.7x tabl_googleblog.skp 0.141203 0.209971
0.8x desk_sfgate.skp 0.229078 0.275248
1.0x desk_chalkboard.skp 2.28916 2.20101
0.9x tabl_googlecalendar.skp 0.687155 0.75413
1.1x tabl_gspro.skp 0.186399 0.169522
0.5x desk_googlespreadsheetdashed.skp 0.512622 1.02815
0.6x desk_mobilenews.skp 0.183506 0.293929
0.8x tabl_techmeme.skp 0.193074 0.229106
1.2x desk_carsvg.skp 5.33047 4.34058
0.5x desk_youtube.skp 0.132663 0.248236
0.7x tabl_pravda.skp 0.21644 0.308788
1.1x desk_gws.skp 0.698107 0.625371
0.5x tabl_mozilla.skp 0.189638 0.386801
1.0x desk_baidu.skp 0.353438 0.355965
1.1x tabl_sahadan.skp 0.476226 0.416737
summary: â‰¥0.3x (A 0.9x, G 0.8x) â‰¤1.3x

Whole SKP playback speed is fine of course.

BUG=skia:2378
R=fmalita@chromium.org, mtklein@google.com

Author: mtklein@chromium.org

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

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

10 years agoAdd timing to dump_record.
commit-bot@chromium.org [Fri, 9 May 2014 14:59:29 +0000 (14:59 +0000)]
Add timing to dump_record.

Prints microsecond timing for each command in the left-hand column:

optimized flat/http___mobile_news_sandbox_google_com_news_pt0_scroll_layer_7.skp
   4.0   1 Save
2075.0   2      DrawRect
 104.0   3      BoundedDrawPosTextH
 135.4   4      DrawRect
   9.4   5      DrawRect
   5.6   6      DrawRect
   8.2   7      DrawRect
   6.8   8      DrawRect
...

(I'm sure Rietveld will just mangle the crap out of that.  It's helpfully right-aligned.)

To do this, I made Draw from SkRecordDraw Skia-public as SkRecords::Draw,
and time it command-by-command.

BUG=skia:2378
R=fmalita@chromium.org, mtklein@google.com

Author: mtklein@chromium.org

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

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

10 years agoupdate canvas-layer-state (which should be deleted soon)
commit-bot@chromium.org [Fri, 9 May 2014 14:39:26 +0000 (14:39 +0000)]
update canvas-layer-state (which should be deleted soon)

R=reed@android.com
TBR=reed
NOTRY=true

Author: caryclark@google.com

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

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

10 years agoQuadTree: don't leak deferred inserts past clear()
commit-bot@chromium.org [Fri, 9 May 2014 14:33:55 +0000 (14:33 +0000)]
QuadTree: don't leak deferred inserts past clear()

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

Author: mtklein@chromium.org

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

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

10 years agocleanup GrContext resource cache api
commit-bot@chromium.org [Fri, 9 May 2014 14:29:32 +0000 (14:29 +0000)]
cleanup GrContext resource cache api

R=robertphillips@google.com

Author: bsalomon@google.com

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

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

10 years agocommit GM actuals as expectations for all tests, marking them as unreviewed
commit-bot@chromium.org [Fri, 9 May 2014 14:16:11 +0000 (14:16 +0000)]
commit GM actuals as expectations for all tests, marking them as unreviewed

BUG=skia:1578
NOTRY=True
R=caryclark@google.com
TBR=caryclark

Author: epoger@google.com

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

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

10 years agoFix typo in SkFontHost_win.cpp in comment.
bungeman@google.com [Fri, 9 May 2014 14:11:19 +0000 (14:11 +0000)]
Fix typo in SkFontHost_win.cpp in comment.

We want the 'size', not the 'side'.

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

10 years agoAdd missing sfnt header files to gyp.
bungeman@google.com [Fri, 9 May 2014 14:08:28 +0000 (14:08 +0000)]
Add missing sfnt header files to gyp.

This makes finding these files easier in IDEs.

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

10 years agoDisable filtering in draw Bitmap operation when pixels are aligned
commit-bot@chromium.org [Fri, 9 May 2014 13:53:38 +0000 (13:53 +0000)]
Disable filtering in draw Bitmap operation when pixels are aligned

The check is now performed in the callers of internalDrawBitmap() to
avoid retrieve the texture with wrong filter mode when the pixels were
aligned.

This bug was spotted in Chrome for Android: Chrome for Androids's
Canvas drawImage loses fidelity in canvases larger than 64k pixels.

BUG=chromium:231916

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

Signed-off-by: Samuel Iglesias Gonsalvez <siglesias@igalia.com>
Author: siglesias@igalia.com

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

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

10 years agoupdate Test-Linux-Bare-NoGPU-Arm8_64-Debug expectations
commit-bot@chromium.org [Fri, 9 May 2014 12:00:19 +0000 (12:00 +0000)]
update Test-Linux-Bare-NoGPU-Arm8_64-Debug expectations

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

Author: caryclark@google.com

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

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

10 years agoPoint to skiadocs in our README.
tfarina@chromium.org [Fri, 9 May 2014 04:30:09 +0000 (04:30 +0000)]
Point to skiadocs in our README.

This seems to be the canonical location for all skia documentation
nowadays.

R=epoger@google.com

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

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

10 years agoreland "extract some common code from PictureRenderer"
commit-bot@chromium.org [Fri, 9 May 2014 03:18:41 +0000 (03:18 +0000)]
reland "extract some common code from PictureRenderer"

re-lands https://codereview.chromium.org/273703006/

NOTRY=True
R=bungeman@google.com, djsollen@google.com

Author: epoger@google.com

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

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

10 years agoSanitizing source files in Housekeeper-Nightly
skia.committer@gmail.com [Fri, 9 May 2014 03:05:30 +0000 (03:05 +0000)]
Sanitizing source files in Housekeeper-Nightly

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

10 years agoRevert of Remove SK_SUPPORT_LEGACY_PUBLICEFFECTCONSTRUCTORS flag from chromium_define...
commit-bot@chromium.org [Fri, 9 May 2014 00:05:52 +0000 (00:05 +0000)]
Revert of Remove SK_SUPPORT_LEGACY_PUBLICEFFECTCONSTRUCTORS flag from chromium_defines. (https://codereview.chromium.org/265973002/)

Reason for revert:
This is breaking compilation of cc_unittests in Chrome

Original issue's description:
> Remove SK_SUPPORT_LEGACY_PUBLICEFFECTCONSTRUCTORS flag from chromium_defines.
>
> Chrome/Blink doesn't use any of the legacy methods behind this flag any more.
>
> BUG=skia:2187
>
> Committed: http://code.google.com/p/skia/source/detail?r=14651

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

Author: robertphillips@google.com

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

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

10 years agoFix rendering artifacts in pull-saveLayers-forward mode
commit-bot@chromium.org [Thu, 8 May 2014 23:24:05 +0000 (23:24 +0000)]
Fix rendering artifacts in pull-saveLayers-forward mode

This CL fixes 2 bugs in the pre-rendering of saveLayers:

1) The drawBitmapRect call wasn't occurring in device space so could sometimes be double transformed

2) The BBH op skipping in SkPicturePlayback could sometimes mess up the SkPictureStateTree's state

It also reduces the number of layers that are pre-rendered when a BBH is not in use.

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

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

Author: robertphillips@google.com

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

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

10 years agorebaseline_server JSON: pass category values as values, not keys
commit-bot@chromium.org [Thu, 8 May 2014 21:15:20 +0000 (21:15 +0000)]
rebaseline_server JSON: pass category values as values, not keys

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

Author: epoger@google.com

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

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

10 years agoRemove non-existent header file from Android opts.
bungeman@google.com [Thu, 8 May 2014 19:41:48 +0000 (19:41 +0000)]
Remove non-existent header file from Android opts.

R=djsollen@google.com

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

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

10 years agoreturn null if subshader fails
commit-bot@chromium.org [Thu, 8 May 2014 19:22:52 +0000 (19:22 +0000)]
return null if subshader fails

TBR=scroggo@google.com

Author: reed@google.com

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

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

10 years agoSplit Star into Star (stores nothing) and List (stores matches).
commit-bot@chromium.org [Thu, 8 May 2014 18:58:32 +0000 (18:58 +0000)]
Split Star into Star (stores nothing) and List (stores matches).

Just noticed that the cull noop pattern (PushCull, Star<NoOp>, PopCull)
would trigger the Star that stores matches.  We certainly don't need
those matches here, so instead of magically determining which Star you
need, we'll make you tell us which one you want.

No one but List's unit test needs List.  I'll leave it for now, but we
might find it's not useful.

BUG=skia:2378
R=fmalita@chromium.org, mtklein@google.com, bungeman@google.com

Author: mtklein@chromium.org

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

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

10 years agoRevert r14650 (Fix rendering artifacts in pull-saveLayers-forward mode - https:/...
robertphillips@google.com [Thu, 8 May 2014 18:45:20 +0000 (18:45 +0000)]
Revert r14650 (Fix rendering artifacts in pull-saveLayers-forward mode - https://codereview.chromium.org/267293007) due to unit test failures

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

10 years agoNoop away PushCull/PopCull pairs with nothing between them.
commit-bot@chromium.org [Thu, 8 May 2014 18:17:51 +0000 (18:17 +0000)]
Noop away PushCull/PopCull pairs with nothing between them.

BUG=skia:2378
R=fmalita@chromium.org, mtklein@google.com

Author: mtklein@chromium.org

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

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

10 years agostore map ptr in GrGeometryBuffer base class
commit-bot@chromium.org [Thu, 8 May 2014 18:13:23 +0000 (18:13 +0000)]
store map ptr in GrGeometryBuffer base class

R=robertphillips@google.com

Author: bsalomon@google.com

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

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

10 years agoRemove SK_SUPPORT_LEGACY_PUBLICEFFECTCONSTRUCTORS flag from chromium_defines.
commit-bot@chromium.org [Thu, 8 May 2014 18:13:15 +0000 (18:13 +0000)]
Remove SK_SUPPORT_LEGACY_PUBLICEFFECTCONSTRUCTORS flag from chromium_defines.

Chrome/Blink doesn't use any of the legacy methods behind this flag any more.

BUG=skia:2187
R=reed@google.com

Author: dominikg@chromium.org

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

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

10 years agoFix rendering artifacts in pull-saveLayers-forward mode
commit-bot@chromium.org [Thu, 8 May 2014 18:13:07 +0000 (18:13 +0000)]
Fix rendering artifacts in pull-saveLayers-forward mode

This CL fixes 2 bugs in the pre-rendering of saveLayers:

1) The drawBitmapRect call wasn't occurring in device space so could sometimes be double transformed

2) The BBH op skipping in SkPicturePlayback could sometimes mess up the SkPictureStateTree's state

It also reduces the number of layers that are pre-rendered when a BBH is not in use.

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

Author: robertphillips@google.com

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

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

10 years agoRevert of extract some common code from PictureRenderer (https://codereview.chromium...
commit-bot@chromium.org [Thu, 8 May 2014 16:23:56 +0000 (16:23 +0000)]
Revert of extract some common code from PictureRenderer (https://codereview.chromium.org/273703006/)

Reason for revert:
breaking the android debug builds.  It seems to be the result of android using shared libs and not lumping everything into one big static lib or executable. So we have more than one copy of SkString which is the root of the issue.

Original issue's description:
> extract some common code from PictureRenderer
>
> Committed: http://code.google.com/p/skia/source/detail?r=14638

R=borenet@google.com, epoger@google.com
TBR=borenet@google.com, epoger@google.com
NOTREECHECKS=true
NOTRY=true

Author: djsollen@google.com

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

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

10 years agorebaseline_server: add link to other result sets from main UI
commit-bot@chromium.org [Thu, 8 May 2014 15:40:06 +0000 (15:40 +0000)]
rebaseline_server: add link to other result sets from main UI

BUG=skia:2529
NOTREECHECKS=True
NOTRY=True
(SkipBuildbotRuns)

R=caryclark@google.com

Author: epoger@google.com

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

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

10 years agoFix Moz2D build
commit-bot@chromium.org [Thu, 8 May 2014 15:29:57 +0000 (15:29 +0000)]
Fix Moz2D build

BUG=skia:2221
R=george@mozilla.com, reed@google.com

Author: borenet@google.com

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

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

10 years agoRevert of Improved x86 SSE build and run-time checks. (https://codereview.chromium...
commit-bot@chromium.org [Thu, 8 May 2014 15:27:52 +0000 (15:27 +0000)]
Revert of Improved x86 SSE build and run-time checks. (https://codereview.chromium.org/272503006/)

Reason for revert:
Windows builders breaking.  :(

Original issue's description:
> Improved x86 SSE build and run-time checks.
>
> Replaces the current build/run-time checks for SSE level in
> opts_check_x86.cpp with a simpler and more future-proof version.
> Also adds SSE versions 4.1 and 4.2 to the config file.
>
> Author: henrik.smiding@intel.com
>
> Signed-off-by: Henrik Smiding <henrik.smiding@intel.com>
>
> Committed: http://code.google.com/p/skia/source/detail?r=14644

R=reed@google.com, tomhudson@google.com, djsollen@google.com, joakim.landberg@intel.com, henrik.smiding@intel.com
TBR=djsollen@google.com, henrik.smiding@intel.com, joakim.landberg@intel.com, reed@google.com, tomhudson@google.com
NOTREECHECKS=true
NOTRY=true

Author: mtklein@google.com

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

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

10 years agoGreenify bench bot Perf-Android-Nexus7-Tegra3-Arm7-Release at build 2816
commit-bot@chromium.org [Thu, 8 May 2014 15:27:02 +0000 (15:27 +0000)]
Greenify bench bot Perf-Android-Nexus7-Tegra3-Arm7-Release at build 2816
R=bsalomon@google.com
TBR=bsalomon@google.com
NOTRY=true

Author: djsollen@google.com

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

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

10 years agoImproved x86 SSE build and run-time checks.
commit-bot@chromium.org [Thu, 8 May 2014 15:17:50 +0000 (15:17 +0000)]
Improved x86 SSE build and run-time checks.

Replaces the current build/run-time checks for SSE level in
opts_check_x86.cpp with a simpler and more future-proof version.
Also adds SSE versions 4.1 and 4.2 to the config file.

Author: henrik.smiding@intel.com

Signed-off-by: Henrik Smiding <henrik.smiding@intel.com>
R=reed@google.com, mtklein@google.com, tomhudson@google.com, djsollen@google.com, joakim.landberg@intel.com

Author: henrik.smiding@intel.com

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

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

10 years agoGreenify bench bot Perf-Android-Nexus4-Adreno320-Arm7-Release at build 1643
commit-bot@chromium.org [Thu, 8 May 2014 15:14:57 +0000 (15:14 +0000)]
Greenify bench bot Perf-Android-Nexus4-Adreno320-Arm7-Release at build 1643
R=bsalomon@google.com
TBR=bsalomon@google.com
NOTRY=true

Author: djsollen@google.com

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

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

10 years agoFix some references to Makefiles in gyp_skia.
tfarina@chromium.org [Thu, 8 May 2014 15:13:52 +0000 (15:13 +0000)]
Fix some references to Makefiles in gyp_skia.

BUG=None
TEST=None, no functional changes. Just updating documentation.
R=epoger@google.com

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

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

10 years agoRemove global SkRandoms from ChartBench.
commit-bot@chromium.org [Thu, 8 May 2014 15:12:43 +0000 (15:12 +0000)]
Remove global SkRandoms from ChartBench.

TSAN picked them up while running DM.

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

Author: mtklein@chromium.org

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

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

10 years agoGreenify bench bot Perf-Android-Nexus7-Tegra3-Arm7-Release at build 2818
commit-bot@chromium.org [Thu, 8 May 2014 15:00:24 +0000 (15:00 +0000)]
Greenify bench bot Perf-Android-Nexus7-Tegra3-Arm7-Release at build 2818
R=bsalomon@google.com, bensong@google.com
TBR=bsalomon@google.com
NOTRY=true

Author: bsalomon@google.com

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

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

10 years agoGreenify bench bot Perf-Android-Nexus4-Adreno320-Arm7-Release at build 1644
commit-bot@chromium.org [Thu, 8 May 2014 14:57:58 +0000 (14:57 +0000)]
Greenify bench bot Perf-Android-Nexus4-Adreno320-Arm7-Release at build 1644
R=bsalomon@google.com, bensong@google.com
TBR=bsalomon@google.com
NOTRY=true

Author: bsalomon@google.com

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

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

10 years agoextract some common code from PictureRenderer
commit-bot@chromium.org [Thu, 8 May 2014 14:48:44 +0000 (14:48 +0000)]
extract some common code from PictureRenderer

R=borenet@google.com

Author: epoger@google.com

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

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

10 years agoGreenify bench bot Perf-Win7-ShuttleA-HD2000-x86-Release at build 2207
commit-bot@chromium.org [Thu, 8 May 2014 04:57:58 +0000 (04:57 +0000)]
Greenify bench bot Perf-Win7-ShuttleA-HD2000-x86-Release at build 2207
R=bsalomon@google.com
TBR=bsalomon@google.com
NOTRY=true

Author: bensong@google.com

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

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

10 years agoSanitizing source files in Housekeeper-Nightly
skia.committer@gmail.com [Thu, 8 May 2014 03:05:29 +0000 (03:05 +0000)]
Sanitizing source files in Housekeeper-Nightly

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

10 years agobench rebase after 37f6e62
commit-bot@chromium.org [Thu, 8 May 2014 02:25:44 +0000 (02:25 +0000)]
bench rebase after 37f6e62
R=robertphillips@google.com
TBR=robertphillips@google.com
NOTRY=true

Author: bensong@google.com

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

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

10 years agoMove paints to the front of draw structs.
commit-bot@chromium.org [Wed, 7 May 2014 22:59:38 +0000 (22:59 +0000)]
Move paints to the front of draw structs.

The order of arguments in these structs is arbitrary, so we might as well arrange them to optimize something.  Putting the paints at the front means the logic to find the paint is a lot more concise: it's usually just ptr+0, or *(ptr+0) when the SkPaint is optional.

This considerably reduces the size of the jump table in IsDraw::operator().

BUG=skia:2378
R=fmalita@chromium.org, mtklein@google.com

Author: mtklein@chromium.org

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

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

10 years agoadd localmatrix-shader
commit-bot@chromium.org [Wed, 7 May 2014 22:26:37 +0000 (22:26 +0000)]
add localmatrix-shader

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

Author: reed@google.com

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

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

10 years agoFirst pass at pre-rendering saveLayers for GPU
robertphillips@google.com [Wed, 7 May 2014 21:31:09 +0000 (21:31 +0000)]
First pass at pre-rendering saveLayers for GPU

https://codereview.chromium.org/261663003/

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

10 years agoConvert all SkRecordPattern matchers into SkRecord mutators.
commit-bot@chromium.org [Wed, 7 May 2014 21:16:09 +0000 (21:16 +0000)]
Convert all SkRecordPattern matchers into SkRecord mutators.

  - Allow any return type from SkRecord mutators and visitors;
  - update existing calls to mutate and visit;
  - convert match to operator() in SkRecordPattern;
  - tidy up a few inelegant bits of old code in tests.

The net result is that the generated code is much clearer.  All the mutate() calls
inline as you'd hope, and you can now actually follow along with the disassembly.

BUG=skia:2378
R=fmalita@chromium.org, bungeman@google.com, mtklein@google.com

Author: mtklein@chromium.org

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

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

10 years agoSome small cleanups for image filter code.
senorblanco@chromium.org [Wed, 7 May 2014 20:56:08 +0000 (20:56 +0000)]
Some small cleanups for image filter code.

Use the 2-param flavour of mapVector instead of the 3-param, where possible.
Add an SkMatrixImageFilter test case to the tiling unit test.

R=junov@chromium.org

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

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

10 years agoremove SK_SUPPORT_LEGACY_BLURMASKFILTER_STYLE
reed@google.com [Wed, 7 May 2014 20:54:58 +0000 (20:54 +0000)]
remove SK_SUPPORT_LEGACY_BLURMASKFILTER_STYLE

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

10 years agoRename from "(un)lock" to "(un)map" for geometry buffers.
commit-bot@chromium.org [Wed, 7 May 2014 20:51:05 +0000 (20:51 +0000)]
Rename from "(un)lock" to "(un)map" for geometry buffers.

This better reflects OpenGL terminology and is less overloaded ("lock" is used w.r.t. the resource cache).

R=robertphillips@google.com

Author: bsalomon@google.com

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

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

10 years agoFix for SkDisplacementMap::onFilterBounds().
senorblanco@chromium.org [Wed, 7 May 2014 20:00:04 +0000 (20:00 +0000)]
Fix for SkDisplacementMap::onFilterBounds().

Two problems: we were not applying the CTM to the scale parameter when
modifying clip bounds, and the recursion for onFilterBounds() must be done
in the reverse order.

BUG=370914
R=junov@chromium.org

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

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

10 years agoStatically initialize those zero-size singletons.
commit-bot@chromium.org [Wed, 7 May 2014 19:56:27 +0000 (19:56 +0000)]
Statically initialize those zero-size singletons.

This way GCC/Clang don't generate the magic static code to call the
pointless T::T() once in a threadsafe way.  = {} is plenty initialized.

BUG=skia:2378

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

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

Author: mtklein@chromium.org

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

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