platform/upstream/libSkiaSharp.git
10 years agocreate struct to hold all the params passed around for shader::context
commit-bot@chromium.org [Thu, 1 May 2014 19:31:31 +0000 (19:31 +0000)]
create struct to hold all the params passed around for shader::context

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

Author: reed@google.com

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

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

10 years agoAdd more detailed logging when things go wrong in android_run_skia
commit-bot@chromium.org [Thu, 1 May 2014 19:30:35 +0000 (19:30 +0000)]
Add more detailed logging when things go wrong in android_run_skia

R=borenet@google.com, halcanary@google.com

Author: djsollen@google.com

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

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

10 years agotweaks to tab CSS for rebaseline server page; moar pretty
commit-bot@chromium.org [Thu, 1 May 2014 19:29:53 +0000 (19:29 +0000)]
tweaks to tab CSS for rebaseline server page; moar pretty

BUG=skia:
R=epoger@google.com

Author: humper@google.com

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

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

10 years agogyp rules for ubsan
commit-bot@chromium.org [Thu, 1 May 2014 19:29:46 +0000 (19:29 +0000)]
gyp rules for ubsan

BUG=skia:2481
R=bungeman@google.com, mtklein@google.com

Author: mtklein@chromium.org

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

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

10 years agorebaseline simpleblurroundrect
commit-bot@chromium.org [Thu, 1 May 2014 19:22:41 +0000 (19:22 +0000)]
rebaseline simpleblurroundrect

BUG=skia:
TBR=epoger

Author: humper@google.com

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

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

10 years agoRoll ANGLE.
commit-bot@chromium.org [Thu, 1 May 2014 19:22:22 +0000 (19:22 +0000)]
Roll ANGLE.

BUG=skia:2272

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

R=bsalomon@chromium.org, bsalomon@google.com

Author: geofflang@chromium.org

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

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

10 years agoRemove SkDEBUGFAILs that are problematic for Chromium
commit-bot@chromium.org [Thu, 1 May 2014 19:21:41 +0000 (19:21 +0000)]
Remove SkDEBUGFAILs that are problematic for Chromium

BUG=368107
R=egdaniel@google.com

Author: bsalomon@google.com

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

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

10 years agoMove to a theme that more closely matches what was there before.
commit-bot@chromium.org [Thu, 1 May 2014 19:21:33 +0000 (19:21 +0000)]
Move to a theme that more closely matches what was there before.

Also, since it's a variation of "default" we can drop a whole file.

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

Author: jcgregorio@google.com

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

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

10 years agoDM: Push GPU-parent child tasks to the front of the queue.
commit-bot@chromium.org [Thu, 1 May 2014 17:41:32 +0000 (17:41 +0000)]
DM: Push GPU-parent child tasks to the front of the queue.

Like yesterday's change to run CPU-parent child tasks serially in thread, this
reduces peak memory usage by improving the temporaly locality of the bitmaps we
create.

E.g. Let's say we start with tasks A B C and D
    Queue: [ A B C D ]
Running A creates A' and A", which depend on a bitmap created by A.
    Queue: [ B C D A' A" * ]
That bitmap now needs sit around in RAM while B C and D run pointlessly and can
only be destroyed at *.  If instead we do this and push dependent child tasks
to the front of the queue, the queue and bitmap lifetime looks like this:
    Queue: [ A' A" * B C D ]

This is much, much worse in practice because the queue is often several thousand
tasks long.  100s of megs of bitmaps can pile up for 10s of seconds pointlessly.

To make this work we add addNext() to SkThreadPool and its cousin DMTaskRunner.
I also took the opportunity to swap head and tail in the threadpool
implementation so it matches the comments and intuition better: we always pop
the head, add() puts it at the tail, addNext() at the head.

Before
  Debug:   49s, 1403352k peak
  Release: 16s, 2064008k peak

After
  Debug:   49s, 1234788k peak
  Release: 15s, 1903424k peak

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

Author: mtklein@chromium.org

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

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

10 years agoWhitespace change to test the Skia CQ
commit-bot@chromium.org [Thu, 1 May 2014 16:30:30 +0000 (16:30 +0000)]
Whitespace change to test the Skia CQ

(SkipBuildbotRuns)

BUG=skia:
R=fmalita@chromium.org

Author: rmistry@google.com

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

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

10 years agoremove code behind SK_SUPPORT_LEGACY_PROCXFERMODE
reed@google.com [Thu, 1 May 2014 15:56:52 +0000 (15:56 +0000)]
remove code behind SK_SUPPORT_LEGACY_PROCXFERMODE

BUG=skia:
R=scroggo@google.com

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

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

10 years agoImprove Android gdbserver and gdb_ex scripts
commit-bot@chromium.org [Thu, 1 May 2014 15:34:28 +0000 (15:34 +0000)]
Improve Android gdbserver and gdb_ex scripts
- Don't complain if $GDB_TMP_DIR already exists
- Since $GDB_TMP_DIR/gdb.setup might already exist, overwrite the file.

NOTRY=true
R=djsollen@google.com

Author: halcanary@google.com

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

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

10 years agoreduce total number of segments in giant-dash
commit-bot@chromium.org [Thu, 1 May 2014 15:30:52 +0000 (15:30 +0000)]
reduce total number of segments in giant-dash

still exercises large coordinates, but won't kill our memory usage on 32bit bots

NOTRY=True
R=caryclark@google.com

Author: reed@google.com

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

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

10 years agoexpose ConvertRadiusToSigma to aid clients in the API transition to sigma
commit-bot@chromium.org [Thu, 1 May 2014 15:24:55 +0000 (15:24 +0000)]
expose ConvertRadiusToSigma to aid clients in the API transition to sigma

NOTRY=True
R=robertphillips@google.com, fmalita@chromium.org

Author: reed@google.com

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

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

10 years agoRevert of re-enable simpleblurroundrect gm (https://codereview.chromium.org/263823002/)
commit-bot@chromium.org [Thu, 1 May 2014 15:18:06 +0000 (15:18 +0000)]
Revert of re-enable simpleblurroundrect gm (https://codereview.chromium.org/263823002/)

Reason for revert:
broken rebaseline.

Original issue's description:
> re-enable simpleblurroundrect gm
>
> BUG=skia:
> TBR=caryclark
>
> Committed: http://code.google.com/p/skia/source/detail?r=14498

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

Author: humper@google.com

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

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

10 years agoUse CodeMirror for WebTry snippets.
fmalita@google.com [Thu, 1 May 2014 15:14:56 +0000 (15:14 +0000)]
Use CodeMirror for WebTry snippets.

CodeMirror (http://codemirror.net) is a rich code editor. This CL
adds the needed library and wires it with some minimal features, to
update the editor appearance to something like this:
http://codemirror.net/demo/theme.html?ambiance

We can later add Skia-customized features (custom keyword highlighting,
code completion, etc.).

(only tested locally)

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

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

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

10 years agore-enable simpleblurroundrect gm
commit-bot@chromium.org [Thu, 1 May 2014 14:38:45 +0000 (14:38 +0000)]
re-enable simpleblurroundrect gm

BUG=skia:
R=caryclark@google.com
TBR=caryclark

Author: humper@google.com

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

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

10 years agoAdd a fix for a spurious assert in SkMatrixConvolutionImageFilter.
senorblanco@chromium.org [Thu, 1 May 2014 14:03:41 +0000 (14:03 +0000)]
Add a fix for a spurious assert in SkMatrixConvolutionImageFilter.

When matrix convolution processes border pixels with zero width, it
asserts in getAddr32() with an invalid x coordinate. The assert is
harmless, since the returned pointer is never accessed (the next line
is a loop from left to right, which does nothing, since left == right).
However, the fix is simple: early out on an empty rect before entering
the outer loop.

R=sugoi@chromium.org

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

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

10 years agoAdd fmalita@chromium.org to CQ_COMMITTERS.
fmalita@google.com [Thu, 1 May 2014 13:32:21 +0000 (13:32 +0000)]
Add fmalita@chromium.org to CQ_COMMITTERS.

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

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

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

10 years agoSupport unknown Android target devices via environment variables.
commit-bot@chromium.org [Thu, 1 May 2014 12:56:19 +0000 (12:56 +0000)]
Support unknown Android target devices via environment variables.

This is useful for working with prototypes and uncommon devices.

NOTRY=true
R=djsollen@google.com, scroggo@google.com

Author: halcanary@google.com

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

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

10 years agoDisplay the color at a point.
commit-bot@chromium.org [Thu, 1 May 2014 12:54:25 +0000 (12:54 +0000)]
Display the color at a point.

BUG=skia:
R=mtklein@google.com

Author: jcgregorio@google.com

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

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

10 years agoRename expectations for Valgrind bot: ATI5770->GTX550Ti
commit-bot@chromium.org [Thu, 1 May 2014 12:32:39 +0000 (12:32 +0000)]
Rename expectations for Valgrind bot: ATI5770->GTX550Ti

This will likely cause GM failures, since the GPU has changed.

BUG=skia:2506
R=caryclark@google.com

Author: borenet@google.com

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

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

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

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

10 years agoadd default impl for context methods on shader
commit-bot@chromium.org [Wed, 30 Apr 2014 23:29:02 +0000 (23:29 +0000)]
add default impl for context methods on shader

These are reasonable return values, since both of these methods can return a known value (0)
which means that no context can be created. This also makes it easier for chrome's subclasses
which already do not want to create a context, but having them actually overridden makes
changing the virtual signatures much harder.

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

Author: reed@google.com

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

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

10 years agoRebaseline tileimagefilter on IntelRhb post-r14432.
senorblanco@chromium.org [Wed, 30 Apr 2014 21:46:38 +0000 (21:46 +0000)]
Rebaseline tileimagefilter on IntelRhb post-r14432.

TBR=caryclark@google.com
BUG=skia:

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

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

10 years agoremove unneeded SK_SUPPORT_LEGACY_PROCXFERMODE
commit-bot@chromium.org [Wed, 30 Apr 2014 21:34:14 +0000 (21:34 +0000)]
remove unneeded SK_SUPPORT_LEGACY_PROCXFERMODE

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

Author: reed@google.com

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

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

10 years agorebaseline_server: ignore builders we don't have any expectations for
commit-bot@chromium.org [Wed, 30 Apr 2014 21:32:37 +0000 (21:32 +0000)]
rebaseline_server: ignore builders we don't have any expectations for

(SkipBuildbotRuns)
BUG=skia:2486
NOTREECHECKS=True
NOTRY=True
R=rmistry@google.com
TBR=rmistry

Author: epoger@google.com

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

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

10 years agoone valgrind.supp to rule them all.
commit-bot@chromium.org [Wed, 30 Apr 2014 21:10:13 +0000 (21:10 +0000)]
one valgrind.supp to rule them all.

R=mtklein@google.com, robertphillips@google.com, borenet@google.com

Author: bsalomon@google.com

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

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

10 years agoDM: run child tasks that are already on the CPU threadpool serially
commit-bot@chromium.org [Wed, 30 Apr 2014 20:47:30 +0000 (20:47 +0000)]
DM: run child tasks that are already on the CPU threadpool serially

These tasks tend to do similar things with similar sized bitmaps, so running
them serially means we tend to hold 2x bitmaps at a time (golden and
comparison) instead of (1+k)x bitmaps (golden and k concurrent comparisons).

We still migrate GPU task's children over to the main CPU thread pool,
because they'll run faster there and free up capacity on the GPU thread.

Before
  Debug: 54s, 2.9G peak
  Release: 13s, 2.4G peak

After
  Debug: 48s, 1.5G peak
  Release: 15s, 2.0G peak

BUG=skia:2478
R=borenet@google.com, mtklein@google.com

Author: mtklein@chromium.org

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

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

10 years agotry /LARGEADDRESSAWARE to give Windows 32-bit builds access to more RAM.
commit-bot@chromium.org [Wed, 30 Apr 2014 20:47:22 +0000 (20:47 +0000)]
try /LARGEADDRESSAWARE to give Windows 32-bit builds access to more RAM.

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

Author: mtklein@chromium.org

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

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

10 years agomake gitignore happy with symlinks
commit-bot@chromium.org [Wed, 30 Apr 2014 19:55:51 +0000 (19:55 +0000)]
make gitignore happy with symlinks

See https://codereview.chromium.org/139473002 for similar CL.

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

Author: halcanary@google.com

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

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

10 years agoSet ulimits for the compile stage.
commit-bot@chromium.org [Wed, 30 Apr 2014 19:38:51 +0000 (19:38 +0000)]
Set ulimits for the compile stage.

BUG=skia:
R=mtklein@google.com

Author: jcgregorio@google.com

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

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

10 years agoAdd Valgrind suppressions for nvidia driver.
commit-bot@chromium.org [Wed, 30 Apr 2014 19:20:42 +0000 (19:20 +0000)]
Add Valgrind suppressions for nvidia driver.

Did we recently switch the machine Valgrind runs on from an ati card to
an nvidia one?

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

Author: mtklein@chromium.org

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

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

10 years agoRename SK_DEBUGBREAK to SK_ALWAYSBREAK
commit-bot@chromium.org [Wed, 30 Apr 2014 18:24:16 +0000 (18:24 +0000)]
Rename SK_DEBUGBREAK to SK_ALWAYSBREAK

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

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

Author: bsalomon@google.com

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

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

10 years agoFix bug with starting at the home page.
commit-bot@chromium.org [Wed, 30 Apr 2014 16:54:00 +0000 (16:54 +0000)]
Fix bug with starting at the home page.

BUG=skia:
R=mtklein@google.com

Author: jcgregorio@google.com

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

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

10 years agoAvoid undefined behavior and enable asserts in ClampTest.
commit-bot@chromium.org [Wed, 30 Apr 2014 16:51:51 +0000 (16:51 +0000)]
Avoid undefined behavior and enable asserts in ClampTest.

The old code here wasn't being careful to avoid int32_t overflow in slow_check.  Fix that.

R_ASSERT hasn't been doing anything for a while.  As a result, there are a couple bugs in SkClampRange, marked as such and commented out.  The asserts also weren't quite passing, so I fixed them up (allowing 0xFFFF to be considered either as part of the ramp or part of V1.)

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

Author: mtklein@chromium.org

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

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

10 years agoTurn off warnings as errors, also allow c++11 extensions.
commit-bot@chromium.org [Wed, 30 Apr 2014 16:49:34 +0000 (16:49 +0000)]
Turn off warnings as errors, also allow c++11 extensions.

BUG=skia:
R=mtklein@google.com

Author: jcgregorio@google.com

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

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

10 years agoremove inverse_paths from ignored tests
commit-bot@chromium.org [Wed, 30 Apr 2014 16:34:43 +0000 (16:34 +0000)]
remove inverse_paths from ignored tests

new expectations have been checked in

https://codereview.chromium.org/264653003

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

Author: caryclark@google.com

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

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

10 years agoRevert of Rename SK_DEBUGBREAK to SK_ALWAYSBREAK (https://codereview.chromium.org...
commit-bot@chromium.org [Wed, 30 Apr 2014 16:27:21 +0000 (16:27 +0000)]
Revert of Rename SK_DEBUGBREAK to SK_ALWAYSBREAK (https://codereview.chromium.org/263553012/)

Reason for revert:
Broke windows bot

Original issue's description:
> Rename SK_DEBUGBREAK to SK_ALWAYSBREAK
>
> Committed: http://code.google.com/p/skia/source/detail?r=14473

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

Author: bsalomon@google.com

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

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

10 years agounsuppress inverse_paths gm images
commit-bot@chromium.org [Wed, 30 Apr 2014 16:15:49 +0000 (16:15 +0000)]
unsuppress inverse_paths gm images

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

Author: caryclark@google.com

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

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

10 years agoSet TCP keepalive for MySQL connections.
commit-bot@chromium.org [Wed, 30 Apr 2014 16:14:10 +0000 (16:14 +0000)]
Set TCP keepalive for MySQL connections.

BUG=skia:
R=mtklein@google.com

Author: jcgregorio@google.com

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

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

10 years agoRename SK_DEBUGBREAK to SK_ALWAYSBREAK
commit-bot@chromium.org [Wed, 30 Apr 2014 16:14:04 +0000 (16:14 +0000)]
Rename SK_DEBUGBREAK to SK_ALWAYSBREAK

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

Author: bsalomon@google.com

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

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

10 years agoUpdate the list of bots in rebaseline.py.
senorblanco@chromium.org [Wed, 30 Apr 2014 16:11:50 +0000 (16:11 +0000)]
Update the list of bots in rebaseline.py.

This script was not updated for the new Ubuntu bots. Also, it was
attempting to rebaseline some bots for which we don't keep results
(NVPR, ASAN).

R=borenet@google.com, borenet

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

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

10 years agoEliminate SkPicture-related flags Chromium no longer requires
commit-bot@chromium.org [Wed, 30 Apr 2014 15:55:14 +0000 (15:55 +0000)]
Eliminate SkPicture-related flags Chromium no longer requires

This was originally reverted in r14349 (Restore SkPicture-related guard flags for Chrome - https://codereview.chromium.org/255543004/) due to Android WebView compile failures. These failures should be fixed in Chromium r267210 (Fixing Chromium Android build's reliance on old SkPicture recording API - https://codereview.chromium.org/251843005/)

R=bsalomon@google.com

Author: robertphillips@google.com

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

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

10 years agoUse SkDebugf to print when GrContext can't be created for a config.
commit-bot@chromium.org [Wed, 30 Apr 2014 15:54:29 +0000 (15:54 +0000)]
Use SkDebugf to print when GrContext can't be created for a config.

Also, don't label it an "Error". New output matches that of GM.

BUG=skia:2494
R=caryclark@google.com, bensong@google.com

Author: bsalomon@google.com

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

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

10 years agorefresh expectations for android intel conical gradients
commit-bot@chromium.org [Wed, 30 Apr 2014 15:29:53 +0000 (15:29 +0000)]
refresh expectations for android intel conical gradients

TBR=reed
NOTRY=true

Author: caryclark@google.com

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

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

10 years agoun-ignore image-surface GM results
commit-bot@chromium.org [Wed, 30 Apr 2014 15:22:47 +0000 (15:22 +0000)]
un-ignore image-surface GM results

NOTRY=True
TBR=reed@google.com

Author: reed@google.com

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

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

10 years agoNew Ubuntu baselines for tileimagefilter GM post-r14432.
senorblanco@chromium.org [Wed, 30 Apr 2014 15:18:53 +0000 (15:18 +0000)]
New Ubuntu baselines for tileimagefilter GM post-r14432.

TBR=caryclark@google.com

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

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

10 years agorebaseline but with known incorrectness
commit-bot@chromium.org [Wed, 30 Apr 2014 15:17:01 +0000 (15:17 +0000)]
rebaseline but with known incorrectness

BUG=skia:2497
NOTRY=True
TBR=bsalomon@google.com

Author: reed@google.com

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

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

10 years agorebaseline GM changes to image-surface
commit-bot@chromium.org [Wed, 30 Apr 2014 14:59:38 +0000 (14:59 +0000)]
rebaseline GM changes to image-surface

NOTRY=True
TBR=caryclark@google.com

Author: reed@google.com

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

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

10 years agoCleanup of SSE optimization files.
commit-bot@chromium.org [Wed, 30 Apr 2014 14:58:46 +0000 (14:58 +0000)]
Cleanup of SSE optimization files.

General cleanup of optimization files for x86/SSEx.
Renamed the opts_check_SSE2.cpp file to _x86, since it's not specific
to SSE2. Commented out the ColorRect32 optimization, since it's
disabled anyway, to make it more visible.
Also fixed a lot of indentation, inclusion guards, spelling,
copyright headers, braces, whitespace, and sorting of includes.

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/264603002

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

10 years agoDM: when 32-bit, disable pathops internal threading by default
commit-bot@chromium.org [Wed, 30 Apr 2014 14:56:29 +0000 (14:56 +0000)]
DM: when 32-bit, disable pathops internal threading by default

Before this change, when limited to 4G, pathops threaded tests were the weak
link RAM-consumption-wise (in thread-local font caches) up until about 12
cores, where other problems begin to pile up too.

Tested by running:
    ulimit -Sv 4194304
    out/Debug/dm --threads N [--pathOpsSingleThread]

After this, we're _probably_ good to go on 32-bit machines with 8 cores.

BUG=skia:2478
R=borenet@google.com, mtklein@google.com

Author: mtklein@chromium.org

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

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

10 years agoNew baselines for tileimagefilter GM after r14432.
senorblanco@chromium.org [Wed, 30 Apr 2014 14:35:57 +0000 (14:35 +0000)]
New baselines for tileimagefilter GM after r14432.

TBR=caryclark@google.com

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

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

10 years agoremove rebaselined tests from ignore file
commit-bot@chromium.org [Wed, 30 Apr 2014 14:33:04 +0000 (14:33 +0000)]
remove rebaselined tests from ignore file

this removes the lines corresponding to this rebaseline CL

Change committed as 14447

https://codereview.chromium.org/263473002/

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

Author: caryclark@google.com

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

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

10 years agoConvert GrCrash->SkFAIL GrDebugCrash->SkDEBUGFAIL
commit-bot@chromium.org [Wed, 30 Apr 2014 14:17:00 +0000 (14:17 +0000)]
Convert GrCrash->SkFAIL GrDebugCrash->SkDEBUGFAIL

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

Author: bsalomon@google.com

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

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

10 years agoRebaselines post-r14433.
senorblanco@chromium.org [Wed, 30 Apr 2014 14:11:29 +0000 (14:11 +0000)]
Rebaselines post-r14433.

Update testimagefilters GM on SGX540 GPU and matrixconvolution GM on
Win7 (perspective).

TBR=caryclark@google.com
BUG=skia:

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

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

10 years agofix cubic/line intersection; add skp tests
commit-bot@chromium.org [Wed, 30 Apr 2014 13:37:48 +0000 (13:37 +0000)]
fix cubic/line intersection; add skp tests

BUG=skia:2488
TBR=reed

Author: caryclark@google.com

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

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

10 years agoTurn on quilt mode in DM.
commit-bot@chromium.org [Wed, 30 Apr 2014 13:20:45 +0000 (13:20 +0000)]
Turn on quilt mode in DM.
  - Rename TileGrid -> Quilt to avoid the name overload.
  - Tag all failing GMs with kSkipTiled_Flag.

You may be wondering, do any GMs pass?  Yes, some do!  And that trends towards all of them as we increase --quiltTile.

Two GMs only fail in --quilt mode in 565.  Otherwise all GMs which fail are skipped, and those which don't fail aren't. (The 8888 variants of those two GMs are skipped even though they pass.)

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

Author: mtklein@chromium.org

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

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

10 years agoRemove the unused SkCachePreload_arm
commit-bot@chromium.org [Wed, 30 Apr 2014 13:19:28 +0000 (13:19 +0000)]
Remove the unused SkCachePreload_arm

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

Author: kevin.petit@arm.com

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

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

10 years agoBackfill unit tests for SkRecord
commit-bot@chromium.org [Wed, 30 Apr 2014 13:18:12 +0000 (13:18 +0000)]
Backfill unit tests for SkRecord

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

Author: mtklein@chromium.org

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

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

10 years agobench rebase after eeaeafe
commit-bot@chromium.org [Wed, 30 Apr 2014 12:56:40 +0000 (12:56 +0000)]
bench rebase after eeaeafe
R=robertphillips@google.com
TBR=robertphillips@google.com
NOTRY=true
NOTREECHECKS=true

Author: bensong@google.com

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

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

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

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

10 years agoFail to create GrContext when we get a NULL for a GL/GLSL version string
commit-bot@chromium.org [Wed, 30 Apr 2014 01:26:04 +0000 (01:26 +0000)]
Fail to create GrContext when we get a NULL for a GL/GLSL version string

BUG=368107
R=jvanverth@google.com

Author: bsalomon@google.com

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

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

10 years agoIncrease picture bench inner loop iteration to 10 for testing number stableness.
commit-bot@chromium.org [Wed, 30 Apr 2014 00:46:29 +0000 (00:46 +0000)]
Increase picture bench inner loop iteration to 10 for testing number stableness.

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

Author: bensong@google.com

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

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

10 years agofix bitmap shader leak in gm
commit-bot@chromium.org [Tue, 29 Apr 2014 21:46:59 +0000 (21:46 +0000)]
fix bitmap shader leak in gm

this was introduced in https://codereview.chromium.org/245963010

this was detected by caryclark@ by examining the result of valgrind on the skia buildbots

BUG=skia:1976
R=caryclark@google.com

Author: dominikg@chromium.org

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

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

10 years agorebaseline perf expectations after r14443
commit-bot@chromium.org [Tue, 29 Apr 2014 21:28:04 +0000 (21:28 +0000)]
rebaseline perf expectations after r14443

bench rebase test 4610a46
BUG=skia:2466
R=bensong@google.com
TBR=robertphillips@google.com
NOTRY=true
NOTREECHECKS=true

Author: epoger@google.com

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

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

10 years agoRevert of Improve Android scripts: (https://codereview.chromium.org/254923003/)
commit-bot@chromium.org [Tue, 29 Apr 2014 21:00:06 +0000 (21:00 +0000)]
Revert of Improve Android scripts: (https://codereview.chromium.org/254923003/)

Reason for revert:
errors

Original issue's description:
> Improve Android scripts:
>
> * android_gdbserver:
>   - Don't complain if $GDB_TMP_DIR already exists
>   - Since $GDB_TMP_DIR/gdb.setup might already exist, overwrite the file.
>
> * android_run_skia:
>   - Move resource files onto the device.
>   - Change to tmp directory before executing program.
>
> * android_make, android_setup:
>   - stop assuming PWD is skia source directory.
>
> NOTRY=true
>
> Committed: http://code.google.com/p/skia/source/detail?r=14442

R=djsollen@google.com, scroggo@google.com
TBR=djsollen@google.com, scroggo@google.com
NOTREECHECKS=true
NOTRY=true

Author: halcanary@google.com

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

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

10 years agoupdate yunchao suppressed expectations
commit-bot@chromium.org [Tue, 29 Apr 2014 20:44:12 +0000 (20:44 +0000)]
update yunchao suppressed expectations

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

Author: caryclark@google.com

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

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

10 years agorebaseline perf expectations after r14443
commit-bot@chromium.org [Tue, 29 Apr 2014 20:32:36 +0000 (20:32 +0000)]
rebaseline perf expectations after r14443

bench rebase test 4610a46
R=robertphillips@google.com
TBR=robertphillips@google.com
NOTRY=true
NOTREECHECKS=true

Author: epoger@google.com

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

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

10 years agoadb_push_if_needed - make target directory if it does not exist.
commit-bot@chromium.org [Tue, 29 Apr 2014 20:12:21 +0000 (20:12 +0000)]
adb_push_if_needed - make target directory if it does not exist.

(fixes error introduced in 21a6d13.)

NOTRY=true
R=djsollen@google.com

Author: halcanary@google.com

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

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

10 years agoMove GrGLInterface function ptr setup into a common function for all OpenGL GrGLInter...
commit-bot@chromium.org [Tue, 29 Apr 2014 20:06:22 +0000 (20:06 +0000)]
Move GrGLInterface function ptr setup into a common function for all OpenGL GrGLInterface factories (but not GLES yet).

R=robertphillips@google.com

Author: bsalomon@google.com

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

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

10 years agoreland r14391 ('fix contents of render_pictures JSON summary')
commit-bot@chromium.org [Tue, 29 Apr 2014 19:39:22 +0000 (19:39 +0000)]
reland r14391 ('fix contents of render_pictures JSON summary')

BUG=skia:2043,skia:2044,skia:1942,skia:2466
R=caryclark@google.com
TBR=caryclark

Author: epoger@google.com

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

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

10 years agoImprove Android scripts:
commit-bot@chromium.org [Tue, 29 Apr 2014 19:09:58 +0000 (19:09 +0000)]
Improve Android scripts:

* android_gdbserver:
  - Don't complain if $GDB_TMP_DIR already exists
  - Since $GDB_TMP_DIR/gdb.setup might already exist, overwrite the file.

* android_run_skia:
  - Move resource files onto the device.
  - Change to tmp directory before executing program.

* android_make, android_setup:
  - stop assuming PWD is skia source directory.

NOTRY=true
R=djsollen@google.com, scroggo@google.com

Author: halcanary@google.com

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

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

10 years agoandroid_gdbserver - fix warning about missing shared library symbols
commit-bot@chromium.org [Tue, 29 Apr 2014 19:04:21 +0000 (19:04 +0000)]
android_gdbserver - fix warning about missing shared library symbols

The warning was:
warning: Could not load shared library symbols for 4 libraries,
e.g. /system/bin/linker.  Use the "info sharedlibrary" command to
see the complete listing.

Fixed by grabbing all of those libraries along with libc.so.

NOTRY=true
R=scroggo@google.com, djsollen@google.com

Author: halcanary@google.com

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

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

10 years agoMove setup of SkPictCopyInfo into SkPicture::clone
commit-bot@chromium.org [Tue, 29 Apr 2014 17:22:54 +0000 (17:22 +0000)]
Move setup of SkPictCopyInfo into SkPicture::clone

This refactoring seemed sufficiently fraught that I've broken it out of a larger patch to move fBitmapHeap out of SkPictureRecord/SkPicturePlayback and into SkPicture.

SkPicturePlayback's friending of SkPicture should be temporary and just allows SkPicture to access items that will be moving into it soon.

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

Author: robertphillips@google.com

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

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

10 years agoFix buildbot_globals.py: Retrieve the file from our repo mirror
commit-bot@chromium.org [Tue, 29 Apr 2014 17:11:19 +0000 (17:11 +0000)]
Fix buildbot_globals.py: Retrieve the file from our repo mirror

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

Author: borenet@google.com

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

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

10 years agoDM's rtree picture replay mode can ignore kSkippedTiled_Flag safely.
commit-bot@chromium.org [Tue, 29 Apr 2014 16:44:00 +0000 (16:44 +0000)]
DM's rtree picture replay mode can ignore kSkippedTiled_Flag safely.

tested: out/Debug/dm && out/Release/dm

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

Author: mtklein@chromium.org

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

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

10 years agofix cycle_colortypes so we can test 565
commit-bot@chromium.org [Tue, 29 Apr 2014 16:25:36 +0000 (16:25 +0000)]
fix cycle_colortypes so we can test 565

TBR=bsalomon@google.com

Author: reed@google.com

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

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

10 years agoRename expectations for renamed bots (Ubuntu13.10)
commit-bot@chromium.org [Tue, 29 Apr 2014 15:56:36 +0000 (15:56 +0000)]
Rename expectations for renamed bots (Ubuntu13.10)

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

Author: borenet@google.com

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

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

10 years agoARM Skia NEON patches - 36 - Color32
commit-bot@chromium.org [Tue, 29 Apr 2014 15:36:33 +0000 (15:36 +0000)]
ARM Skia NEON patches - 36 - Color32

Convert Color32 to intrinsics

This change is performance-neutral for high values of count and is
a big improvement for values smaller than 64.

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

Author: kevin.petit@arm.com

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

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

10 years agoFixed issue found by clusterfuzz
commit-bot@chromium.org [Tue, 29 Apr 2014 15:35:23 +0000 (15:35 +0000)]
Fixed issue found by clusterfuzz

An integer overflow was causing an issue when reading a string with a very large (or negative) size.

BUG=367764
R=senorblanco@google.com, senorblanco@chromium.org, reed@google.com, borenet@google.com

Author: sugoi@chromium.org

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

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

10 years agoDon't bother doing the empty clip check in SkRecordDraw.
commit-bot@chromium.org [Tue, 29 Apr 2014 15:34:03 +0000 (15:34 +0000)]
Don't bother doing the empty clip check in SkRecordDraw.

On Mike's suggestion, I tested out not doing any empty-clip check at all in
SkRecordDraw, given that mostly we'll do that again anyway inside SkCanvas.

Most SKPs are identical to the status quo, whether bot or silk, played back in tiles
or full.  Average playback performance, both arithmetic and geometric mean, is also
unchanged.

A handful of SKPs do draw faster or slower reliably, particularly when tiled.  E.g. a
cnn tile draws about 40% faster, a cuteoverload tile about 20% slower.  Their profiles
look pretty much the same before and after, so I can't really explain the changes.

I'd say, given that performance is mostly identical and very identical in bulk,
we might as well remove this code.  It's nice to keep SkRecordDraw as dumb as possible.

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

Author: mtklein@chromium.org

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

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

10 years agoImplement bounds traversals for tile and matrix convolution filters.
senorblanco@chromium.org [Tue, 29 Apr 2014 15:20:39 +0000 (15:20 +0000)]
Implement bounds traversals for tile and matrix convolution filters.

Add a new GM that exercises tiled drawing all pixel-moving filters
(and some non-pixel-moving ones) and compares it against non-tiled
drawing of the same filters. Fixing this test revealed that tile and
matrix convolution filters had no onFilterBounds() traversals
(test-driven development FTW). Tile requires (conservatively) the
bounds to include the whole source rect, since it may end up in the
result. Matrix convolution requires the bounds to be offset by the
kernel size and target.

R=reed@google.com
BUG=skia:

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

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

10 years agoadd asAShadowBlur for android to drawlooper
reed@google.com [Tue, 29 Apr 2014 15:20:16 +0000 (15:20 +0000)]
add asAShadowBlur for android to drawlooper

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

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

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

10 years agobench rebase test 1c74374
commit-bot@chromium.org [Tue, 29 Apr 2014 12:18:38 +0000 (12:18 +0000)]
bench rebase test 1c74374
R=robertphillips@google.com
TBR=robertphillips@google.com
NOTRY=true
NOTREECHECKS=true

Author: bensong@google.com

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

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

10 years agofix leaks in blurrrect -- this should address the valgrind issues.
commit-bot@chromium.org [Tue, 29 Apr 2014 04:41:33 +0000 (04:41 +0000)]
fix leaks in blurrrect -- this should address the valgrind issues.

BUG=skia:
R=caryclark@google.com
TBR=caryclark

Author: humper@google.com

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

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

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

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

10 years agoExplicitly include gyp_gen/android_framework_gyp.
commit-bot@chromium.org [Tue, 29 Apr 2014 00:39:45 +0000 (00:39 +0000)]
Explicitly include gyp_gen/android_framework_gyp.

Add __init__.py to gyp_gen and explicitly include android_framework_gyp
from there.

Fixes a bug where after moving that file, python was looking for it in
its old location, where there was a compiled version of the file.

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

Author: scroggo@google.com

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

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

10 years agoRemove dependency of views on angle
commit-bot@chromium.org [Tue, 29 Apr 2014 00:38:39 +0000 (00:38 +0000)]
Remove dependency of views on angle

Just like with http://code.google.com/p/skia/source/detail?r=13618,
listing angle.gyp as a dependency when in the Android tree (i.e.
w/o third_party/externals) causes gyp_to_android.py to fail.

(See http://108.170.220.27:10115/builders/Housekeeper-PerCommit-AndroidRoll/builds/229/steps/Merge/logs/stdio
for example failure.)

Fortunately, views (and views_animated) has no need to depend directly
on angle.

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

BUG=skia:2447

Author: scroggo@google.com

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

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

10 years agoadd SK_API to include/record
commit-bot@chromium.org [Mon, 28 Apr 2014 21:04:32 +0000 (21:04 +0000)]
add SK_API to include/record

Failed builders https://codereview.chromium.org/225023031/ at PS 24 look like they're failing because we're not exposing these symbols when Skia's a dynamic library.

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

Author: mtklein@chromium.org

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

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

10 years agoAdd a magnifier lens to the image.
commit-bot@chromium.org [Mon, 28 Apr 2014 21:03:34 +0000 (21:03 +0000)]
Add a magnifier lens to the image.

BUG=skia:
R=mtklein@google.com

Author: jcgregorio@google.com

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

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

10 years agofix android logal gm for filltypespersp
commit-bot@chromium.org [Mon, 28 Apr 2014 20:34:48 +0000 (20:34 +0000)]
fix android logal gm for filltypespersp

BUG=skia:
NOTRY=true
TBR=borenet

Author: caryclark@google.com

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

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

10 years agoAdds a blacklist of SKPs to exclude from bench expectations.
commit-bot@chromium.org [Mon, 28 Apr 2014 20:20:43 +0000 (20:20 +0000)]
Adds a blacklist of SKPs to exclude from bench expectations.

BUG=skia:2469
NOTRY=true
NOTREECHECKS=true
R=caryclark@google.com
TBR=caryclark@google.com

Author: bensong@google.com

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

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

10 years agoRefactor SkPictureStateTree::Iterator to avoid use of kClip_SaveFlag.
commit-bot@chromium.org [Mon, 28 Apr 2014 20:17:48 +0000 (20:17 +0000)]
Refactor SkPictureStateTree::Iterator to avoid use of kClip_SaveFlag.

The current implementation relies on soon-to-be-deprecated
kClip_SaveFlag behavior. Updated to use default save flags
(kMatrixClip_SaveFlag) and stop assuming that the matrix survives
restore() calls.

R=junov@chromium.org, reed@google.com, robertphillips@chromium.org, robertphillips@google.com

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

Author: fmalita@chromium.org

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

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

10 years agoFast path for blurred round rects -- blur a small 9patch rect on the CPU
commit-bot@chromium.org [Mon, 28 Apr 2014 19:49:24 +0000 (19:49 +0000)]
Fast path for blurred round rects -- blur a small 9patch rect on the CPU
And nonlinearly stretch the resulting texture across proxy geometry.

BUG=

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

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

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

Author: humper@google.com

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

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

10 years agoSimplify empty-clip check.
commit-bot@chromium.org [Mon, 28 Apr 2014 19:49:00 +0000 (19:49 +0000)]
Simplify empty-clip check.

No measurable playback speed difference, but simpler code.

This makes sense: I'm seeing SkRecordDraw at ~0.25% of playback cost.  We can
pretty much do whatever we want in there for free if it helps avoid real work.

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

Author: mtklein@chromium.org

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

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

10 years agoRename GR_GL_GET_PROC macros in mesa and unix GrGLInterface factories to be more...
commit-bot@chromium.org [Mon, 28 Apr 2014 19:41:24 +0000 (19:41 +0000)]
Rename GR_GL_GET_PROC macros in mesa and unix GrGLInterface factories to be more consistent with other platforms.

R=robertphillips@google.com

Author: bsalomon@google.com

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

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

10 years agoAdd --timescale to bench_record and bench_playback.
commit-bot@chromium.org [Mon, 28 Apr 2014 19:41:17 +0000 (19:41 +0000)]
Add --timescale to bench_record and bench_playback.

Default bench_record to report us, bench_playback ms.

BUG=skia:2378
R=mtklein@google.com

Author: mtklein@chromium.org

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

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

10 years agoFirst version of gpuveto testing script
commit-bot@chromium.org [Mon, 28 Apr 2014 19:41:01 +0000 (19:41 +0000)]
First version of gpuveto testing script

This CL just sketches out the structure of the gpuveto testing process. Two big areas for improvement are:
  render the picture in tiles and label as unsuitable if any gpu tile is slower than raster
  decide on whether tilegrid is used or not

As expected the current gpuveto heuristic isn't so good:

predicted     suitable    unsuitable
-------------------------------------------
actual suitable      10           17
actual unsuitable    15           27

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

Author: robertphillips@google.com

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

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

10 years agoAdd chrome_changes file
commit-bot@chromium.org [Mon, 28 Apr 2014 19:40:53 +0000 (19:40 +0000)]
Add chrome_changes file

BUG=skia:2457
R=epoger@google.com, bsalomon@google.com, robertphillips@google.com, bungeman@google.com, reed@google.com, skia-team@google.com

Author: borenet@google.com

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

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