Use texture barriers to read directly from the RT

src/gpu/gl/GrGLGpu.cpp

 /*
  * Copyright 2011 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 
 #include "GrGLGpu.h"
 #include "GrGLStencilAttachment.h"
@@ skipping 2750 matching lines @@
             if(!SkIRect::IntersectsNoEmptyCheck(dstRect, srcRect)) {
                 return true;
             }
         } else {
             return true;
         }
     }
     return false;
 }
 
+void GrGLGpu::setXferBarrier(GrXferBarrierType type) {
+    switch (type) {
+        case kTexture_GrXferBarrierType:
+            SkASSERT(this->caps()->textureBarrierSupport());

[Chris Dalton, 2015/04/17 08:06:05]

I can update this to an if instead of an assert, to protect from Xfer barriers
that request nonsupported barriers.

[bsalomon, 2015/04/21 15:19:27]

I think an assert is fine. Technically a client could derive their own XP
classes, and if that was common it might be nice to fail in a friendlier manner,
but it's not. Asserting seems ok.

+            GL_CALL(TextureBarrier());
+            return;
+    }
+}
+
 void GrGLGpu::didAddGpuTraceMarker() {
     if (this->caps()->gpuTracingSupport()) {
         const GrTraceMarkerSet& markerArray = this->getActiveTraceMarkers();
         SkString markerString = markerArray.toStringLast();
 #if GR_FORCE_GPU_TRACE_DEBUGGING
         SkDebugf("%s\n", markerString.c_str());
 #else
         GL_CALL(PushGroupMarker(0, markerString.c_str()));
 #endif
     }
@@ skipping 35 matching lines @@
             this->setVertexArrayID(gpu, 0);
         }
         int attrCount = gpu->glCaps().maxVertexAttributes();
         if (fDefaultVertexArrayAttribState.count() != attrCount) {
             fDefaultVertexArrayAttribState.resize(attrCount);
         }
         attribState = &fDefaultVertexArrayAttribState;
     }
     return attribState;
 }