c0b642ca48d58416409e555549434066f09692b7 |
|
27-Mar-2017 |
Brian Salomon <bsalomon@google.com> |
Split GrPipelineInput into separate color and coverage types, the latter of which is just an enum. Assign names that indicate that they aren't just for the input phase since I plan to use them at the boundary between FPs and XPs as well. Renamed GrProcOptInfo to GrColorFragmentProcessorAnalysis. This is now only used on the color side and the new name seems clearer to me. Change GrMeshDrawOp::getFragmentProcessorAnalysisInputs to use the new color/coverage types directly rather than a class that has been reduced to simply bundling them together. Change-Id: If93bae74c9d590486eecdf63f302418c96deab65 Reviewed-on: https://skia-review.googlesource.com/10161 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Greg Daniel <egdaniel@google.com>
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
54d212e1bfaea0be88c3c40820d0b1ae0daebecf |
|
21-Mar-2017 |
Brian Salomon <bsalomon@google.com> |
Revert "Revert "Remove GrPipeline from GrDrawOp."" This reverts commit c48af934608bbb65650641f66adb51f2102d4274. Change-Id: I4ba78fd7e5a7d406b88223ca6f7245c029b60f76 Reviewed-on: https://skia-review.googlesource.com/9981 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
c48af934608bbb65650641f66adb51f2102d4274 |
|
16-Mar-2017 |
Brian Salomon <bsalomon@google.com> |
Revert "Remove GrPipeline from GrDrawOp." This reverts commit 2bf4b3a97b770811d9e0558dbbfbdb57cfafbdb7. Reason for revert: nanobench assertion Original change's description: > Remove GrPipeline from GrDrawOp. > > GrDrawOp subclasses are now free to construct their pipelines at flush time and now in theory could use multiple GrPipelines for multipass rendering. > > GrProcessorSet may be used to retain the processors from a GrPaint with "pending execution" style refs. > > NVPR and Instanced rendering are updated to create their pipelines at flush time without a GrPipelineBuilder. > > The monolithic pipeline creation/management that was on GrDrawOp is moved to GrMeshDrawOp. However, this is temporary and will be removed in coming changes. > > Change-Id: I124282e3cea5d070970b5460c8a679fcaf7a8eff > Reviewed-on: https://skia-review.googlesource.com/7279 > Commit-Queue: Brian Salomon <bsalomon@google.com> > Reviewed-by: Robert Phillips <robertphillips@google.com> > TBR=bsalomon@google.com,robertphillips@google.com,csmartdalton@google.com,reviews@skia.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Change-Id: I1bc64f6cbbd5f482417637a034342c2b5371dc5c Reviewed-on: https://skia-review.googlesource.com/9817 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
2bf4b3a97b770811d9e0558dbbfbdb57cfafbdb7 |
|
16-Mar-2017 |
Brian Salomon <bsalomon@google.com> |
Remove GrPipeline from GrDrawOp. GrDrawOp subclasses are now free to construct their pipelines at flush time and now in theory could use multiple GrPipelines for multipass rendering. GrProcessorSet may be used to retain the processors from a GrPaint with "pending execution" style refs. NVPR and Instanced rendering are updated to create their pipelines at flush time without a GrPipelineBuilder. The monolithic pipeline creation/management that was on GrDrawOp is moved to GrMeshDrawOp. However, this is temporary and will be removed in coming changes. Change-Id: I124282e3cea5d070970b5460c8a679fcaf7a8eff Reviewed-on: https://skia-review.googlesource.com/7279 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
780b41fc10fb212ec6d9a9aeb7ca4cc0292fb85e |
|
13-Mar-2017 |
Brian Salomon <bsalomon@google.com> |
Add query to GrXPFactory about coverage-as-alpha optimization This will be needed to have GrDrawOps that haven't yet built pipelines. Change-Id: If5292aaa5dc9f98dccbe27be98960b630332158d Reviewed-on: https://skia-review.googlesource.com/9480 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
42c456fd20bd45dd02f6d5eb0af7acf04341b1ee |
|
06-Mar-2017 |
Brian Salomon <bsalomon@google.com> |
Remove PLS path renderer Change-Id: Ib727b0749a5a7da95832970e79804417e8b6a247 Reviewed-on: https://skia-review.googlesource.com/9300 Reviewed-by: Ethan Nicholas <ethannicholas@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
5298dc8bf30f580f551d130346c007efaf4b2098 |
|
22-Feb-2017 |
Brian Salomon <bsalomon@google.com> |
Make GrPipelineAnalysis a nested class of GrProcessorSet. It is renamed to FragmentProcessorAnalysis since it represents the outputs of the final FPs. It now stores the analysis results that are subsequently needed rather than exposing GrProcOptInfo. GrProcOptInfo is now only used on color FPs (not coverage). Miscellaneous related renamings. Change-Id: I95c518a7a76df6dc294a9fa67c611f8f653247bc Reviewed-on: https://skia-review.googlesource.com/8534 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
00b2cca30033c984662ccc7d1eaed962b3714ad7 |
|
21-Feb-2017 |
Brian Salomon <bsalomon@google.com> |
Simplify GrPaint::isConstantBlendedColor to avoid FP analysis and XP virtual calls This was the only non-test user of GrXPFactory::isConstantPreCoverageBlendedColor which is now removed. Change-Id: Ic1c130d30a44e1a955b89f8912433a3c9df2e61e Reviewed-on: https://skia-review.googlesource.com/8776 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
9a51498720e234b413e5a3c46d38ab40bda131de |
|
14-Feb-2017 |
Brian Salomon <bsalomon@google.com> |
Remove component flags from GrXPFactory output analysis. Change-Id: Ieb8dab564e6e593dca2e092d352756052dadfd90 Reviewed-on: https://skia-review.googlesource.com/8354 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Greg Daniel <egdaniel@google.com>
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
5be6c95fa95a518ab109339df4695d6dda6e2419 |
|
20-Jan-2017 |
Brian Salomon <bsalomon@google.com> |
Revert "Revert "Make it possible to query GrXPFactory for dst texture without GrPipelineAnalysis."" This reverts commit 3329cceab5feca230df1caee16be045249228dc1. Reason for revert: Bot failures are unrelated to the original change. Change-Id: I21b5927dc4384a25930bdefe16e57bcc9276ffa4 Reviewed-on: https://skia-review.googlesource.com/7347 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
3329cceab5feca230df1caee16be045249228dc1 |
|
20-Jan-2017 |
Brian Salomon <bsalomon@google.com> |
Revert "Make it possible to query GrXPFactory for dst texture without GrPipelineAnalysis." This reverts commit f833215420847565b4c9945aebdc2e7ae182937f. Reason for revert: <INSERT REASONING HERE> Original change's description: > Make it possible to query GrXPFactory for dst texture without GrPipelineAnalysis. > > Change-Id: I8c140eb4e3e5f2d21ecbf8f8f3c8533dc7f50e7c > Reviewed-on: https://skia-review.googlesource.com/7316 > Commit-Queue: Brian Salomon <bsalomon@google.com> > Reviewed-by: Greg Daniel <egdaniel@google.com> > TBR=egdaniel@google.com,bsalomon@google.com,reviews@skia.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Change-Id: I790afb9a01422cb4c2d3a4be4ecd20e8c4466b29 Reviewed-on: https://skia-review.googlesource.com/7342 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
f833215420847565b4c9945aebdc2e7ae182937f |
|
20-Jan-2017 |
Brian Salomon <bsalomon@google.com> |
Make it possible to query GrXPFactory for dst texture without GrPipelineAnalysis. Change-Id: I8c140eb4e3e5f2d21ecbf8f8f3c8533dc7f50e7c Reviewed-on: https://skia-review.googlesource.com/7316 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Greg Daniel <egdaniel@google.com>
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
a16339297859f37df69230e64f05624cef511ad3 |
|
09-Jan-2017 |
Brian Salomon <bsalomon@google.com> |
Revert "Revert "Removing ref counting from GrXPFactory."" This reverts commit 003312a211e65f35e402d6fe80a32e23d4c94ac4. Change-Id: Ib41065e5c356d1dd99e70fa10611ac6756c2b79d Reviewed-on: https://skia-review.googlesource.com/6803 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
003312a211e65f35e402d6fe80a32e23d4c94ac4 |
|
09-Jan-2017 |
Brian Salomon <bsalomon@google.com> |
Revert "Removing ref counting from GrXPFactory." This reverts commit a8f80de2bc17672b4b6f26d3cf6b38123ac850c9. Reason for revert: nanobench failing on windows bots, possibly others Change-Id: Iacb8c650064a28654c165665be057377ffb02ba5 Reviewed-on: https://skia-review.googlesource.com/6802 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
a8f80de2bc17672b4b6f26d3cf6b38123ac850c9 |
|
07-Jan-2017 |
Brian Salomon <bsalomon@google.com> |
Removing ref counting from GrXPFactory. All GrXPFactory instances are static constexpr. Change-Id: If1086b08534166201e53b3fd9379104e361eb5e6 Reviewed-on: https://skia-review.googlesource.com/6701 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
c747bb6526e36ccf775182345116bf9ade73732b |
|
06-Jan-2017 |
Brian Salomon <bsalomon@google.com> |
Make all GrXPFactory instances statics. Change-Id: I423b821d96d483db8c43a6d5530adbc19cd68912 Reviewed-on: https://skia-review.googlesource.com/6693 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Greg Daniel <egdaniel@google.com>
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
92aee3d6857386f2b5b8e1148e680a7b58e9b1fc |
|
21-Dec-2016 |
Brian Salomon <bsalomon@google.com> |
This renames methods and classes that relate to static analysis of combinations of GrDrawOps and GrPipelines. Change-Id: I737b901a19d3c67d2ff7f95802fb4df35656beb2 Reviewed-on: https://skia-review.googlesource.com/6199 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
06ca8ec87cf6fab57cadd043a5ac18c4154a4129 |
|
09-Jun-2016 |
bungeman <bungeman@google.com> |
sk_sp for Ganesh. Convert use of GrFragmentProcessor, GrGeometryProcessor, and GrXPFactory to sk_sp. This clarifies ownership and should reduce reference count churn by moving ownership. GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2041113004 Review-Url: https://codereview.chromium.org/2041113004
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
3ccf2e75ed92adb3c2c289e0cb95901d162df394 |
|
06-May-2016 |
cdalton <cdalton@nvidia.com> |
Remove hasMixedSamples() from GrPipelineBuilder This info is unknown until after any clip has been applied to the stencil settings, so it's misleading to include in the builder. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1952323002 Review-Url: https://codereview.chromium.org/1952323002
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
9d524f22bfde5dc3dc8f48e1be39bdebd3bb0304 |
|
29-Mar-2016 |
halcanary <halcanary@google.com> |
Style bikeshed - remove extraneous whitespace GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1842753002 Review URL: https://codereview.chromium.org/1842753002
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
2279325d539700ee3da29d6e874b3b3ce1dcf49c |
|
30-Jan-2016 |
ethannicholas <ethannicholas@google.com> |
added support for PLS path rendering BUG=skia:3555 GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1541903002 Committed: https://skia.googlesource.com/skia/+/7df3f5e127f8016d17b637cc48a6a4718f1a6822 Review URL: https://codereview.chromium.org/1541903002
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
5366a09ed07e886dd5fd1b94828241c53df3726d |
|
22-Jan-2016 |
ethannicholas <ethannicholas@google.com> |
Revert of added support for PLS path rendering (patchset #16 id:360001 of https://codereview.chromium.org/1541903002/ ) Reason for revert: ASAN failure at src/gpu/GrXferProcessor.cpp:224 Original issue's description: > added support for PLS path rendering > > BUG=skia:3555 > GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1541903002 > > Committed: https://skia.googlesource.com/skia/+/7df3f5e127f8016d17b637cc48a6a4718f1a6822 TBR=bsalomon@google.com,egdaniel@google.com,joshualitt@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia:3555 GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1626553002 Review URL: https://codereview.chromium.org/1626553002
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
7df3f5e127f8016d17b637cc48a6a4718f1a6822 |
|
22-Jan-2016 |
ethannicholas <ethannicholas@google.com> |
added support for PLS path rendering BUG=skia:3555 GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1541903002 Review URL: https://codereview.chromium.org/1541903002
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
de4166a3b45d859512e27e3257841f064d67549c |
|
30-Nov-2015 |
ethannicholas <ethannicholas@google.com> |
APIs which took colorPOI / coveragePOI pairs updated to take a GrPipelineOptimizations struct TBR=bsalomon@google.com Review URL: https://codereview.chromium.org/1480353002
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
c4b72720e75313079212e69e46a5ef7c474b2305 |
|
23-Nov-2015 |
egdaniel <egdaniel@google.com> |
Don't create a GXPFactory when blend is SrcOver BUG=skia: Review URL: https://codereview.chromium.org/1471053002
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
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/GrDisableColorXP.h
|
86ae0a9e465f157eaa263ef7515e10619946ff83 |
|
09-Jun-2015 |
cdalton <cdalton@nvidia.com> |
Add mixed samples support to XPs BUG=skia: Review URL: https://codereview.chromium.org/1164973002
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
1fa4572d1a32e9fbf102d7388efe5e44bef460f1 |
|
02-Jun-2015 |
cdalton <cdalton@nvidia.com> |
Update XPF invariant info to not account for conflation Renames getInvariantOutput to getInvariantBlendedColor on GrXPFactory and redefines it to not account for coverage conflation. This is the information that all the callsites actually wanted to know. BUG=skia: Review URL: https://codereview.chromium.org/1161273005
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
6a44c6a7d89b748fb040d41697a337d357d7fa22 |
|
26-May-2015 |
bsalomon <bsalomon@google.com> |
Move DstCoordTexture to GrXP, rename and remove the word "copy" from dstcopytexture names. Committed: https://skia.googlesource.com/skia/+/bb106b5f6ee18f7453b63f3a95f421b60a957f39 Review URL: https://codereview.chromium.org/1132093004
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
d23a395d519835f78630aaea3f2a2c30ecdffe87 |
|
23-May-2015 |
bungeman <bungeman@google.com> |
Revert of Move DstCoordTexture to GrXP, rename and remove the word "copy" from dstcopytexture names. (patchset #6 id:100001 of https://codereview.chromium.org/1132093004/) Reason for revert: This is asserting on GTX660 bots with '!dst'. This may just be catching an existing issue. https://build.chromium.org/p/client.skia/builders/Test-Ubuntu-GCC-ShuttleA-GPU-GTX660-x86_64-Debug/builds/622/steps/dm/logs/stdio https://build.chromium.org/p/client.skia/builders/Test-Win7-MSVC-ShuttleA-GPU-HD2000-x86-Debug-ANGLE/builds/209/steps/dm/logs/stdio https://build.chromium.org/p/client.skia/builders/Test-Win8-MSVC-ShuttleA-GPU-GTX660-x86_64-Debug/builds/318/steps/dm/logs/stdio Original issue's description: > Move DstCoordTexture to GrXP, rename and remove the word "copy" from dstcopytexture names. > > Committed: https://skia.googlesource.com/skia/+/bb106b5f6ee18f7453b63f3a95f421b60a957f39 TBR=egdaniel@google.com,bsalomon@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/1158453004
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
bb106b5f6ee18f7453b63f3a95f421b60a957f39 |
|
22-May-2015 |
bsalomon <bsalomon@google.com> |
Move DstCoordTexture to GrXP, rename and remove the word "copy" from dstcopytexture names. Review URL: https://codereview.chromium.org/1132093004
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
4b91f768b348aa1cebeb54f3ff9331938734c242 |
|
19-May-2015 |
bsalomon <bsalomon@google.com> |
rename GrDrawTargetCaps to GrCaps Review URL: https://codereview.chromium.org/1133123009
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
99bc9fdfb555f1cf8e426329633e735e3020b829 |
|
04-May-2015 |
egdaniel <egdaniel@google.com> |
Remove canTweakAlphaForCoverage from XP's since batch reads flag. BUG=skia: Review URL: https://codereview.chromium.org/1127693002
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
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/GrDisableColorXP.h
|
0bdeec9640640922eb6b0a9f798bcd85eb898f29 |
|
23-Feb-2015 |
egdaniel <egdaniel@google.com> |
Remove canApplyCoverage from XP and all related functions in gpu code. BUG=skia: Review URL: https://codereview.chromium.org/919683002
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
3ad6570e94965d54871eaea8b314f560b82ca792 |
|
17-Feb-2015 |
egdaniel <egdaniel@google.com> |
Use dst copies in porter duffer XP to correctly render certain blends. BUG=skia: Committed: https://skia.googlesource.com/skia/+/997c6358d94e188b1a7b89a4f86e24cbe0f5a164 Review URL: https://codereview.chromium.org/914003003
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
2b816bacc0696f03d88c8060b21eda1e5cc7e8b1 |
|
13-Feb-2015 |
egdaniel <egdaniel@google.com> |
Revert of Use dst copies in porter duffer XP to correctly render certain blends. (patchset #4 id:60001 of https://codereview.chromium.org/914003003/) Reason for revert: Failing GLProgramTest passing in stupid coeffs Original issue's description: > Use dst copies in porter duffer XP to correctly render certain blends. > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/997c6358d94e188b1a7b89a4f86e24cbe0f5a164 TBR=bsalomon@google.com,joshualitt@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/923153003
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
997c6358d94e188b1a7b89a4f86e24cbe0f5a164 |
|
13-Feb-2015 |
egdaniel <egdaniel@google.com> |
Use dst copies in porter duffer XP to correctly render certain blends. BUG=skia: Review URL: https://codereview.chromium.org/914003003
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
e36914cb205699526988127a827d1a76c9a98d39 |
|
13-Feb-2015 |
egdaniel <egdaniel@google.com> |
Pass in ProcOptInfos into willNeedDstCopy on XPs BUG=skia: Review URL: https://codereview.chromium.org/912413002
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
41d4f09356567ead0216e1a7e4110bd58822b81f |
|
09-Feb-2015 |
egdaniel <egdaniel@google.com> |
Move GrXferProcessor subclasses into cpp files BUG=skia: Review URL: https://codereview.chromium.org/860383007
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
50785a3d10b53bea5beb6e18431a2449860be237 |
|
06-Feb-2015 |
bsalomon <bsalomon@google.com> |
Revert of Revert of Move DstCopy on gpu into the GrXferProcessor. (patchset #1 id:1 of https://codereview.chromium.org/901663007/) Reason for revert: The revert didn't help the 10.9 bot. Unreverting by reverting the revert (which is basically relanding the original patch which itself was a revert of a revert). Revert. Original issue's description: > Revert of Move DstCopy on gpu into the GrXferProcessor. (patchset #11 id:200001 of https://codereview.chromium.org/885923002/) > > Reason for revert: > Testing to see if reverting fixes 10.9 bots. > > Original issue's description: > > Move DstCopy on gpu into the GrXferProcessor. > > > > BUG=skia: > > > > Committed: https://skia.googlesource.com/skia/+/74a11753604768bf461b80cabb66060e8564d82c > > > > Committed: https://skia.googlesource.com/skia/+/5e1378d0e075a323144ba14e0a4cbcca35eccc69 > > TBR=joshualitt@google.com,egdaniel@google.com > NOPRESUBMIT=true > NOTREECHECKS=true > NOTRY=true > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/21b2c53218ab25f4268e3992e51d916076a2a7ee TBR=joshualitt@google.com,egdaniel@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/873723009
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
21b2c53218ab25f4268e3992e51d916076a2a7ee |
|
05-Feb-2015 |
bsalomon <bsalomon@google.com> |
Revert of Move DstCopy on gpu into the GrXferProcessor. (patchset #11 id:200001 of https://codereview.chromium.org/885923002/) Reason for revert: Testing to see if reverting fixes 10.9 bots. Original issue's description: > Move DstCopy on gpu into the GrXferProcessor. > > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/74a11753604768bf461b80cabb66060e8564d82c > > Committed: https://skia.googlesource.com/skia/+/5e1378d0e075a323144ba14e0a4cbcca35eccc69 TBR=joshualitt@google.com,egdaniel@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/901663007
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
5e1378d0e075a323144ba14e0a4cbcca35eccc69 |
|
05-Feb-2015 |
egdaniel <egdaniel@google.com> |
Move DstCopy on gpu into the GrXferProcessor. BUG=skia: Committed: https://skia.googlesource.com/skia/+/74a11753604768bf461b80cabb66060e8564d82c Review URL: https://codereview.chromium.org/885923002
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
4dce32c46675efb901618161771d450241af4307 |
|
04-Feb-2015 |
egdaniel <egdaniel@google.com> |
Revert "Move DstCopy on gpu into the GrXferProcessor." This reverts commit 74a11753604768bf461b80cabb66060e8564d82c. TBR=joshualitt@google.com,bsalomon@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Committed: https://skia.googlesource.com/skia/+/3e9dfdb3784c0cbfecf7589a74aa9aff7ef40abd Review URL: https://codereview.chromium.org/896163003
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
7adb35559430303a25be37f21af56c0df6c62f2b |
|
04-Feb-2015 |
egdaniel <egdaniel@google.com> |
Revert of Revert "Move DstCopy on gpu into the GrXferProcessor." (patchset #1 id:1 of https://codereview.chromium.org/896163003/) Reason for revert: failed on my manual revert Original issue's description: > Revert "Move DstCopy on gpu into the GrXferProcessor." > > This reverts commit 74a11753604768bf461b80cabb66060e8564d82c. > > TBR=joshualitt@google.com,bsalomon@google.com > NOPRESUBMIT=true > NOTREECHECKS=true > NOTRY=true > BUG=skia: > > Committed: https://skia.googlesource.com/skia/+/3e9dfdb3784c0cbfecf7589a74aa9aff7ef40abd TBR= NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/900913002
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
3e9dfdb3784c0cbfecf7589a74aa9aff7ef40abd |
|
04-Feb-2015 |
egdaniel <egdaniel@google.com> |
Revert "Move DstCopy on gpu into the GrXferProcessor." This reverts commit 74a11753604768bf461b80cabb66060e8564d82c. TBR=joshualitt@google.com,bsalomon@google.com NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=skia: Review URL: https://codereview.chromium.org/896163003
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
74a11753604768bf461b80cabb66060e8564d82c |
|
04-Feb-2015 |
egdaniel <egdaniel@google.com> |
Move DstCopy on gpu into the GrXferProcessor. BUG=skia: Review URL: https://codereview.chromium.org/885923002
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
71e236c03e65ff6b48a3d0eb091f814dd3e3a928 |
|
20-Jan-2015 |
egdaniel <egdaniel@google.com> |
Remove willReadDst from GrFragmentProcessor. Since only XP's can read dst now, there is no reason to have this query on GrFP. This also triggered a chain reaction of cleaning up/removing unnecessary code elsewhere. BUG=skia: Review URL: https://codereview.chromium.org/851143003
/external/skia/src/gpu/effects/GrDisableColorXP.h
|
080e673b10ac607305f140ddb245e140ccde40c6 |
|
22-Dec-2014 |
egdaniel <egdaniel@google.com> |
Add XP to handle the cases where we disable color write. BUG=skia: Review URL: https://codereview.chromium.org/787233003
/external/skia/src/gpu/effects/GrDisableColorXP.h
|