remove lcd

TBR=djsollen@google.com

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

Update DM to respect blacklists for tests

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

Revert of Import functionality for new GL backend (patchset #2 id:20001 of https://codereview.chromium.org/1185573003/)

Reason for revert:
breaking multiple builds. For example...

https://uberchromegw.corp.google.com/i/client.skia/builders/Test-Ubuntu-GCC-Golo-GPU-GT610-x86_64-Debug-ASAN/builds/667

Original issue's description:
> Import functionality for new GL backend
>
> Brings in the following functionality:
>
>   ARB_draw_instanced
>   ARB_instanced_arrays
>   NV_bindless_texture
>   EXT_direct_state_access
>   KHR_debug
>
> Also cleans up some of the NVPR extension loading.
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/202c4c75fb806f751cf796850bf67fe9793e0643

TBR=bsalomon@google.com,cdalton@nvidia.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:

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

make emptypath portable; add 565 color utility

R=djsollen@google.com

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

Stop running SKPs in DM on the bots.

We're not triaging them, and we get the same coverage (both Release and Debug)
from nanobench.

BUG=skia:

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

Print command to be executed in verbose mode

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

Import functionality for new GL backend

Brings in the following functionality:

  ARB_draw_instanced
  ARB_instanced_arrays
  NV_bindless_texture
  EXT_direct_state_access
  KHR_debug

Also cleans up some of the NVPR extension loading.

BUG=skia:

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

Disable failing GPU tests on Nexus 6.

BUG=skia:3942

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

update complexclip to portable

TBR=djsollen@google.com

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

Fixing stripe test

I originally thought that there was no harm in reading or skipping
zero lines after we have already reached the end of the image.

However, once we reach the end of the image, onFinish() is
automatically called.  Performing a read or a skip after
the call to onFinish() is invalid and will cause onFinish()
to be called a second time (which is also invalid).

Seems like the code requires good behavior and the test is
wrong.

BUG=skia:

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

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

Implement support for mixed sampled render targets

Adds a new FBO type kStencil_MSFBOType that is selected whenever
NV_framebuffer_mixed_samples extension is available.  In this new
FBO type a non-msaa color buffer is created with a multisampled
stencil buffer attachment.

Replaces numSamples() with separate numColorSamples and numStencilSamples
methods in RenderTarget.

In mixed samples mode non-MSAA codepaths are used to draw simple shapes,
while NVPR-rendered paths and text are rendered with a multisampled
stencil.

BUG=skia:3177

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

convert cubicpaths gm tests to portable

TBR=djsollen

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

Remove ambiguity of SkJpegUtility name.

We have two, one in images/, the other in codec/.  As codec's the new hotness,
I left it's name vanilla and suffixed the old one.

BUG=skia:

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

Don't leak fragment processor in SkColorFilter::asFragmentProcessors implementations.

TBR=reed@google.com

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

Move blend enums into GrBlend.h

Moves blend enums from GrXferProcessor.h to GrBlend.h, makes GrBlend.h
public.

BUG=skia:

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

fix portable text in test

TBR=djsollen@google.com

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

SkiaLab doc: add another step for new Android devices

BUG=skia:3718
NOTRY=true
DOCS_PREVIEW= https://skia.org/?cl=1185663003

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

change Resize to take pixmap for src

BUG=skia:

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

remove unused private Resizer

BUG=skia:
TBR=

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

next batch of portable typeface changes

TBR=djsollen@google.com

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

DM testing for skipping and scaling

BUG=skia:

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

fix deserialization after FirstDirection change to paths

BUG=skia:

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

Update README.chromium.

Capitalize Skia and use new site url.

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

Fix for dashing shader on Mali: use highp where necessary.

The varyings used to compute rect coverage need to be high precision,
since they're offset by the frag coord.

R=egdaniel

BUG=skia:3935

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

Eliminate the check for ANDROID_LARGE_MEMORY in jpeg decoders

BUG=skia:1282

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

remove duplicate assign of fLastGlyphID

Hal found a brain-dead bug on my part. Thanks Hal!

TBR=halcanary@google.com

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

PDF expects fonts to map glyph ID 0 to the missing character.
Add a mapping of glyph ID 0 to the toy font to allow for this.

TBR=bungeman@google.com

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

retro add copyright

BUG=skia:
NOTRY=True
TBR=

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

initialize font metrics for pdf (found by valgrind)

TBR=halcanary@google.com

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

Init GrGLSLCaps fields from inside GrGLCaps

Fixes circular dependencies by allowing the initialization to be done
intertwined.

BUG=skia:

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

SkPath::Direction serves two masters:
- input param to addFoo (e.g. addRect), where only CW or CCW are valid)
- output param from computing functions, that sometimes return kUnknown

This CL's intent is to split these into distinct enums/features:
- Direction (public) loses kUnknown, and is only used for input
- FirstDirection (private) is used for computing the first direction we see when analyzing a contour

BUG=skia:

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

fix broken build by not passing the verbose flag through

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

Remove verbose bash logging

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

Added GrAAFlatteningConvexPathRenderer.

This is an alternate version of GrAAConvexPathRenderer which handles
curves by first flattening them to straight lines.

BUG=skia:

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

Label some unused static vars as SK_UNUSED

Makes my gcc 5.1.1 much happier.

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

[ mtklein mischief below here ]
No public API changes.
TBR=reed@google.com

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

turn on antialiased text

Aliased text didn't work with msaa16, so make it all
antialiased.

TBR=reed@google.com

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

Today's gm include many differences caused by platform font implementations.
This experiment replaces the label used in the aaxfermodes gm with
aliased text generated from paths common to all platforms.

Since there is no way today to generate all dm output from trybots,
this will be checked in to confirm that this strategy provides simpler
output across devices.

This does not introduce a new public interface; instead, dm uses
a extern backdoor to install the SkTypeface::CreateFromName
handler.

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

Update flags passed to Android GPU bots to exclude running skps and image sources.

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

Remove overly-promiscuous SkNx syntax sugar.

I haven't figured out a pithy way to have these apply to only classes
originating from SkNx, so let's just remove them.  There aren't too
many use cases, and it's not really any less readable without them.

Semantically, this is a no-op.

BUG=skia:

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

Towards removing getTexture() on SkImage

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

Use SkMScalar as float by default to match Chromium.

Tweak some test values to pass with floats.

As expected, this regresses matrix44_setconcat_general by about 2x.

BUG=skia:

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

Subset decoding benchmarks

It was my goal to create benchmarks that could measure all
of the use cases that we have identified.  I think single
subsets, translating, and scaling are the important ones.

It might be a good idea to discuss the document in greater
detail as well.  I just wanted to share this to aid the
discussion.
https://docs.google.com/a/google.com/document/d/1OxW96GDMAlw6dnzNXmiNX-F9oDBBlGXzSsgd0DMIkbI/edit?usp=sharing

BUG=skia:

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

add copyright

BUG=skia:
NOTRY=True
TBR=

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

Add android native_app_glue to platform_tools/android/third_party

BUG=skia:

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

Revert[2] SkDraw and all Blitters to use pixmap instead of bitmapi

This reverts commit b3f0ec9f9967da2f80f0d842cb7fd53617b48de3.

BUG=skia:

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

A bunch of little SkMScalar-as-float build fixes.

Landing this before http://crrev.com/1169813006 so the test changes needed
there are clearer.  This just lets things build with floats, and is a no-op for
doubles.

No public API changes we didn't intend to already have...
TBR=reed@google.com

BUG=skia:

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

Fix build for UCLIBC platforms (newlocale/freelocale)

UCLIBC does not have locale support by default.

BUG=skia:

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

Pump up matrix44_setconcat benches 1000x so they can be timed on Android.

BUG=skia:

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

Fill capHeight and maxCharWidth on Mac.

BUG=skia:3924

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

remove subclassing from ImageFilter::Proxy

BUG=skia:

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

remove Proxy entrypoints for canHandleFilter

BUG=skia:

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

Split SetConcatMatrix44Bench to cover both internal paths.

The existing bench only tests the fast path, but we're looking to speed
up the general case.  It'd be nice to be able to measure that speedup.

BUG=skia:

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

Revert of change SkDraw and all Blitters to use pixmap instead of bitmap (patchset #6 id:100001 of https://codereview.chromium.org/1148793007/)

Reason for revert:
speculative revert to try to unblock DEPS roll

Original issue's description:
> change SkDraw and all Blitters to use pixmap instead of bitmap
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/c31af44336f5eb4a50e83e76e51962d46c3ed458

TBR=scroggo@google.com,jvanverth@google.com,reed@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:

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

Add mixed samples support to XPs

BUG=skia:

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

Restore support for 10.6 SDK in header.

Must use ApplicationServices, can't use CoreText directly.

Remove attempted bug mitigation in Mac typeface.

An attempt was made to hold onto the 'sbix' table when it existed
when drawing to mitigate an issue with vanishing font data. This
code has not affected the crash rate, so remove it.

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

Fix regression in GrXferProcessor

Fix segfault introduced in https://codereview.chromium.org/1170553002/

BUG=skia:

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

Provide tag-along for SkCreateTypefaceFromCTFont.

This provides a means of keeping an object alive for the entire
lifetime of an SkTypeface.

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

SkPDF: fix name escape code

BUG=494913

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

Apply coverage in XP base class when using dst reads

Moves the coverage logic into GrGLXferProcessor for XPs that perform
dst reads. XPs that don't use a dst read are still responsible to
handle coverage on their own.

BUG=skia:

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

change SkDraw and all Blitters to use pixmap instead of bitmap

BUG=skia:

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

Add benjaminwagner@ to CQ_COMMITTERS.

BUG=skia:
NOTRY=true

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

Revert "Revert of move erase into SkPixmap (patchset #1 id:1 of https://codereview.chromium.org/1161033005/)"

This reverts commit cca9b3ac908cf3e9a0c5774a5ffd118d8429e08a.

BUG=skia:
TBR=

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

Revert of move erase into SkPixmap (patchset #1 id:1 of https://codereview.chromium.org/1161033005/)

Reason for revert:
broken builds

Original issue's description:
> move erase into SkPixmap
>
> harvested from https://codereview.chromium.org/1148793007
>
> BUG=skia:
> TBR=
>
> Committed: https://skia.googlesource.com/skia/+/0963f5dab079627c5523ce6a443af27a33e361f7

TBR=reed@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:

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

move erase into SkPixmap

harvested from https://codereview.chromium.org/1148793007

BUG=skia:
TBR=

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

Cap color index values

In the provided example the color table has 10 entries but some of the pixels in the bitmap overflow. This CL goes through the pixel values and caps them to the max index.

An alternate approach would be to just have the color table always have 256 entries but zero out the unused ones.

BUG=492265

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

Add bigtileimagefilter GM & improvements to toString

This is all the ancillary code from https://codereview.chromium.org/1152553006/ (Fix dst bound reported by SkTileImageFilter).

TBR=reed@google.com, senorblanco@google.com

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

Added CQ config to skia repo

This must land in sync with https://chromereviews.googleplex.com/204307013 and
revert in sync.

R=rmistry@chromium.org, tandrii@chromium.org
NOTRY=true
NOTREECHECKS=true
BUG=chromium:471767

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

Update SKP version

Automatic commit by the RecreateSKPs bot.

TBR=
NO_MERGE_BUILDS

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

add extractSubset and SkAutoPixmapStorage

extracted from larger CL in progress: https://codereview.chromium.org/1148793007

BUG=skia:
TBR=

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

clarify that accessPixels is for write-access, and peekPixels is RO

BUG=skia:
TBR=

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

Add cmap 3,0 (symbol) as Unicode fallback.

FreeType automatically selects the 'most Unicode' cmap available,
otherwise the default cmap is NULL. When this happens the fallback
should include the 3,0 symbol cmap, as is outlined in the Apple
TrueType documentation. This cmap should effectively be a Unicode
mapping, but exclusively in the private use area.

BUG=skia:1873,chromium:489452

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

Opt for advanced blend at creation time in CrCustomXfermode

Modifies GrCustomXfermode to decide at creation time whether it will
use advanced blend, instead of deciding during getOptimizations.

BUG=skia:

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

delete (unused) TransparentShader

BUG=skia:
TBR=

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

Add ColorMatrix filter tests from Android CTS.

BUG=skia:3848

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

Update how we package skia resource for android testing

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

use pixmaps for dst in sprites -- NO BITMAPS

BUG=skia:
TBR=scroggo@google.com

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

Remove memcmp from GrProgramDesc op== and Less

Works around an ASAN complaint.

BUG=skia:3891

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

Presubmit: add PRESUBMIT=false to non-master branch changes

BUG=skia:

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

Stop running tiles_rt-gpu on the bots

BUG=skia:

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

Remove unused PackBits methods and fix length checks

Also a bit of general cleanup.

BUG=chromium:486944

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

Add prelog flag to DM

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

switch bitmapshader internals over to pixmap

BUG=skia:
NOTRY=True

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

Add documentation for SkiaLab

BUG=skia:3850
NOTRY=true
DOCS_PREVIEW= https://skia.org/?cl=1154623006

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

Disable unused attributes when using copy shader

BUG=chromium:496540

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

Revert of Fix dst bound reported by SkTileImageFilter (patchset #3 id:40001 of https://codereview.chromium.org/1152553006/)

Reason for revert:
Blink

Original issue's description:
> Fix dst bound reported by SkTileImageFilter
>
> In the example from the bug we had the filter DAG:
>
> color filter (table)
>     0: xfermode filter (arith)
>         0: tile filter [0,80,34,114] -> [0,80,800,480]
>              0: color filter (table)
>                  0: bitmap src 34x34 -> [0,80,34,114]
>         1: color filter (table)
>              0: picture filter [0, 80, 800, 480]
>
> computeFastBounds was coming out of the DAG with a bound of [0,80,34,114] which didn't represent the pixels that would be drawn.
>
> This CL updates SkTileImageFilter to correctly set the bound for the pixels it will hit.
>
> BUG=493783
>
> Committed: https://skia.googlesource.com/skia/+/05be93bbdf09576f7903130e3b106b0a8c7c4b4e
>
> Committed: https://skia.googlesource.com/skia/+/0be685755f942baea26c66a87226b569fc17e960

TBR=reed@google.com,senorblanco@google.com,senorblanco@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=493783

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

make test hittestpath random calls deterministic

R=bsalomon@google.com

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

Trivial change to bump minimum block size for Buffer pool

TBR=
BUG=skia:

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

Disable dual source blending support when GLSL version is too old

BUG=skia:3891

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

Fallback bitmap devices should observe the computed pixel geometry

When onCreateDevice() fails, internalSaveLayer() attempts to create a
fallback bitmap device instead - but it ignores the computed pixel geometry
in this case.

BUG=496054
R=reed@google.com

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

Remove SK_ANDROID_FREETYPE_HAS_MM from build.

This flag is now no longer used now that GX variations now seem
to have landed for good.

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

Add GM that tests clipping annotations

TBR=reed@google.com

BUG=skia:3880

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

Fix dst bound reported by SkTileImageFilter

In the example from the bug we had the filter DAG:

color filter (table)
    0: xfermode filter (arith)
        0: tile filter [0,80,34,114] -> [0,80,800,480]
             0: color filter (table)
                 0: bitmap src 34x34 -> [0,80,34,114]
        1: color filter (table)
             0: picture filter [0, 80, 800, 480]

computeFastBounds was coming out of the DAG with a bound of [0,80,34,114] which didn't represent the pixels that would be drawn.

This CL updates SkTileImageFilter to correctly set the bound for the pixels it will hit.

BUG=493783

Committed: https://skia.googlesource.com/skia/+/05be93bbdf09576f7903130e3b106b0a8c7c4b4e

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

Fix edge-line artifacts issue on Mali

Some GMs show edge-line artifacts on Mali at the bottom/right of the canvas.

Change the 'rtAdjustment' uniform precision from 'midiump' to 'highp' if the GPU vendor is ARM.

NOTE: Vertex Shader should have default 'highp' precision, but now we explicit declared some uniform/attribute precision as 'midiump'.

This patch should fix these issues:

Issue 3693: Some GMs produce random artifacts on Mali in MSAA4
    https://code.google.com/p/skia/issues/detail?id=3693

Issue 3685: srcmode GM has top/right line artifacts on Mali T604 in MSAA4
    https://code.google.com/p/skia/issues/detail?id=3685

BUG=skia:

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

refactor bitmapshader to use a controller

BUG=skia:

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

remove stray printf

BUG=skia:
TBR=junov@google.com

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

Remove SK_FREETYPE_HAS_MM.

The SK_FREETYPE_HAS_MM define was added to delay the changes to
support GX variation fonts on platforms which did not compile ftmm.c
into their FreeType build. Now that all builds should have this file
the define can be removed.

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

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

Reenable reordering

TBR=
BUG=skia:

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

Begin tracking driver info in GrGLContextInfo

BUG=skia:

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