| Commit message (Collapse) | Author | Age |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Change a batch of GM tests to convert their background color
so that it is representable in 8888 and 565.
Enable portable text in those same tests to minimize platform
differences.
In a couple of bitmap tests, use portable typefaces instead of
choosing 'Times' which may or may not be available on the platform.
R=borenet@google.com
Review URL: https://codereview.chromium.org/1176243006
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The 'paintingColor' in onGetBitmap() was used to save the entire
gif's background color while filling the 1st frame of a gif image,
since it is 'static' qualified, so when re-entering this function
for subsequent frames, this saved value will be used directly without
re-obtaining it.
But if asynchronous playing multi gif files in single process without
being controlled, this 'static' will make the 'paintingColor'
corrupted, because the different SkGIFMovie instances hold the same
reference of 'paintingColor'.
Signed-off-by: Lu Tong <lu.x.tong@sonymobile.com>;
BUG=skia:
Review URL: https://codereview.chromium.org/1184083002
|
|
|
|
|
|
|
|
|
|
| |
Also, remove some SkPDFDevice functions.
Will fix this GM: http://crrev.com/1159273003
BUG=skia:3872
Review URL: https://codereview.chromium.org/1148263005
|
|
|
|
|
|
|
|
|
| |
Automatic commit by the RecreateSKPs bot.
TBR=
NO_MERGE_BUILDS
Review URL: https://codereview.chromium.org/1183033002
|
|
|
|
|
| |
The 'sdl' directory is lower case, but the gyp files refer to it
with upper case.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Brings in the following functionality:
ARB_draw_instanced
ARB_instanced_arrays
NV_bindless_texture
EXT_direct_state_access
KHR_debug
Also cleans up some of the NVPR extension loading.
BUG=skia:
Committed: https://skia.googlesource.com/skia/+/202c4c75fb806f751cf796850bf67fe9793e0643
Review URL: https://codereview.chromium.org/1185573003
|
|
|
|
|
|
| |
TBR=djsollen@google.com
Review URL: https://codereview.chromium.org/1187603002
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/1164403002
|
|
|
|
|
|
| |
TBR=djsollen@google.com
Review URL: https://codereview.chromium.org/1183753002
|
|
|
|
| |
Review URL: https://codereview.chromium.org/1173363011
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
https://codereview.chromium.org/1185573003/)
Reason for revert:
breaking multiple builds. For example...
https://uberchromegw.corp.google.com/i/client.skia/builders/Test-Ubuntu-GCC-Golo-GPU-GT610-x86_64-Debug-ASAN/builds/667
Original issue's description:
> Import functionality for new GL backend
>
> Brings in the following functionality:
>
> ARB_draw_instanced
> ARB_instanced_arrays
> NV_bindless_texture
> EXT_direct_state_access
> KHR_debug
>
> Also cleans up some of the NVPR extension loading.
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/202c4c75fb806f751cf796850bf67fe9793e0643
TBR=bsalomon@google.com,cdalton@nvidia.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/1184863002
|
|
|
|
|
|
| |
R=djsollen@google.com
Review URL: https://codereview.chromium.org/1182833004
|
|
|
|
|
|
|
|
|
| |
We're not triaging them, and we get the same coverage (both Release and Debug)
from nanobench.
BUG=skia:
Review URL: https://codereview.chromium.org/1178943006
|
|
|
|
| |
Review URL: https://codereview.chromium.org/1180163002
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Brings in the following functionality:
ARB_draw_instanced
ARB_instanced_arrays
NV_bindless_texture
EXT_direct_state_access
KHR_debug
Also cleans up some of the NVPR extension loading.
BUG=skia:
Review URL: https://codereview.chromium.org/1185573003
|
|
|
|
|
|
| |
BUG=skia:3942
Review URL: https://codereview.chromium.org/1185753002
|
|
|
|
|
|
| |
TBR=djsollen@google.com
Review URL: https://codereview.chromium.org/1179713007
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
I originally thought that there was no harm in reading or skipping
zero lines after we have already reached the end of the image.
However, once we reach the end of the image, onFinish() is
automatically called. Performing a read or a skip after
the call to onFinish() is invalid and will cause onFinish()
to be called a second time (which is also invalid).
Seems like the code requires good behavior and the test is
wrong.
BUG=skia:
Review URL: https://codereview.chromium.org/1179213002
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Adds a new FBO type kStencil_MSFBOType that is selected whenever
NV_framebuffer_mixed_samples extension is available. In this new
FBO type a non-msaa color buffer is created with a multisampled
stencil buffer attachment.
Replaces numSamples() with separate numColorSamples and numStencilSamples
methods in RenderTarget.
In mixed samples mode non-MSAA codepaths are used to draw simple shapes,
while NVPR-rendered paths and text are rendered with a multisampled
stencil.
BUG=skia:3177
Review URL: https://codereview.chromium.org/1001503002
|
|
|
|
|
|
| |
TBR=djsollen
Review URL: https://codereview.chromium.org/1182113002
|
|
|
|
|
|
|
|
|
| |
We have two, one in images/, the other in codec/. As codec's the new hotness,
I left it's name vanilla and suffixed the old one.
BUG=skia:
Review URL: https://codereview.chromium.org/1185733002
|
|
|
|
|
|
|
|
| |
implementations.
TBR=reed@google.com
Review URL: https://codereview.chromium.org/1182813002
|
|
|
|
|
|
|
|
|
| |
Moves blend enums from GrXferProcessor.h to GrBlend.h, makes GrBlend.h
public.
BUG=skia:
Review URL: https://codereview.chromium.org/1180713008
|
|
|
|
|
|
| |
TBR=djsollen@google.com
Review URL: https://codereview.chromium.org/1179173003
|
|
|
|
|
|
|
|
| |
BUG=skia:3718
NOTRY=true
DOCS_PREVIEW= https://skia.org/?cl=1185663003
Review URL: https://codereview.chromium.org/1185663003
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/1174293005
|
|
|
|
|
|
|
| |
BUG=skia:
TBR=
Review URL: https://codereview.chromium.org/1177173003
|
|
|
|
|
|
| |
TBR=djsollen@google.com
Review URL: https://codereview.chromium.org/1177303005
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/1175993005
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/1185453003
|
|
|
|
|
|
| |
Capitalize Skia and use new site url.
Review URL: https://codereview.chromium.org/1183503002
|
|
|
|
|
|
|
|
|
|
|
| |
The varyings used to compute rect coverage need to be high precision,
since they're offset by the frag coord.
R=egdaniel
BUG=skia:3935
Review URL: https://codereview.chromium.org/1178123004
|
|
|
|
|
|
| |
BUG=skia:1282
Review URL: https://codereview.chromium.org/1103313002
|
|
|
|
|
|
|
|
| |
Hal found a brain-dead bug on my part. Thanks Hal!
TBR=halcanary@google.com
Review URL: https://codereview.chromium.org/1176423003
|
|
|
|
|
|
|
|
| |
Add a mapping of glyph ID 0 to the toy font to allow for this.
TBR=bungeman@google.com
Review URL: https://codereview.chromium.org/1175243004
|
|
|
|
|
|
|
|
| |
BUG=skia:
NOTRY=True
TBR=
Review URL: https://codereview.chromium.org/1183463002
|
|
|
|
|
|
| |
TBR=halcanary@google.com
Review URL: https://codereview.chromium.org/1175253004
|
|
|
|
|
|
|
|
|
| |
Fixes circular dependencies by allowing the initialization to be done
intertwined.
BUG=skia:
Review URL: https://codereview.chromium.org/1177053002
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- input param to addFoo (e.g. addRect), where only CW or CCW are valid)
- output param from computing functions, that sometimes return kUnknown
This CL's intent is to split these into distinct enums/features:
- Direction (public) loses kUnknown, and is only used for input
- FirstDirection (private) is used for computing the first direction we see when analyzing a contour
BUG=skia:
Review URL: https://codereview.chromium.org/1176953002
|
|
|
|
| |
Review URL: https://codereview.chromium.org/1176543004
|
|
|
|
| |
Review URL: https://codereview.chromium.org/1170283004
|
|
|
|
|
|
|
|
|
| |
This is an alternate version of GrAAConvexPathRenderer which handles
curves by first flattening them to straight lines.
BUG=skia:
Review URL: https://codereview.chromium.org/1158803002
|
|
|
|
|
|
|
|
|
|
|
|
| |
Makes my gcc 5.1.1 much happier.
R=mtklein@google.com,scroggo@google.com
[ mtklein mischief below here ]
No public API changes.
TBR=reed@google.com
Review URL: https://codereview.chromium.org/1170863006
|
|
|
|
|
|
|
|
|
| |
Aliased text didn't work with msaa16, so make it all
antialiased.
TBR=reed@google.com
Review URL: https://codereview.chromium.org/1169173006
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This experiment replaces the label used in the aaxfermodes gm with
aliased text generated from paths common to all platforms.
Since there is no way today to generate all dm output from trybots,
this will be checked in to confirm that this strategy provides simpler
output across devices.
This does not introduce a new public interface; instead, dm uses
a extern backdoor to install the SkTypeface::CreateFromName
handler.
Review URL: https://codereview.chromium.org/1163283002
|
|
|
|
|
|
| |
sources.
Review URL: https://codereview.chromium.org/1176923003
|
|
|
|
|
|
|
|
|
|
|
|
| |
I haven't figured out a pithy way to have these apply to only classes
originating from SkNx, so let's just remove them. There aren't too
many use cases, and it's not really any less readable without them.
Semantically, this is a no-op.
BUG=skia:
Review URL: https://codereview.chromium.org/1167153002
|
|
|
|
| |
Review URL: https://codereview.chromium.org/1166993002
|
|
|
|
|
|
|
|
|
|
| |
Tweak some test values to pass with floats.
As expected, this regresses matrix44_setconcat_general by about 2x.
BUG=skia:
Review URL: https://codereview.chromium.org/1169813006
|