In SkPDFDocumentToBitmap, use SkCreateDataProviderFromStream

CGDataProviderCreateWithData to SkCreateDataProviderFromStream in
SkPDFDocumentToBitmap

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

Fix ability to capture archives in webpages_playback.py

Updated all page sets to match the file name. This is to make it easier to call record_wpr with the page set class name.
Also updated the wait time for baidu to 60 seconds.

BUG=skia:3049

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

Use MacOS's CoreGraphics PDF Rasterizer in DM

BUG=3061

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

Update SKP version

Automatic commit by the RecreateSKPs bot.

TBR=

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

add gm: colorwheel

BUG=skia:3061

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

add saveLayer to lua

BUG=skia:
TBR=

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

Move .js files to their own directory

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

bug fix for k1
TBR=
BUG=skia:

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

Add gpu support for glVertexAttrb1f, 2fv, and 3fv

BUG=skia:

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

Add `SkIRect bounds()` convenience method to SkImageInfo and SkBitmap.

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

Update GM expectations for https://codereview.chromium.org/677463002/

BUG=skia:2935
TBR=mtklein@google.com

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

Doing the "using namespace" penance by adding in v8:: everywhere, like I should have to begin with.

Also:
  - SkWindow now has createSurface, not createCanvas.
  - Add the platform init code v8 now seems to require.
  - Fix library linkage.
  - Call isolate->Enter(); because it doesn't look
    like v8 starts with a default isolate to begin with.

BUG=skia:

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

Delete unused method aliases and texture flag.

Committed: https://skia.googlesource.com/skia/+/7fdffe41494cbd78a8ee2a6ef0509e46ec77039f

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

When bmp is not cached don't upload as index8 format.

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

Add some more volatile paths.

Some clip paths were not marked as volatile, and ending up in the
distance field path renderer when they shouldn't.

BUG=skia:3066

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

tools: Remove reference to svndiff.

svndiff script was archived by commit
https://skia.googlesource.com/skia/+/70242ace3b2ed06b33ee5ea5c9ac3aedea89b194

R=borenet@google.com

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

Added varying struct

TBR=
BUG=skia:

Committed: https://skia.googlesource.com/skia/+/852ae80b9c3c6fd53f993ac35133d80863993cbe

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

move c api into include

BUG=skia:

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

Remove a pointless use of SkWeakRefCnt.

Can't quite get rid of SkWeakRefCnt yet... SkFontMgr_indirect uses it to cache
SkTypefaces, and I don't quite understand it enough yet to cut out the weak refs.

BUG=skia:3065

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

Print GPU cache stats in nanobench/dm with veryVerbose

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

Add harfbuzz to DEPS

https://skia.googlesource.com/third_party/harfbuzz/

BUG=skia:3067

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

Ignore simpleaaclip_path on Win7-gpu for now.

BUG=skia:3066
TBR=mtklein@google.com

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

Build gpu stats tracking in dev builds.

Fix cache stats, add more info.

BUG=skia:2889

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

add color wheel to validate decoding colors

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

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

Revert of Added varying struct (patchset #9 id:160001 of https://codereview.chromium.org/671023002/)

Reason for revert:
may have caused gm change on arm

Original issue's description:
> Added varying struct
>
> TBR=
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/852ae80b9c3c6fd53f993ac35133d80863993cbe

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

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

Add isScale() helper function to SkMatrix44.

This will be used later in Chromium to cleanup gfx::Transform::IsScale2d().

BUG=408710,skia:997
TEST=None
R=bsalomon@google.com,danakj@chromium.org

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

rebaseline for new shaderproc

BUG=skia:

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

Added varying struct

TBR=
BUG=skia:

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

Rebaseline lots of images, associate some with bug 3063.

TBR=
NOTRY=true

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

Set temporary paths volatile so we don't cache them.

Any path that is generated frame-to-frame should not be rendered by using the
DistanceFieldPathRenderer, because generating the initial distance field,
uploading it and rendering it takes longer than the SoftwarePathRenderer.

BUG=skia:2935

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

Add test for new FrontBufferedStream behavior.

Test for https://skia.googlesource.com/skia/+/dd5a1e094c19fa10202c37c50a1f799e5af5dac0

Verify that FrontBufferedStream does not attempt to read beyond the
end of its underlying stream.

Make SkStreamToCGImageSource handle an empty stream better.

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

Don't read random data in ICO check.

Check to ensure SkStream::read succeeds before checking the result.

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

Re-add in v8 to DEPS.

BUG=skia:

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

Update SKP version

Automatic commit by the RecreateSKPs bot.

TBR=

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

Possible optimization to SkString

BUG=skia:

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

Cleanup: Turn GrReducedClip into a class with a static function.

Clean up namespace usage.

Similar to what was done in
https://skia.googlesource.com/skia/+/a5414c4a8efc3119ee20fcee96c0bf68a04909c7

BUG=None
TEST=None
R=bsalomon@google.com

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

Add flag to skimage for decoding into unpremul.

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

squelch filterindiabox too

TBR=bsalomon@google.com

BUG=skia:

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

Fix imagemagnifier GM quality and speed.

The imagemagnifier GM was applying the filter both to a saveLayer()
restore() around the whole scene, as well as to each individual
text draw.

Applying the filter only in the saveLayer() makes it run ~20x faster
on my Linux box, and the quality is improved as well (since the
primitives are not double-filtered).

BUG=skia:

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

Revert "Revert of create shaderproc for nofilter-opaque-dx (patchset #7 id:120001 of https://codereview.chromium.org/664783004/)"

This reverts commit 430b795cc8a1cdbddd8fdc5511a3a523348937f7 and adds suppressions.

BUG=skia:

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

Don't use MIP filter mode on compressed textures

Add compressed texture image downsample GMs

BUG=chromium:426331

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

Remove test expectations from ignored-tests.txt.

TBR=mtklein@google.com
NOTRY=true
BUG=skia:

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

Adding an option to render only the shadow in SkDropShadowImageFilter

This is basically how blink uses the filter. Currently, I can't use it for "ShadowOnly" mode with the filter at all, but instead of copying the code and risking to have the codepaths diverge, I'm simply going to add the option here.

BUG=skia:

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

Rebaselines for tests affected by image filter PDF change.

(aka https://codereview.chromium.org/644323006/)

TBR=mtklein@google.com
NOTRY=true

BUG=skia:

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

Add utils to better quantize grayscale values to three bit indices while
compressing coverage masks.

Signed-off-by: Pavel Krajcevski <pavel@cs.unc.edu>
BUG=skia:

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

add float equivalent in dumpHex output as comment

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

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

Revert of create shaderproc for nofilter-opaque-dx (patchset #7 id:120001 of https://codereview.chromium.org/664783004/)

Reason for revert:
Many GMs fixed.  Needs rebaseline, perhaps layout test rebaselines.

Original issue's description:
> create shaderproc for nofilter-opaque-dx
>
>
> speedup nofilter
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/a40a276bcee2246439dcf816273c1307f5c3c69f

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

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

use SkMaskCache inside SkBlurMaskFilter

depends on https://codereview.chromium.org/670063004/

BUG=skia:

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

Correct typo in SkRemotableFontMgr_win_dw.cpp.

New code was introduced behind a flag, but in that code there is a comma
before a parameter name instead of after.

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

create shaderproc for nofilter-opaque-dx

speedup nofilter

BUG=skia:

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

Revert of Add test for new FrontBufferedStream behavior. (patchset #1 id:1 of https://codereview.chromium.org/641813009/)

Reason for revert:
Test is SkASSERTing on Macs, e.g:

http://chromegw.corp.google.com/i/client.skia/builders/Test-Mac10.6-MacMini4.1-GeForce320M-x86_64-Debug/builds/257/steps/dm/logs/stdio

210 tasks left 1737M peak     1ms test FrontBufferedStream../../src/ports/SkImageDecoder_CG.cpp:43: failed assertion "data"

Signal 11:
_sigtramp (+0x1a)
SkStreamToCGImageSource(SkStream*) (+0x62)
SkImageDecoder_CG::onDecode(SkStream*, SkBitmap*, SkImageDecoder::Mode) (+0x2e)
SkImageDecoder::decode(SkStream*, SkBitmap*, SkColorType, SkImageDecoder::Mode) (+0x81)
SkImageDecoder::DecodeStream(SkStreamRewindable*, SkBitmap*, SkColorType, SkImageDecoder::Mode, SkImageDecoder::Format*) (+0xff)
SkImageDecoder::DecodeStream(SkStreamRewindable*, SkBitmap*) (+0x31)
test_ShortFrontBufferedStream(skiatest::Reporter*) (+0x97)
skiatest::ShortFrontBufferedStreamClass::onRun(skiatest::Reporter*) (+0x19)
skiatest::Test::run() (+0x7c)
DM::CpuTestTask::draw() (+0x5a)
DM::CpuTask::run() (+0x9e)
non-virtual thunk to DM::CpuTask::run() (+0x1c)
(anonymous namespace)::ThreadPool::Loop(void*) (+0xf4)
thread_start(void*) (+0x54)
_pthread_start (+0x14b)

Original issue's description:
> Add test for new FrontBufferedStream behavior.
>
> Test for https://skia.googlesource.com/skia/+/dd5a1e094c19fa10202c37c50a1f799e5af5dac0
>
> Verify that FrontBufferedStream does not attempt to read beyond the
> end of its underlying stream.
>
> Committed: https://skia.googlesource.com/skia/+/da59f05c6738dbb9a92cad21c608cdfae53a76b2

TBR=reed@google.com,scroggo@google.com
NOTREECHECKS=true
NOTRY=true

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

Allocate only the vertices we need for text.

This restructures the vertex allocation for text rendering to compute
the max number of vertices we would need for a line of text, and then
only allocate that much. If this number exceeds the quad index limit,
then it will allocate for the max number of quads, and reallocate for
the rest later.

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

updating k1 expectations

TBR=
BUG=skia:

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

Removed simple tests steps from CanvasTest.

CanvasTest only checks if the state of the different canvas
implementations are consistent with the one of the default
implementation.

Because most of the draw calls are stateless and already tested in GMs
the test steps that exercices them are not useful.

BUG=skia:3054

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

Remove android specific genID from SkPaint.

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

Add test for new FrontBufferedStream behavior.

Test for https://skia.googlesource.com/skia/+/dd5a1e094c19fa10202c37c50a1f799e5af5dac0

Verify that FrontBufferedStream does not attempt to read beyond the
end of its underlying stream.

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

This return_false is now only used on Android.  Squelch a warning.

BUG=skia:

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

Rename Nexus9 expectations

BUG=skia:3047
TBR=djsollen

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

Add SkMaskCache

BUG=skia:

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

Revert of Delete unused method aliases and texture flag. (patchset #1 id:1 of https://codereview.chromium.org/672753002/)

Reason for revert:
Need to remove one more reference to this in blink before relanding.

Original issue's description:
> Delete unused method aliases and texture flag.
>
> Committed: https://skia.googlesource.com/skia/+/7fdffe41494cbd78a8ee2a6ef0509e46ec77039f

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

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

Update fontMgr to take list of bcp47 language tags.

This will enable clients to pass more than one bcp47 tag to ensure
that the most appropriate font is selected.

BUG=chromium:422180

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

Delete unused method aliases and texture flag.

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

fix memory leak

TBR=
BUG=skia:

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

some more expectations, update ignores

TBR=
BUG=skia:

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

New baselines
TBR=

BUG=skia:

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

one more test to ignore

TBR=
BUG=skia:

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

Oval and stroke AA rect now batch

BUG=skia:

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

Committed: https://skia.googlesource.com/skia/+/18055afb838a278b5a8436cd51dbfbb688e1e0a0

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

Revert of Oval and stroke AA rect now batch (patchset #7 id:110001 of https://codereview.chromium.org/664193002/)

Reason for revert:
whoops, accidental commit

Original issue's description:
> Oval and stroke AA rect now batch
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/bc54fab1a4e5b51628a8c1557c62bc74e078870d
>
> Committed: https://skia.googlesource.com/skia/+/18055afb838a278b5a8436cd51dbfbb688e1e0a0

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

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

Oval and stroke AA rect now batch

BUG=skia:

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

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

Revert of Oval and stroke AA rect now batch (patchset #7 id:110001 of https://codereview.chromium.org/664193002/)

Reason for revert:
Many GMs failing, e.g. http://build.chromium.org/p/client.skia.android/builders/Test-Android-Nexus5-Adreno330-Arm7-Release/builds/95/steps/Compare%20GMs/logs/stdio

Original issue's description:
> Oval and stroke AA rect now batch
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/bc54fab1a4e5b51628a8c1557c62bc74e078870d

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

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

Oval and stroke AA rect now batch

BUG=skia:

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

Add SkTypeface::getBounds()

mirrored from https://codereview.chromium.org/666303002/

BUG=skia:
TBR=

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

fix some bit-rot in the ios port of sampleapp

BUG=skia:

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

Qualify the return value of SkImageDecoder::decode

Add a new enum to differentiate between a complete decode and a
partial decode (with the third value being failure). Return this
value from SkImageDecoder::onDecode (in all subclasses, plus
SkImageDecoder_empty) and ::decode.

For convenience, if the enum is treated as a boolean, success and
partial success are both considered true.

Note that the static helper functions (DecodeFile etc) still return
true and false (for one thing, this allows us to continue to use
SkImageDecoder::DecodeMemory as an SkPicture::InstallPixelRefProc in
SkPicture::CreateFromStream).

Also correctly report failure in SkASTCImageDecoder::onDecode when
SkTextureCompressor::DecompressBufferFromFormat fails.

BUG=skia:3037
BUG:b/17419670

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

webtry: Improve error handling/message when creating sqlite tables.

In case they fail, lets fail with an ERROR message rather than an INFO,
and lets make the error message a little bit better.

BUG=None
TEST=None
R=jcgregorio@google.com

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

explicitly size fType in SkRRect, to ensure the are no uninitialized bytes
(so it can be used as a key in a cache)

BUG=skia:

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

SkResourceCache::Key namespace support.

Add a unique-per-subclass namespace tag to make Keys from different
domains comparable.

Also drop the SkPictureShader cache and convert to using the global
resource cache instead.

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

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

Override SkCanvas::drawImage() in SkDeferredCanvas and SkGPipe

Depend on https://codereview.chromium.org/663233002

BUG=skia:2947

Committed: https://skia.googlesource.com/skia/+/687732fe046c9cfec940f2d7f661cd97651d10a8

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

GP Emit Code takes a struct

BUG=skia:

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

More genericity: overload isPaintOpaque(SkPaint, SkBitmap)

Instead of taking a Bitmap as an argument this version
takes only the type of content (None/Opaque/Transparent).
This will be used to check the opaqueness of a SkPaint
that draws a SkImage.

BUG=skia:3042

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

Use correct size for font style name search.

This fixes an out of bounds read introduced with
a4c4a2d8cd65abb1e5ac20813831cdb9ace6c7ee
"Replace SkTypeface::Style with SkFontStyle."

BUG=chromium:425856

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

Picture shader resource caching.

Replace the current/naive shader caching mechanism with a more general
implementation based on SkResourceCache.

Caching the bitmap shader itself (as opposed to just the tile bitmap)
makes for a chunkier key, but OTOH avoids allocating new shaders on
cache hit.

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

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

Ignore failure on internal bot

BUG=skia:3023
TBR=egdaniel@google.com

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

Add back page_sets/skia_googlespreadsheet_desktop

Absolutely could not get the old page to work so I just created a new page.

NOTRY=true
BUG=skia:3008

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

Track nested picture xform state for layer hoisting

The accumulated matrix state of any enclosing SkPictures must be stored separate from the picture-local CTM. Any setMatrix calls inside a layer need to replace the picture-local CTM but concatenate with the enclosing SkPicture transform state (and the transform state needed to translate the layer to the correct location in the cached GrTexture).

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

Update webpages_playback script and all page_sets to use new skpicture_printer benchmark

The purpose of this CL is to turn the RecreateSKPs bot green after
https://codereview.chromium.org/665193002/ lands.

Description of changes in this CL:
* After https://codereview.chromium.org/665193002/ lands we will no longer need to copy the archives and pagesets to telemetry's data directory, I removed the code that did this in https://codereview.chromium.org/655313003/
* page_sets now need unique class names. Updated all page_sets to have unique class names.
* Using the new --page_set_name and --page_set_base_dir and --also-run-disabled-tests parameters

BUG=skia:3046

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

Bug fix for es 3.00 fb fetch

patch for es 300 shader fb fetch

BUG=skia:

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

add round/ceil/etc. for SkMScalar

BUG=skia:
TBR=

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

Draw SKPs in 256x256 tiles in nanobench.

(This CL will certainly trigger performance regression alerts.  Tiled drawing is slower than non-tiled drawing.)

BUG=skia:

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

Implement maxrss for Windows

No more -1 in the first column on Windows!

E.g. http://build.chromium.org/p/client.skia/builders/Perf-Win8-ShuttleA-HD7770-x86_64-Release-Trybot/builds/0/steps/nanobench/logs/stdio

BUG=skia:

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

SkLazyPtr follow ups

 - moves test to LazyPtrTest.cpp
 - adds the ability to use a Create() method with an argument

BUG=skia:

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

Widen the set of nested rects that can be drawn natively on the GPU

GrAARectRenderer::fillAANestedRects only handles thin stroked rects correctly if the margins are all the same. It is also correct if all the margins are >= 1.0f. This CL allows such cases to use the fillAANestedRects fast path.

This seems to halve the gpu rendering time of the picture in bug crbug.com/425427.

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

small tweaks to SkResourceCache::Key
  - remove unused method and obsolete note
  - add a reminder that yes, we are checking counts are equal

BUG=skia:

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

use - and = to zoom text in slides

BUG=skia:
TBR=
NOTRY=True

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

Fix debugger's SaveAs functionality

Without this patch the SaveAs functionality only copies the portion of the picture from the current command in the GUI.

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

Check SkStream::isAtEnd before attempting to read.

In FrontBufferedStream, when attempting to read and buffer more
data, do not try to read if the underlying stream is at the end.

Prevents a bug where we continue trying to read a stream that
has nothing more to read.

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

Add check to see if we need to remove fixed funciton VA in OptDrawState

BUG=skia:

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

Rebaseline dftext GM for new color emoji test.

BUG=skia:3033
TBR=mtklein@google.com

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

Clean up some warnings about unused code.

BUG=skia:

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

add gradients and drawPaint to lua

BUG=skia:
TBR=

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