History log of /external/skia/src/gpu/GrTest.cpp
Revision Date Author Comments (<<< Hide modified files) (Show modified files >>>)
5ec26ae9bfca635ccc98283aad5deda11519d826 25-Feb-2016 bsalomon <bsalomon@google.com> Move Budgeted enum out of SkSurface, use in GrTextureProvider

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1728093005
DOCS_PREVIEW= https://skia.org/?cl=1728093005

Committed: https://skia.googlesource.com/skia/+/57599fe6c0336feaeeeb9b1996e77b70219b483c
CQ_EXTRA_TRYBOTS=client.skia.compile:Build-Ubuntu-GCC-x86_64-Release-CMake-Trybot,Build-Mac-Clang-x86_64-Release-CMake-Trybot

Review URL: https://codereview.chromium.org/1728093005
/external/skia/src/gpu/GrTest.cpp
bd500f09ce5b1ca464431d693f9ec5da60a59230 25-Feb-2016 bsalomon <bsalomon@google.com> Revert of Move Budgeted enum out of SkSurface, use in GrTextureProvider (patchset #6 id:100001 of https://codereview.chromium.org/1728093005/ )

Reason for revert:
Need workaround for chrome to build

Original issue's description:
> Move Budgeted enum out of SkSurface, use in GrTextureProvider
>
> BUG=skia:
> GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1728093005
> DOCS_PREVIEW= https://skia.org/?cl=1728093005
>
> Committed: https://skia.googlesource.com/skia/+/57599fe6c0336feaeeeb9b1996e77b70219b483c

TBR=reed@google.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:

Review URL: https://codereview.chromium.org/1734043002
/external/skia/src/gpu/GrTest.cpp
57599fe6c0336feaeeeb9b1996e77b70219b483c 25-Feb-2016 bsalomon <bsalomon@google.com> Move Budgeted enum out of SkSurface, use in GrTextureProvider

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1728093005
DOCS_PREVIEW= https://skia.org/?cl=1728093005

Review URL: https://codereview.chromium.org/1728093005
/external/skia/src/gpu/GrTest.cpp
f7b8b8affec91fcfab0d79199e466c16c254fe56 24-Feb-2016 ericrk <ericrk@chromium.org> Add wrapBackendTextureAsRenderTarget API

Skia's GrTextureProvider currently exposes two APIs for wrapping backend
objects:
* wrapBackendTexture - wraps a texture into a GrTexture. Depending on
flags, this GrTexture can be converted to a GrRenderTarget. Skia
manages the render target objects it may create to provide a render
target for the texture. This allows Skia to create stencil buffers
if needed and manager MSAA resolves.
* wrapBackendRenderTarget - wraps a FBO into a GrRenderTarget. This
object cannot be converted to a GrTexture. Skia does not manage
the render target objects for such a GrRenderTarget, and as such
cannot attach stencil buffers or perform MSAA resolves on the
created GrRenderTarget.

Given these two options, wrapBackendTexture provides more versatility
and allows Skia more room for optimization. Chrome currently uses
wrapBackendTexture for this reason.

While these two functions cover most cases, they do not provide a way
for Skia to wrap a texture into a render target (and gain the MSAA and
stencil buffer management), without also creating a GrTexture. This is
problematic in cases where a texture can be bound to a render target,
but cannot be textured from, as is the case in Chrome's limited support
for GL_TEXTURE_RECTANGLE.

To address this, a new function is created:
* wrapBackendTextureAsRenderTarget - wraps a texture into a
GrRenderTarget. As with wrapBackendTexture, the created render
target objects are fully managed by Skia. Unlike wrapBackendTexture
no GrTexture is created, and the created object will never be
textured from.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1709163003

Review URL: https://codereview.chromium.org/1709163003
/external/skia/src/gpu/GrTest.cpp
7023a00c35d904e4ccff09c377e9ba26abba6181 22-Feb-2016 jvanverth <jvanverth@google.com> Use unorm shorts for texture coordinates when rendering text.

There are a couple of reasons for this:
- Vulkan does not guarantee conversions from integral vertex attributes
to floating point shader variables
- This may be faster and more precise on some platforms, as it avoids
the aforementioned conversion and changes a multiply by a very small
value to a multiply by a medium-sized value.
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1713693002
TBR=bsalomon@google.com

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

Review URL: https://codereview.chromium.org/1713693002
/external/skia/src/gpu/GrTest.cpp
d106c2c64012688f8d1439543d942f8f66a142e0 19-Feb-2016 jvanverth <jvanverth@google.com> Revert of Use unorm shorts for texture coordinates when rendering text. (patchset #3 id:40001 of https://codereview.chromium.org/1713693002/ )

Reason for revert:
Causing issues with text on Mali 400s. Examples: largeglyphblur, imageblurtiled. It appears that there are precision problems.

Original issue's description:
> Use unorm shorts for texture coordinates when rendering text.
>
> There are a couple of reasons for this:
> - Vulkan does not guarantee conversions from integral vertex attributes
> to floating point shader variables
> - This may be faster and more precise on some platforms, as it avoids
> the aforementioned conversion and changes a multiply by a very small
> value to a multiply by a medium-sized value.
> GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1713693002
> TBR=bsalomon@google.com
>
> Committed: https://skia.googlesource.com/skia/+/e507ff0460f4f878214b9454fb5b9ab8d64d8063

TBR=joshualitt@google.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

Review URL: https://codereview.chromium.org/1709133003
/external/skia/src/gpu/GrTest.cpp
e507ff0460f4f878214b9454fb5b9ab8d64d8063 19-Feb-2016 jvanverth <jvanverth@google.com> Use unorm shorts for texture coordinates when rendering text.

There are a couple of reasons for this:
- Vulkan does not guarantee conversions from integral vertex attributes
to floating point shader variables
- This may be faster and more precise on some platforms, as it avoids
the aforementioned conversion and changes a multiply by a very small
value to a multiply by a medium-sized value.
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1713693002
TBR=bsalomon@google.com

Review URL: https://codereview.chromium.org/1713693002
/external/skia/src/gpu/GrTest.cpp
ed828001da75f4a3a5d31d9f1c1935e3595ce920 16-Feb-2016 cblume <cblume@chromium.org> There is an unused rowBytes parameter being passed along.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1636873002

Review URL: https://codereview.chromium.org/1636873002
/external/skia/src/gpu/GrTest.cpp
e63ffef6248bd103b5f7827f1e4bc75e47ca9e20 05-Feb-2016 bsalomon <bsalomon@google.com> Remove deferred clear from SkGpuDevice

Add combining to GrClearBatch

Fix issue with state tracking in GrGLGpu::createTestingOnlyBackendTexture

Add tests for clearing GPU SkSurfaces and add tests for GrDrawContext::clear().

Add comment that SkCanvas::flush will resolve the RT in the GPU case.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1658823002

Review URL: https://codereview.chromium.org/1658823002
/external/skia/src/gpu/GrTest.cpp
6121405df5689875c7309c20632a82897fce4127 26-Jan-2016 cblume <cblume@chromium.org> Cleaning up trailing spaces and typos.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1631993002

Review URL: https://codereview.chromium.org/1631993002
/external/skia/src/gpu/GrTest.cpp
de83b41cc7fc5bd1398e06c011e698215261665f 14-Jan-2016 joshualitt <joshualitt@chromium.org> Add Audittrail for path renderers

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1588683002

Review URL: https://codereview.chromium.org/1588683002
/external/skia/src/gpu/GrTest.cpp
f5883a6d75a18c581507214d3057b5c721ed1ffc 13-Jan-2016 joshualitt <joshualitt@chromium.org> Hide drawBatch on GrDrawTarget from test Methods

TBR=bsalomon@google.com
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1574383002

Review URL: https://codereview.chromium.org/1574383002
/external/skia/src/gpu/GrTest.cpp
41e4384ea0208631ab7847266c3369b17ed7fbf3 08-Jan-2016 bsalomon <bsalomon@google.com> Move config texturability/renderability to config table.

GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1563443002

Committed: https://skia.googlesource.com/skia/+/32a3cd2f2860b15a842a6aa49e4e6a3bed04f949

Review URL: https://codereview.chromium.org/1563443002
/external/skia/src/gpu/GrTest.cpp
4999df8ca2d53d7f34e0f6d0c70ab620bef2fe13 08-Jan-2016 egdaniel <egdaniel@google.com> Revert of Move config texturability/renderability to config table (patchset #8 id:140001 of https://codereview.chromium.org/1563443002/ )

Reason for revert:
speculative revert for breaking iOS writePixels tests

Original issue's description:
> Move config texturability/renderability to config table.
>
> GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1563443002
>
> Committed: https://skia.googlesource.com/skia/+/32a3cd2f2860b15a842a6aa49e4e6a3bed04f949

TBR=jvanverth@google.com,bsalomon@google.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

Review URL: https://codereview.chromium.org/1569103003
/external/skia/src/gpu/GrTest.cpp
32a3cd2f2860b15a842a6aa49e4e6a3bed04f949 07-Jan-2016 bsalomon <bsalomon@google.com> Move config texturability/renderability to config table.

GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1563443002

Review URL: https://codereview.chromium.org/1563443002
/external/skia/src/gpu/GrTest.cpp
17aa047066eaaa56637c4c2b93c8c4c374127dbf 05-Jan-2016 jvanverth <jvanverth@google.com> More framework support for TransferBuffers

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1534123003

Review URL: https://codereview.chromium.org/1534123003
/external/skia/src/gpu/GrTest.cpp
e804292e805917002cc3d7baa7f967fb20d2c7cb 11-Dec-2015 joshualitt <joshualitt@chromium.org> Move all text stuff to its own folder

BUG=skia:

Review URL: https://codereview.chromium.org/1521453002
/external/skia/src/gpu/GrTest.cpp
0671b967eb02d44c8951dc4dc39df09fac15b097 09-Dec-2015 jvanverth <jvanverth@google.com> Move texture drawing utility method to SkGpuDevice

BUG=skia:4542

Review URL: https://codereview.chromium.org/1506203002
/external/skia/src/gpu/GrTest.cpp
6dea83f244cfdea52901eef6b31cee60b07a8ea0 03-Dec-2015 bsalomon <bsalomon@google.com> Add option to draw wireframe batch bounds

Committed: https://skia.googlesource.com/skia/+/26489ef21ff5df33b8cb5943fddfd4604e203960

Review URL: https://codereview.chromium.org/1494473005
/external/skia/src/gpu/GrTest.cpp
e2725f584095cd0a2cc5100fb6f1dc6a1740284e 03-Dec-2015 scroggo <scroggo@google.com> Revert of Add option to draw wireframe batch bounds (patchset #6 id:100001 of https://codereview.chromium.org/1494473005/ )

Reason for revert:
Looks to be responsible for breaking builds e.g. https://uberchromegw.corp.google.com/i/client.skia/builders/Test-Mac10.9-Clang-MacMini6.2-GPU-HD4000-x86_64-Debug/builds/3030/steps/dm/logs/stdio

Original issue's description:
> Add option to draw wireframe batch bounds
>
> Committed: https://skia.googlesource.com/skia/+/26489ef21ff5df33b8cb5943fddfd4604e203960

TBR=joshualitt@google.com,bsalomon@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

Review URL: https://codereview.chromium.org/1492633007
/external/skia/src/gpu/GrTest.cpp
26489ef21ff5df33b8cb5943fddfd4604e203960 03-Dec-2015 bsalomon <bsalomon@google.com> Add option to draw wireframe batch bounds

Review URL: https://codereview.chromium.org/1494473005
/external/skia/src/gpu/GrTest.cpp
73063dc517f424ad5660db0fbc5fe6fcc13f77f7 03-Dec-2015 jvanverth <jvanverth@google.com> Add transfer buffer support.

BUG=skia:4604

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

Review URL: https://codereview.chromium.org/1490473003
/external/skia/src/gpu/GrTest.cpp
53c5d5fb795fe04bec050c0583223027c25b839b 03-Dec-2015 egdaniel <egdaniel@google.com> Revert of Add transfer buffer support. (patchset #9 id:160001 of https://codereview.chromium.org/1490473003/ )

Reason for revert:
speculative revert for deps roll failures

Original issue's description:
> Add transfer buffer support.
>
> BUG=skia:4604
>
> Committed: https://skia.googlesource.com/skia/+/fa498fe12239988578465d0dee69f0d5645bb361

TBR=bsalomon@google.com,jvanverth@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:4604

Review URL: https://codereview.chromium.org/1496843003
/external/skia/src/gpu/GrTest.cpp
dc5685ac3752e90dd68179e9f1675ff6f15ed600 02-Dec-2015 joshualitt <joshualitt@chromium.org> Wire up resource cache stats dumping in nanobench

TBR=bsalomon@google.com
BUG=skia:

Review URL: https://codereview.chromium.org/1495543003
/external/skia/src/gpu/GrTest.cpp
fa498fe12239988578465d0dee69f0d5645bb361 02-Dec-2015 jvanverth <jvanverth@google.com> Add transfer buffer support.

BUG=skia:4604

Review URL: https://codereview.chromium.org/1490473003
/external/skia/src/gpu/GrTest.cpp
e45c81c8f3bb62e45f2ff3b8772b4b23f1ddc6ca 02-Dec-2015 joshualitt <joshualitt@chromium.org> Began logging more gpu stats from nanobench

BUG=skia:

Review URL: https://codereview.chromium.org/1489033004
/external/skia/src/gpu/GrTest.cpp
69cfe95b7be386cf7d349b61388e93ea7c3cd386 30-Nov-2015 bsalomon <bsalomon@google.com> Add debug option to clip each GrBatch to its device bounds

Review URL: https://codereview.chromium.org/1471083002
/external/skia/src/gpu/GrTest.cpp
504ce5dc772e14be2f5697b00ac82fcf82127763 16-Nov-2015 robertphillips <robertphillips@google.com> Optionally pass rendertarget to getTestTarget

This shouldn't really make any difference but allocating and holding on to a GrRenderTarget for each test target generates image differences for Mali GPUs. This CL allows an existing render target to be used for the test target.

TBR=bsalomon@google.com

Review URL: https://codereview.chromium.org/1447113002
/external/skia/src/gpu/GrTest.cpp
0dfa62c97707ac9b0fc0bdba7acfc8405db5c836 16-Nov-2015 robertphillips <robertphillips@google.com> Update testing frameworks/tests for MDB

This CL:

Fixes an ordering problem w.r.t. drawTarget clean up in GrContext::abandonContext (for text test that abandons context)
Fixes when the lastDrawTarget field is set on a RenderTarget (now in GrDrawTarget ctor) due to GrTestTarget use case
Updates the ProgramUnitTest to use multiple drawTargets
Adds renderTarget creation to GrTestTargets (in MDB drawTargets require them)

BUG=skia:4094

Committed: https://skia.googlesource.com/skia/+/9c8605144a0f15e3e69a4e1dcd5d3e63f339380e

Review URL: https://codereview.chromium.org/1441533003
/external/skia/src/gpu/GrTest.cpp
1d5127327111e00d0e4530adae73b11ad2ee3f42 13-Nov-2015 egdaniel <egdaniel@google.com> Revert of Update testing frameworks/tests for MDB (patchset #4 id:60001 of https://codereview.chromium.org/1441533003/ )

Reason for revert:
Speculative revert for leaks based on ASAN bot (and some local valgrind testing)

Original issue's description:
> Update testing frameworks/tests for MDB
>
> This CL:
>
> Fixes an ordering problem w.r.t. drawTarget clean up in GrContext::abandonContext (for text test that abandons context)
> Fixes when the lastDrawTarget field is set on a RenderTarget (now in GrDrawTarget ctor) due to GrTestTarget use case
> Updates the ProgramUnitTest to use multiple drawTargets
> Adds renderTarget creation to GrTestTargets (in MDB drawTargets require them)
>
> BUG=skia:4094
>
> Committed: https://skia.googlesource.com/skia/+/9c8605144a0f15e3e69a4e1dcd5d3e63f339380e

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

Review URL: https://codereview.chromium.org/1450513002
/external/skia/src/gpu/GrTest.cpp
9c8605144a0f15e3e69a4e1dcd5d3e63f339380e 13-Nov-2015 robertphillips <robertphillips@google.com> Update testing frameworks/tests for MDB

This CL:

Fixes an ordering problem w.r.t. drawTarget clean up in GrContext::abandonContext (for text test that abandons context)
Fixes when the lastDrawTarget field is set on a RenderTarget (now in GrDrawTarget ctor) due to GrTestTarget use case
Updates the ProgramUnitTest to use multiple drawTargets
Adds renderTarget creation to GrTestTargets (in MDB drawTargets require them)

BUG=skia:4094

Review URL: https://codereview.chromium.org/1441533003
/external/skia/src/gpu/GrTest.cpp
caef3450488f98aa0bc429c4e2d8e29d6a7fece4 11-Nov-2015 robertphillips <robertphillips@google.com> Readd "immediate" mode

This isn't an exact replacement. The accumulated batches are now flushed at drawContext-entry-point granularity (via the AutoCheckFlush objects) rather than per batch.

TBR=bsalomon@google.com

Review URL: https://codereview.chromium.org/1439533003
/external/skia/src/gpu/GrTest.cpp
67d7620285cdfa60158add6615db03bd48e6d8b0 11-Nov-2015 bsalomon <bsalomon@google.com> Fix leaks in unit tests of GrGLTextureInfos

TBR=egdaniel@google.com

Review URL: https://codereview.chromium.org/1433353005
/external/skia/src/gpu/GrTest.cpp
60029a5397f75aae4bdb994f26bd297edc3e433c 09-Nov-2015 robertphillips <robertphillips@google.com> Update Layer Hoisting to store its atlas texture in the resource cache

BUG=skia:4346

Committed: https://skia.googlesource.com/skia/+/42597bc99f00553825843b5ed41e81b121773368

Review URL: https://codereview.chromium.org/1406013006
/external/skia/src/gpu/GrTest.cpp
629162dd8ef127f793c9cedf0f47a1d4de4d3a3d 08-Nov-2015 jvanverth <jvanverth@google.com> Add text animation sample; tweak DrawShip sample

Committed: https://skia.googlesource.com/skia/+/3b484a40b3be7f0262afadeaf6b741ba5cedcfe1

Review URL: https://codereview.chromium.org/1410663005
/external/skia/src/gpu/GrTest.cpp
d75ccc6a0a2eb166234d919ffd3f62ed39dd3a6e 07-Nov-2015 jvanverth <jvanverth@google.com> Revert of Add text animation sample; tweak DrawShip sample (patchset #3 id:40001 of https://codereview.chromium.org/1410663005/ )

Reason for revert:
CrOS bots failing.

Original issue's description:
> Add text animation sample; tweak DrawShip sample
>
> Committed: https://skia.googlesource.com/skia/+/3b484a40b3be7f0262afadeaf6b741ba5cedcfe1

TBR=robertphillips@google.com,bsalomon@google.com,reed@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

Review URL: https://codereview.chromium.org/1408063015
/external/skia/src/gpu/GrTest.cpp
3b484a40b3be7f0262afadeaf6b741ba5cedcfe1 06-Nov-2015 jvanverth <jvanverth@google.com> Add text animation sample; tweak DrawShip sample

Review URL: https://codereview.chromium.org/1410663005
/external/skia/src/gpu/GrTest.cpp
cf1d19805a289c612532cf2d4505bb348b78ba77 06-Nov-2015 robertphillips <robertphillips@google.com> Revert of Update Layer Hoisting to store its atlas texture in the resource cache (patchset #6 id:100001 of https://codereview.chromium.org/1406013006/ )

Reason for revert:
Android

Original issue's description:
> Update Layer Hoisting to store its atlas texture in the resource cache
>
> BUG=skia:4346
>
> Committed: https://skia.googlesource.com/skia/+/42597bc99f00553825843b5ed41e81b121773368

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

Review URL: https://codereview.chromium.org/1413483004
/external/skia/src/gpu/GrTest.cpp
42597bc99f00553825843b5ed41e81b121773368 06-Nov-2015 robertphillips <robertphillips@google.com> Update Layer Hoisting to store its atlas texture in the resource cache

BUG=skia:4346

Review URL: https://codereview.chromium.org/1406013006
/external/skia/src/gpu/GrTest.cpp
648c696438410fe0f0f5db9bb7935006fecf9cad 23-Oct-2015 bsalomon <bsalomon@google.com> Add immediate mode option for gpu configs in dm

Review URL: https://codereview.chromium.org/1421853002
/external/skia/src/gpu/GrTest.cpp
77a2e52c7d5c77de73ecab5fec79072ee4460706 17-Oct-2015 robertphillips <robertphillips@google.com> Remove DrawingMgr shims from GrContext

BUG=skia:4094

Review URL: https://codereview.chromium.org/1413673002
/external/skia/src/gpu/GrTest.cpp
a106c627532ad669cf7d879955ae8ea6a53233c1 16-Oct-2015 robertphillips <robertphillips@google.com> Loosen requirement that there be only one GrDrawTarget

BUG=skia:4094

Review URL: https://codereview.chromium.org/1406173003
/external/skia/src/gpu/GrTest.cpp
ec00d94199fad7723b5987b86c1abef8ddafe2d8 14-Sep-2015 egdaniel <egdaniel@google.com> Move some of the adding stencil attachment logic of Gpu and into Render Target.

The new flow of calls for attaching a Stencil looks like:

Client
rt->attachStencilAttachment()
gpu->getStencilAttachment()
glgpu->createStencilAttachment()
glrt->completeStencilAttachment() //actually attaches

BUG=skia:

Review URL: https://codereview.chromium.org/1333383002
/external/skia/src/gpu/GrTest.cpp
465283cdf98ed9ab5285ca7b9814e430fca1d452 11-Sep-2015 joshualitt <joshualitt@chromium.org> Remove batchtracker

BUG=skia:

Review URL: https://codereview.chromium.org/1332923003
/external/skia/src/gpu/GrTest.cpp
512be5340c2a29550053d35bc058d3aecd5c1fc7 10-Sep-2015 bsalomon <bsalomon@google.com> Cleanup GrDrawTarget now that all paths lead to GrBatch

Review URL: https://codereview.chromium.org/1315513008
/external/skia/src/gpu/GrTest.cpp
87a5c9fa0ff6fdf5225df53666798a2d513ade6d 08-Sep-2015 joshualitt <joshualitt@chromium.org> Add number of draws to GrGpu stats

BUG=skia:

Review URL: https://codereview.chromium.org/1333543002
/external/skia/src/gpu/GrTest.cpp
dac7005b877c7c1c6f36b6e298133dfc7b85b9c0 01-Sep-2015 joshualitt <joshualitt@chromium.org> Remove GrGpuTraceMarker hooks until we rethink the design

TBR=bsalomon@google.com
BUG=526308

Review URL: https://codereview.chromium.org/1323823003
/external/skia/src/gpu/GrTest.cpp
96fcdcc219d2a0d3579719b84b28bede76efba64 27-Aug-2015 halcanary <halcanary@google.com> Style Change: NULL->nullptr
DOCS_PREVIEW= https://skia.org/?cl=1316233002

Review URL: https://codereview.chromium.org/1316233002
/external/skia/src/gpu/GrTest.cpp
385fe4d4b62d7d1dd76116dd570df3290a2f487b 26-Aug-2015 halcanary <halcanary@google.com> Style Change: SkNEW->new; SkDELETE->delete
DOCS_PREVIEW= https://skia.org/?cl=1316123003

Review URL: https://codereview.chromium.org/1316123003
/external/skia/src/gpu/GrTest.cpp
1cbdcde9116e9efb514236faf8cfa42649a041d1 21-Aug-2015 joshualitt <joshualitt@chromium.org> Fix for copy surface not handling dirty context

BUG=521943

Review URL: https://codereview.chromium.org/1304263003
/external/skia/src/gpu/GrTest.cpp
7f9c9eb91e65f2d83474f2b34f9ed0d9525245ae 21-Aug-2015 joshualitt <joshualitt@chromium.org> wire up reduced atlas and zero textblob cache to test

BUG=skia:

Review URL: https://codereview.chromium.org/1300813007
/external/skia/src/gpu/GrTest.cpp
da04e0e80a0113e1d838f7406cd8a8d545b8c02b 19-Aug-2015 joshualitt <joshualitt@chromium.org> Allow setting of GrBatchFontCache atlas sizes

BUG=skia:

Review URL: https://codereview.chromium.org/1255943006
/external/skia/src/gpu/GrTest.cpp
cb02b38b2c48bfde333ce3c699dd0451e2d867fa 12-Aug-2015 bsalomon <bsalomon@google.com> Check for xfer barriers in GrBatch, auto-issue barriers in GrGpu

Review URL: https://codereview.chromium.org/1287973003
/external/skia/src/gpu/GrTest.cpp
51c8d409ad55a72eceb46ec7b65702d6d540aa4f 06-Aug-2015 egdaniel <egdaniel@google.com> Move some work from backend onClear to base class clear

BUG=skia:

Review URL: https://codereview.chromium.org/1275543005
/external/skia/src/gpu/GrTest.cpp
17d833b05b1554fd1c54144b4bd2f4f23946b7b8 03-Aug-2015 joshualitt <joshualitt@chromium.org> Add abliity to set textblob cache budget to GrContext

BUG=skia:

Review URL: https://codereview.chromium.org/1264283002
/external/skia/src/gpu/GrTest.cpp
6cb3cbe8e67db5fb94ba7d98f60833229b008544 30-Jul-2015 bsalomon <bsalomon@google.com> Make GrGpu read/write pixels take GrSurface

Review URL: https://codereview.chromium.org/1262473004
/external/skia/src/gpu/GrTest.cpp
f0674516cb18f6b13e34f404ff5793d9b9ebb56c 28-Jul-2015 bsalomon <bsalomon@google.com> Move draw on upload decision in GrGpu

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/0341b4427e5f037e3b501ed6e57dfdb7b40f150e

Review URL: https://codereview.chromium.org/1257073003
/external/skia/src/gpu/GrTest.cpp
c3fb74693e25cf6dffc8add8669c54c4b7ace3a2 28-Jul-2015 bsalomon <bsalomon@google.com> Revert of Move draw on upload decision in GrGpu (patchset #4 id:50001 of https://codereview.chromium.org/1257073003/)

Reason for revert:
breaking write pixels test on bots

Original issue's description:
> Move draw on upload decision in GrGpu
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/0341b4427e5f037e3b501ed6e57dfdb7b40f150e

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

Review URL: https://codereview.chromium.org/1260293004
/external/skia/src/gpu/GrTest.cpp
b542bae1f5a2726e52a15fa90acec7e7b935cb71 28-Jul-2015 joshualitt <joshualitt@chromium.org> Rename GrInOrderDrawBuffer to GrBufferedDrawTarget

BUG=skia:

Review URL: https://codereview.chromium.org/1261033002
/external/skia/src/gpu/GrTest.cpp
0341b4427e5f037e3b501ed6e57dfdb7b40f150e 28-Jul-2015 bsalomon <bsalomon@google.com> Move draw on upload decision in GrGpu

BUG=skia:

Review URL: https://codereview.chromium.org/1257073003
/external/skia/src/gpu/GrTest.cpp
398260262f8508687614ec496a23b8152bcdc967 23-Jul-2015 bsalomon <bsalomon@google.com> Attempt to somewhat simplify GrContext::readSurfacePixels interaction with GrGpu.

Review URL: https://codereview.chromium.org/1255483005
/external/skia/src/gpu/GrTest.cpp
889579287770ba35156a73aa02d9ef5d2313c490 14-Jul-2015 jvanverth <jvanverth@google.com> Rename backing texture testing routines

Review URL: https://codereview.chromium.org/1230193006
/external/skia/src/gpu/GrTest.cpp
1c735488cbec9abdb25c9a12e0c11af9c302a776 13-Jul-2015 joshualitt <joshualitt@chromium.org> Start to propagate constness of GrPipelineBuilder up the stack

BUG=skia:

Review URL: https://codereview.chromium.org/1229143007
/external/skia/src/gpu/GrTest.cpp
672bb7fc6640e3fc68107354ed4ae45a2a1e2d29 13-Jul-2015 jvanverth <jvanverth@google.com> Remove GL-specific code from GMs and tests

TBR=bsalomon@google.com

Review URL: https://codereview.chromium.org/1232173002
/external/skia/src/gpu/GrTest.cpp
f9ad558943ee14bdff824ebfb33988bece8ed3db 26-Jun-2015 reed <reed@chromium.org> Revert of Revert of Return GrGLContext from GrTestTarget (patchset #1 id:1 of https://codereview.chromium.org/1211123004/)

Reason for revert:
un-referting, did not help DEPS roll

Original issue's description:
> Revert of Return GrGLContext from GrTestTarget (patchset #3 id:40001 of https://codereview.chromium.org/1207393002/)
>
> Reason for revert:
> speculative revert to unblock DEPS roll (layout-tests)
>
> Original issue's description:
> > Return GrGLContext from GrTestTarget
> >
> > BUG=skia:
> >
> > Committed: https://skia.googlesource.com/skia/+/b6d93ea428ca5d13641c35faca7d0380980a3a17
>
> TBR=bsalomon@google.com,joshualitt@google.com,joshualitt@chromium.org
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/1852ec2b46a2a0efbb3550ff4d8b640f56a810e9

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

Review URL: https://codereview.chromium.org/1215593002
/external/skia/src/gpu/GrTest.cpp
1852ec2b46a2a0efbb3550ff4d8b640f56a810e9 26-Jun-2015 reed <reed@chromium.org> Revert of Return GrGLContext from GrTestTarget (patchset #3 id:40001 of https://codereview.chromium.org/1207393002/)

Reason for revert:
speculative revert to unblock DEPS roll (layout-tests)

Original issue's description:
> Return GrGLContext from GrTestTarget
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/b6d93ea428ca5d13641c35faca7d0380980a3a17

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

Review URL: https://codereview.chromium.org/1211123004
/external/skia/src/gpu/GrTest.cpp
b6d93ea428ca5d13641c35faca7d0380980a3a17 26-Jun-2015 joshualitt <joshualitt@chromium.org> Return GrGLContext from GrTestTarget

BUG=skia:

Review URL: https://codereview.chromium.org/1207393002
/external/skia/src/gpu/GrTest.cpp
6dc6f5f4a153d33ed91565cb3cd397a310a937d0 18-Jun-2015 bsalomon <bsalomon@google.com> Add support for creating texture backed images where Skia will delete the texture.

Review URL: https://codereview.chromium.org/1187523005
/external/skia/src/gpu/GrTest.cpp
8b08159aa7d109bc787705105036c54c96287323 02-Jun-2015 joshualitt <joshualitt@chromium.org> Fix for crashing bots

TBR=bsalomon@google.com
BUG=skia:

Review URL: https://codereview.chromium.org/1159183006
/external/skia/src/gpu/GrTest.cpp
6df86409ca586c3cb34f616f03501bd96181f9e4 01-Jun-2015 bsalomon <bsalomon@google.com> Revert "Revert of Move copy-surface-as-draw fallback to GrGLGpu. (patchset #12 id:220001 of https://codereview.chromium.org/1144433002/)"

This reverts commit 404b3b264b833eb7cffdc833c7e2ebbd708e123a.

Review URL: https://codereview.chromium.org/1161063003
/external/skia/src/gpu/GrTest.cpp
cabe20cafd5f091a68bbc2c0c48755ba9b61b0b7 01-Jun-2015 kkinnunen <kkinnunen@nvidia.com> Refactor GrGpu path rendering functions to GrPathRendering

GrGpu, GrGLGpu, GrPathRendering, GrGLPathRendering all duplicated
each others' path rendering related member functions. Make
GrPathRending the logical extension of GrGpu by removing the methods
from GrGpu. Similarly to the GL variants.

Changes includes and forward declarations due to removing
GrDrawTarget.h include from GrGpu.h. This was necessary due to
GrDrawTarget.h including GrPathRendering.h, and now GrPathRendering.h
includes GrGpu.h. (Also GrGpu.h does not need GrDrawTarget.h anymore).

Very slight runtime improvement on x86_64:
desk_fontwipe.skp_1 836us -> 841us 1.01x
desk_silkfinance.skp_1 2.01ms -> 2.01ms 1x
desk_forecastio.skp_1 7.03ms -> 7.05ms 1x
desk_weather.skp_1 3.74ms -> 3.74ms 1x
desk_twitter.skp_1 8.02ms -> 8.01ms 1x
desk_mapsvg.skp_1 5.24ms -> 5.23ms 1x
desk_pokemonwiki.skp_1 7.06ms -> 7.03ms 1x
desk_sfgate.skp_1 3.19ms -> 3.17ms 1x
tabl_cuteoverload.skp_1 2.66ms -> 2.64ms 0.99x
...
tabl_transformice.skp_1 3.06ms -> 2.98ms 0.98x
tabl_googlecalendar.skp_1 11.3ms -> 11ms 0.97x
tabl_gamedeksiam.skp_1 12.8ms -> 12.4ms 0.97x
desk_samoasvg.skp_1 10.6ms -> 10.3ms 0.97x
tabl_worldjournal.skp_1 4.44ms -> 4.3ms 0.97x

Review URL: https://codereview.chromium.org/1157683006
/external/skia/src/gpu/GrTest.cpp
993a4216a6014b9de8f4d8120360c94550dc6761 29-May-2015 bsalomon <bsalomon@google.com> SkImage::NewFromYUVTexturesCopy

Review URL: https://codereview.chromium.org/1149553002
/external/skia/src/gpu/GrTest.cpp
4ee6bd86c500d9b464e2f8cb3565544b8c97c0a9 27-May-2015 bsalomon <bsalomon@google.com> Add caps overrides to GMs

Review URL: https://codereview.chromium.org/1158963002
/external/skia/src/gpu/GrTest.cpp
ea4615034498aca2f9ca1753fb9a1ef10508d8cc 26-May-2015 robertphillips <robertphillips@google.com> Split drawing functionality out of GrContext and into new GrDrawContext

This is mainly a mechanical CL. There were some fiddly bits in GrContext.cpp where it no longer had access to the GrDrawTarget (and had to use the new GrDrawContext).

I've converted GrAARectRenderer & GrOvalRenderer into static classes so I could stop allocating them.

Review URL: https://codereview.chromium.org/1151283004
/external/skia/src/gpu/GrTest.cpp
682c269a1511200322916af83053e26004c0ec40 22-May-2015 bsalomon <bsalomon@google.com> Store context options on caps.

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

Review URL: https://codereview.chromium.org/1158433006
/external/skia/src/gpu/GrTest.cpp
08bf86d1b7b03cc6829f2488d9adcb0502e84b97 22-May-2015 bsalomon <bsalomon@google.com> Revert of Store context options on caps. (patchset #3 id:40001 of https://codereview.chromium.org/1158433006/)

Reason for revert:
breaking chromeos build (???)

Original issue's description:
> Store context options on caps.
>
> Committed: https://skia.googlesource.com/skia/+/f28cff71db2cbb1ff18a8fbf1e80ca761d1f69bc

TBR=joshualitt@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

Review URL: https://codereview.chromium.org/1151603005
/external/skia/src/gpu/GrTest.cpp
f28cff71db2cbb1ff18a8fbf1e80ca761d1f69bc 22-May-2015 bsalomon <bsalomon@google.com> Store context options on caps.

Review URL: https://codereview.chromium.org/1158433006
/external/skia/src/gpu/GrTest.cpp
4b91f768b348aa1cebeb54f3ff9331938734c242 19-May-2015 bsalomon <bsalomon@google.com> rename GrDrawTargetCaps to GrCaps

Review URL: https://codereview.chromium.org/1133123009
/external/skia/src/gpu/GrTest.cpp
404b3b264b833eb7cffdc833c7e2ebbd708e123a 18-May-2015 mtklein <mtklein@google.com> Revert of Move copy-surface-as-draw fallback to GrGLGpu. (patchset #12 id:220001 of https://codereview.chromium.org/1144433002/)

Reason for revert:
Many GMs changed, some totally busted.

Original issue's description:
> Move copy-surface-as-draw fallback to GrGLGpu.
>
> Committed: https://skia.googlesource.com/skia/+/5df6fee929823f50c55cc50f7c882a309c1b1de9

TBR=joshualitt@google.com,egdaniel@google.com,bsalomon@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

Review URL: https://codereview.chromium.org/1142003003
/external/skia/src/gpu/GrTest.cpp
5df6fee929823f50c55cc50f7c882a309c1b1de9 18-May-2015 bsalomon <bsalomon@google.com> Move copy-surface-as-draw fallback to GrGLGpu.

Review URL: https://codereview.chromium.org/1144433002
/external/skia/src/gpu/GrTest.cpp
231c5fd590c898957d65cbfbc51040c7fe236af6 13-May-2015 cdalton <cdalton@nvidia.com> Don't issue texture barriers for RT's with separate storage

BUG=skia:

Review URL: https://codereview.chromium.org/1137663004
/external/skia/src/gpu/GrTest.cpp
e64eb570a5b9480bc24d0656ccabcff1ab13a229 07-May-2015 bsalomon <bsalomon@google.com> Iterate over instanced draws in GrGpu rather than above GrBatchTarget

Review URL: https://codereview.chromium.org/1127273007
/external/skia/src/gpu/GrTest.cpp
e40d39725908d99d255496025411e5c9ca32663e 07-May-2015 robertphillips <robertphillips@google.com> Isolate GrBufferAllocPools inside GrBatchTarget

This CL refactors the location of the GrBufferAllocPools so they reside entirely inside the GrBatchTarget. This is in preparation for making them use scratch resources.

Review URL: https://codereview.chromium.org/1131553002
/external/skia/src/gpu/GrTest.cpp
cb8979d088a66ebaf41f10ba6f5c830615aa0e03 05-May-2015 bsalomon <bsalomon@google.com> Move DrawInfo out from GrDrawTarget and rename to GrVertices.

Review URL: https://codereview.chromium.org/1124733004
/external/skia/src/gpu/GrTest.cpp
9954bc38c498f6b9e9d8c0bcc5cd00d45bfc6e23 29-Apr-2015 cdalton <cdalton@nvidia.com> Use texture barriers to read directly from the RT

Updates GrXferProcessor to read directly from the RT texture when
texture barriers are supported and it needs to know the dst color.
Also adds the notion of an Xfer barrier and uses it to issue texture
barriers when the XP will read the RT.

BUG=skia:

Review URL: https://codereview.chromium.org/1040303002
/external/skia/src/gpu/GrTest.cpp
72e3ae486c66871c2043eac4f08d85d419fbca2a 28-Apr-2015 bsalomon <bsalomon@google.com> Remove vertex/index buffer factilities from GrDrawTarget.

Review URL: https://codereview.chromium.org/1105263002
/external/skia/src/gpu/GrTest.cpp
b0e1be207f6b5a5346641b7b675bb9bd1993f9df 22-Apr-2015 egdaniel <egdaniel@google.com> Refactor createTexture and onCreateTexture

BUG=skia:

Review URL: https://codereview.chromium.org/1102663002
/external/skia/src/gpu/GrTest.cpp
8dc7c3a839b38b73af34cc2674a06f49eb1ce527 16-Apr-2015 egdaniel <egdaniel@google.com> Rename GrStencilBuffer to GrStencilAttachment

BUG=skia:

Review URL: https://codereview.chromium.org/1083133002
/external/skia/src/gpu/GrTest.cpp
36352bf5e38f45a70ee4f4fc132a38048d38206d 26-Mar-2015 mtklein <mtklein@chromium.org> C++11 override should now be supported by all of {bots,Chrome,Android,Mozilla}

NOPRESUBMIT=true

BUG=skia:
DOCS_PREVIEW= https://skia.org/?cl=1037793002

Review URL: https://codereview.chromium.org/1037793002
/external/skia/src/gpu/GrTest.cpp
44701df5ce572ac3cccec785cf52103d3d5d14a5 23-Feb-2015 joshualitt <joshualitt@chromium.org> Move clip off of draw target

BUG=skia:

Review URL: https://codereview.chromium.org/947443003
/external/skia/src/gpu/GrTest.cpp
52057c870777b72971962e90f13475b31c9f635f 23-Feb-2015 bsalomon <bsalomon@google.com> Fix release-developer build.

TBR=humper@google.com

Review URL: https://codereview.chromium.org/951613002
/external/skia/src/gpu/GrTest.cpp
9e5fc72d638af5740b0b12a0ec12cf1f561f6a4d 23-Feb-2015 bsalomon <bsalomon@google.com> Add stencil buffer create tracking to GPU stats.

Review URL: https://codereview.chromium.org/949953002
/external/skia/src/gpu/GrTest.cpp
ddf30e64fe474847b204d7062fad3341d245062c 19-Feb-2015 bsalomon <bsalomon@google.com> Handle the case when the GrResourceCache timestamp wraps.

NOTREECHECKS=true

Review URL: https://codereview.chromium.org/916103006
/external/skia/src/gpu/GrTest.cpp
02a44a488605112aa6683c9d919e13b188112ce1 19-Feb-2015 bsalomon <bsalomon@google.com> Recycle stencil buffers across render targets.

Review URL: https://codereview.chromium.org/939093002
/external/skia/src/gpu/GrTest.cpp
297aaf97a32ac16a2bc3f4bbc231d5de859ac02d 19-Feb-2015 kkinnunen <kkinnunen@nvidia.com> Avoid wrong memory access due to double substitution and %%

Use the form SkDebugf("%s", arbitraryString) instead of
SkDebugf(arbitraryString).

Fixes the case where SkString::appendf-ing a string with "%%" and then
printing the string with SkDebugf would cause uninitialized read and
corrupted debug print.

ninja -C out/Debug tools && valgrind --leak-check=full
./out/Debug/render_pictures --config gpu -w q -r ...

...

==7307== Conditional jump or move depends on uninitialised value(s)
==7307== at 0x6908475: __printf_fp (printf_fp.c:1180)
==7307== by 0x6904267: vfprintf (vfprintf.c:1629)
==7307== by 0x6906E53: buffered_vfprintf (vfprintf.c:2313)
==7307== by 0x690188D: vfprintf (vfprintf.c:1316)
==7307== by 0x67E8F5: SkDebugf(char const*, ...) (SkDebug_stdio.cpp:18)
==7307== by 0x7983F1: GrContext::printCacheStats() const (GrTest.cpp:54)
==7307== by 0x408ECF: tool_main(int, char**) (render_pictures_main.cpp:480)
==7307== by 0x40913E: main (render_pictures_main.cpp:511)
==7307==
Budget: 2048 items 100663296 bytes
Entry Count: current 652 (651 budgeted, 0 wrapped, 297 locked, 638 scratch 32 0.000000ull), high 652
Entry Bytes: current 51087658 (budgeted 49826658, 49 0.000000ull, 1261000 unbudgeted) high 51087658

(observe "ull" instead of "% full")

(from mtklein)
This CL is not editing public API.
TBR=reed@google.com

Review URL: https://codereview.chromium.org/943453002
/external/skia/src/gpu/GrTest.cpp
f320e04c50a1c8a861bc1d8f50bf732044ff9843 18-Feb-2015 bsalomon <bsalomon@google.com> Use an array of nonpurgeable resources in GrResourceCache

Review URL: https://codereview.chromium.org/932863004
/external/skia/src/gpu/GrTest.cpp
3582d3ee9fffdec715f5e4949a241ab08e6271ec 13-Feb-2015 bsalomon <bsalomon@google.com> Split out methods in GrGpuResource::CacheAccess that can be called outside of the cache.

Review URL: https://codereview.chromium.org/923143002
/external/skia/src/gpu/GrTest.cpp
0ea80f43a1af05b8157a4ef387223bb5b0da35ed 11-Feb-2015 bsalomon <bsalomon@google.com> Rename GrResourceCache2->GrResourceCache

TBR=robertphillips@google.com

Review URL: https://codereview.chromium.org/921453002
/external/skia/src/gpu/GrTest.cpp
50785a3d10b53bea5beb6e18431a2449860be237 06-Feb-2015 bsalomon <bsalomon@google.com> Revert of Revert of Move DstCopy on gpu into the GrXferProcessor. (patchset #1 id:1 of https://codereview.chromium.org/901663007/)

Reason for revert:
The revert didn't help the 10.9 bot. Unreverting by reverting the revert (which is basically relanding the original patch which itself was a revert of a revert). Revert.

Original issue's description:
> Revert of Move DstCopy on gpu into the GrXferProcessor. (patchset #11 id:200001 of https://codereview.chromium.org/885923002/)
>
> Reason for revert:
> Testing to see if reverting fixes 10.9 bots.
>
> Original issue's description:
> > Move DstCopy on gpu into the GrXferProcessor.
> >
> > BUG=skia:
> >
> > Committed: https://skia.googlesource.com/skia/+/74a11753604768bf461b80cabb66060e8564d82c
> >
> > Committed: https://skia.googlesource.com/skia/+/5e1378d0e075a323144ba14e0a4cbcca35eccc69
>
> TBR=joshualitt@google.com,egdaniel@google.com
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/21b2c53218ab25f4268e3992e51d916076a2a7ee

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

Review URL: https://codereview.chromium.org/873723009
/external/skia/src/gpu/GrTest.cpp
21b2c53218ab25f4268e3992e51d916076a2a7ee 05-Feb-2015 bsalomon <bsalomon@google.com> Revert of Move DstCopy on gpu into the GrXferProcessor. (patchset #11 id:200001 of https://codereview.chromium.org/885923002/)

Reason for revert:
Testing to see if reverting fixes 10.9 bots.

Original issue's description:
> Move DstCopy on gpu into the GrXferProcessor.
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/74a11753604768bf461b80cabb66060e8564d82c
>
> Committed: https://skia.googlesource.com/skia/+/5e1378d0e075a323144ba14e0a4cbcca35eccc69

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

Review URL: https://codereview.chromium.org/901663007
/external/skia/src/gpu/GrTest.cpp
5e1378d0e075a323144ba14e0a4cbcca35eccc69 05-Feb-2015 egdaniel <egdaniel@google.com> Move DstCopy on gpu into the GrXferProcessor.

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/74a11753604768bf461b80cabb66060e8564d82c

Review URL: https://codereview.chromium.org/885923002
/external/skia/src/gpu/GrTest.cpp
4dce32c46675efb901618161771d450241af4307 04-Feb-2015 egdaniel <egdaniel@google.com> Revert "Move DstCopy on gpu into the GrXferProcessor."

This reverts commit 74a11753604768bf461b80cabb66060e8564d82c.

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

Committed: https://skia.googlesource.com/skia/+/3e9dfdb3784c0cbfecf7589a74aa9aff7ef40abd

Review URL: https://codereview.chromium.org/896163003
/external/skia/src/gpu/GrTest.cpp
7adb35559430303a25be37f21af56c0df6c62f2b 04-Feb-2015 egdaniel <egdaniel@google.com> Revert of Revert "Move DstCopy on gpu into the GrXferProcessor." (patchset #1 id:1 of https://codereview.chromium.org/896163003/)

Reason for revert:
failed on my manual revert

Original issue's description:
> Revert "Move DstCopy on gpu into the GrXferProcessor."
>
> This reverts commit 74a11753604768bf461b80cabb66060e8564d82c.
>
> TBR=joshualitt@google.com,bsalomon@google.com
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/3e9dfdb3784c0cbfecf7589a74aa9aff7ef40abd

TBR=
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:

Review URL: https://codereview.chromium.org/900913002
/external/skia/src/gpu/GrTest.cpp
3e9dfdb3784c0cbfecf7589a74aa9aff7ef40abd 04-Feb-2015 egdaniel <egdaniel@google.com> Revert "Move DstCopy on gpu into the GrXferProcessor."

This reverts commit 74a11753604768bf461b80cabb66060e8564d82c.

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

Review URL: https://codereview.chromium.org/896163003
/external/skia/src/gpu/GrTest.cpp
74a11753604768bf461b80cabb66060e8564d82c 04-Feb-2015 egdaniel <egdaniel@google.com> Move DstCopy on gpu into the GrXferProcessor.

BUG=skia:

Review URL: https://codereview.chromium.org/885923002
/external/skia/src/gpu/GrTest.cpp
b12ea41286ce36e085c5a14711da0cf9f240fdf1 03-Feb-2015 bsalomon <bsalomon@google.com> Add texture create/upload stats and make nanobench have explicit gpu stats flag

Review URL: https://codereview.chromium.org/891973002
/external/skia/src/gpu/GrTest.cpp
b9eb4ac0f1c29d6fe10ad7ff81ed8326ac1ea043 03-Feb-2015 mtklein <mtklein@chromium.org> patch from issue 886233004 at patchset 40001 (http://crrev.com/886233004#ps40001)

... with changes proposed in the review.

BUG=skia:

Review URL: https://codereview.chromium.org/894013002
/external/skia/src/gpu/GrTest.cpp
36c57dfb4fe9bbaca436942d5eaa75b142ba251d 27-Jan-2015 kkinnunen <kkinnunen@nvidia.com> Make stencil buffers uncached for uncached render target textures

Make new stencil buffers of uncached render target textures not affect the
cache budgets. This is consistent with render buffer storage of uncached
render target textures.

Affects only newly created stencil buffers. An uncached render target
might still receive a cached stencil buffer if such is available from
cache.

BUG=skia:3119
BUG=skia:3301

Review URL: https://codereview.chromium.org/859013002
/external/skia/src/gpu/GrTest.cpp
8dd688b7569df569a672a8a67b2db86a9d376cfc 22-Jan-2015 egdaniel <egdaniel@google.com> Rename GrOptDrawState to GrPipeline and GrDrawState to GrPipelineBuilder

BUG=skia:

Review URL: https://codereview.chromium.org/858343002
/external/skia/src/gpu/GrTest.cpp
17e7314e0e3085ec6021997d7c0593c339ba6a2e 21-Jan-2015 joshualitt <joshualitt@chromium.org> remove drawtype

BUG=skia:

Review URL: https://codereview.chromium.org/862933005
/external/skia/src/gpu/GrTest.cpp
873ad0e0b4d67bdc7bad025018f597450e7004c6 20-Jan-2015 joshualitt <joshualitt@chromium.org> BUG=skia:

Review URL: https://codereview.chromium.org/815643005
/external/skia/src/gpu/GrTest.cpp
5236cf480daf82b2f36e42795abdbbc915533a59 14-Jan-2015 bsalomon <bsalomon@google.com> Make uncached textures uncached from the get go.

This avoids the problem of a newly created uncached texture causing a purge of cached resources.

BUG=chromium:445885

Review URL: https://codereview.chromium.org/846303002
/external/skia/src/gpu/GrTest.cpp
3e79124a69d4806f0a1a776090bff718e1b90970 17-Dec-2014 bsalomon <bsalomon@google.com> Stop creating GrODS for stencilPath commands.

Review URL: https://codereview.chromium.org/816513003
/external/skia/src/gpu/GrTest.cpp
d95263c60e1c3b9e337d777d6f3cf286c1dc43f6 16-Dec-2014 bsalomon <bsalomon@google.com> Remove GrGpu::flushGraphicsState

Review URL: https://codereview.chromium.org/809843002
/external/skia/src/gpu/GrTest.cpp
dafa4d09cbe66943b2414bdeaa0f6ef7f900dc4a 04-Dec-2014 joshualitt <joshualitt@chromium.org> move program descriptor generation to flush

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/829e1b80b1020b17f2078020c990e079b70c077c

Review URL: https://codereview.chromium.org/777673003
/external/skia/src/gpu/GrTest.cpp
f78c60c92a8c212fc462262f2ce81e584d0f02f8 04-Dec-2014 joshualitt <joshualitt@google.com> Revert of move program descriptor generation to flush (patchset #7 id:120001 of https://codereview.chromium.org/777673003/)

Reason for revert:
breaking linux build

Original issue's description:
> move program descriptor generation to flush
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/829e1b80b1020b17f2078020c990e079b70c077c

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

Review URL: https://codereview.chromium.org/776243005
/external/skia/src/gpu/GrTest.cpp
829e1b80b1020b17f2078020c990e079b70c077c 03-Dec-2014 joshualitt <joshualitt@chromium.org> move program descriptor generation to flush

BUG=skia:

Review URL: https://codereview.chromium.org/777673003
/external/skia/src/gpu/GrTest.cpp
f90a02b42ac7a1ed59460760c6ce03f6f975f22b 26-Nov-2014 bsalomon <bsalomon@google.com> some cleanup around GrGpu/GrDrawTarget copySurface

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

Review URL: https://codereview.chromium.org/749903003
/external/skia/src/gpu/GrTest.cpp
8ee4e601f9aa4199e62b57260780207fd26e446a 26-Nov-2014 bsalomon <bsalomon@google.com> Revert of some cleanup around GrGpu/GrDrawTarget copySurface (patchset #3 id:40001 of https://codereview.chromium.org/749903003/)

Reason for revert:
likely causing es rendering errors.

Original issue's description:
> some cleanup around GrGpu/GrDrawTarget copySurface
>
> Committed: https://skia.googlesource.com/skia/+/e9aa5dc4d5906788eaf691d7c69f1494928f401d

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

Review URL: https://codereview.chromium.org/763593002
/external/skia/src/gpu/GrTest.cpp
e9aa5dc4d5906788eaf691d7c69f1494928f401d 26-Nov-2014 bsalomon <bsalomon@google.com> some cleanup around GrGpu/GrDrawTarget copySurface

Review URL: https://codereview.chromium.org/749903003
/external/skia/src/gpu/GrTest.cpp
9176e2c159089458b1e2226a94fab1af0fba32ac 20-Nov-2014 joshualitt <joshualitt@chromium.org> dstCopy on optdrawstate

BUG=skia:

Review URL: https://codereview.chromium.org/735363003
/external/skia/src/gpu/GrTest.cpp
54e0c12a5ab2d83fe249dd199d6879e8c0f04404 19-Nov-2014 joshualitt <joshualitt@chromium.org> scissor state on optstate

waiting

BUG=skia:

Review URL: https://codereview.chromium.org/742763002
/external/skia/src/gpu/GrTest.cpp
9853ccef19c200be93a6211f32589fa82a53067c 17-Nov-2014 joshualitt <joshualitt@chromium.org> Drawstate on stack

BUG=skia:

Review URL: https://codereview.chromium.org/732693002
/external/skia/src/gpu/GrTest.cpp
71cb0c241e439b6ed746b90294d0b6916644a644 14-Nov-2014 bsalomon <bsalomon@google.com> Replace GrResourceCache with GrResourceCache2.

BUG=skia:2889

Committed: https://skia.googlesource.com/skia/+/66a450f21a3da174b7eed89a1d5fc8591e8b6ee6

Committed: https://skia.googlesource.com/skia/+/407aa584d183c1bf314f5defd1cf0202e8a96c89

Review URL: https://codereview.chromium.org/716143004
/external/skia/src/gpu/GrTest.cpp
ac49acda520292b059bc37fd368dff912626c3d9 14-Nov-2014 bsalomon <bsalomon@google.com> Revert of Replace GrResourceCache with GrResourceCache2. (patchset #7 id:120001 of https://codereview.chromium.org/716143004/)

Reason for revert:
broken again

Original issue's description:
> Replace GrResourceCache with GrResourceCache2.
>
> BUG=skia:2889
>
> Committed: https://skia.googlesource.com/skia/+/66a450f21a3da174b7eed89a1d5fc8591e8b6ee6
>
> Committed: https://skia.googlesource.com/skia/+/407aa584d183c1bf314f5defd1cf0202e8a96c89

TBR=robertphillips@google.com
NOTREECHECKS=true
NOTRY=true
BUG=skia:2889

Review URL: https://codereview.chromium.org/726913002
/external/skia/src/gpu/GrTest.cpp
407aa584d183c1bf314f5defd1cf0202e8a96c89 14-Nov-2014 bsalomon <bsalomon@google.com> Replace GrResourceCache with GrResourceCache2.

BUG=skia:2889

Committed: https://skia.googlesource.com/skia/+/66a450f21a3da174b7eed89a1d5fc8591e8b6ee6

Review URL: https://codereview.chromium.org/716143004
/external/skia/src/gpu/GrTest.cpp
f21dab95405979a4aff9f6a741c818304a67f220 13-Nov-2014 bsalomon <bsalomon@google.com> Revert of Replace GrResourceCache with GrResourceCache2. (patchset #6 id:100001 of https://codereview.chromium.org/716143004/)

Reason for revert:
Breaking stuff

Original issue's description:
> Replace GrResourceCache with GrResourceCache2.
>
> BUG=skia:2889
>
> Committed: https://skia.googlesource.com/skia/+/66a450f21a3da174b7eed89a1d5fc8591e8b6ee6

TBR=robertphillips@google.com
NOTREECHECKS=true
NOTRY=true
BUG=skia:2889

Review URL: https://codereview.chromium.org/715333003
/external/skia/src/gpu/GrTest.cpp
66a450f21a3da174b7eed89a1d5fc8591e8b6ee6 13-Nov-2014 bsalomon <bsalomon@google.com> Replace GrResourceCache with GrResourceCache2.

BUG=skia:2889

Review URL: https://codereview.chromium.org/716143004
/external/skia/src/gpu/GrTest.cpp
d53a82706ebd227bc0e4dd535118cdd8298131bd 11-Nov-2014 joshualitt <joshualitt@chromium.org> Snap optdrawstate in inorder draw buffer and pass into gpu

BUG=skia:

Review URL: https://codereview.chromium.org/709133003
/external/skia/src/gpu/GrTest.cpp
4b68ec08c69ee9f17e13fce2094bb04b12b54856 07-Nov-2014 joshualitt <joshualitt@chromium.org> simple cl to cleanup friends in GrGpu

BUG=skia:

Review URL: https://codereview.chromium.org/709003006
/external/skia/src/gpu/GrTest.cpp
3322fa432a96fdc94d54f2475faf918dfa05b919 07-Nov-2014 joshualitt <joshualitt@chromium.org> Refactor DrawTarget and GPU to be independent

BUG=skia:

Review URL: https://codereview.chromium.org/705593002
/external/skia/src/gpu/GrTest.cpp
334355742137cc238887d4f85856e6c4d9ec8aa8 05-Nov-2014 bsalomon <bsalomon@google.com> Add mock context and use in ResourceCacheTest.

BUG=skia:2889

Committed: https://skia.googlesource.com/skia/+/820dd6c335411aad889c1d7e8a857642ecd87e30

Review URL: https://codereview.chromium.org/702083003
/external/skia/src/gpu/GrTest.cpp
fdcf2c0863cdd5779bd6203d45b77d2daf21f159 05-Nov-2014 bsalomon <bsalomon@google.com> Revert of Add mock context and use in ResourceCacheTest. (patchset #4 id:60001 of https://codereview.chromium.org/702083003/)

Reason for revert:
Breaking tests

Original issue's description:
> Add mock context and use in ResourceCacheTest.
>
> BUG=skia:2889
>
> Committed: https://skia.googlesource.com/skia/+/820dd6c335411aad889c1d7e8a857642ecd87e30

TBR=robertphillips@google.com
NOTREECHECKS=true
NOTRY=true
BUG=skia:2889

Review URL: https://codereview.chromium.org/704563004
/external/skia/src/gpu/GrTest.cpp
820dd6c335411aad889c1d7e8a857642ecd87e30 05-Nov-2014 bsalomon <bsalomon@google.com> Add mock context and use in ResourceCacheTest.

BUG=skia:2889

Review URL: https://codereview.chromium.org/702083003
/external/skia/src/gpu/GrTest.cpp
50408adfd23994cdbb5c531ff6366e4e9d29ee6d 03-Nov-2014 joshualitt <joshualitt@chromium.org> Test factory uses the inorder draw buffer

BUG=skia:

Review URL: https://codereview.chromium.org/696353002
/external/skia/src/gpu/GrTest.cpp
95c2003740c4cd01fd1b02ed93b9de7227b1d0f5 09-May-2014 commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> cleanup GrContext resource cache api

R=robertphillips@google.com

Author: bsalomon@google.com

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

git-svn-id: http://skia.googlecode.com/svn/trunk@14669 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/gpu/GrTest.cpp
94d8f1ed6116780346ffa39ed683f1977786434a 18-Dec-2013 robertphillips@google.com <robertphillips@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> Update bench pictures to time image decode & upload costs

https://codereview.chromium.org/117583002/



git-svn-id: http://skia.googlecode.com/svn/trunk@12742 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/gpu/GrTest.cpp
78a1078f17f4f0ae63415298517262a64f706af6 21-Aug-2013 commit-bot@chromium.org <commit-bot@chromium.org@2bbb7eff-a529-9590-31e7-b0007b416f81> Create new target to hold gpu test code, enable direct testing of GrEffects in GM.

R=robertphillips@google.com, jvanverth@google.com, egdaniel@google.com

Author: bsalomon@google.com

Review URL: https://chromiumcodereview.appspot.com/23352003

git-svn-id: http://skia.googlecode.com/svn/trunk@10866 2bbb7eff-a529-9590-31e7-b0007b416f81
/external/skia/src/gpu/GrTest.cpp