improve precision of gradients (disabled for now)

BUG=skia:2898
TBR=caryclark

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

rebase
NOTRY=True
TBR=
BUG=skia:

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

I could create new GMs if you prefer.

BUG=skia:

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

adding bleed back to ignore

TBR=
NOTRY=True
BUG=skia:

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

rebase

TBR=
NOTRY=True
BUG=skia:

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

Remove GrCoordTransform::kPosition_CoordSet

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

remove dead code for SK_SUPPORT_LEGACY_TEXTRENDERMODE

BUG=skia:
TBR=robertphilips

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

Change to create device coord coordset

BUG=skia:

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

Various complex clip rebases

TBR=
NOTRY=

BUG=skia:

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

rebaseline after perspective/conic fix

BUG=skia:
TBR=
NOTRY=True

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

Fix complexclip3 test on gpu

BUG=skia:3260
R=egdaniel@google.com

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

Cleanup: Remove a bunch of SkFontHost.h includes (unused).

Nobody that is including SkFontHost is using SkFontHost API, so lets
remove this includes, since the API per se is deprecated.

BUG=None
R=reed@google.com

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

One more pass at getting half float texture configs correct.

Added in check for sized internal format.
Made choice between RED and ALPHA orthogonal to HALF_FLOAT and
HALF_FLOAT_OES.
Enabled rendertarget support on ES 2.0.

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

Update SKP version

Automatic commit by the RecreateSKPs bot.

TBR=

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

Cleanup: More override fixes.

Fixes errors like:

../../src/gpu/GrGeometryProcessor.h:172:25: error: 'name' overrides a member function but is not marked 'override' [-Werror,-Winconsistent-missing-override]
    virtual const char* name() const = 0;
                            ^
../../include/gpu/GrProcessor.h:66:25: note: overridden virtual function is here
                                virtual const char* name() const = 0;

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

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

rebase

TBR=
NOTRY=True
BUG=skia:

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

remove view matrix from context

BUG=skia:

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

use conics for addRRect

BUG=skia:

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

add inverted path draw cases to complexclip gm

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

Ignore buggy clip gm on Mac 10.9

TBR=
NOTRY=true

BUG=skia:

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

Stop creating GrODS for stencilPath commands.

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

Add Coverage Drawing XP

BUG=skia:

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

rebaseline after perspective/conic fix

BUG=skia:
TBR=
NOTRY=True

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

version 0 of imgconv tool

BUG=skia:

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

Rebaseline HD7770 GMs for https://codereview.chromium.org/811133002.

TBR=bensong@google.com

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

rebaseline after perspective/conic change

BUG=skia:
TBR=
NOTRY=True

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

Turn off warnings as errors for the debugger

Compiling the debugger on Linux was running afoul of warnings-as-errors

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

Add new complexclip3 gm.

This GM is used to test the combined clipping of a complex clip (packman shape)
and a simple one (circle). We loop over all combinations of clip ops, aa/bw clip,
and inverse/non-inverse clips.

This GM triggers a current bug in the gpu clipping code which fires an assert. Thus
the skipGPU flag is set until that bug is fixed.

BUG=skia:

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

Mark flaky gm as ignore-failure on mac 10.9

BUG=skia:3256

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

Re-enable default sized formats, handling BGRA exception.

Sized texture formats are accepted by OpenGL desktop and ES 3.0,
except when using BGRA with GL_EXT_texture_format_BGRA8888.
In that case we fall back to using the unsized format.

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

rebaseline after perspective/conic fix

BUG=skia:
TBR=
NOTRY=True

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

Stop calling flushGLState in onStencilPath

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

remove flags that are now in chrome's userconfig

BUG=skia:

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

rebaseline after perspective fix (using conics)

BUG=skia:
TBR=
NOTRY=True

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

use conics

BUG=skia:

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

rebaseline after fix to cornerpatheffect

BUG=skia:
TBR=
NOTRY=True

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

Fix a case of kPartialSuccess.

Missed in https://codereview.chromium.org/647023006/

When JPEG decoder does not have the remainder of the image, it should
return kPartialSuccess.

BUG:b/17419670
BUG:b/18769124

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

Rebaseline HD7770 images after sized texture format revert.

TBR=bensong@google.com

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

fix end-point and conic bugs in cornerpatheffect

fixes bug in GM/patheffect -- will need to be rebaselined

BUG=skia:

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

Update SKP version

Automatic commit by the RecreateSKPs bot.

TBR=

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

webpages_playback.py: Page sets to use changed telemetry API (AddUserStory)

Make page sets to use telemetry.page.page_set.AddUserStory instead of
AddPage.

BUG=chromium:439512
BUG=skia:3196

NOTRY=true

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

add arcto patheffect

BUG=skia:
TBR=

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

Whitespace change

To re-trigger the build bots.

TBR=bensong@google.com
NOTREECHECKS=true
NOTRY=true

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

Give the hoisting surfaces surface props

This mimics how layers are set up in SkGpuDevice::onCreateCompatibleDevice

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

Revert using sized formats for everything except ES 2.0.

Temporary fix for WritePixels test on Nexus 5/Nexus 7-2013.

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

Remove GrGpu::flushGraphicsState

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

Catch the stragglers from skbug.com/3241

Update expectations for tests that were using SkLCGRandom (and now
use SkRandom).

TBR=benchen
BUG=skia:3241

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

Update expectations for Win8-HD7770.

TBR=bensong@google.com

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

Massive rebaseline for SkRandom.

All of the following GMs have been switched from SkLCGRandom to
SkRandom. Update their expectations for the new images.

-beziers
-circles
-convexpaths
-complexclip2
-complexclip2_rect_aa
-complexclip2_rect_bw
-complexclip2_path_aa
-complexclip2_path_bw
-complexclip2_rrect_aa
-complexclip2_rrect_bw
-cubicpath
-cubicclosepath
-degeneratesegments
-emptypath
-getpostextpath
-hittestpath
-linepath
-lineclosepath
-points
-polygons
-quadpath
-quadclosepath
-strokerects
-strokes_round
-strokes_poly

BUG=skia:3241
TBR=mtklein

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

Change desktop and ES 3.0 to always use sized internal texture formats.

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

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

Alter GM's --mpd option to write out images

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

SkMatrix44(const SkMatrix&) needs to initialize the type mask

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

Fix layer hoisting image filter corner cases

This CL fixes 5 bugs related to hoisting image filters:

For image filters the src layer (the one prior to filtering) often needs to be
smaller then the final layer. This requires the saveLayer's optional bounds
to be stored (in SkLayerInfo.h and SkRecordDraw.cpp) and then used in
compute_source_rect and carried around in GrCachedLayer.

The image filters can add an extra offset to the final draw operation.
This is now computed in GrLayerHoister::FilterLayer and  carried around in
GrCachedLayer.

Filtered layers must use exact matches. This is now done in GrLayerCache::lock.

The filter cache requires a valid matrix so it can compute the correct offset.
This is now done in GrLayerHoister::FilterLayer.

Filtered layers need to be drawn with drawSprite while unfiltered (and therefore
hopefully atlased) layers can be drawn with drawBitmap. This is now done in
draw_replacement_bitmap.

Committed: https://skia.googlesource.com/skia/+/702eb9622102599d94ab6798e6227cf29f48c2d3

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

Rename GrGpuGL_program.cpp to GrGLGpuProgramCache.cpp

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

add sample for arcTo

BUG=skia:
TBR=

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

Revert of Fix layer hoisting image filter corner cases (patchset #3 id:40001 of https://codereview.chromium.org/803183003/)

Reason for revert:
Breaking DM

Original issue's description:
> Fix layer hoisting image filter corner cases
>
> This CL fixes 5 bugs related to hoisting image filters:
>
> For image filters the src layer (the one prior to filtering) often needs to be
> smaller then the final layer. This requires the saveLayer's optional bounds
> to be stored (in SkLayerInfo.h and SkRecordDraw.cpp) and then used in
> compute_source_rect and carried around in GrCachedLayer.
>
> The image filters can add an extra offset to the final draw operation.
> This is now computed in GrLayerHoister::FilterLayer and  carried around in
> GrCachedLayer.
>
> Filtered layers must use exact matches. This is now done in GrLayerCache::lock.
>
> The filter cache requires a valid matrix so it can compute the correct offset.
> This is now done in GrLayerHoister::FilterLayer.
>
> Filtered layers need to be drawn with drawSprite while unfiltered (and therefore
> hopefully atlased) layers can be drawn with drawBitmap. This is now done in
> draw_replacement_bitmap.
>
> Committed: https://skia.googlesource.com/skia/+/702eb9622102599d94ab6798e6227cf29f48c2d3

TBR=bsalomon@google.com
NOTREECHECKS=true
NOTRY=true

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

Rename GrGpuGL to GrGLGpu for consistency

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

Fix layer hoisting image filter corner cases

This CL fixes 5 bugs related to hoisting image filters:

For image filters the src layer (the one prior to filtering) often needs to be
smaller then the final layer. This requires the saveLayer's optional bounds
to be stored (in SkLayerInfo.h and SkRecordDraw.cpp) and then used in
compute_source_rect and carried around in GrCachedLayer.

The image filters can add an extra offset to the final draw operation.
This is now computed in GrLayerHoister::FilterLayer and  carried around in
GrCachedLayer.

Filtered layers must use exact matches. This is now done in GrLayerCache::lock.

The filter cache requires a valid matrix so it can compute the correct offset.
This is now done in GrLayerHoister::FilterLayer.

Filtered layers need to be drawn with drawSprite while unfiltered (and therefore
hopefully atlased) layers can be drawn with drawBitmap. This is now done in
draw_replacement_bitmap.

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

Revert of Use SkPaint::getFontBounds() for text bounding boxes in pictures. (patchset #1 id:1 of https://codereview.chromium.org/805983003/)

Reason for revert:
Test-Win7-ShuttleA-HD2000-x86-Debug-GDI

Original issue's description:
> Use SkPaint::getFontBounds() for text bounding boxes in pictures.
>
> Now that SkTextBlobs have landed, this is a perf no-op, but it at least lets us eliminate a bunch of questionable hacks.
>
> CQ_EXTRA_TRYBOTS=client.skia:Test-Win7-ShuttleA-HD2000-x86-Debug-Trybot,Test-Mac10.8-MacMini4.1-GeForce320M-x86_64-Debug-Trybot
>
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/02d2b9831579173e783569530ab7bae08de907e9

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

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

Revert of Change desktop and ES 3.0 to always use sized internal texture formats. (patchset #2 id:20001 of https://codereview.chromium.org/806943002/)

Reason for revert:
Failing GMs on Mac, Windows and Ubuntu12.

Original issue's description:
> Change desktop and ES 3.0 to always use sized internal texture formats.
>
> Committed: https://skia.googlesource.com/skia/+/bc02bf0ee4221604796cd6d0394ca3af60c0a579

TBR=bsalomon@google.com
NOTREECHECKS=true
NOTRY=true

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

Use SkPaint::getFontBounds() for text bounding boxes in pictures.

Now that SkTextBlobs have landed, this is a perf no-op, but it at least lets us eliminate a bunch of questionable hacks.

CQ_EXTRA_TRYBOTS=client.skia:Test-Win7-ShuttleA-HD2000-x86-Debug-Trybot,Test-Mac10.8-MacMini4.1-GeForce320M-x86_64-Debug-Trybot

BUG=skia:

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

Change desktop and ES 3.0 to always use sized internal texture formats.

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

rebase gms and update ignore

NOTRY=True
TBR=
BUG=skia:

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

whitespace change to context to force rebuild of skia

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

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

This change will ultimately pull uniform color, and to a much lesser degree uniform coverage, into GPs.  There are still some loose ends because drawstate has the ability to override the GP, but fixing these cleanly will have to wait until we have deferred geometry in place and can make attribute / uniform decisions on the fly.

BUG=skia:

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

Move all non-program cache functions out of GrGpuGL_program.cpp

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

pass dst-rect through from drawBitmapRect for better precision

BUG=skia:

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

Look at DYNAMIC_ANNOTATIONS_ENABLED now.

Defining SK_DYNAMIC_ANNOTATIONS_ENABLED as 1 whenever DYNAMIC_ANNOTATIONS_ENABLED was 1
seems to be working fine for Chrome.  Should be we can just use DYNAMIC_ANNOTATIONS_ENABLED.

BUG=skia:

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

add paint::getFontBounds()

BUG=skia:

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

Add missed convexpaths.

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

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

Delete old bench expectation code.

Just noticed these were still here.

BUG=skia:

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

devirtualize some functions on GrGpu

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

Rolling libwebp broke our iOS builds.  Silence warnings instead.

BUG=skia:

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

There can be only one (SkRandom)!

Remove SkLCGRandom. We already decided the new one was better, which is
why we wrote the new SkRandom.

Convert GMs that were using SkLCGRandom to use the improved SkRandom.
Motivated by the fact that these GMs draw differently on some runs. We
believe this to be a result of using the old SkLCGRandom.

Add each of the tests that were using SkLCGRandom to ignore-tests.txt,
since we expect they'll draw differently using SkRandom.

Move a trimmed down version of SkLCGRandom into SkDiscretePathEffect.
In order to preserve the old behavior, trim down SkLCGRandom to only
the methods used by SkDiscretePathEffect, and hide it in
SkDiscretePathEffect's cpp file.

BUG=skia:3241

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

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