rmistry [Mon, 13 Jul 2015 13:35:15 +0000 (06:35 -0700)]
Whitespace change to test Skia's new bugdroid configuration
BUG=chromium:509664
NOTRY=true
TBR=
Review URL: https://codereview.chromium.org/
1237653002
nodir [Mon, 13 Jul 2015 13:23:24 +0000 (06:23 -0700)]
Skia project metadata
Added project.cfg metadata file for chrome-infra
R=borenet@google.com, rmistry@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=507723
Review URL: https://codereview.chromium.org/
1231383004
mtklein [Mon, 13 Jul 2015 13:18:39 +0000 (06:18 -0700)]
Fix up -Winconsistent-missing-override
(and a couple presubmit fixes)
This allows us to turn back on -Werror for LLVM coverage builds,
and more generally supports building with Clang 3.7.
No public API changes.
TBR=reed@google.com
BUG=skia:
Review URL: https://codereview.chromium.org/
1232463006
mtklein [Mon, 13 Jul 2015 13:15:36 +0000 (06:15 -0700)]
Use SK_DECLARE_STATIC_MUTEX for global mutex.
This initializes the mutex at link time on non-Windows platforms,
rather than at pre-main-runtime like `static SkMutex gFoo;`
BUG=skia:
Review URL: https://codereview.chromium.org/
1226403007
Derek Sollenberger [Mon, 13 Jul 2015 12:47:21 +0000 (08:47 -0400)]
Cleanup skia test harness due to changes to DisplayListCanvas
Review URL: https://codereview.chromium.org/
1235823003 .
caryclark [Mon, 13 Jul 2015 12:39:47 +0000 (05:39 -0700)]
make convex poly clip portable
TBR=reed@google.com
Review URL: https://codereview.chromium.org/
1238483002
skia.buildbots [Sun, 12 Jul 2015 07:19:05 +0000 (00:19 -0700)]
Update SKP version
Automatic commit by the RecreateSKPs bot.
TBR=
NO_MERGE_BUILDS
Review URL: https://codereview.chromium.org/
1231323002
ericrk [Fri, 10 Jul 2015 22:05:13 +0000 (15:05 -0700)]
Bilinear optimization for 1D convolution.
Splits GrGLConvolutionEffect into GrGLBilerpConvolutionEffect and
GrGLBoundedConvolutionEffect. When doing a non-bounded convolution we now
always use the GrGLBilerpConvolutionEffect which uses bilinear filtering to
perform half as many samples in the texture.
BUG=skia:3986
Review URL: https://codereview.chromium.org/
1216623003
joshualitt [Fri, 10 Jul 2015 21:14:03 +0000 (14:14 -0700)]
remove some unused stuff
BUG=skia:
Review URL: https://codereview.chromium.org/
1231163002
reed [Fri, 10 Jul 2015 19:08:38 +0000 (12:08 -0700)]
add option to use global mutex-lock in mac-scalercontext
BUG=413332
Review URL: https://codereview.chromium.org/
1220873009
scroggo [Fri, 10 Jul 2015 19:07:02 +0000 (12:07 -0700)]
Allow creating multiple scanline decoders.
Make getScanlineDecoder return a new object each time, which is
owned by the caller, and independent from any existing scanline
decoders and the SkCodec itself.
Since the SkCodec already contains the entire state machine, and it
is used by the scanline decoders, simply create a new SkCodec which
is now owned by the scanline decoder.
Move code that cleans up after using a scanline decoder into its
destructor
One side effect is that creating the first scanline decoder requires
a duplication of the stream and re-reading the header. (With some
more complexity/changes, we could pass the state machine to the
scanline decoder and make the SkCodec recreate its own state machine
instead.) The typical client of the scanline decoder (region decoder)
uses an SkMemoryStream, so the duplication is cheap, although we
should consider the extra time to reread the header/recreate the state
machine. (If/when we use the scanline decoder for other purposes,
where the stream may not be cheaply duplicated, we should consider
passing the state machine.)
One (intended) result of this change is that a client can create a
new scanline decoder in a new thread, and decode different pieces of
the image simultaneously.
In SkPngCodec::decodePalette, use fBitDepth rather than a parameter.
Review URL: https://codereview.chromium.org/
1230033004
joshualitt [Fri, 10 Jul 2015 18:31:01 +0000 (11:31 -0700)]
Trivial cleanup
TBR=robertphillips@google.com
BUG=skia:
Review URL: https://codereview.chromium.org/
1229913007
scroggo [Fri, 10 Jul 2015 18:05:10 +0000 (11:05 -0700)]
Move android_gdb_tmp into out/
This way it will be ignored.
Review URL: https://codereview.chromium.org/
1230773003
msarett [Fri, 10 Jul 2015 17:59:47 +0000 (10:59 -0700)]
Fix msvs builds
BUG=skia:
Review URL: https://codereview.chromium.org/
1229303002
scroggo [Fri, 10 Jul 2015 16:32:09 +0000 (09:32 -0700)]
Use the upstream version of libwebp, v0.4.3.
DEPS:
Update to pull v0.4.3 of libwebp from upstream
gyp/libwebp.gyp:
Add new files, as referenced by the gyp file used by Chromium.
resource/tests:
Add regression tests for particular images.
BUG=skia:3442
BUG=skia:3315
BUG=skia:3429
Committed: https://skia.googlesource.com/skia/+/
3aa0fb4d80c76b559ff4b82d5e569993aea06da1
Review URL: https://codereview.chromium.org/
1178013008
joshualitt [Fri, 10 Jul 2015 16:05:57 +0000 (09:05 -0700)]
fix for valgrind issue
BUG=skia:
Review URL: https://codereview.chromium.org/
1227933003
dongseong.hwang [Fri, 10 Jul 2015 15:32:23 +0000 (08:32 -0700)]
Fix componene debug build failure in chromium
The error log is as follows:
../../third_party/skia/include/core/SkSpinlock.h:24: error: undefined reference to 'SkPODSpinlock::contendedAcquire()'
collect2: error: ld returned 1 exit status
[mtklein added below here]
Despite the presence in include/ and the added SK_API, this file is not part of Skia's public API... it's just used by files which are.
TBR=reed@google.com
Review URL: https://codereview.chromium.org/
1229003004
emmaleer [Fri, 10 Jul 2015 14:48:03 +0000 (07:48 -0700)]
Changing name of has_transpareny_in_palette to has_transparency_in_tRNS
This function checks for transparency info in the tRNS chunk
Other images besides paletted images can have tRNS chunks,
including Gray8 and RGB
has_transparency_in_tRNS is a better name for the function
BUG=skia:
Review URL: https://codereview.chromium.org/
1224453002
joshualitt [Fri, 10 Jul 2015 14:26:21 +0000 (07:26 -0700)]
Move GrProcessorDataManager to GrMemoryPool
TBR=bsalomon@google.com
BUG=skia:
Review URL: https://codereview.chromium.org/
1225363002
joel.liang [Fri, 10 Jul 2015 02:46:18 +0000 (19:46 -0700)]
Workaround for blacklist KHR_blend_equation_advanced on ARM GPU
ARM driver will check the fragment shader compatiblity for KHR_blend_equation_advanced even if blending is disabled.
Workaround: Set blending equation to any basic equation when we disable blending.
https://code.google.com/p/skia/issues/detail?id=3943
BUG=skia:3943
Review URL: https://codereview.chromium.org/
1216963004
fmalita [Fri, 10 Jul 2015 00:10:31 +0000 (17:10 -0700)]
Fix SkImage::asLegacyBitmap() rowBytes assert
When reusing the pixel ref, we need to observe its rowBytes.
R=reed@google.com
Review URL: https://codereview.chromium.org/
1227413002
mtklein [Thu, 9 Jul 2015 22:41:11 +0000 (15:41 -0700)]
Let's not leak this SkData.
BUG=skia:
Review URL: https://codereview.chromium.org/
1231903006
reed [Thu, 9 Jul 2015 21:08:49 +0000 (14:08 -0700)]
remove getter for imagegenerator factory, update test
BUG=skia:
Review URL: https://codereview.chromium.org/
1223413002
herb [Thu, 9 Jul 2015 20:44:32 +0000 (13:44 -0700)]
Create a template AutoTAcquire, and specialize an SkMutex version.
Add copyright notice.
BUG=skia:
Review URL: https://codereview.chromium.org/
1230583008
joshualitt [Thu, 9 Jul 2015 20:25:02 +0000 (13:25 -0700)]
Remove setFromPaint from GrPipelineBuilder
TBR=bsalomon@google.com
BUG=skia:
Review URL: https://codereview.chromium.org/
1227153003
joshualitt [Thu, 9 Jul 2015 19:17:44 +0000 (12:17 -0700)]
Trivial cleanup of copying GrPipelineBuilder
TBR=robertphillips@google.com
BUG=skia:
Review URL: https://codereview.chromium.org/
1229983003
reed [Thu, 9 Jul 2015 18:47:36 +0000 (11:47 -0700)]
add runtime option to provide data->imagegenerator factory
BUG=skia:
Review URL: https://codereview.chromium.org/
1229933003
herb [Thu, 9 Jul 2015 18:37:35 +0000 (11:37 -0700)]
Add RAII for other types of locks. Change SkGlyphCache_Globals to
use a SkSpinlock instead of SkMutex. This results in significant
performance increase for mpd in nanobench.
BUG=skia:
Review URL: https://codereview.chromium.org/
1210143004
joshualitt [Thu, 9 Jul 2015 17:55:28 +0000 (10:55 -0700)]
Remove init function from GrTextContext.h
BUG=skia:
Review URL: https://codereview.chromium.org/
1231923002
mtklein [Thu, 9 Jul 2015 17:51:36 +0000 (10:51 -0700)]
Outline SkSpinlock::acquire().
The proportion of time spent doing useful work is well over 99% in acquire(),
so outlining it doesn't hurt speed at all, and makes it much easier to pick out
on a profile.
It'd be about 50/50 work/overhead if we outlined the extremely-cheap release().
I also tried outlining some SkRefCnt methods with similar mixed results.
BUG=skia:
No public API changes.
TBR=reed@google.com
Review URL: https://codereview.chromium.org/
1212253013
herb [Thu, 9 Jul 2015 17:50:24 +0000 (10:50 -0700)]
Change to use mean and to use stderr.
BUG=skia:
Review URL: https://codereview.chromium.org/
1228783003
joshualitt [Thu, 9 Jul 2015 17:24:35 +0000 (10:24 -0700)]
More threading of GrProcessorDataManager
TBR=bsalomon@google.com
BUG=skia:
Review URL: https://codereview.chromium.org/
1230813003
mtklein [Thu, 9 Jul 2015 17:04:56 +0000 (10:04 -0700)]
Have nanobench pay attention to --threads.
TBR=
BUG=skia:
Review URL: https://codereview.chromium.org/
1229953002
scroggo [Thu, 9 Jul 2015 16:08:00 +0000 (09:08 -0700)]
Remove SkImageGenerator pieces only for SkCodec.
Follow up to the split between SkImageGenerator and SkCodec. Now that
SkCodec does not inherit from SkImageGenerator, SkImageGenerator no
longer needs Options or Result, which were added for SkCodec. Remove
them, but keep them behind a flag, since Chromium has its own
subclasses of SkImageGenerator which assume the old signature for
onGetPixels.
Review URL: https://codereview.chromium.org/
1226023003
jvanverth [Thu, 9 Jul 2015 16:04:16 +0000 (09:04 -0700)]
Add normal map sample
This adds an example of an SkShader that does normal
mapping. It has a single directional light and an
ambient light.
Committed: https://skia.googlesource.com/skia/+/
8e0da72ba890de395c9946ec6639c9e1e7b16027
Review URL: https://codereview.chromium.org/
1212813009
msarett [Thu, 9 Jul 2015 15:23:50 +0000 (08:23 -0700)]
Temporarily suppress valgrind errors in tip of tree libjpeg-turbo
BUG=skia:4030
Review URL: https://codereview.chromium.org/
1231613004
scroggo [Thu, 9 Jul 2015 15:16:03 +0000 (08:16 -0700)]
SkCodec no longer inherits from SkImageGenerator.
SkImageGenerator makes some assumptions that are not necessarily valid
for SkCodec. For example, SkCodec does not assume that it can always be
rewound.
We also have an ongoing question of what an SkCodec should report as
its default settings (i.e. the return from getInfo). It makes sense for
an SkCodec to report that its pixels are unpremultiplied, if that is
the case for the underlying data, but if a client of SkImageGenerator
uses the default settings (as many do), they will receive
unpremultiplied pixels which cannot (currently) be drawn with Skia. We
may ultimately decide to revisit SkCodec reporting an SkImageInfo, but
I have left it unchanged for now.
Import features of SkImageGenerator used by SkCodec into SkCodec.
I have left SkImageGenerator unchanged for now, but it no longer needs
Result or Options. This will require changes to Chromium.
Manually handle the lifetime of fScanlineDecoder, so SkScanlineDecoder.h
can include SkCodec.h (where Result is), and SkCodec.h does not need
to include it (to delete fScanlineDecoder).
In many places, make the following simple changes:
- Now include SkScanlineDecoder.h, which is no longer included by
SkCodec.h
- Use the enums in SkCodec, rather than SkImageGenerator
- Stop including SkImageGenerator.h where no longer needed
Review URL: https://codereview.chromium.org/
1220733013
msarett [Thu, 9 Jul 2015 14:55:01 +0000 (07:55 -0700)]
Do not try to compile x86 and x86_64 .asm files on Android
This is currently only works on Linux hosts because we have checked
in a binary and does not work on Mac hosts. We are disabling until
we find a suitable solution.
BUG=skia:4028
Review URL: https://codereview.chromium.org/
1228823006
joshualitt [Thu, 9 Jul 2015 14:31:31 +0000 (07:31 -0700)]
YUV to RGB Texture threading GrProcessorDataManager
BUG=skia:
Review URL: https://codereview.chromium.org/
1230803002
halcanary [Thu, 9 Jul 2015 13:58:06 +0000 (06:58 -0700)]
doc: more changes
NOTRY=true
DOCS_PREVIEW= https://skia.org/user/api/skcanvas?cl=
1228063002
DOCS_PREVIEW= https://skia.org/user/tips?cl=
1228063002
TBR=
Review URL: https://codereview.chromium.org/
1228063002
joshualitt [Thu, 9 Jul 2015 13:28:14 +0000 (06:28 -0700)]
rename GrShaderDataManager -> GrProcessorDataManager
BUG=skia:
Review URL: https://codereview.chromium.org/
1228683002
robertphillips [Wed, 8 Jul 2015 21:46:12 +0000 (14:46 -0700)]
Revert of Add normal map sample (patchset #6 id:100001 of https://codereview.chromium.org/
1212813009/)
Reason for revert:
Compilation failures:
../../../../../samplecode/SampleLighting.cpp:13:18: fatal error: SkGr.h: No such file or directory
Original issue's description:
> Add normal map sample
>
> This adds an example of an SkShader that does normal
> mapping. It has a single directional light and an
> ambient light.
>
> Committed: https://skia.googlesource.com/skia/+/
8e0da72ba890de395c9946ec6639c9e1e7b16027
TBR=reed@google.com,bsalomon@google.com,jvanverth@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/
1230603002
joshualitt [Wed, 8 Jul 2015 21:26:19 +0000 (14:26 -0700)]
fix up test create functions
TBR=bsalomon@google.com
BUG=skia:
Review URL: https://codereview.chromium.org/
1213623022
jvanverth [Wed, 8 Jul 2015 21:21:47 +0000 (14:21 -0700)]
Add normal map sample
This adds an example of an SkShader that does normal
mapping. It has a single directional light and an
ambient light.
Review URL: https://codereview.chromium.org/
1212813009
halcanary [Wed, 8 Jul 2015 20:58:45 +0000 (13:58 -0700)]
SkAdvancedTypefaceMetrics zeroing constructor
Review URL: https://codereview.chromium.org/
1221903004
dcheng [Wed, 8 Jul 2015 20:25:23 +0000 (13:25 -0700)]
Remove another assertion that a reference is non-null.
This tickles a warning in Clang:
reference cannot be bound to dereferenced null pointer in well-defined
C++ code; pointer may be assumed to always convert to true
[-Wundefined-bool-conversion]
BUG=none
Review URL: https://codereview.chromium.org/
1225053003
ethannicholas [Wed, 8 Jul 2015 20:19:51 +0000 (13:19 -0700)]
disabled GrAALinearizingConvexRenderer for now
BUG=505579
Review URL: https://codereview.chromium.org/
1219653004
joshualitt [Wed, 8 Jul 2015 19:54:04 +0000 (12:54 -0700)]
add ability to get FBO ID to Surface
BUG=skia:
Review URL: https://codereview.chromium.org/
1220733007
reed [Wed, 8 Jul 2015 19:46:22 +0000 (12:46 -0700)]
Add image->bitmap
BUG=skia:
patch from issue
1212163012 at patchset 1 (http://crrev.com/
1212163012#ps1)
Review URL: https://codereview.chromium.org/
1208993017
halcanary [Wed, 8 Jul 2015 18:34:36 +0000 (11:34 -0700)]
doc: embed fiddle
NOTRY=true
DOCS_PREVIEW= https://skia.org/user/api/skcanvas?cl=
1149633009
DOCS_PREVIEW= https://skia.org/user/api/skpaint?cl=
1149633009
Review URL: https://codereview.chromium.org/
1149633009
fmalita [Wed, 8 Jul 2015 18:34:20 +0000 (11:34 -0700)]
Remove SkImage::NewFromBitmap encoded data helper
Attempting to instantiate image decoders at this stage introduces
embedder dependencies.
Instead, we can just let callers set up any needed generators on the
bitmap.
R=reed@google.com
Review URL: https://codereview.chromium.org/
1218663012
bsalomon [Wed, 8 Jul 2015 18:26:37 +0000 (11:26 -0700)]
Makes GrPipelineInfo a class with query functions used by GrBatch subclasses.
Committed: https://skia.googlesource.com/skia/+/
f5179a4c490bc787190321bd8ffdb0e6a4efa9ac
Review URL: https://codereview.chromium.org/
1213383005
scroggo [Wed, 8 Jul 2015 17:57:22 +0000 (10:57 -0700)]
Revert of Use the upstream version of libwebp, v0.4.3. (patchset #6 id:70001 of https://codereview.chromium.org/
1178013008/)
Reason for revert:
Breaking the build e.g. http://build.chromium.org/p/client.skia/builders/Perf-Mac10.8-Clang-MacMini4.1-GPU-GeForce320M-x86_64-Release/builds/1082/steps/build%20nanobench/logs/stdio
../../../third_party/externals/libwebp/src/utils/./endian_inl.h:51:10: error: use of unknown builtin '__builtin_bswap16' [-Wimplicit-function-declaration]
return __builtin_bswap16(x);
^
1 error generated.
Original issue's description:
> Use the upstream version of libwebp, v0.4.3.
>
> DEPS:
> Update to pull v0.4.3 of libwebp from upstream
>
> gyp/libwebp.gyp:
> Add new files, as referenced by the gyp file used by Chromium.
>
> resource/tests:
> Add regression tests for particular images.
>
> BUG=skia:3442
> BUG=skia:3315
> BUG=skia:3429
>
> Committed: https://skia.googlesource.com/skia/+/
3aa0fb4d80c76b559ff4b82d5e569993aea06da1
TBR=djsollen@google.com,jzern@chromium.org,msarett@google.com,emmaleer@google.com,scroggo@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:3442
Review URL: https://codereview.chromium.org/
1223583004
halcanary [Wed, 8 Jul 2015 17:56:01 +0000 (10:56 -0700)]
doc: add instructions to capture a `.skp` from chromium
NOTRY=true
DOCS_PREVIEW= https://skia.org/user/tips?cl=
1220783005
Review URL: https://codereview.chromium.org/
1220783005
djsollen [Wed, 8 Jul 2015 17:50:42 +0000 (10:50 -0700)]
Add gradle wrappers for Android app builds
These wrappers are approxiately 56K in size and are the recommened
way to use Gradle. It also ensures that developers wanting to
build the app don't need install an additional dependency.
Review URL: https://codereview.chromium.org/
1227723002
scroggo [Wed, 8 Jul 2015 17:48:40 +0000 (10:48 -0700)]
Use the upstream version of libwebp, v0.4.3.
DEPS:
Update to pull v0.4.3 of libwebp from upstream
gyp/libwebp.gyp:
Add new files, as referenced by the gyp file used by Chromium.
resource/tests:
Add regression tests for particular images.
BUG=skia:3442
BUG=skia:3315
BUG=skia:3429
Review URL: https://codereview.chromium.org/
1178013008
bsalomon [Wed, 8 Jul 2015 17:20:21 +0000 (10:20 -0700)]
Revert of Rework GrPipelineInfo (patchset #7 id:120001 of https://codereview.chromium.org/
1213383005/)
Reason for revert:
breaking stuff!
Original issue's description:
> Makes GrPipelineInfo a class with query functions used by GrBatch subclasses.
>
> Committed: https://skia.googlesource.com/skia/+/
f5179a4c490bc787190321bd8ffdb0e6a4efa9ac
TBR=joshualitt@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/
1213013003
mtklein [Wed, 8 Jul 2015 17:06:01 +0000 (10:06 -0700)]
Add tools/flatten to flatten nested pictures in .skps.
Tested by running on skps/sp_desk_nytimes.skp.
The output .skp had no nested draw picture calls, and the files were both 9.3M.
BUG=skia:
Review URL: https://codereview.chromium.org/
1221303020
joshualitt [Wed, 8 Jul 2015 16:36:59 +0000 (09:36 -0700)]
more threading of GrShaderDataManager
TBR=bsalomon@google.com
BUG=skia:
Review URL: https://codereview.chromium.org/
1215643006
joshualitt [Wed, 8 Jul 2015 16:10:03 +0000 (09:10 -0700)]
Thread GrShaderDataManager through Image filters
TBR=bsalomon@google.com
BUG=skia:
Review URL: https://codereview.chromium.org/
1229613002
mtklein [Wed, 8 Jul 2015 15:49:20 +0000 (08:49 -0700)]
DM runs more than 100k tests now.
BUG=skia:
Review URL: https://codereview.chromium.org/
1217493006
mtklein [Wed, 8 Jul 2015 15:26:39 +0000 (08:26 -0700)]
Fix leak in PictureTest.cpp
TBR=schenney@chromium.org
BUG=skia:
Review URL: https://codereview.chromium.org/
1210053008
joshualitt [Wed, 8 Jul 2015 14:58:18 +0000 (07:58 -0700)]
More threading of GrShaderDataManager
TBR=bsalomon@google.com
BUG=skia:
Review URL: https://codereview.chromium.org/
1213613016
bsalomon [Wed, 8 Jul 2015 14:55:59 +0000 (07:55 -0700)]
Makes GrPipelineInfo a class with query functions used by GrBatch subclasses.
Review URL: https://codereview.chromium.org/
1213383005
bsalomon [Wed, 8 Jul 2015 14:53:10 +0000 (07:53 -0700)]
Add SK_API to GrSurface
TBR=robertphillips@google.com
Review URL: https://codereview.chromium.org/
1209043012
mtklein [Wed, 8 Jul 2015 14:25:27 +0000 (07:25 -0700)]
DM: swizzle BGRA to RGBA before calculating pixel MD5.
We name our .pngs by pixel hashes for gold. For 8888 images, we're hashing
SkPMColors, which have platform-dependent order: BGRA on Linux and Windows,
RGBA otherwise. This means we can end up with pixel-identical pngs with
different hashes, which is confusing.
This CL standardizes on RGBA for 8888 configs, arbitrarily chosen so that
Android ends up a no-op. Long-term, this should eliminate most of the
0-pixel-diff problems we see on gold.skia.org. There are other ways to end up
with the same .png from different SkBitmaps (think, red 565 square vs. red 8888
square) but they're rather less common / likely.
This will temporarily create a giant 0-pixel-diff problem on gold.skia.org.
Any Linux or Windows images which are not already pixel-identical to a Mac or
Android image should show up as untriaged hashes that are pixel-identical to
their version just before landing (we're only changing the hash, not the .png).
This means anything vaguely platform dependent (fonts, GPUs) will probably show
up as needing a triage but with a zero diff from a previous image.
If this goes well, we might do the same for 565. Just want to leave them out
for now to cut down on the triaging I need to do in one go.
BUG=skia:
Review URL: https://codereview.chromium.org/
1226933005
joshualitt [Wed, 8 Jul 2015 13:51:43 +0000 (06:51 -0700)]
Initial CL to create dummy GrShaderDataManager and thread it through
TBR=bsalomon@google.com
BUG=skia:
Review URL: https://codereview.chromium.org/
1225673007
dcheng [Wed, 8 Jul 2015 02:35:53 +0000 (19:35 -0700)]
Remove &glyph from SkASSERT.
This tickles a warning in Clang:
reference cannot be bound to dereferenced null pointer in well-defined
C++ code; pointer may be assumed to always convert to true
[-Wundefined-bool-conversion]
BUG=none
Review URL: https://codereview.chromium.org/
1220113003
caryclark [Wed, 8 Jul 2015 01:09:32 +0000 (18:09 -0700)]
fix pathops battle test
The intermediate subtraction threw away bits causing
the subsequent cross product to fail. Defer the
bit truncation until after the vector is computed.
TBR=reed@google.com
Review URL: https://codereview.chromium.org/
1226933003
wfh [Wed, 8 Jul 2015 01:06:19 +0000 (18:06 -0700)]
Add check that Dwrite glyph fits in unicode table.
BUG=470146
Review URL: https://codereview.chromium.org/
1218043004
schenney [Tue, 7 Jul 2015 21:27:10 +0000 (14:27 -0700)]
Allow reset of the SkPictureRecorder cull rect and other parameters during endRecording.
For some users of SkPictureRecorder, the cull rect is more efficiently
determined while drawing is in progress, rather than when recording starts.
The existing API requires the cull rect at start time, even though the
information is not used for any culling purpose until the end of recording.
This patch provides a means to reset the cull rect when recording ends,
allowing users to update the rect based on information learned during
drawing and for the new rect to be used as the culling bound. A valid
bound is still required on the beginRecording call because
it sizes the underlying canvas and sets the aspect ratio for any bounding
box hierarchy. The bounding box factory can also be specified and parameters
that control SkPicture creation.
R=mtklein, reed1
BUG=skia:3919
Review URL: https://codereview.chromium.org/
1178673007
ethannicholas [Tue, 7 Jul 2015 19:41:52 +0000 (12:41 -0700)]
When GrAAConvesTesselator was merging the first and last point of the array, it was deleting the last point without checking to see if any of the previous points shared the same index. This led to continuing to use the index of a deleted point and thus referencing memory past the end of the vertex array.
BUG=chromium:505227
Review URL: https://codereview.chromium.org/
1225683008
mtklein [Tue, 7 Jul 2015 19:21:21 +0000 (12:21 -0700)]
Remove SkThread.h, include SkMutex.h or SkAtomics.h as appropriate.
SkThread.h doesn't do anything anymore execept include those two,
and thankfully, it doesn't seem to be mentioned outside Skia.
No public API changes.
TBR=reed@google.com
BUG=skia:
Committed: https://skia.googlesource.com/skia/+/
c50acf2321d7a934c80d754e9cbe936dfb8eb4cc
Review URL: https://codereview.chromium.org/
1215393002
robertphillips [Tue, 7 Jul 2015 17:28:43 +0000 (10:28 -0700)]
Propagate SkSurfaceProps when known
This CL just closes the loop on some low hanging fruit w.r.t. propagating surface properties.
Review URL: https://codereview.chromium.org/
1214853025
reed [Tue, 7 Jul 2015 17:22:31 +0000 (10:22 -0700)]
add matrix options to drawDrawable
BUG=skia:
Review URL: https://codereview.chromium.org/
1224783002
robertphillips [Tue, 7 Jul 2015 17:05:18 +0000 (10:05 -0700)]
style nit cleanups. e.g. remove 'virtual' on override declarations
Review URL: https://codereview.chromium.org/
1216433005
mtklein [Tue, 7 Jul 2015 16:43:28 +0000 (09:43 -0700)]
Make sp- via support SkDrawables; add a GM to test it.
BUG=skia:
Review URL: https://codereview.chromium.org/
1212363005
msarett [Tue, 7 Jul 2015 15:50:01 +0000 (08:50 -0700)]
Add libjpeg-turbo library (depends on yasm)
Mangle external function names to avoid conflict with libjpeg
Take advantage of direct color conversion (RGBA, BGRA, 565)
Prepare to use jpeg_skip_scanlines (when it is upstreamed)
BUG=skia:
Committed: https://skia.googlesource.com/skia/+/
b60c3f8291529303299262dba19b1a896060bd2d
Committed: https://skia.googlesource.com/skia/+/
f8bf9181d7b0463c8e371755cfbb9ece90b34fc5
Committed: https://skia.googlesource.com/skia/+/
e9e3ee33f30c14c31afd5fc3fe4dda7f15783c75
Committed: https://skia.googlesource.com/skia/+/
40141b57f061fbfcc2fa38da942d9efe25aca4d0
Review URL: https://codereview.chromium.org/
1180983002
reed [Tue, 7 Jul 2015 13:11:19 +0000 (06:11 -0700)]
add SkImage::NewFromBitmap
BUG=skia:
Review URL: https://codereview.chromium.org/
1217053003
scroggo [Tue, 7 Jul 2015 13:09:08 +0000 (06:09 -0700)]
Revert of Revert of Fixing libpng transform use (patchset #1 id:1 of https://codereview.chromium.org/
1213743004/)
Reason for revert:
This appears to have been reverted in order to fix the DEPS roll in https://codereview.chromium.org/
1214943004/
However, it only affects SkCodec, which is not used by Chromium. Relanding
Original issue's description:
> Revert of Fixing libpng transform use (patchset #5 id:80001 of https://codereview.chromium.org/
1214203005/)
>
> Reason for revert:
> DEPS roll failing
>
> Original issue's description:
> > This change:
> > - supports kGray correctly
> > - avoid extra call to png_get_IHDR by storing the bit depth
> > - call transforms as needed
> > - checks for tRNS alpha value in RGB and GRAY color types
> >
> >
> > BUG=skia:
> >
> > Committed: https://skia.googlesource.com/skia/+/
9693037fd41b7ce545b44beaa3489dcfd915018c
>
> TBR=scroggo@google.com,msarett@google.com,emmaleer@google.com
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/
6c90e09575c1a77aee060aa475fdb3d25a17d6a0
TBR=msarett@google.com,emmaleer@google.com,jvanverth@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/
1215853005
robertphillips [Mon, 6 Jul 2015 19:15:34 +0000 (12:15 -0700)]
Fix up SkMatrix::isFinite call
Review URL: https://codereview.chromium.org/
1228473004
caryclark [Mon, 6 Jul 2015 18:38:33 +0000 (11:38 -0700)]
When three or more edges are coincident, the logic needs
to compute the overlapping ranges and combine the winding
into a single destination.
This computes coincidence more rigorously, fixing the
edge cases exposed by this bug.
Also, add the ability to debug and dump pathop structures
from the coincident context.
TBR=reed@google.com
BUG=skia:3651
Review URL: https://codereview.chromium.org/
1182493015
borenet [Mon, 6 Jul 2015 18:18:45 +0000 (11:18 -0700)]
Add nanobench-style JSON output to llvm_coverage_run
Runs the build and run scripts for coverage, then dumps the results into
a file in nanobench-compatible format.
BUG=skia:2430
Review URL: https://codereview.chromium.org/
1227523004
fmalita [Mon, 6 Jul 2015 15:20:15 +0000 (08:20 -0700)]
Use kLow_SkFilterQuality in DrawBitmapAABench
BUG=skia:4001
R=reed@google.com
Review URL: https://codereview.chromium.org/
1222713007
robertphillips [Mon, 6 Jul 2015 14:59:09 +0000 (07:59 -0700)]
Revert of Revert of Revert of Fix SkTileImageFilter clipping/cropRect interaction issue (patchset #1 id:1 of https://codereview.chromium.org/
1207353004/)
Reason for revert:
Still want to disable due to pref regression
Original issue's description:
> Revert of Revert of Fix SkTileImageFilter clipping/cropRect interaction issue (patchset #1 id:1 of https://codereview.chromium.org/
1219193002/)
>
> Reason for revert:
> Blocking the roll
>
> Original issue's description:
> > Revert of Fix SkTileImageFilter clipping/cropRect interaction issue (patchset #2 id:30001 of https://codereview.chromium.org/
1210053003/)
> >
> > Reason for revert:
> > Perf regression: https://code.google.com/p/chromium/issues/detail?id=505564
> >
> > Original issue's description:
> > > Fix SkTileImageFilter clipping/cropRect interaction issue
> > >
> > > BUG=499499
> > >
> > > Committed: https://skia.googlesource.com/skia/+/
157bcd0840b578060dbc3365daafffc6837da391
> >
> > TBR=reed@google.com,senorblanco@google.com,senorblanco@chromium.org,robertphillips@google.com
> > NOPRESUBMIT=true
> > NOTREECHECKS=true
> > NOTRY=true
> > BUG=499499
> >
> > Committed: https://skia.googlesource.com/skia/+/
ebaf6a69bf604c85185e23aca3fb93308e747ff5
>
> TBR=reed@google.com,senorblanco@google.com,senorblanco@chromium.org,robertphillips@google.com,reed@chromium.org,bsalomon@google.com
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=499499
>
> Committed: https://skia.googlesource.com/skia/+/
835510085062f055c04d8ea46d82831cfbe51793
TBR=reed@google.com,senorblanco@google.com,senorblanco@chromium.org,reed@chromium.org,bsalomon@google.com,jvanverth@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=499499
Review URL: https://codereview.chromium.org/
1217303004
borenet [Mon, 6 Jul 2015 14:43:58 +0000 (07:43 -0700)]
Add scripts for running LLVM coverage
BUG=skia:2430
Review URL: https://codereview.chromium.org/
1213063009
fmalita [Mon, 6 Jul 2015 12:25:17 +0000 (05:25 -0700)]
Add a bench for measuring drawBitmap anti-aliasing overhead
R=reed@google.com,mtklein@google.com
Review URL: https://codereview.chromium.org/
1207403007
skia.buildbots [Mon, 6 Jul 2015 12:25:09 +0000 (05:25 -0700)]
Update SKP version
Automatic commit by the RecreateSKPs bot.
TBR=
NO_MERGE_BUILDS
Review URL: https://codereview.chromium.org/
1222483003
reed [Sat, 4 Jul 2015 04:01:10 +0000 (21:01 -0700)]
add colortable param to newrastercopy
BUG=skia:
TBR=
NOTREECHECKS=True
Review URL: https://codereview.chromium.org/
1222683004
reed [Fri, 3 Jul 2015 17:43:43 +0000 (10:43 -0700)]
explicitly bump legacy genID on gpu surface
Previous attempt to "test" getTextureHandle was giving false-positive because we had an active image-snapshot, which was incidentally also triggering a new genID.
I think this CL will also enable this change: https://codereview.chromium.org/
1222243002/
BUG=skia:
TBR=bsalomon@google.com
NOTREECHECKS=True
Review URL: https://codereview.chromium.org/
1208993008
jvanverth [Fri, 3 Jul 2015 12:48:53 +0000 (05:48 -0700)]
Revert of Revert of Fix SkTileImageFilter clipping/cropRect interaction issue (patchset #1 id:1 of https://codereview.chromium.org/
1219193002/)
Reason for revert:
Blocking the roll
Original issue's description:
> Revert of Fix SkTileImageFilter clipping/cropRect interaction issue (patchset #2 id:30001 of https://codereview.chromium.org/
1210053003/)
>
> Reason for revert:
> Perf regression: https://code.google.com/p/chromium/issues/detail?id=505564
>
> Original issue's description:
> > Fix SkTileImageFilter clipping/cropRect interaction issue
> >
> > BUG=499499
> >
> > Committed: https://skia.googlesource.com/skia/+/
157bcd0840b578060dbc3365daafffc6837da391
>
> TBR=reed@google.com,senorblanco@google.com,senorblanco@chromium.org,robertphillips@google.com
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=499499
>
> Committed: https://skia.googlesource.com/skia/+/
ebaf6a69bf604c85185e23aca3fb93308e747ff5
TBR=reed@google.com,senorblanco@google.com,senorblanco@chromium.org,robertphillips@google.com,reed@chromium.org,bsalomon@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=499499
Review URL: https://codereview.chromium.org/
1207353004
jvanverth [Thu, 2 Jul 2015 19:53:27 +0000 (12:53 -0700)]
Revert of Implement support for CHROMIUM_path_rendering pseudo extension (patchset #4 id:60001 of https://codereview.chromium.org/
1192663002/)
Reason for revert:
DEPS roll failing
Original issue's description:
> Implement support for CHROMIUM_path_rendering pseudo extension
>
> Implement support for path rendering in Chromium through
> CHROMIUM_path_rendering pseudo extension.
>
> The extension defines a new pseudo-gl function,
> BindFragmentInputLocation. This behaves similarly to the
> BindUniformLocation pseudo-gl function. The idea is to assign fragment
> input location to a fragment input before linking the program.
>
> BUG=chromium:344330
>
> Committed: https://skia.googlesource.com/skia/+/
eeef46d181f9f8db388ecea81df699fc1b3c9280
TBR=bsalomon@google.com,joshualitt@google.com,kkinnunen@nvidia.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=chromium:344330
Review URL: https://codereview.chromium.org/
1223673002
jvanverth [Thu, 2 Jul 2015 17:40:24 +0000 (10:40 -0700)]
Revert of Switch SkJpegCode to libjpeg-turbo (patchset #29 id:750001 of https://codereview.chromium.org/
1180983002/)
Reason for revert:
DEPS roll failing
Original issue's description:
> Add libjpeg-turbo library (depends on yasm)
> Mangle external function names to avoid conflict with libjpeg
> Take advantage of direct color conversion (RGBA, BGRA, 565)
> Prepare to use jpeg_skip_scanlines (when it is upstreamed)
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/
b60c3f8291529303299262dba19b1a896060bd2d
>
> Committed: https://skia.googlesource.com/skia/+/
f8bf9181d7b0463c8e371755cfbb9ece90b34fc5
>
> Committed: https://skia.googlesource.com/skia/+/
e9e3ee33f30c14c31afd5fc3fe4dda7f15783c75
>
> Committed: https://skia.googlesource.com/skia/+/
40141b57f061fbfcc2fa38da942d9efe25aca4d0
TBR=scroggo@google.com,djsollen@google.com,emmaleer@google.com,msarett@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/
1226543003
jvanverth [Thu, 2 Jul 2015 17:35:25 +0000 (10:35 -0700)]
Revert of Fixing libpng transform use (patchset #5 id:80001 of https://codereview.chromium.org/
1214203005/)
Reason for revert:
DEPS roll failing
Original issue's description:
> This change:
> - supports kGray correctly
> - avoid extra call to png_get_IHDR by storing the bit depth
> - call transforms as needed
> - checks for tRNS alpha value in RGB and GRAY color types
>
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/
9693037fd41b7ce545b44beaa3489dcfd915018c
TBR=scroggo@google.com,msarett@google.com,emmaleer@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/
1213743004
jvanverth [Thu, 2 Jul 2015 17:35:08 +0000 (10:35 -0700)]
Revert of Fix libjpeg-turbo deps file (patchset #2 id:2 of https://codereview.chromium.org/
1216303007/)
Reason for revert:
DEPS roll failing
Original issue's description:
> Fix libjpeg-turbo deps file
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/
4380f06a124c28a47dc99956d48c2d4c5bcb78e0
TBR=msarett@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/
1226453003
jvanverth [Thu, 2 Jul 2015 13:42:49 +0000 (06:42 -0700)]
Revert of Remove SkThread.h, include SkMutex.h or SkAtomics.h as appropriate. (patchset #4 id:60001 of https://codereview.chromium.org/
1215393002/)
Reason for revert:
Breaking the roll.
E.g. on android_chromium_gn_compile_dbg:
FAILED: /b/build/goma/gomacc ../../third_party/android_tools/ndk/toolchains/arm-linux-androideabi-4.9/prebuilt/linux-x86_64/bin/arm-linux-androideabi-g++ -MMD -MF obj/skia/ext/libskia.SkMemory_new_handler.o.d -DV8_DEPRECATION_WARNINGS -DCLD_VERSION=1 -DENABLE_NOTIFICATIONS -DENABLE_BROWSER_CDMS -DENABLE_PRINTING=1 -DENABLE_BASIC_PRINTING=1 -DDONT_EMBED_BUILD_METADATA -DUSE_OPENSSL=1 -DUSE_OPENSSL_CERTS=1 -DNO_TCMALLOC -DDISABLE_NACL -DENABLE_CONFIGURATION_POLICY -DENABLE_SUPERVISED_USERS=1 -DENABLE_AUTOFILL_DIALOG=1 -DUSE_PROPRIETARY_CODECS -DV8_USE_EXTERNAL_STARTUP_DATA -DVIDEO_HOLE=1 -DMOBILE_SAFE_BROWSING -DSAFE_BROWSING_SERVICE -DCHROMIUM_BUILD -D_FILE_OFFSET_BITS=64 -DANDROID -DHAVE_SYS_UIO_H -DCOMPONENT_BUILD -D__GNU_SOURCE=1 -D_DEBUG -DDYNAMIC_ANNOTATIONS_ENABLED=1 -DWTF_USE_DYNAMIC_ANNOTATIONS=1 -DSKIA_IMPLEMENTATION=1 -DSK_ARM_HAS_OPTIONAL_NEON -DSK_GAMMA_APPLY_TO_A8 -DSK_GAMMA_EXPONENT=1.4 -DSK_GAMMA_CONTRAST=0.0 -DSK_DEFAULT_FONT_CACHE_LIMIT=
1048576 -DSK_IGNORE_LINEONLY_AA_CONVEX_PATH_OPTS -DSKIA_DLL -DGR_GL_IGNORE_ES3_MSAA=0 -DSK_SUPPORT_GPU=1 -DSK_BUILD_FOR_ANDROID -DUSE_CHROMIUM_SKIA -DXML_STATIC -I../.. -Igen -I../../third_party/skia/include/private -I../../third_party/skia/src/core -I../../third_party/skia/src/image -I../../third_party/skia/src/opts -I../../third_party/skia/src/pdf -I../../third_party/skia/src/ports -I../../third_party/skia/src/sfnt -I../../third_party/skia/src/utils -I../../third_party/skia/src/lazy -I../../skia/config -I../../skia/ext -I../../third_party/skia/include/c -I../../third_party/skia/include/config -I../../third_party/skia/include/core -I../../third_party/skia/include/effects -I../../third_party/skia/include/images -I../../third_party/skia/include/lazy -I../../third_party/skia/include/pathops -I../../third_party/skia/include/pdf -I../../third_party/skia/include/pipe -I../../third_party/skia/include/ports -I../../third_party/skia/include/utils -I../../third_party/skia/include/gpu -I../../third_party/skia/src/gpu -I../../third_party/zlib -I../../third_party/expat/files/lib -I../../third_party/freetype-android/include -I../../third_party/freetype-android/src/include -I../../third_party/android_tools/ndk/sources/android/cpufeatures -fno-strict-aliasing -march=armv7-a -mfloat-abi=softfp -mtune=generic-armv7-a -mthumb -mthumb-interwork -fno-tree-sra -fno-caller-saves -funwind-tables -fPIC -pipe -ffunction-sections -funwind-tables -fno-short-enums -finline-limit=64 -mfpu=vfpv3-d16 -isystem../../third_party/android_tools/ndk/sources/cxx-stl/llvm-libc++/libcxx/include -isystem../../third_party/android_tools/ndk/sources/cxx-stl/llvm-libc++abi/libcxxabi/include -isystem../../third_party/android_tools/ndk/sources/android/support/include -fvisibility=hidden --sysroot=/b/build/slave/android_chromium_gn/build/src/third_party/android_tools/ndk/platforms/android-16/arch-arm -Os -fdata-sections -ffunction-sections -fomit-frame-pointer -g1 -Wno-format -Wendif-labels -Werror -Wno-missing-field-initializers -Wno-unused-parameter -Wno-psabi -Wno-extra -Wno-ignored-qualifiers -Wno-type-limits -Wno-unused-local-typedefs -fno-threadsafe-statics -fvisibility-inlines-hidden -std=gnu++11 -fno-rtti -fno-exceptions -Wno-deprecated -Wno-narrowing -Wno-literal-suffix -Wno-error=c++0x-compat -Wno-non-virtual-dtor -Wno-sign-promo -c ../../skia/ext/SkMemory_new_handler.cpp -o obj/skia/ext/libskia.SkMemory_new_handler.o
../../skia/ext/SkMemory_new_handler.cpp:12:52: fatal error: third_party/skia/include/core/SkThread.h: No such file or directory
#include "third_party/skia/include/core/SkThread.h"
Original issue's description:
> Remove SkThread.h, include SkMutex.h or SkAtomics.h as appropriate.
>
> SkThread.h doesn't do anything anymore execept include those two,
> and thankfully, it doesn't seem to be mentioned outside Skia.
>
> No public API changes.
> TBR=reed@google.com
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/
c50acf2321d7a934c80d754e9cbe936dfb8eb4cc
TBR=mtklein@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/
1214603003
kkinnunen [Thu, 2 Jul 2015 10:01:43 +0000 (03:01 -0700)]
Implement support for CHROMIUM_path_rendering pseudo extension
Implement support for path rendering in Chromium through
CHROMIUM_path_rendering pseudo extension.
The extension defines a new pseudo-gl function,
BindFragmentInputLocation. This behaves similarly to the
BindUniformLocation pseudo-gl function. The idea is to assign fragment
input location to a fragment input before linking the program.
BUG=chromium:344330
Review URL: https://codereview.chromium.org/
1192663002
Matt Sarett [Wed, 1 Jul 2015 22:56:52 +0000 (18:56 -0400)]
Fix libjpeg-turbo deps file
BUG=skia:
Review URL: https://codereview.chromium.org/
1216303007.
Matt Sarett [Wed, 1 Jul 2015 22:21:42 +0000 (18:21 -0400)]
whitespace
emmaleer [Wed, 1 Jul 2015 21:44:32 +0000 (14:44 -0700)]
This change:
- supports kGray correctly
- avoid extra call to png_get_IHDR by storing the bit depth
- call transforms as needed
- checks for tRNS alpha value in RGB and GRAY color types
BUG=skia:
Review URL: https://codereview.chromium.org/
1214203005