Revert of Move DstCopy on gpu into the GrXferProcessor.

Revert of Move DstCopy on gpu into the GrXferProcessor. (patchset #8 id:140001 of https://codereview.chromium.org/885923002/)

Reason for revert:
Breaking xfermodes on some android devices

Original issue's description:
> Move DstCopy on gpu into the GrXferProcessor.
> 
> BUG=skia:
> 
> Committed: https://skia.googlesource.com/skia/+/74a11753604768bf461b80cabb66060e8564d82c

TBR=joshualitt@google.com,bsalomon@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:

[egdaniel, 2015-02-04 14:04:23]

Created Revert of Move DstCopy on gpu into the GrXferProcessor.

[commit-bot: I haz the power, 2015-02-04 14:05:06]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/900883002/1

[commit-bot: I haz the power, 2015-02-04 14:05:21]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-02-04 14:05:22]

Failed to apply patch for src/gpu/GrBatchTarget.cpp:
While running git apply --index -3 -p1;
  error: patch failed: src/gpu/GrBatchTarget.cpp:37
  Falling back to three-way merge...
  Applied patch to 'src/gpu/GrBatchTarget.cpp' with conflicts.
  U src/gpu/GrBatchTarget.cpp

Patch:       src/gpu/GrBatchTarget.cpp
Index: src/gpu/GrBatchTarget.cpp
diff --git a/src/gpu/GrBatchTarget.cpp b/src/gpu/GrBatchTarget.cpp
index
d5d8d8f28508a386f9e35e4df85fbea5e7a801a9..6a3135310a7c611f1742f44497bfba71fbc8ca60
100644
--- a/src/gpu/GrBatchTarget.cpp
+++ b/src/gpu/GrBatchTarget.cpp
@@ -21,7 +21,8 @@
         BufferedFlush* bf = iter.get();
         const GrPipeline* pipeline = bf->fPipeline;
         const GrPrimitiveProcessor* primProc = bf->fPrimitiveProcessor.get();
-        fGpu->buildProgramDesc(&desc, *primProc, *pipeline, bf->fBatchTracker);
+        fGpu->buildProgramDesc(&desc, *primProc, *pipeline,
pipeline->descInfo(),
+                               bf->fBatchTracker);
 
         GrGpu::DrawArgs args(primProc, pipeline, &desc, &bf->fBatchTracker);
         for (int i = 0; i < bf->fDraws.count(); i++) {
@@ -37,7 +38,8 @@
     BufferedFlush* bf = fIter.get();
     const GrPipeline* pipeline = bf->fPipeline;
     const GrPrimitiveProcessor* primProc = bf->fPrimitiveProcessor.get();
-    fGpu->buildProgramDesc(&desc, *primProc, *pipeline, bf->fBatchTracker);
+    fGpu->buildProgramDesc(&desc, *primProc, *pipeline, pipeline->descInfo(),
+                           bf->fBatchTracker);
 
     GrGpu::DrawArgs args(primProc, pipeline, &desc, &bf->fBatchTracker);
     for (int i = 0; i < bf->fDraws.count(); i++) {

[joshualitt, 2015-02-24 21:45:14]

On 2015/02/04 14:05:22, I haz the power (commit-bot) wrote:
> Failed to apply patch for src/gpu/GrBatchTarget.cpp:
> While running git apply --index -3 -p1;
>   error: patch failed: src/gpu/GrBatchTarget.cpp:37
>   Falling back to three-way merge...
>   Applied patch to 'src/gpu/GrBatchTarget.cpp' with conflicts.
>   U src/gpu/GrBatchTarget.cpp
> 
> Patch:       src/gpu/GrBatchTarget.cpp
> Index: src/gpu/GrBatchTarget.cpp
> diff --git a/src/gpu/GrBatchTarget.cpp b/src/gpu/GrBatchTarget.cpp
> index
>
d5d8d8f28508a386f9e35e4df85fbea5e7a801a9..6a3135310a7c611f1742f44497bfba71fbc8ca60
> 100644
> --- a/src/gpu/GrBatchTarget.cpp
> +++ b/src/gpu/GrBatchTarget.cpp
> @@ -21,7 +21,8 @@
>          BufferedFlush* bf = iter.get();
>          const GrPipeline* pipeline = bf->fPipeline;
>          const GrPrimitiveProcessor* primProc = bf->fPrimitiveProcessor.get();
> -        fGpu->buildProgramDesc(&desc, *primProc, *pipeline,
bf->fBatchTracker);
> +        fGpu->buildProgramDesc(&desc, *primProc, *pipeline,
> pipeline->descInfo(),
> +                               bf->fBatchTracker);
>  
>          GrGpu::DrawArgs args(primProc, pipeline, &desc, &bf->fBatchTracker);
>          for (int i = 0; i < bf->fDraws.count(); i++) {
> @@ -37,7 +38,8 @@
>      BufferedFlush* bf = fIter.get();
>      const GrPipeline* pipeline = bf->fPipeline;
>      const GrPrimitiveProcessor* primProc = bf->fPrimitiveProcessor.get();
> -    fGpu->buildProgramDesc(&desc, *primProc, *pipeline, bf->fBatchTracker);
> +    fGpu->buildProgramDesc(&desc, *primProc, *pipeline, pipeline->descInfo(),
> +                           bf->fBatchTracker);
>  
>      GrGpu::DrawArgs args(primProc, pipeline, &desc, &bf->fBatchTracker);
>      for (int i = 0; i < bf->fDraws.count(); i++) {

Greg, is it okay to close this?