kkinnunen [Wed, 30 Jul 2014 07:04:16 +0000 (00:04 -0700)]
Rename GrGLUniformManager to GrGLProgramDataManager
Rename GrGLUniformManager to GrGLProgramDataManager in anticipation that the
class would be used to manage shader resources that are not uniforms.
This is needed in order to implement NVPR on GLES.
R=bsalomon@google.com
Author: kkinnunen@nvidia.com
Review URL: https://codereview.chromium.org/
365853002
borenet [Wed, 30 Jul 2014 05:57:43 +0000 (22:57 -0700)]
Update SKP version to 63
Automatic commit by the RecreateSKPs bot.
TBR=
Author: borenet@google.com
Review URL: https://codereview.chromium.org/
426183002
cdalton [Tue, 29 Jul 2014 22:25:51 +0000 (15:25 -0700)]
Incorporate glStencilThenCover* nvpr methods
Adds the glStencilThenCover* nvpr methods to GrGLInterface and starts
using them. When drawing multible paths, this will make it so we only
have to send the index/transform data once. It will also allow the
driver to save time internally.
The glStencilThenCover* methods are a newer addition, so they aren't
available on every driver. In the event that they are not present, we
emulate them using the existing glStencil*/glCover* methods.
BUG=skia:
R=markkilgard@gmail.com, bsalomon@google.com
Author: cdalton@nvidia.com
Review URL: https://codereview.chromium.org/
423173004
krajcevski [Tue, 29 Jul 2014 20:19:05 +0000 (13:19 -0700)]
fRight -> fBottom
R=robertphillips@google.com
Author: krajcevski@google.com
Review URL: https://codereview.chromium.org/
428963005
egdaniel [Tue, 29 Jul 2014 20:15:47 +0000 (13:15 -0700)]
Remove support in the gpu to use hardware AA lines. Current code paths could not reach this code and thus was never used.
The blend opt flag kDisableBlend should now be able to be removed, however it is left in this CL and will removed in its own.
BUG=skia:
R=bsalomon@google.com
Author: egdaniel@google.com
Review URL: https://codereview.chromium.org/
427823002
joshualitt [Tue, 29 Jul 2014 19:59:27 +0000 (12:59 -0700)]
Adding repeat mode to texture domain
BUG=skia:
R=bsalomon@chromium.org, senorblanco@chromium.org, bsalomon@google.com, junov@chromium.org
Author: joshualitt@chromium.org
Review URL: https://codereview.chromium.org/
422123003
djsollen [Tue, 29 Jul 2014 19:38:25 +0000 (12:38 -0700)]
Add ability to generate/use FDO profiles for Android
R=scroggo@google.com, bungeman@google.com
Author: djsollen@google.com
Review URL: https://codereview.chromium.org/
420803003
caryclark [Tue, 29 Jul 2014 19:13:28 +0000 (12:13 -0700)]
add cubic red option to pathops tool
R=reed@google.com
TBR=reed@google.com
NOTREECHECKS=true
NOTRY=true
Author: caryclark@google.com
Review URL: https://codereview.chromium.org/
428963004
krajcevski [Tue, 29 Jul 2014 18:44:26 +0000 (11:44 -0700)]
Add query for block dimensions of a given format
R=robertphillips@google.com
Author: krajcevski@google.com
Review URL: https://codereview.chromium.org/
422023006
robertphillips [Tue, 29 Jul 2014 17:37:02 +0000 (10:37 -0700)]
Optimize correct picture in bench_pictures when using --bbh
Almost all PictureRenderer::init implementations have the power to overwrite the picture. Retrieve the correct SkPicture from the renderer for optimization.
R=bsalomon@google.com
Author: robertphillips@google.com
Review URL: https://codereview.chromium.org/
428883008
bensong [Tue, 29 Jul 2014 17:35:46 +0000 (10:35 -0700)]
Revert of Add effect caching to distance field text. (https://codereview.chromium.org/
424103002/)
Reason for revert:
breaking Chrome canary.
Original issue's description:
> Add effect caching to distance field text.
>
> This also is a step towards unifying GrDistanceFieldTextureEffect and GrDistanceFieldLCDTextureEffect.
>
> Committed: https://skia.googlesource.com/skia/+/
137bac067306c5446bc4f9797bedc3bbaf302822
R=robertphillips@google.com, jvanverth@google.com
TBR=jvanverth@google.com, robertphillips@google.com
NOTREECHECKS=true
NOTRY=true
Author: bensong@google.com
Review URL: https://codereview.chromium.org/
424173002
egdaniel [Tue, 29 Jul 2014 15:52:23 +0000 (08:52 -0700)]
Revert of Remove gpu support for willUseHWAALines. (https://codereview.chromium.org/
423943003/)
Reason for revert:
Breaking windows 7 compare gm's
Original issue's description:
> Remove gpu support for willUseHWAALines.
>
> By removing willUseHWAALines, we also no long reference the blend opt flag kDisableBlend so it has been removed
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/
e6dfba868c19b00562f8c733b9bf37dd4ec9e68c
R=bsalomon@google.com
TBR=bsalomon@google.com
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Author: egdaniel@google.com
Review URL: https://codereview.chromium.org/
430493003
jvanverth [Tue, 29 Jul 2014 15:29:50 +0000 (08:29 -0700)]
Add effect caching to distance field text.
This also is a step towards unifying GrDistanceFieldTextureEffect and GrDistanceFieldLCDTextureEffect.
R=robertphillips@google.com
Author: jvanverth@google.com
Review URL: https://codereview.chromium.org/
424103002
epoger [Tue, 29 Jul 2014 15:27:11 +0000 (08:27 -0700)]
enable SKP rendering on 2 platforms, 2 configs each
The presence of the expectations file (even if empty) causes the bots to render the SKPs and record results.
2 platforms, 2 configs each should be good for exercising rebaselining tools.
BUG=skia:1942
NOTRY=true
R=rmistry@google.com
TBR=rmistry
Author: epoger@google.com
Review URL: https://codereview.chromium.org/
426023002
bsalomon [Tue, 29 Jul 2014 15:01:52 +0000 (08:01 -0700)]
Some fixes around GrContext::abandonContext:
Fix debug crash when GrResourceCache is destroyed after GrContext is abandoned while GrTextures are in the exlusive list.
Notify debug GL context that GL resources are expected to remain undeleted when context is destroyed after being abandoned.
Stop leaking program cache entries when context is abandoned.
R=robertphillips@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/
422323002
egdaniel [Tue, 29 Jul 2014 14:51:02 +0000 (07:51 -0700)]
Remove gpu support for willUseHWAALines.
By removing willUseHWAALines, we also no long reference the blend opt flag kDisableBlend so it has been removed
BUG=skia:
R=bsalomon@google.com
Author: egdaniel@google.com
Review URL: https://codereview.chromium.org/
423943003
krajcevski [Tue, 29 Jul 2014 14:24:58 +0000 (07:24 -0700)]
Add function for creating ASTC blitter
R=robertphillips@google.com
Author: krajcevski@google.com
Review URL: https://codereview.chromium.org/
427693002
krajcevski [Tue, 29 Jul 2014 14:21:41 +0000 (07:21 -0700)]
Pass clip blitter rows preserved request
R=robertphillips@google.com, reed@google.com
Author: krajcevski@google.com
Review URL: https://codereview.chromium.org/
413403005
robertphillips [Tue, 29 Jul 2014 14:14:49 +0000 (07:14 -0700)]
Now always use Atlas for layer cache
This is setup for running cluster telemetry tests
R=bsalomon@google.com
Author: robertphillips@google.com
Review URL: https://codereview.chromium.org/
429593002
tomhudson [Tue, 29 Jul 2014 13:52:38 +0000 (06:52 -0700)]
Fix debug output compile error and minor style nits
R=djsollen@google.com, tomhudson@google.com
Author: tomhudson@chromium.org
Review URL: https://codereview.chromium.org/
420333002
robertphillips [Tue, 29 Jul 2014 13:17:49 +0000 (06:17 -0700)]
Finish removing GrTHashTable
This class is no longer used in Ganesh (in favor of SkTDynamicHash)
R=jvanverth@google.com
Author: robertphillips@google.com
Review URL: https://codereview.chromium.org/
421253003
kkinnunen [Tue, 29 Jul 2014 13:12:49 +0000 (06:12 -0700)]
Use if instead of ifdef SK_SUPPORT_GPU
The SK_SUPPORT_GPU is always defined.
R=reed@google.com, bsalomon@google.com
Author: kkinnunen@nvidia.com
Review URL: https://codereview.chromium.org/
331353003
robertphillips [Tue, 29 Jul 2014 13:07:19 +0000 (06:07 -0700)]
Add plot-based purging to GrLayerCache
This CL allows a GrPlot full of atlased layer to be purged from the atlas to make room for new layers.
R=jvanverth@google.com
Author: robertphillips@google.com
Review URL: https://codereview.chromium.org/
411703003
borenet [Tue, 29 Jul 2014 07:56:39 +0000 (00:56 -0700)]
Update SKP version to 62
Automatic commit by the RecreateSKPs bot.
TBR=
Author: borenet@google.com
Review URL: https://codereview.chromium.org/
428733004
tfarina [Tue, 29 Jul 2014 02:26:58 +0000 (19:26 -0700)]
Cleanup: Rename SkOSPath functions.
Mostly for brevity and matches better with Python:
Python | Old C++ | New C++
os.path.join | SkOSPath::SkPathJoin | SkOSPath::Join
os.path.basename | SkOSPath::SkBasename | SkOSPath::Basename
BUG=None
TEST=make all
R=mtklein@google.com, bsalomon@google.com
Author: tfarina@chromium.org
Review URL: https://codereview.chromium.org/
428443002
yunchao.he [Tue, 29 Jul 2014 02:18:49 +0000 (19:18 -0700)]
round and bevel hairline rects show the same as miter hairline rects
So round and bevel hairline rects can generate the vertices and indices the same as miter rects do. The original code behaves the same as bevel rects for round and bevel hairline rects.
This small CL can save (vertices and indices) buffer memory as well as improve performance (draw less triangles).
gm cases show no difference after this CL is applied.
BUG=skia:
R=robertphillips@google.com
Author: yunchao.he@intel.com
Review URL: https://codereview.chromium.org/
417113002
cdalton [Mon, 28 Jul 2014 22:39:05 +0000 (15:39 -0700)]
Rebaseline needed nvpr gms after text change
Updates a few nvpr expectations for slight pixel diffs after a change
in the math that positions glyphs.
BUG=skia:
R=jvanverth@google.com, bungeman@google.com
Author: cdalton@nvidia.com
Review URL: https://codereview.chromium.org/
421423002
krajcevski [Mon, 28 Jul 2014 21:14:16 +0000 (14:14 -0700)]
Generalize compressed blitter into its own templated class
R=robertphillips@google.com
Author: krajcevski@google.com
Review URL: https://codereview.chromium.org/
421593004
bsalomon [Mon, 28 Jul 2014 20:48:36 +0000 (13:48 -0700)]
Test abandoning GL context in dm/nanobench.
Rename GrContext::contextDestroyed to GrContext::abandonContext.
Remove GrContext::resetContext.
R=robertphillips@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/
422903002
bsalomon [Mon, 28 Jul 2014 20:43:02 +0000 (13:43 -0700)]
Revert of Remove relative path to GrColor.h in SkShader.h (https://codereview.chromium.org/
422023005/)
Reason for revert:
As Ben suspected, this breaks the chrome build.
Original issue's description:
> Remove relative path to GrColor.h in SkShader.h
>
> Committed: https://skia.googlesource.com/skia/+/
939f430f660d26e46116a38d2a8436afbd55dfb9
R=bungeman@google.com, george@mozilla.com
TBR=bungeman@google.com, george@mozilla.com
NOTREECHECKS=true
NOTRY=true
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/
423943005
bsalomon [Mon, 28 Jul 2014 20:27:35 +0000 (13:27 -0700)]
Remove relative path to GrColor.h in SkShader.h
R=bungeman@google.com, george@mozilla.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/
422023005
halcanary [Mon, 28 Jul 2014 17:54:26 +0000 (10:54 -0700)]
disable new test on windows
R=bensong@google.com, djsollen@google.com
TBR=bensong@google.com, djsollen@google.com
CQ_EXTRA_TRYBOTS=tryserver.skia:Test-Win7-ShuttleA-HD2000-x86_64-Release-Trybot
NOTREECHECKS=true
Author: halcanary@google.com
Review URL: https://codereview.chromium.org/
424833002
halcanary [Mon, 28 Jul 2014 15:56:35 +0000 (08:56 -0700)]
If decoder doesn't ouput indexed color, don't check colortable
R=djsollen@google.com, bungeman@google.com
TBR=bungeman@google.com, djsollen@google.com
Author: halcanary@google.com
Review URL: https://codereview.chromium.org/
427593002
halcanary [Mon, 28 Jul 2014 15:23:55 +0000 (08:23 -0700)]
SkDiscardablePixelRef returns correct indexed color on relock.
R=djsollen@google.com
Author: halcanary@google.com
Review URL: https://codereview.chromium.org/
428603002
bungeman [Mon, 28 Jul 2014 15:19:45 +0000 (08:19 -0700)]
Only lowercase names in Android font config parser.
BUG=chromium:397549
R=tomhudson@chromium.org, tomhudson@google.com
Author: bungeman@google.com
Review URL: https://codereview.chromium.org/
420223003
djsollen [Mon, 28 Jul 2014 14:44:55 +0000 (07:44 -0700)]
Remove code only used by SkCreateTypefaceForScript which has already been deleted
R=tomhudson@google.com, bungeman@google.com
Author: djsollen@google.com
Review URL: https://codereview.chromium.org/
425753002
halcanary [Mon, 28 Jul 2014 14:21:24 +0000 (07:21 -0700)]
dm prints out number of matching skp files
R=bungeman@google.com, mtklein@google.com
Author: halcanary@google.com
Review URL: https://codereview.chromium.org/
421713002
Derek Sollenberger [Mon, 28 Jul 2014 14:04:55 +0000 (10:04 -0400)]
Remove redundant helper function as all clients pass locale
R=tomhudson@google.com
Review URL: https://codereview.chromium.org/
426643003
Derek Sollenberger [Mon, 28 Jul 2014 13:54:10 +0000 (09:54 -0400)]
Remove unused code now that the android framework has move this logic elsewhere.
R=tomhudson@google.com
Review URL: https://codereview.chromium.org/
424663005
borenet [Sun, 27 Jul 2014 05:45:10 +0000 (22:45 -0700)]
Update SKP version to 60
Automatic commit by the RecreateSKPs bot.
TBR=
Author: borenet@google.com
Review URL: https://codereview.chromium.org/
420133002
borenet [Sat, 26 Jul 2014 07:32:37 +0000 (00:32 -0700)]
Update SKP version to 59
Automatic commit by the RecreateSKPs bot.
TBR=
Author: borenet@google.com
Review URL: https://codereview.chromium.org/
422643002
jvanverth [Sat, 26 Jul 2014 02:03:34 +0000 (19:03 -0700)]
Add vertex color support to distance field text.
BUG=skia:
R=bsalomon@google.com
Author: jvanverth@google.com
Review URL: https://codereview.chromium.org/
421563002
jvanverth [Fri, 25 Jul 2014 23:46:53 +0000 (16:46 -0700)]
Move vertex buffer setup out of drawPackedGlyph().
This gets a very welcome ~10% speedup on my Mac.
Committed: https://skia.googlesource.com/skia/+/
1d38619389cc1100bd516b9acbf17b7bd42d33ce
R=bsalomon@google.com
Author: jvanverth@google.com
Review URL: https://codereview.chromium.org/
414573002
bungeman [Fri, 25 Jul 2014 22:11:03 +0000 (15:11 -0700)]
Fix thread unsafe mutex initialization.
BUG=skia:2779
Committed: https://skia.googlesource.com/skia/+/
d6aeb6dc8fe21066f1a2c4813a4256a3acd3edf5
R=robertphillips@google.com, mtklein@google.com, reed@android.com, bsalomon@google.com
Author: bungeman@google.com
Review URL: https://codereview.chromium.org/
419113002
halcanary [Fri, 25 Jul 2014 21:17:35 +0000 (14:17 -0700)]
Fix crash in DM/SKPTask
R=bungeman@google.com
Author: halcanary@google.com
Review URL: https://codereview.chromium.org/
416333007
cdalton [Fri, 25 Jul 2014 21:13:57 +0000 (14:13 -0700)]
Send less transform data when drawing text with nvpr
Before this change, GrStencilAndCoverTextContext would send 6-float
affine transforms to drawPaths for every glyph. This updates it to
concat the text scale onto the context matrix, and then only send
2-float translates (or 1-float x-translates when possible).
Also adds a glyph_pos_align test to gm that exercises the newly added
codepaths, and starts ignoring a few gm tests with benign pixel diffs
until we can rebaseline.
BUG=skia:
R=bsalomon@google.com, kkinnunen@nvidia.com, jvanverth@google.com, bungeman@google.com
Author: cdalton@nvidia.com
Review URL: https://codereview.chromium.org/
406523003
scroggo [Fri, 25 Jul 2014 20:54:43 +0000 (13:54 -0700)]
Fix image decoder memory overwrite bug.
In SkPNGImageDecoder::onDecodeSubset, use png_read_rows to write to
the scratch memory provided. This is what we should have been doing
anyway. Further, writing directly to the bitmap can cause writing
to the wrong memory since the bitmap may not be as big as the
scratch memory in the case of sampling with a short bitmap.
Bug=b/
13921093
R=djsollen@google.com
Author: scroggo@google.com
Review URL: https://codereview.chromium.org/
423473003
bungeman [Fri, 25 Jul 2014 18:52:47 +0000 (11:52 -0700)]
Fix thread unsafe mutex initialization.
BUG=skia:2779
R=robertphillips@google.com, mtklein@google.com, reed@android.com, bsalomon@google.com
Author: bungeman@google.com
Review URL: https://codereview.chromium.org/
419113002
jvanverth [Fri, 25 Jul 2014 18:52:16 +0000 (11:52 -0700)]
Revert of Move vertex buffer setup out of drawPackedGlyph(). (https://codereview.chromium.org/
414573002/)
Reason for revert:
Breaking Ubuntu and Windows bots.
Original issue's description:
> Move vertex buffer setup out of drawPackedGlyph().
>
> This gets a very welcome ~10% speedup on my Mac.
>
> Committed: https://skia.googlesource.com/skia/+/
1d38619389cc1100bd516b9acbf17b7bd42d33ce
R=bsalomon@google.com
TBR=bsalomon@google.com
NOTREECHECKS=true
NOTRY=true
Author: jvanverth@google.com
Review URL: https://codereview.chromium.org/
419943002
jvanverth [Fri, 25 Jul 2014 18:31:13 +0000 (11:31 -0700)]
Move vertex buffer setup out of drawPackedGlyph().
This gets a very welcome ~10% speedup on my Mac.
R=bsalomon@google.com
Author: jvanverth@google.com
Review URL: https://codereview.chromium.org/
414573002
dandov [Fri, 25 Jul 2014 17:44:53 +0000 (10:44 -0700)]
Added classes SkPatch and SkPatchMesh which help encapsulate and generalize this new primitive. The functionality and responsability of each class is better explained in the comments of the files.
Each patch defines a method genMesh that produces the geometry to draw. To do this they receive a SkPatchMesh object which they need to initialize in order to set up how the data is going to be formatted. Later they call function like setColor or pointAt to set the values at a specific index, the SkMeshPatch object handles the indices based on the format and makes it transparent to the client.
Added a slide to sample app to show how to set up this classes and how they interact.
BUG=skia:
R=jvanverth@google.com, egdaniel@google.com, bsalomon@google.com
Author: dandov@google.com
Review URL: https://codereview.chromium.org/
405163003
scroggo [Fri, 25 Jul 2014 17:26:32 +0000 (10:26 -0700)]
Fix assert in skimage test.
The width and height may not match expected width and height when
sampleSize is not 1.
R=djsollen@google.com
Author: scroggo@google.com
Review URL: https://codereview.chromium.org/
418363002
krajcevski [Fri, 25 Jul 2014 17:23:42 +0000 (10:23 -0700)]
Some small ASTC optimizations
R=robertphillips@google.com
Author: krajcevski@google.com
Review URL: https://codereview.chromium.org/
417123003
bsalomon [Fri, 25 Jul 2014 15:35:45 +0000 (08:35 -0700)]
Rename GrGpuObject to GrGpuResource
R=robertphillips@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/
418143004
kelvinly [Fri, 25 Jul 2014 15:30:11 +0000 (08:30 -0700)]
Fix the other microbench schema
BUG=skia:
R=bensong@google.com, jcgregorio@google.com
Author: kelvinly@google.com
Review URL: https://codereview.chromium.org/
420723004
krajcevski [Fri, 25 Jul 2014 14:33:01 +0000 (07:33 -0700)]
Add preliminary ASTC encoder
R=robertphillips@google.com
Author: krajcevski@google.com
Review URL: https://codereview.chromium.org/
417933005
bsalomon [Fri, 25 Jul 2014 14:32:33 +0000 (07:32 -0700)]
Merge GrGpuObject and GrCacheable.
We want to create a new base class for "meta" gr resources as part of the GrResourceCache rewrite and this is an iterim step towards that goal.s
R=robertphillips@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/
414013005
robertphillips [Fri, 25 Jul 2014 12:52:38 +0000 (05:52 -0700)]
Alter Default & Software path renderers to better handle AA hairline paths
For some (as yet unknown reason) the guard in ConicPathBench that disables the path_hairline_big_AA_conic case isn't working. As an alternate fix, this CL makes the Default path renderer confess its lack of support for conics and allows the Software path renderer to handle non-AA cases. This will work but the real fix is to enable conics in the Default path renderer.
Here is the situation we're seeing in the path_hairline_big_AA_conic case:
the path is a hairline drawn with AA
the render target is multisampled (so AA on the draw gets disabled)
StencilAndCover path renderer doesn't handle hairlines
AAHairLine path renderer only handles AA paths (but can handle conics)
AAConvexPath renderer also only handles AA paths
Default path renderer only handles non-AA paths (but doesn't handle conics)
Software path renderer - was disallowed from handling non-AA paths
BUG=skia:2078
R=bsalomon@google.com
Author: robertphillips@google.com
Review URL: https://codereview.chromium.org/
418073002
robertphillips [Fri, 25 Jul 2014 11:34:40 +0000 (04:34 -0700)]
Rebaseline GN & N4 perf trigger flakes
R=borenet@google.com
TBR=borenet@google.com
NOTRY=true
NOTREECHECKS=true
Author: robertphillips@google.com
Review URL: https://codereview.chromium.org/
419683005
borenet [Fri, 25 Jul 2014 05:58:07 +0000 (22:58 -0700)]
Update SKP version to 58
Automatic commit by the RecreateSKPs bot.
TBR=
Author: borenet@google.com
Review URL: https://codereview.chromium.org/
420693002
Brian Salomon [Thu, 24 Jul 2014 21:23:44 +0000 (17:23 -0400)]
Add initial baselines for varied_text* GMs
Review URL: https://codereview.chromium.org/
416983002
djsollen [Thu, 24 Jul 2014 20:53:56 +0000 (13:53 -0700)]
Enable the SSSE3 compile time check on all platforms (4th attempt)
BUG=skia:2746
R=bungeman@google.com, robertphillips@google.com, mtklein@google.com
Author: djsollen@google.com
Review URL: https://codereview.chromium.org/
414033002
epoger [Thu, 24 Jul 2014 18:46:47 +0000 (11:46 -0700)]
check in empty SKP expectations for a single platform
This should trigger the http://108.170.220.120:10117/builders/Test-Mac10.8-MacMini4.1-GeForce320M-x86_64-Debug
to start recording images from the SKPs.
BUG=skia:1942
NOTRY=True
R=rmistry@google.com
TBR=rmistry
Author: epoger@google.com
Review URL: https://codereview.chromium.org/
410423006
epoger [Thu, 24 Jul 2014 17:39:01 +0000 (10:39 -0700)]
roll common DEPS
R=borenet@google.com
TBR=borenet
Author: epoger@google.com
Review URL: https://codereview.chromium.org/
411063006
krajcevski [Thu, 24 Jul 2014 15:16:00 +0000 (08:16 -0700)]
Add ASTC to list of alpha compressed formats
R=bsalomon@google.com, robertphillips@google.com
Author: krajcevski@google.com
Review URL: https://codereview.chromium.org/
412113004
bungeman [Thu, 24 Jul 2014 15:05:09 +0000 (08:05 -0700)]
SkFontMgr for Android.
Committed: https://skia.googlesource.com/skia/+/
4f9a01e03ef3bbe7dd8f9bebdcb3555226087e06
R=djsollen@google.com, tomhudson@google.com, robertphillips@google.com
Author: bungeman@google.com
Review URL: https://codereview.chromium.org/
414483002
scroggo [Thu, 24 Jul 2014 13:55:31 +0000 (06:55 -0700)]
skimage expectations for Android-Reference-Unknown
Fixes build.
NOTRY=true
R=borenet@google.com, robertphillips@google.com
Author: scroggo@google.com
Review URL: https://codereview.chromium.org/
416083002
robertphillips [Thu, 24 Jul 2014 12:30:39 +0000 (05:30 -0700)]
Suppress Win8 raster verylargebitmap GM flakes
BUG=skia:2775
R=bsalomon@google.com
TBR=bsalomon@google.com
NOTRY=true
NOTREECHECKS=true
Author: robertphillips@google.com
Review URL: https://codereview.chromium.org/
418863003
bungeman [Wed, 23 Jul 2014 20:31:06 +0000 (13:31 -0700)]
Get additional DW font metrics when available.
BUG=chromium:395043
R=eae@chromium.org, caryclark@google.com
Author: bungeman@google.com
Review URL: https://codereview.chromium.org/
412993002
bungeman [Wed, 23 Jul 2014 18:28:18 +0000 (11:28 -0700)]
Revert of Enable the SSSE3 compile time check on all platforms. (https://codereview.chromium.org/
403583002/)
Reason for revert:
This is blocking the roll. Chromium Windows trybots (like win_chromium_x64_rel) are crashing in the SSSE3 code (for example SkCanvasVideoRenderTest.CroppedFrame).
Original issue's description:
> Enable the SSSE3 compile time check on all platforms (3rd attempt)
>
> BUG=skia:2746
>
> Committed: https://skia.googlesource.com/skia/+/
933834851f9d48fbd85b728cc92e1f0134bfaa4e
R=halcanary@google.com, mtklein@google.com, djsollen@google.com
TBR=djsollen@google.com, halcanary@google.com, mtklein@google.com
NOTREECHECKS=true
NOTRY=true
BUG=skia:2746
Author: bungeman@google.com
Review URL: https://codereview.chromium.org/
418523002
robertphillips [Wed, 23 Jul 2014 11:55:09 +0000 (04:55 -0700)]
Fix some flaky perf triggers
R=borenet@google.com
TBR=borenet@google.com
NOTRY=true
NOTREECHECKS=true
Author: robertphillips@google.com
Review URL: https://codereview.chromium.org/
416483002
robertphillips [Tue, 22 Jul 2014 23:36:13 +0000 (16:36 -0700)]
Revert of SkFontMgr for Android. (https://codereview.chromium.org/
414483002/)
Reason for revert:
Reverting to unblock DEPS roll.
Original issue's description:
> SkFontMgr for Android.
>
> Committed: https://skia.googlesource.com/skia/+/
4f9a01e03ef3bbe7dd8f9bebdcb3555226087e06
R=djsollen@google.com, tomhudson@google.com, bungeman@google.com
TBR=bungeman@google.com, djsollen@google.com, tomhudson@google.com
NOTREECHECKS=true
NOTRY=true
Author: robertphillips@google.com
Review URL: https://codereview.chromium.org/
408143010
robertphillips [Tue, 22 Jul 2014 23:34:37 +0000 (16:34 -0700)]
Revert "Update fontmgr expectations."
This reverts commit
ff4b56b2d52b0e0042d8b2201fd125a6258dd791.
R=bungeman@gmail.com
TBR=bungeman@gmail.com
NOTRY=true
NOTREECHECKS=true
Author: robertphillips@google.com
Review URL: https://codereview.chromium.org/
406323004
bungeman [Tue, 22 Jul 2014 22:21:24 +0000 (18:21 -0400)]
Update fontmgr expectations.
piotaixr [Tue, 22 Jul 2014 22:02:05 +0000 (15:02 -0700)]
Now able to set the localMatrix when creating a SkShader from a SkImage
BUG=skia:2771
R=junov@chromium.org, reed@chromium.org, bsalomon@chromium.org, bsalomon@google.com
Author: piotaixr@chromium.org
Review URL: https://codereview.chromium.org/
409653003
djsollen [Tue, 22 Jul 2014 20:56:34 +0000 (13:56 -0700)]
fix android_framework build do to missing library at link time
R=scroggo@google.com
Author: djsollen@google.com
Review URL: https://codereview.chromium.org/
407073006
jvanverth [Tue, 22 Jul 2014 20:25:26 +0000 (13:25 -0700)]
Replace use of GrTHashTable in GrFontCache with SkTDynamicHash.
Searching the font cache for existing text strikes was showing up
as a hotspot on Android. This change reduces that cost.
R=bsalomon@google.com, robertphillips@google.com, mtklein@google.com
Author: jvanverth@google.com
Review URL: https://codereview.chromium.org/
390103002
bsalomon [Tue, 22 Jul 2014 20:09:46 +0000 (13:09 -0700)]
Revert "Revert of Use the GrCacheable ID to eliminate the need for notifications to GrGpuGL when textures and RTs areā¦ (https://codereview.chromium.org/
376703009/)"
This reverts commit
249171e7d29b5559f3eefe9dbd437030bfad3fda.
Uses 32 bit id instead of 64. Renamed instanceID to uniqueID to match existing code.
R=robertphillips@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/
414493002
bsalomon [Tue, 22 Jul 2014 20:09:05 +0000 (13:09 -0700)]
Make nanobench setup configs outside of loop over benchmarks
R=mtklein@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/
410683005
bungeman [Tue, 22 Jul 2014 20:02:58 +0000 (16:02 -0400)]
Suppress fontmgr gm changes while Android catches up.
senorblanco [Tue, 22 Jul 2014 19:49:25 +0000 (12:49 -0700)]
Remove the single-sigma version of SkDropShadowImageFilter::Create().
It's no longer used in Chrome.
BUG=skia:
R=bsalomon@google.com
Author: senorblanco@chromium.org
Review URL: https://codereview.chromium.org/
407203002
scroggo [Tue, 22 Jul 2014 19:38:55 +0000 (12:38 -0700)]
Run CanvasState test across a library boundary.
Refactor CanvasStateTest to pull out functions which can
either be called directly or from a shared library. Add a
command line flag to pass in the path to a library to open
to call those functions from.
Separate different CanvasTest tests into separate DEF_TEST
tests. This allows them to be run in parallel.
Move the ifdefs outside of function declarations to skip
running CanvasTest tests which do nothing.
Add the canvas_state_lib target. It is a shared library that
exports some functions to be called by another version of
Skia.
BUG=b/
15693384
R=djsollen@google.com, reed@google.com
Author: scroggo@google.com
Review URL: https://codereview.chromium.org/
400043003
scroggo [Tue, 22 Jul 2014 19:09:30 +0000 (12:09 -0700)]
Introduce skia_pic gyp variable.
skia_pic tells ninja to use -fPIC when building for position
independent code.
Set skia_pic to true when building our existing targets that
require position independent code.
Also use skia_pic when building for Android.
R=mtklein@google.com, djsollen@google.com
Author: scroggo@google.com
Review URL: https://codereview.chromium.org/
406613003
bungeman [Tue, 22 Jul 2014 19:07:14 +0000 (12:07 -0700)]
SkFontMgr for Android.
R=djsollen@google.com, tomhudson@google.com
Author: bungeman@google.com
Review URL: https://codereview.chromium.org/
414483002
bsalomon [Tue, 22 Jul 2014 18:17:14 +0000 (11:17 -0700)]
Remove unused enum value and member of GrGpuObject
R=robertphillips@google.com
TBR=robertphillips@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/
413523002
caryclark [Tue, 22 Jul 2014 17:54:07 +0000 (10:54 -0700)]
remove command line args
R=robertphillips@google.com
TBR=robertphillips
NOTRY=true
NOTREECHECKS=true
Author: caryclark@google.com
Review URL: https://codereview.chromium.org/
408243003
caryclark [Tue, 22 Jul 2014 17:40:06 +0000 (10:40 -0700)]
fix nacl
R=robertphillips@google.com
TBR=robertphillips
NOTRY=true
NOTREECHECKS=true
Author: caryclark@google.com
Review URL: https://codereview.chromium.org/
407163002
robertphillips [Tue, 22 Jul 2014 17:18:06 +0000 (10:18 -0700)]
Add auto purging for SkPicture-related Ganesh resources (esp. layers)
This is intended to lower the bookkeeping burden for the Layer Caching feature. Cached layers are now automatically purged when a picture is deleted.
R=bsalomon@google.com
Author: robertphillips@google.com
Review URL: https://codereview.chromium.org/
408923002
caryclark [Tue, 22 Jul 2014 17:15:34 +0000 (10:15 -0700)]
share dm and command flags
Share command flags between dm and unit tests.
Also, allow dm's core to be included by itself and iOSShell.
Command line flags that are the same (or nearly the same) in DM
and in skia_tests have been moved to common_flags. Authors,
please check to see that the shared common flag is correct for
the tool.
For iOS, the 'tool_main' entry point has a wrapper to allow multiple
tools to be statically linked in the iOSShell.
Since SkCommandLineFlags::Parse can only be called once, these calls
are disabled in the IOS build.
Since the iOS app directory is dynamically assigned a name, use '@' to
select it. (This is the same convention chosen by the Mobile Harness
iOS file system utilities.)
Move the heart of dm.gyp into dm.gypi so that it can be included by
itself and iOSShell.gyp.
Add tools/flags/SkCommonFlags.* to define and declare common
command line flags.
Add support for dm to iOSShell.
BUG=skia:
R=scroggo@google.com, mtklein@google.com, jvanverth@google.com, bsalomon@google.com
Author: caryclark@google.com
Review URL: https://codereview.chromium.org/
389653004
joshualitt [Tue, 22 Jul 2014 16:52:11 +0000 (09:52 -0700)]
Initial change to move 2D kernel to its own file.
BUG=skia:
R=bsalomon@chromium.org, senorblanco@chromium.org, bsalomon@google.com
Author: joshualitt@chromium.org
Review URL: https://codereview.chromium.org/
379253003
krajcevski [Tue, 22 Jul 2014 14:44:00 +0000 (07:44 -0700)]
Refactor texture compressors into separate files
R=robertphillips@google.com
Author: krajcevski@google.com
Review URL: https://codereview.chromium.org/
403383003
sugoi [Tue, 22 Jul 2014 14:26:35 +0000 (07:26 -0700)]
Added count validation in SkImageFilter::Common::unflatten()
BUG=395641
R=reed@google.com, senorblanco@google.com, senorblanco@chromium.org
Author: sugoi@chromium.org
Review URL: https://codereview.chromium.org/
405983006
djsollen [Tue, 22 Jul 2014 14:20:18 +0000 (07:20 -0700)]
Enable the SSSE3 compile time check on all platforms (3rd attempt)
BUG=skia:2746
R=halcanary@google.com, mtklein@google.com
Author: djsollen@google.com
Review URL: https://codereview.chromium.org/
403583002
egdaniel [Tue, 22 Jul 2014 13:46:45 +0000 (06:46 -0700)]
Rebaseline from recent gpu optimization changes
R=bsalomon@google.com
TBR=bsalomon@google.com
BUG=skia:
Author: egdaniel@google.com
Review URL: https://codereview.chromium.org/
408203002
robertphillips [Tue, 22 Jul 2014 12:02:50 +0000 (05:02 -0700)]
Rebaseline some flaky perf triggers
R=borenet@google.com
TBR=borenet@google.com
NOTRY=true
NOTREECHECKS=true
Author: robertphillips@google.com
Review URL: https://codereview.chromium.org/
411513002
cdalton [Mon, 21 Jul 2014 22:32:44 +0000 (15:32 -0700)]
Add a GrPathRange class
Adds a GrPathRange object that represents a range of paths on the gpu.
Updates GrDrawTarget::drawPaths and supporting code to use GrPathRange
instead of an array of GrPath objects.
Change-Id: I67845f3893cd4d955db947d699aa3733cbb081e0
BUG=skia:
R=bsalomon@google.com, jvanverth@google.com, kkinnunen@nvidia.com
Author: cdalton@nvidia.com
Review URL: https://codereview.chromium.org/
400713003
bsalomon [Mon, 21 Jul 2014 21:24:01 +0000 (14:24 -0700)]
Make GrCacheable implement its own ref counting.
R=robertphillips@google.com
Author: bsalomon@google.com
Review URL: https://codereview.chromium.org/
392333008
epoger [Mon, 21 Jul 2014 20:55:57 +0000 (13:55 -0700)]
roll common DEPS
R=borenet@google.com
TBR=borenet
Author: epoger@google.com
Review URL: https://codereview.chromium.org/
404823005
egdaniel [Mon, 21 Jul 2014 20:47:57 +0000 (13:47 -0700)]
Add macro so that gpu trace marker can add an arg to the TRACE_EVENT it makes
BUG=skia:
R=bsalomon@google.com
Author: egdaniel@google.com
Review URL: https://codereview.chromium.org/
379223004
robertphillips [Mon, 21 Jul 2014 19:40:57 +0000 (12:40 -0700)]
Move GrTMultiMap to src/core and rename it
R=bsalomon@google.com
Author: robertphillips@google.com
Review URL: https://codereview.chromium.org/
401343002