Axis aligned shader-based rect drawing

src/gpu/GrContext.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 "GrContext.h"
@@ skipping 790 matching lines @@
                 combinedMatrix.mapVectors(&strokeSize, 1);
                 strokeSize.setAbs(strokeSize);
             } else {
                 strokeSize.set(SK_Scalar1, SK_Scalar1);
             }
             fAARectRenderer->strokeAARect(this->getGpu(), target, devRect,
                                           strokeSize, useVertexCoverage);
         } else {
             // filled AA rect
 #ifdef SHADER_AA_FILL_RECT
-            fAARectRenderer->shaderFillAARect(this->getGpu(), target,
-                                              rect, combinedMatrix, devRect,
-                                              useVertexCoverage);
+            if (combinedMatrix.rectStaysRect()) {

[bsalomon, 2013/04/29 19:44:50]

Seems cleaner to me make axis-aligned vs not an implementation detail of the
renderer class rather than part of its API.

[robertphillips, 2013/05/02 13:56:39]

Done. Note that this change will make more sense if/when the rotated geometric
AA rects CL lands. 

+                fAARectRenderer->shaderFillAAAARect(this->getGpu(), target,
+                                                    combinedMatrix, devRect);
+            } else {
+                fAARectRenderer->shaderFillAARect(this->getGpu(), target,
+                                                  rect, combinedMatrix, devRect);
+            }
 #else
             fAARectRenderer->fillAARect(this->getGpu(), target,
                                         devRect, useVertexCoverage);
 #endif
         }
         return;
     }
 
     if (width >= 0) {
         // TODO: consider making static vertex buffers for these cases.
@@ skipping 980 matching lines @@
         return srcTexture;
     }
 }
 
 ///////////////////////////////////////////////////////////////////////////////
 #if GR_CACHE_STATS
 void GrContext::printCacheStats() const {
     fTextureCache->printStats();
 }
 #endif