| Commit message (Collapse) | Author | Age |
|
|
|
|
|
| |
TBR=
Review URL: https://codereview.chromium.org/637293003
|
|
|
|
|
|
| |
TBR=mtklein@google.com
Review URL: https://codereview.chromium.org/635293003
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Make the Sk GL context class, SkGLNativeContext, an abstract base class. Before,
it depended on ifdefs to implement the platform dependent polymorphism. Move
the logic to subclasses of the various platform implementations.
This a step to enable Skia embedders to compile dm and bench_pictures. The
concrete goal is to support running these test apps with Chromium command buffer.
With this change, Chromium can implement its own version of SkGLNativeContext
that uses command buffer, and host the implementation in its own repository.
Implements the above by renaming the SkGLContextHelper to SkGLContext and
removing the unneeded SkGLNativeContext. Also removes
SkGLNativeContext::AutoRestoreContext functionality, it appeared to be unused:
no use in Skia code, and no tests.
BUG=skia:2992
Review URL: https://codereview.chromium.org/630843002
|
|
|
|
|
|
|
|
| |
Automatic commit by the RecreateSKPs bot.
TBR=
Review URL: https://codereview.chromium.org/637173002
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
this is a huge refactor and cleanup of the gl shader building system in
Skia. The entire shader building pipeline is now part of
GrGLProgramCreator, which takes a gp, and some fps, and creates a
program. I added some subclasses of GrGLProgram to handle the
eccentricities of Nvpr/Nvpres. Outside of the builders folder
and GrGLPrograms, this change is basically just a rename
solo gp
BUG=skia:
Committed: https://skia.googlesource.com/skia/+/fe1233c3f12f81bb675718516bbb32f72af726ec
Review URL: https://codereview.chromium.org/611653002
|
|
|
|
|
|
| |
BUG=skia:2995
Review URL: https://codereview.chromium.org/639523002
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
https://codereview.chromium.org/611653002/)
Reason for revert:
Seems to have messed up windows 7 gms
Original issue's description:
> Cleanup of shader building system
>
> this is a huge refactor and cleanup of the gl shader building system in
> Skia. The entire shader building pipeline is now part of
> GrGLProgramCreator, which takes a gp, and some fps, and creates a
> program. I added some subclasses of GrGLProgram to handle the
> eccentricities of Nvpr/Nvpres. Outside of the builders folder
> and GrGLPrograms, this change is basically just a rename
>
>
> solo gp
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/fe1233c3f12f81bb675718516bbb32f72af726ec
TBR=bsalomon@google.com
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/635533005
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
this is a huge refactor and cleanup of the gl shader building system in
Skia. The entire shader building pipeline is now part of
GrGLProgramCreator, which takes a gp, and some fps, and creates a
program. I added some subclasses of GrGLProgram to handle the
eccentricities of Nvpr/Nvpres. Outside of the builders folder
and GrGLPrograms, this change is basically just a rename
solo gp
BUG=skia:
Review URL: https://codereview.chromium.org/611653002
|
|
|
|
| |
This reverts commit d14e1a27643125bfef37fa0ed314b64c1fae22b7.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
SkTileGrid::insert() is about 15% of recording time before
this CL, which reduces it to ~10%. Next steps are looking
into some of the TODOs I've left myself, and vectorizing
the math.
Most of the win here comes from converting integer
divisions into float multiplies.
BUG=skia:1021
Review URL: https://codereview.chromium.org/615853007
|
|
|
|
|
|
| |
TBR=egdaniel@google.com
Review URL: https://codereview.chromium.org/632363004
|
|
|
|
|
|
| |
TBR=robertphillips@google.com
Review URL: https://codereview.chromium.org/638683002
|
|
|
|
|
|
|
|
|
|
|
|
| |
We use this on Linux already, but for whatever reason wasn't a problem on iOS
until using it in SkPaint. Mac 10.7 and 10.8 are showing this warning too,
but seems -Werror is not enabled.
CQ_EXTRA_TRYBOTS=tryserver.skia:Build-Mac10.7-Clang-Arm7-Release-iOS-Trybot
BUG=skia:
Review URL: https://codereview.chromium.org/637593002
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/637583002
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
https://codereview.chromium.org/628633003/)
Reason for revert:
breaks angle bot
Original issue's description:
> gl programs rewrite
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/07a255310aca9f3e83bf741dc663a58818ad681c
TBR=bsalomon@google.com,egdaniel@google.com
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/631183003
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/628633003
|
|
|
|
| |
Review URL: https://codereview.chromium.org/636933003
|
|
|
|
| |
Review URL: https://codereview.chromium.org/634923002
|
|
|
|
|
|
| |
BUG=skia:2889
Review URL: https://codereview.chromium.org/608883003
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/630063002
|
|
|
|
|
|
|
|
| |
BUG=skia:
TBR=
NOTRY=True
Review URL: https://codereview.chromium.org/633923002
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
id:1 of https://codereview.chromium.org/635573004/)
Reason for revert:
This fails on some Android devices, missing part of tileimagefilter GM which were previously triggering the readback/upload code path.
Original issue's description:
> Don't readback and reupload texture-backed bitmaps
>
> Committed: https://skia.googlesource.com/skia/+/b9ab5631703c258f1aae7a3ca1bb4b6596b5b42c
TBR=robertphillips@google.com
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/632293002
|
|
|
|
|
|
|
|
|
|
| |
These are rebaselines for: Create MSAA render target for blur mask texture - https://codereview.chromium.org/616923004
BUG=skia:2995
TBR=bsalomon@google.com
NOTRY=true
Review URL: https://codereview.chromium.org/628813003
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add a flush writes pixel ops flag.
Add an explicit flush writes for GrSurface.
BUG=skia:2977
Committed: https://skia.googlesource.com/skia/+/cf99b00980b6c9c557e71abf1a7c9f9b21217262
Review URL: https://codereview.chromium.org/622663002
|
|
|
|
| |
Review URL: https://codereview.chromium.org/635573004
|
|
|
|
|
|
|
| |
TBR=reed@google.com
NOTRY=true
Review URL: https://codereview.chromium.org/635813003
|
|
|
|
|
|
|
|
|
| |
This should be safe after: Update suppressed baseline images - https://codereview.chromium.org/605973003/
TBR=reed@google.com
NOTRY=true
Review URL: https://codereview.chromium.org/628013003
|
|
|
|
|
|
|
| |
BUG=skia:
TBR=bsalomon@google.com
Review URL: https://codereview.chromium.org/633643004
|
|
|
|
|
|
|
|
| |
Automatic commit by the RecreateSKPs bot.
TBR=
Review URL: https://codereview.chromium.org/635723002
|
|
|
|
|
|
|
| |
BUG=skia:
TBR=
Review URL: https://codereview.chromium.org/636723002
|
|
|
|
|
|
|
| |
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/630093003
|
|
|
|
|
|
|
| |
TBR=robertphillips@google.com
NOTRY=true
Review URL: https://codereview.chromium.org/631993002
|
|
|
|
|
|
|
|
|
|
| |
This updates the non-multipicturedraw GM images suppressed in:
Don't turn on dither for saveLayers which have no paint - https://codereview.chromium.org/619363002/
TBR=reed@google.com
NOTRY=true
Review URL: https://codereview.chromium.org/605973003
|
|
|
|
| |
Review URL: https://codereview.chromium.org/630103003
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/628353002
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/623173004
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/631943002
|
|
|
|
|
|
|
| |
TBR=reed@google.com
NOTRY=true
Review URL: https://codereview.chromium.org/631973002
|
|
|
|
| |
Review URL: https://codereview.chromium.org/630173002
|
|
|
|
|
|
| |
Original Author: Henry Song <henrysong@samsung.com>
Review URL: https://codereview.chromium.org/616923004
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/628293002
|
|
|
|
|
|
| |
Having a layer is a prerequisite for having it be hoisted.
Review URL: https://codereview.chromium.org/632743002
|
|
|
|
|
|
|
|
|
|
|
|
| |
This updates the multipicturedraw_* GM images for:
Reduce overdraw in MultiPictureDraw Sierpinski GM - https://codereview.chromium.org/582633003/
Don't turn on dither for saveLayers which have no paint - https://codereview.chromium.org/619363002/
TBR=reed@google.com
NOTRY=true
Review URL: https://codereview.chromium.org/632703003
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Used to be:
0 -> run on main thread plus an autodetected number of extra threads (default)
N -> run on main thread plus N extra threads
Now it's:
-1 -> run on main thread plus an autodetected number of extra threads (default)
0 -> run on main thread
N -> run on main thread plus N extra threads
BUG=skia:
Review URL: https://codereview.chromium.org/636593002
|
|
|
|
|
|
|
| |
NOTRY=True
TBR=
Review URL: https://codereview.chromium.org/629903004
|
|
|
|
|
|
|
|
|
| |
New baseline images for multipicturedraw_biglayer* GMs added in (Add a MultiPictureDraw GM to exercise layer clipping - https://codereview.chromium.org/617723004/).
NOTRY=true
TBR=reed@google.com
Review URL: https://codereview.chromium.org/629293002
|
|
|
|
|
|
| |
TBR=reed@google.com
Review URL: https://codereview.chromium.org/634623002
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Since we just 'define' them, but not attribute anything to them, like
'1' for example, cpp expands it to nothing and that breaks the "#if"
clauses.
To fix that, uses "#if defined(...)" which will correctly check if your
macro name was defined or not.
BUG=skia:2850
TEST=make most
R=robertphillips@google.com
Review URL: https://codereview.chromium.org/628763005
|
|
|
|
|
|
|
| |
BUG=skia:
TBR=tfarina
Review URL: https://codereview.chromium.org/626623002
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/631693003
|