skimage, die die die

DM's running with --images on all the bots now.  They look happy.

BUG=skia:3235

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

Add xp optimization for RGB coverage.

This is needed since clearColorStages is being changed to ignore color input. For RGB coverage,
we want to clear all the color stages (since we know the final output color), but we don't want
to ignore the color input since it is needed.

In future we will change this so the XP stores the color internally and thus can tell the GP to
simiply ignore color.

BUG=skia:

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

Fix rrects that are large enough that we lose/gain a bit when we add the radius to a bounds coordinate.

add test that triggers assert in addRRect

BUG=skia:3239

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

Silence warnings from libnanomsg on Mac like we do on Linux.

This will silence these warnings:

../../third_party/externals/nanomsg/src/core/global.c:162:34: warning: missing field 'unused' initializer [-Wmissing-field-initializers]
static struct nn_global self = {0};

../../third_party/externals/nanomsg/src/utils/clock.c:44:61: warning: missing field 'denom' initializer [-Wmissing-field-initializers]
static mach_timebase_info_data_t nn_clock_timebase_info = {0};

BUG=skia:

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

Revert of Roll libwebp to v0.4.2 (latest stable) to fix annoying build warning. (patchset #1 id:1 of https://codereview.chromium.org/807553002/)

Reason for revert:
uses __builtins not available on all our compilers

Original issue's description:
> Roll libwebp to v0.4.2 (latest stable) to fix annoying build warning.
>
> This warning should now go away:
>
> ../../third_party/externals/libwebp/src/enc/quant.c:105:23: warning: unused variable 'kCoeffThresh' [-Wunused-const-variable]
> static const uint16_t kCoeffThresh[16] = {
>
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/e655557a551c545f4153b0764269298cd01cd0c0

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

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

Roll libwebp to v0.4.2 (latest stable) to fix annoying build warning.

This warning should now go away:

../../third_party/externals/libwebp/src/enc/quant.c:105:23: warning: unused variable 'kCoeffThresh' [-Wunused-const-variable]
static const uint16_t kCoeffThresh[16] = {

BUG=skia:

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

Rewrite NaN checks in terms of SkScalarIsNaN()

We are trying to replace Skia's NaN checker with our own in Mozilla,
so it would be nice to have to patch a single place by making sure
these NaN checks used SkScalarIsNaN().

BUG=skia:

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

Remove SkCanvas::isDrawingToLayer

BUG=3245

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

Update SKP version

Automatic commit by the RecreateSKPs bot.

TBR=

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

final circles?

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

WAE on Macs too.  That leaves only Android framework builds.

Will rebase after yours lands.

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/161b6dee11a67e877ef5b339b17775b8e44c5927

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

Revert of WAE on Macs too.  That leaves only Android framework builds. (patchset #3 id:40001 of https://codereview.chromium.org/804063002/)

Reason for revert:
Forgot CrOS and NaCl builds.

Original issue's description:
> WAE on Macs too.  That leaves only Android framework builds.
>
> Will rebase after yours lands.
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/161b6dee11a67e877ef5b339b17775b8e44c5927

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

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

WAE on Macs too.  That leaves only Android framework builds.

Will rebase after yours lands.

BUG=skia:

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

circles circles circles

TBR=bensong@google.com

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

fix last warnings on w64 and turn on w.a.e.

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

Add SK_OVERRIDE to a few places that are missing it.

This was done automatically by running clang-modernize -add-override -override-macros.

It's not clear to me whether include/ is already clean or I just can't get the tool to work for headers.

BUG=skia:

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

Add more filetypes to DM's image tests.

ico, wbmp, plus the alternate suffix jpeg.

Also check for capitalized versions, since files sometimes use
capitalized suffixes.

BUG=skia:3235

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

More circle rebaselines on gpu

TBR=bensong@google.com

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

Rebaseline after circle normalization change.

TBR=bensong@google.com

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

add dumpHex option to rect and rrect, to match path

BUG=skia:

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

Do circle anti-aliasing in normalized space to avoid precision issues with half-floats on Adreno.

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

ensure 4-byte alignment for storage, as SkWriterBuffer requires it

BUG=skia:3244

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

Cleanup: Mark some overridden methods with 'SK_OVERRIDE'.

This fixes errors like this:

../../include/gpu/effects/GrPorterDuffXferProcessor.h:27:25: error:
'name' overrides a member function but is not marked 'override'
[-Werror,-Winconsistent-missing-override]

BUG=skia:3075
TEST=ninja -C out/Debug skia_lib
TBR=bsalomon@google.com

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

add get9 and set9 to matrix, to aid in making keys

BUG=skia:
TBR=
NOTREECHECKS=True

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

GLPrograms leak fix

NOTREECHECKS=True
TBR=
BUG=skia:

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

DM warning-free on win64

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

Add image decoding mode to DM.

This is meant to supplant skimage.

BUG=skia:3235

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

Even more win64 warning fixes

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

Rebaselines.

TBR=borenet

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

Check in skimage expectations for nexus player.

TBR=borenet

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

improve cull, so we don't create empty segments

BUG=skia:3224

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

Harden validation for SkPathRef

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

Add Herb as a Skia committer

TBR=hcm@google.com

NOTRY=true

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

Handle SkBitmaps which have no pixels in SkPDFImage.

A test that ensures this no longer crashes has been added.

BUG=skia:3232

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

Update expectations for Test-Mac10.9-MacMini6.2-HD4000-x86_64-Debug

TBR=borenet

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

Remove gms from ignore-tests.

TBR=

BUG=skia:

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

Blindly check in GM and skimage expectations for new bots

BUG=skia:

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

Extract cubic classification from gpu into geometry

Move code from the GPU path utilities into geometry so that
path ops can share.

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

Pass gyp_source_dir to generate_tool.

When generating the tool makefiles, which also calls
android_framework_gyp, we also need to pass the gyp directory.

TBR=borenet
NOTRY=True
NOTREECHECKS=True

BUG=skia:3233

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

Update SKP version

Automatic commit by the RecreateSKPs bot.

TBR=

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

Update some Galaxy S4 expectations.

TBR=djsollen@google.com

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

Ignore some expectations for Nexus 10.

BUG=skia:3205
TBR=bsalomon@google.com

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

More win64 warning fixes

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

Roll ANGLE

BUG=skia:3192

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

Cull pushCull and popCull from Skia.

These calls are unused and going away.  Waiting on crrev.com/796083002.

BUG=skia:

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

SkRecord: increase min block to 512B, remove max.

When we added the 64K allocation cap, the bots showed we took a perf hit
on some large .skps like desk_pokemonwiki.skp, despite not seeing a local
effect.  I'm still not seeing that locally, but I'd like to try removing the cap on
the bots to see what happens.  For big monolithic pictures, really packing into
memory tightly is probably not as important as it is for tiny ones.

Similarly, we're probably being too cautious about making tiny allocations.
Today we start at 16 bytes, which isn't really enough to record anything.
Even the smallest picture, say,
    save
        clipRect
        drawRect
    restore
requires ~200 bytes, so we might as well move our minimum block size up
near there.

I don't know if 16 bytes is too small to start for GrTextStrikes, so I've left the
behavior the same (though the max is still gone).

Local recording performance is neutral-to-positive:
             tabl_deviantart.skp  126us ->  129us 1.02x
                tabl_nytimes.skp  110us ->  112us 1.02x
           tabl_cuteoverload.skp  521us ->  530us 1.02x
             desk_mobilenews.skp  673us ->  682us 1.01x
             desk_chalkboard.skp  843us ->  854us 1.01x
                 desk_sfgate.skp  528us ->  535us 1.01x
            desk_silkfinance.skp 68.2us ->   69us 1.01x
                desk_youtube.skp  623us ->  629us 1.01x
                desk_blogger.skp  472us ->  475us 1.01x
     desk_jsfiddlehumperclip.skp 42.2us -> 42.5us 1.01x
                   desk_espn.skp  255us ->  256us 1.01x
                   desk_ebay.skp  174us ->  174us 1x
                desk_twitter.skp  454us ->  455us 1x
                 tabl_pravda.skp  200us ->  201us 1x
              desk_wordpress.skp  782us ->  784us 1x
               desk_samoasvg.skp  762us ->  761us 1x
                tabl_mozilla.skp 1.58ms -> 1.58ms 1x
               tabl_slashdot.skp  107us ->  107us 1x
               tabl_techmeme.skp  102us ->  102us 0.99x
            tabl_gamedeksiam.skp  729us ->  724us 0.99x
                 tabl_nofolo.skp 65.3us -> 64.7us 0.99x
            desk_gmailthread.skp  339us ->  336us 0.99x
                tabl_sahadan.skp   91us ->   90us 0.99x
           desk_yahooanswers.skp  144us ->  142us 0.99x
                   tabl_cnet.skp  143us ->  141us 0.99x
             tabl_googleblog.skp  206us ->  203us 0.99x
                    tabl_cnn.skp  160us ->  158us 0.99x
               tabl_frantzen.skp 50.5us -> 49.6us 0.98x
               desk_linkedin.skp  328us ->  323us 0.98x
                   tabl_digg.skp  790us ->  769us 0.97x
         desk_jsfiddlebigcar.skp 40.6us -> 39.5us 0.97x
                 desk_mapsvg.skp 1.57ms -> 1.52ms 0.97x
                  tabl_gmail.skp 19.4us -> 18.6us 0.96x
                   tabl_hsfi.skp 9.81us -> 9.11us 0.93x

BUG=skia:

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

valgrind static uninit fix

TBR=
BUG=skia:

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

Add some debugging information to gyp_to_android.

Temporary, to help diagnose skbug.com/3233

TBR=borenet
NOTRY=True
NOTREECHECKS=True
BUG=skia:3233

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

Fix float and half float support on mobile.

It's unclear what params should be used for half float alpha
rendertargets, so they are disabled for the moment.

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

Fix GpuLayerCacheTest compile error

Fix GpuLayerCacheTest compile error:
GpuLayerCacheTest.cpp:120:34: error: comparison between signed and
unsigned integer expressions [-Werror=sign-compare]

gcc --version
gcc (Ubuntu/Linaro 4.6.4-1ubuntu1~12.04) 4.6.4

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

use SkScalar instead of float

BUG=skia:
TBR=

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

adding some flaky gms to ignore

TBR=
NOTRY=True
BUG=skia:

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

rebase and remove ignores

NOTRY=True
TBR=
BUG=skia:

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

Remove GP from drawstate, revision of invariant output for GP

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/c6bc58eded89b0c0a36b8e20e193c200f297a0da

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

Adding a custom data field to GrGpuResource

Motivation: allow Blink to track pre-existing mailbox names for
textures that are recycled by the ganesh scratch texture pool.

BUG=440462
TBR=bsalomon@google.com

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

Update SKP version

Automatic commit by the RecreateSKPs bot.

TBR=

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

one more ignore for hairlines

TBR=
NOTRY=True
BUG=skia:

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

Fix some win64 warnings.

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

Increase timeout on sfgate pageset

TBR=scroggo
NOTRY=true
BUG=skia:3196

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

Create xfer processor backend.

This includes:
-Having an actual XP stage at the end of the gl pipeline.
-All Blending work is handled by XP until actually setting GL blend states
-GLPrograms test to test XP

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/4dffc940c430eec66d4707490eace19c9b3f7904

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

Update GM to permit correctness testing of MPD on the GMs

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

new hairlines gm unstable on hd2000

NOTRY=True
TBR=
BUG=skia:

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

Add parameter to specify gyp dir to gyp_to_android.

NOTRY=true
NOTREECHECKS=True
BUG=skia:3233

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

Enforce thread-safety of bitmaps in pictures via the type.

No runtime difference here, but it makes it impossible to forget to make
a shallow copy; you can't get at the full bitmap without it.

NOTREECHECKS=true

BUG=skia:

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

Force embedding full font when serializing pictures.

We can't do this unconditionally or pipe will become stupidly slow.

DM's serialize mode fails subtly on Mac when we force embedding, so I've
#ifdef'd that away.  Other platforms look fine.

BUG=skia:

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

Replace EncodeBitmap with an interface.

Gives more flexibility to the caller to decide whether to use the
encoded data returned by refEncodedData().

Provides an implementation that supports the old version of
SkPicture::serialize().

TODO: Update Chrome, so we can remove SK_LEGACY_ENCODE_BITMAP entirely

BUG=skia:3190

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

Making GrGpuResource::release public for special case in Blink

BUG=440462

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

Remove action_runner steps for worldjournal pageset to prevent crashes

BUG=skia:3196
NOTRY=true

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

hmm one more hairlines 'straggler' might be flipflopping

TBR=
NOTRY=
BUG=skia:

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

straggler hairline

NOTRY=True
TBR=
BUG=skia:

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

Poppler rebaseline for clipped-bitmap-shaders-clamp

Follow-up for https://codereview.chromium.org/787043006.

TBR=scroggo@google.com

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

Apply the layer's image filter to the hoisted image

Exposing SkSurface_Gpu makes me sad and I would welcome alternatives.

This change is desireable since it greatly decreases the render target swaps.

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

rebase cpu hairlines

NOTRY=True
NOTREECHECKS=True
TBR=
BUG=skia:

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

update test after I added a clip call, making the count odd

BUG=skia:
TBR=
NOTREECHECKS=True
NOTRY=True

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

Defer saves() until they're needed

patch from issue 759443006 at patchset 40001 (http://crrev.com/759443006#ps40001)

BUG=skia:

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

rebase hairlines

TBR=
BUG=skia:

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

Really fix valgrind issue

TBR=
BUG=skia:

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

More rebaselines.

TBR=robertphillips
BUG=skia:

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

Mark nexus10 images as flaky.

TBR=robertphillips
BUG=skia:3226

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

Additional rebaselines for 3490263287e9.

TBR=bungeman1

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

Replace use of deprecated CG methods.

OSX10.9 and iOS7.0 deprecated CGContextShowGlyphsAtPoint so a new API
should be used. OSX10.7 and iOS4.2 replace CGContextShowGlyphsAtPoint with
CTFontDrawGlyphs. OSX10.5 and iOS2.0 have CGContextShowGlyphsAtPositions
which works similarly to CTFontDrawGlyphs and has not yet been deprecated.
This change allows the use of CTFontDrawGlyphs when it is available,
falling back to CGContextShowGlyphsAtPositions when it isn't.

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

Revert of Create xfer processor backend. (patchset #6 id:100001 of https://codereview.chromium.org/764643004/)

Reason for revert:
CL breaks windows DM

Original issue's description:
> Create xfer processor backend.
>
> This includes:
> -Having an actual XP stage at the end of the gl pipeline.
> -All Blending work is handled by XP until actually setting GL blend states
> -GLPrograms test to test XP
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/4dffc940c430eec66d4707490eace19c9b3f7904

TBR=bsalomon@google.com,joshualitt@chromium.org,egdaniel@google.com
NOTREECHECKS=true
NOTRY=true
BUG=skia:

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

Revert of Add more ignore gm's for recent Xfer Processor CL (patchset #1 id:1 of https://codereview.chromium.org/786083004/)

Reason for revert:
Preventing revert of cl breaking windows DM

Original issue's description:
> Add more ignore gm's for recent Xfer Processor CL
>
> TBR=bsalomon@google.com
> NOTRY=true
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/125251133a62a07225b7119ab3ac90153ba4c58a

TBR=bsalomon@google.com,egdaniel@google.com
NOTREECHECKS=true
NOTRY=true
BUG=skia:

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

Revert of Fix to set correct output type when blending when we've read dst (patchset #4 id:60001 of https://codereview.chromium.org/791143002/)

Reason for revert:
Breaks DM on windows

Original issue's description:
> Fix to set correct output type when blending when we've read dst
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/733fc2054044377e914559fa4f46db66ab7d9cea

TBR=bsalomon@google.com,egdaniel@google.com
NOTREECHECKS=true
NOTRY=true
BUG=skia:

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

fix for chrome bug with sub pixel hairlines

BUG=skia:

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

Initial baselines for tall_stretched_bitmaps GM

BUG=skia:

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

Revert of Remove GP from drawstate, revision of invariant output for GP (patchset #9 id:160001 of https://codereview.chromium.org/791743003/)

Reason for revert:
breaks mac

Original issue's description:
> Remove GP from drawstate, revision of invariant output for GP
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/c6bc58eded89b0c0a36b8e20e193c200f297a0da

TBR=bsalomon@google.com,egdaniel@google.com,joshualitt@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=skia:

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

Remove GP from drawstate, revision of invariant output for GP

BUG=skia:

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

Don't store an SkMatrix in BitmapShaderKey.

SkMatrix makes for a poor key component due to mutable/cache fields.

Use its canonical scalars instead.

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

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

Fix to set correct output type when blending when we've read dst

BUG=skia:

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

fix for valgrind uninit variables

TBR=bsalomon@google.com
BUG=skia:

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

Rebaseline imagefiltersscaled images (and rm from ignored-tests file)

Rebaselines for https://codereview.chromium.org/785893004/ (Fix MatrixImageFilter computeFastBounds)

TBR=scroggo@google.com

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

More rebaselines.

TBR=bsalomon
BUG=skia:

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

Remove MULTILINE, ^, and $ from re finding actual_results.json.

MULTILINE, ^, and $ work based on the platform, but the files being
parsed are from other platforms. As a result, the current code for
extracting the actual_results.json version will not find it in
logfiles produced on Windows when run on Mac.

The code for extracting the exact actual_results.json file to use
from a logfile is itself something of a hack, as this information
should be provided ina more structured manner. This proposed
method of finding the exact file is no worse than the old one, and
in cases like above, better.

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

Detect discarded SkPictureShader pixel refs.

This doesn't address the more general problem, but reduces the race
window significantly.

BUG=440282
R=reed@google.com

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

Rebaseline.

All these images barely differ.

TBR=bsalomon

BUG=skia:

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

Include SkTypes.h from SkDynamicAnnotations.h

This allows us to control SK_DYNAMIC_ANNOTATIONS_ENABLED by Sk*Config.h files.
This is a no-op today, because we control it from the compiler command line.

BUG=430815

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

Gradient shaders: make fPtsToUnit const, pre-cache getType().

This prevents races when calling fPtsToUnit.getType() from multiple threads.

This introduces a small amount of redundant code in SkTwoPointRadialGradient,
but it's probably optimized together into no extra run-time work.

BUG=437511

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

Change how SkDebugf is sent to stdout on Android.

Previously, a function was called using dlsym in skia_launcher.

Add a static initializer that changes the setting, and include that for
the tools we automate for testing.

Also only do va_copy if we actually use it.

BUG=skia:2454

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

dd readPixels to SkImage

patch from issue 789673007 at patchset 1 (http://crrev.com/789673007#ps1)

BUG=skia:

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