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
|