| 7023a00c35d904e4ccff09c377e9ba26abba6181 |
|
22-Feb-2016 |
jvanverth <jvanverth@google.com> |
Use unorm shorts for texture coordinates when rendering text.
There are a couple of reasons for this:
- Vulkan does not guarantee conversions from integral vertex attributes
to floating point shader variables
- This may be faster and more precise on some platforms, as it avoids
the aforementioned conversion and changes a multiply by a very small
value to a multiply by a medium-sized value.
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1713693002
TBR=bsalomon@google.com
Committed: https://skia.googlesource.com/skia/+/e507ff0460f4f878214b9454fb5b9ab8d64d8063
Review URL: https://codereview.chromium.org/1713693002
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| d106c2c64012688f8d1439543d942f8f66a142e0 |
|
19-Feb-2016 |
jvanverth <jvanverth@google.com> |
Revert of Use unorm shorts for texture coordinates when rendering text. (patchset #3 id:40001 of https://codereview.chromium.org/1713693002/ )
Reason for revert:
Causing issues with text on Mali 400s. Examples: largeglyphblur, imageblurtiled. It appears that there are precision problems.
Original issue's description:
> Use unorm shorts for texture coordinates when rendering text.
>
> There are a couple of reasons for this:
> - Vulkan does not guarantee conversions from integral vertex attributes
> to floating point shader variables
> - This may be faster and more precise on some platforms, as it avoids
> the aforementioned conversion and changes a multiply by a very small
> value to a multiply by a medium-sized value.
> GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1713693002
> TBR=bsalomon@google.com
>
> Committed: https://skia.googlesource.com/skia/+/e507ff0460f4f878214b9454fb5b9ab8d64d8063
TBR=joshualitt@google.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/1709133003
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| e507ff0460f4f878214b9454fb5b9ab8d64d8063 |
|
19-Feb-2016 |
jvanverth <jvanverth@google.com> |
Use unorm shorts for texture coordinates when rendering text.
There are a couple of reasons for this:
- Vulkan does not guarantee conversions from integral vertex attributes
to floating point shader variables
- This may be faster and more precise on some platforms, as it avoids
the aforementioned conversion and changes a multiply by a very small
value to a multiply by a medium-sized value.
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1713693002
TBR=bsalomon@google.com
Review URL: https://codereview.chromium.org/1713693002
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 8528541dd7f09f5bd76f3f1ce5f45d08ac7347c7 |
|
18-Feb-2016 |
cdalton <cdalton@nvidia.com> |
Add more specialized fragment builders
Adds specialized fragment builders for primitive and fragment
processors. Removes fragment-specific functionality from the Xfer
fragment builder.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1709153002
Review URL: https://codereview.chromium.org/1709153002
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| d9d30f7b577711a200395f7b62acc0d82a19f9c3 |
|
08-Dec-2015 |
joshualitt <joshualitt@chromium.org> |
Allow LCD text to batch across colorchanges. This will always use
color vertices, even when we can't batch across color changes
BUG=skia:
Review URL: https://codereview.chromium.org/1502253003
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 7ea439b2203855db97330b25945b87dd4b170b8b |
|
03-Dec-2015 |
egdaniel <egdaniel@google.com> |
Create GLSLUniformHandler class for gpu backend
BUG=skia:
Review URL: https://codereview.chromium.org/1490283004
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 0eafe79f42e3c675f3c504aed4a41abf511df2b7 |
|
20-Nov-2015 |
egdaniel <egdaniel@google.com> |
Create GrGLSLVaryingHandler class for program building
BUG=skia:
Review URL: https://codereview.chromium.org/1462123003
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 4ca2e6034365ad280ec64473f7f1d72ebd8335e4 |
|
18-Nov-2015 |
egdaniel <egdaniel@google.com> |
Add ShaderBuilders to EmitArgs and remove gettings from ProgBuilder.
BUG=skia:
Review URL: https://codereview.chromium.org/1457543003
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 57d3b039c635945e1dc2fcbac3462ed8bfedb068 |
|
13-Nov-2015 |
egdaniel <egdaniel@google.com> |
Rename some processor functions from GL to GLSL
TBR=bsalomon@google.com
BUG=skia:
Review URL: https://codereview.chromium.org/1443743002
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| e659a581f63fdccb64dce2dc8a478cf56831feea |
|
13-Nov-2015 |
egdaniel <egdaniel@google.com> |
Move GrGLPrimitive/GeometryProc to GLSL
BUG=skia:
Review URL: https://codereview.chromium.org/1441683008
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 64c4728c70001ed074fecf5c4e083781987b12e9 |
|
13-Nov-2015 |
egdaniel <egdaniel@google.com> |
Make all GrFragmentProcessors GL independent.
TBR=bsalomon@google.com
BUG=skia:
Review URL: https://codereview.chromium.org/1434313002
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 2d721d33aad192cc8a7a1321504b39bdca2a57ce |
|
11-Nov-2015 |
egdaniel <egdaniel@google.com> |
Move all ShaderBuilder files to GLSL
BUG=skia:
Review URL: https://codereview.chromium.org/1438003003
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 8dcdedc4a087ea46ce1e2458d335d60918e56310 |
|
11-Nov-2015 |
egdaniel <egdaniel@google.com> |
Make GrGLSLProgramBuilder base class for GrGLProgramBuilder.
This CL still keeps the weird diamond shape we have for all our ProgramBuilders.
However, the GrGLSL base class will allow us to pull multiple other parts
of our program setup away from GL which will eventually allow us to break up
the diamond.
As part of this all ShaderBuilder subclass have been made gl independent,
however I will move them to GLSL files/class names in a follow on CL.
BUG=skia:
Review URL: https://codereview.chromium.org/1416423003
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 018fb62d12d1febf121fe265da5b6117b86a6541 |
|
28-Oct-2015 |
egdaniel <egdaniel@google.com> |
Create GLSL base class for ProgramDataManager
BUG=skia:
Review URL: https://codereview.chromium.org/1428543003
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 27b6335c71bccb96206ff0495793433bfa982777 |
|
15-Sep-2015 |
egdaniel <egdaniel@google.com> |
On gpu, use max(r,g,b) for coverage alpha in LCD and update lcd blend gm.
BUG=skia:
Review URL: https://codereview.chromium.org/1339213004
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 465283cdf98ed9ab5285ca7b9814e430fca1d452 |
|
11-Sep-2015 |
joshualitt <joshualitt@chromium.org> |
Remove batchtracker
BUG=skia:
Review URL: https://codereview.chromium.org/1332923003
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| c21b09eec91c9e263cb0b88467ea44e348ed4962 |
|
29-Aug-2015 |
bsalomon <bsalomon@google.com> |
Change SkShader;asFragmentProcessor signature to no longer take skpaint\grcolor*
Committed: https://skia.googlesource.com/skia/+/ecfdc251be71f3d634e76afdd6375bf55fc061aa
Review URL: https://codereview.chromium.org/1316513002
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| a511e6ad1042093bea5b015c22a920313fc57c0b |
|
29-Aug-2015 |
rmistry <rmistry@google.com> |
Revert of Change SkShader;asFragmentProcessor signature to no longer take skpaint\grcolor* (patchset #8 id:140001 of https://codereview.chromium.org/1316513002/ )
Reason for revert:
Primary suspect in failing DEPS rolls:
* https://codereview.chromium.org/1315753006
* https://codereview.chromium.org/1308323006
* https://codereview.chromium.org/1320903004
Primary suspect because the failing win bots did not fail in https://codereview.chromium.org/1315753005
Original issue's description:
> Change SkShader;asFragmentProcessor signature to no longer take skpaint\grcolor*
>
> Committed: https://skia.googlesource.com/skia/+/ecfdc251be71f3d634e76afdd6375bf55fc061aa
TBR=joshualitt@google.com,wangyix@google.com,robertphillips@google.com,bsalomon@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/1313573005
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| ecfdc251be71f3d634e76afdd6375bf55fc061aa |
|
28-Aug-2015 |
bsalomon <bsalomon@google.com> |
Change SkShader;asFragmentProcessor signature to no longer take skpaint\grcolor*
Review URL: https://codereview.chromium.org/1316513002
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 96fcdcc219d2a0d3579719b84b28bede76efba64 |
|
27-Aug-2015 |
halcanary <halcanary@google.com> |
Style Change: NULL->nullptr
DOCS_PREVIEW= https://skia.org/?cl=1316233002
Review URL: https://codereview.chromium.org/1316233002
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 385fe4d4b62d7d1dd76116dd570df3290a2f487b |
|
26-Aug-2015 |
halcanary <halcanary@google.com> |
Style Change: SkNEW->new; SkDELETE->delete
DOCS_PREVIEW= https://skia.org/?cl=1316123003
Review URL: https://codereview.chromium.org/1316123003
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| da04e0e80a0113e1d838f7406cd8a8d545b8c02b |
|
19-Aug-2015 |
joshualitt <joshualitt@chromium.org> |
Allow setting of GrBatchFontCache atlas sizes
BUG=skia:
Review URL: https://codereview.chromium.org/1255943006
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 7375d6bab2ee8b02da276597ed4d60f22f54eb89 |
|
07-Aug-2015 |
joshualitt <joshualitt@chromium.org> |
fixup precision with configurable atlas
BUG=skia:
Review URL: https://codereview.chromium.org/1276383003
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 922c8b13c512c3287509936795735c1b31bedba9 |
|
07-Aug-2015 |
joshualitt <joshualitt@chromium.org> |
Break LCD and Bitmap text dependency on hardcoded atlas values
BUG=skia:
Committed: https://skia.googlesource.com/skia/+/846b022f6b469cfde285372f26e0d5c593d122ac
Review URL: https://codereview.chromium.org/1271873002
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 9162ea9444ac3127c3a93a880dc77efc839391b3 |
|
05-Aug-2015 |
joshualitt <joshualitt@google.com> |
Revert of Break LCD and Bitmap text dependency on hardcoded atlas values (patchset #2 id:20001 of https://codereview.chromium.org/1271873002/ )
Reason for revert:
Breaks S3
Original issue's description:
> Break LCD and Bitmap text dependency on hardcoded atlas values
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/846b022f6b469cfde285372f26e0d5c593d122ac
TBR=jvanverth@google.com,bsalomon@google.com,joshualitt@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/1271163003
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 846b022f6b469cfde285372f26e0d5c593d122ac |
|
04-Aug-2015 |
joshualitt <joshualitt@chromium.org> |
Break LCD and Bitmap text dependency on hardcoded atlas values
BUG=skia:
Review URL: https://codereview.chromium.org/1271873002
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 6af0c9394301b9f976813cc6dbdbbfe779030fe1 |
|
22-Jul-2015 |
wangyix <wangyix@google.com> |
Moved GrGLFragmentProcessor definition to its own file
BUG=skia:
Review URL: https://codereview.chromium.org/1246193002
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 0067ff5e0f85084dd2b5ad9886b526482b89a116 |
|
08-Jul-2015 |
joshualitt <joshualitt@chromium.org> |
fix up test create functions
TBR=bsalomon@google.com
BUG=skia:
Review URL: https://codereview.chromium.org/1213623022
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| cba99b88fd5fb39def7a094dc32c0745c7a1cfea |
|
24-Jun-2015 |
jvanverth <jvanverth@google.com> |
Move GLSL-specific routines/classes to separate glsl directory
The purpose is to begin separating our GLSL-specific code from
our GL-specific code, so it can be used with the GL45 platform
Review URL: https://codereview.chromium.org/1202293002
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 4b91f768b348aa1cebeb54f3ff9331938734c242 |
|
19-May-2015 |
bsalomon <bsalomon@google.com> |
rename GrDrawTargetCaps to GrCaps
Review URL: https://codereview.chromium.org/1133123009
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| b8c241ad099f3f0c2cbf3e7c10f5f6207175d490 |
|
19-May-2015 |
joshualitt <joshualitt@chromium.org> |
Preliminary attempt to remove batch tracker
BUG=skia:
Committed: https://skia.googlesource.com/skia/+/cbfe91d82500f4ae8c3ff7bd74b3021a4b89fd84
Review URL: https://codereview.chromium.org/1139723004
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| d3a560fa80bfb3e2d2e989f951bb3b1c52316654 |
|
19-May-2015 |
joshualitt <joshualitt@google.com> |
Revert of Preliminary attempt to remove batch tracker (patchset #3 id:40001 of https://codereview.chromium.org/1139723004/)
Reason for revert:
breaking bots
Original issue's description:
> Preliminary attempt to remove batch tracker
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/cbfe91d82500f4ae8c3ff7bd74b3021a4b89fd84
TBR=robertphillips@google.com,joshualitt@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/1132323003
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| cbfe91d82500f4ae8c3ff7bd74b3021a4b89fd84 |
|
19-May-2015 |
joshualitt <joshualitt@chromium.org> |
Preliminary attempt to remove batch tracker
BUG=skia:
Review URL: https://codereview.chromium.org/1139723004
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| e3ababe44315452cd33b96a18ce316ede09ff3c3 |
|
15-May-2015 |
joshualitt <joshualitt@chromium.org> |
remove localmatrix from GrGeometryProcessor base class
BUG=skia:
Review URL: https://codereview.chromium.org/1131513005
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| e578a95d3ab16544794b94da4e7ae13fc2ce6244 |
|
14-May-2015 |
joshualitt <joshualitt@chromium.org> |
Remove viewmatrix from GrGeometryProcessor base class
BUG=skia:
Review URL: https://codereview.chromium.org/1127953003
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 88c23fc6e8ed0243159fd17d815b4f813805647b |
|
13-May-2015 |
joshualitt <joshualitt@chromium.org> |
remove color from GrGeometryProcessor
BUG=skia:
Review URL: https://codereview.chromium.org/1140983002
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 1ba8cc9193f13a812401e14b8037fdcf3f3acbac |
|
13-May-2015 |
joshualitt <joshualitt@chromium.org> |
removing equality / compute invariant loops from GrGeometryProcessors
BUG=skia:
Committed: https://skia.googlesource.com/skia/+/ef292a0901205b9785a30daae2c036aa34a970ca
Review URL: https://codereview.chromium.org/1111603004
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 29bee0fe657fabf7c396502b69c9167fba13eaaa |
|
29-Apr-2015 |
egdaniel <egdaniel@google.com> |
Make XPFragmentBuilder only Builder with access to DstCopy.
Plus a bunch of renaming.
BUG=skia:
Review URL: https://codereview.chromium.org/1110033004
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 4eaf9cef5a76098f78efac30beb966ac833d32c2 |
|
28-Apr-2015 |
joshualitt <joshualitt@chromium.org> |
create GrTestUtils.h, move some common functions into it
BUG=skia:
Review URL: https://codereview.chromium.org/1117443002
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 50cb76b2bb0ffa607a1409f77be0ae7d64e31436 |
|
28-Apr-2015 |
joshualitt <joshualitt@google.com> |
Revert of removing equality / compute invariant loops from GrGeometryProcessors (patchset #2 id:20001 of https://codereview.chromium.org/1111603004/)
Reason for revert:
breaks gl programs
Original issue's description:
> removing equality / compute invariant loops from GrGeometryProcessors
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/ef292a0901205b9785a30daae2c036aa34a970ca
TBR=bsalomon@google.com,joshualitt@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/1110993002
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| ef292a0901205b9785a30daae2c036aa34a970ca |
|
28-Apr-2015 |
joshualitt <joshualitt@chromium.org> |
removing equality / compute invariant loops from GrGeometryProcessors
BUG=skia:
Review URL: https://codereview.chromium.org/1111603004
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| cfc18867d982119d9dc2888bf09f1093012daadd |
|
28-Apr-2015 |
jvanverth <jvanverth@google.com> |
Use GLSLCaps for creating processor keys and GLSL-specific programs
Effectively all this does is future-proof any GLSL-specific code, as
GLSLCaps is just a typedef of GLCaps.
BUG=skia:
Review URL: https://codereview.chromium.org/1109863004
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 36352bf5e38f45a70ee4f4fc132a38048d38206d |
|
26-Mar-2015 |
mtklein <mtklein@chromium.org> |
C++11 override should now be supported by all of {bots,Chrome,Android,Mozilla}
NOPRESUBMIT=true
BUG=skia:
DOCS_PREVIEW= https://skia.org/?cl=1037793002
Review URL: https://codereview.chromium.org/1037793002
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| cb251f1d0a8aa7f3e6f685b45046173fc05f4b92 |
|
11-Mar-2015 |
jvanverth <jvanverth@google.com> |
Adjust atlas sizes to fix Mali400 precision issues
The previous change to atlas sizes introduced an issue where texture
coordinates for glyphs stored on the right side of the atlas were being
computed in an imprecise manner on Mali400 chips.
The only reasonable fix appears to be to use power-of-two texture sizes.
This widens the glyph atlas to the next power of 2 (or 2048) for A8 glyphs
only, and widens each GrPlot by 2x as well. By doing this, we can fit 3-4 large
distance field glyphs into a single GrPlot, which gives us 100-128 total large
glyphs at one time.
The 565 and 8888 glyph atlases are kept in their original size to preserve
space.
BUG=skia:3523
Review URL: https://codereview.chromium.org/994303003
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 5a105ff05303ac82a867b8b84a1edd145bd46218 |
|
18-Feb-2015 |
jvanverth <jvanverth@google.com> |
Use uint16s for texture coordinates when rendering text.
Allows us to push more vertices into a given vertex buffer, with
a slight performance improvement.
Committed: https://skia.googlesource.com/skia/+/059034d252007d0dd86fff5ffdbb53cbcb10d34b
Review URL: https://codereview.chromium.org/917373002
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| e4ef1ca5be11aed67c0ed0c7eb1862696fb063e3 |
|
18-Feb-2015 |
reed <reed@chromium.org> |
Revert of Use uint16s for texture coordinates when rendering text. (patchset #5 id:80001 of https://codereview.chromium.org/917373002/)
Reason for revert:
speculative revert for DEPS failures
https://codereview.chromium.org/932973002/
Original issue's description:
> Use uint16s for texture coordinates when rendering text.
>
> Allows us to push more vertices into a given vertex buffer, with
> a slight performance improvement.
>
> Committed: https://skia.googlesource.com/skia/+/059034d252007d0dd86fff5ffdbb53cbcb10d34b
TBR=joshualitt@google.com,robertphillips@google.com,bsalomon@google.com,reed@google.com,djsollen@google.com,jvanverth@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/920333003
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 059034d252007d0dd86fff5ffdbb53cbcb10d34b |
|
17-Feb-2015 |
jvanverth <jvanverth@google.com> |
Use uint16s for texture coordinates when rendering text.
Allows us to push more vertices into a given vertex buffer, with
a slight performance improvement.
Review URL: https://codereview.chromium.org/917373002
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| dd2198701b0ec9da61ecf73418cad03642d715e7 |
|
12-Feb-2015 |
joshualitt <joshualitt@chromium.org> |
fix for uniform view matrix being uploaded but not used
As discussed, I'll follow this up by removing localmatrix/uniform view matrix from the base classe
BUG=skia:
Review URL: https://codereview.chromium.org/920933002
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 02b05015b55d1900a9e34039942101da189053ce |
|
11-Feb-2015 |
joshualitt <joshualitt@chromium.org> |
Small change to use a GrGeometryProcessor for all BitmapText draw calls
BUG=skia:
Review URL: https://codereview.chromium.org/914723002
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 4d8da81562852e0ff7e18b66ee1cebd50ad81ee8 |
|
28-Jan-2015 |
joshualitt <joshualitt@chromium.org> |
GrBatchPrototype
BUG=skia:
Committed: https://skia.googlesource.com/skia/+/d15e4e45374275c045572b304c229237c4a82be4
Committed: https://skia.googlesource.com/skia/+/d5a7db4a867c7e6ccf8451a053d987b470099198
Review URL: https://codereview.chromium.org/845103005
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| c2893c5e3870d7e9a37ca146e7da88fba54977d5 |
|
28-Jan-2015 |
joshualitt <joshualitt@google.com> |
Revert of GrBatchPrototype (patchset #32 id:630001 of https://codereview.chromium.org/845103005/)
Reason for revert:
One last try to fix mac perf regression
Original issue's description:
> GrBatchPrototype
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/d15e4e45374275c045572b304c229237c4a82be4
>
> Committed: https://skia.googlesource.com/skia/+/d5a7db4a867c7e6ccf8451a053d987b470099198
TBR=bsalomon@google.com,kkinnunen@nvidia.com,joshualitt@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/877393002
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| d5a7db4a867c7e6ccf8451a053d987b470099198 |
|
28-Jan-2015 |
joshualitt <joshualitt@chromium.org> |
GrBatchPrototype
BUG=skia:
Committed: https://skia.googlesource.com/skia/+/d15e4e45374275c045572b304c229237c4a82be4
Review URL: https://codereview.chromium.org/845103005
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| ca0a1799ffdceb4ccd998468566f39abac2f0b55 |
|
27-Jan-2015 |
joshualitt <joshualitt@google.com> |
Revert of GrBatchPrototype (patchset #30 id:570001 of https://codereview.chromium.org/845103005/)
Reason for revert:
creates large performance regression
Original issue's description:
> GrBatchPrototype
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/d15e4e45374275c045572b304c229237c4a82be4
TBR=bsalomon@google.com,joshualitt@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/862823004
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| d15e4e45374275c045572b304c229237c4a82be4 |
|
26-Jan-2015 |
joshualitt <joshualitt@chromium.org> |
GrBatchPrototype
BUG=skia:
Review URL: https://codereview.chromium.org/845103005
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 46d36f0e7b709a077c647841eee23bd3efdc4117 |
|
18-Jan-2015 |
robertphillips <robertphillips@google.com> |
Refactor position computation to enable device space "nudge"
To match raster's handling of BW geometry we want to be able to perform a device space "nudge" on all geometry. This CL sets us up to be able to do that in GrGLVertexBuilder::transformToNormalizedDeviceSpace.
BUG=423834
TBR=bsalomon@google.com
Review URL: https://codereview.chromium.org/854013002
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 71c9260e6fa1798ad1e41b2c2ae9b3cce08bb610 |
|
14-Jan-2015 |
joshualitt <joshualitt@chromium.org> |
More changes to bring together path / geo procs
BUG=skia:
Review URL: https://codereview.chromium.org/820783005
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| abb52a1a70a81915c6196e0fb3e9bcb05e8be14d |
|
14-Jan-2015 |
joshualitt <joshualitt@chromium.org> |
Move most of the transform logic into the primitive processors
BUG=skia:
Review URL: https://codereview.chromium.org/822423004
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 72c9faab45124e08c85f70ca38536914862d947c |
|
09-Jan-2015 |
mtklein <mtklein@chromium.org> |
Fix up all the easy virtual ... SK_OVERRIDE cases.
This fixes every case where virtual and SK_OVERRIDE were on the same line,
which should be the bulk of cases. We'll have to manually clean up the rest
over time unless I level up in regexes.
for f in (find . -type f); perl -p -i -e 's/virtual (.*)SK_OVERRIDE/\1SK_OVERRIDE/g' $f; end
BUG=skia:
Review URL: https://codereview.chromium.org/806653007
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| ee2af95db72152dfa61c841875df0594ca93437d |
|
30-Dec-2014 |
joshualitt <joshualitt@chromium.org> |
ViewMatrix uniform upload moved to GeometryProcessor
BUG=skia:
Review URL: https://codereview.chromium.org/827973002
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 8059eb9f6e24ed609393fbda4ad71edea03ac258 |
|
30-Dec-2014 |
joshualitt <joshualitt@chromium.org> |
Move ViewMatrix off of drawstate
BUG=skia:
Review URL: https://codereview.chromium.org/815553003
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 8fc6c2d82c1f30ff82274334c01f0799def6a609 |
|
23-Dec-2014 |
joshualitt <joshualitt@chromium.org> |
This CL cleans up the last remaining users of localCoordChange on paint
NOTREECHECKS=True
BUG=skia:
Review URL: https://codereview.chromium.org/817853002
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 290c09b8bbd8d221d363150e2ce87158f4668df0 |
|
19-Dec-2014 |
joshualitt <joshualitt@chromium.org> |
initial changes to add local matrix to primitive processor
BUG=skia:
Review URL: https://codereview.chromium.org/820523002
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 9b98932adaceb7ad0a617ade16616923f6bffe84 |
|
15-Dec-2014 |
joshualitt <joshualitt@chromium.org> |
This change will ultimately pull uniform color, and to a much lesser degree uniform coverage, into GPs. There are still some loose ends because drawstate has the ability to override the GP, but fixing these cleanly will have to wait until we have deferred geometry in place and can make attribute / uniform decisions on the fly.
BUG=skia:
Review URL: https://codereview.chromium.org/746423007
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 56995b5cc00c9c83bd5fcf86bca9a67e939a96cb |
|
12-Dec-2014 |
joshualitt <joshualitt@chromium.org> |
Remove GP from drawstate, revision of invariant output for GP
BUG=skia:
Committed: https://skia.googlesource.com/skia/+/c6bc58eded89b0c0a36b8e20e193c200f297a0da
Review URL: https://codereview.chromium.org/791743003
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 8c0f615fdd41b8b8048bf398791bb2138c511826 |
|
10-Dec-2014 |
joshualitt <joshualitt@google.com> |
Revert of Remove GP from drawstate, revision of invariant output for GP (patchset #9 id:160001 of https://codereview.chromium.org/791743003/)
Reason for revert:
breaks mac
Original issue's description:
> Remove GP from drawstate, revision of invariant output for GP
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/c6bc58eded89b0c0a36b8e20e193c200f297a0da
TBR=bsalomon@google.com,egdaniel@google.com,joshualitt@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/794843002
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| c6bc58eded89b0c0a36b8e20e193c200f297a0da |
|
10-Dec-2014 |
joshualitt <joshualitt@chromium.org> |
Remove GP from drawstate, revision of invariant output for GP
BUG=skia:
Review URL: https://codereview.chromium.org/791743003
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 2e3b3e369d79e78f7635d4c20e83a47ab571bdf2 |
|
09-Dec-2014 |
joshualitt <joshualitt@chromium.org> |
This cl moves color and coverage off of drawstate. In an effort to keep this CL manageable, I have left the compute invariant input / output in a bit of a strange state(fixing this will be complicated).
In addition, NVPR makes this very complicated, and I haven't quite figured out a good way to handle it, so for now color and coverage DO live on optstate, but I will figure out some way to refactor that in future CLs.
BUG=skia:
Review URL: https://codereview.chromium.org/783763002
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|
| 309e346744f57fadd66ba3c2f14875c86311682e |
|
09-Dec-2014 |
egdaniel <egdaniel@google.com> |
Rename CustomCoordTextureEffect to GrBitmapTextGeoProc.
Also add in explicit set for LCD text in invariantOutput.
BUG=skia:
Review URL: https://codereview.chromium.org/786293002
/external/skia/src/gpu/effects/GrBitmapTextGeoProc.cpp
|