Move some work from backend onClear to base class clear

src/gpu/GrGpu.cpp

 
 /*
  * Copyright 2010 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 
 #include "GrGpu.h"
@@ skipping 200 matching lines @@
     this->handleDirtyContext();
     GrIndexBuffer* ib = this->onCreateIndexBuffer(size, dynamic);
     if (!this->caps()->reuseScratchBuffers()) {
         ib->resourcePriv().removeScratchKey();
     }
     return ib;
 }
 
 void GrGpu::clear(const SkIRect* rect,
                   GrColor color,
-                  bool canIgnoreRect,
                   GrRenderTarget* renderTarget) {

[bsalomon, 2015/08/06 16:04:14]

Shouldn't this now just require a rect param (const & instead of const *)? Also
could assert that rect is contained in the rt rect.

[egdaniel, 2015/08/06 17:02:53]

Done.

     SkASSERT(renderTarget);
     this->handleDirtyContext();
-    this->onClear(renderTarget, rect, color, canIgnoreRect);
+    this->onClear(renderTarget, rect, color);
 }
 
 void GrGpu::clearStencilClip(const SkIRect& rect,
                              bool insideClip,
                              GrRenderTarget* renderTarget) {
     SkASSERT(renderTarget);
     this->handleDirtyContext();
     this->onClearStencilClip(renderTarget, rect, insideClip);
 }
 
@@ skipping 144 matching lines @@
 ////////////////////////////////////////////////////////////////////////////////
 
 void GrGpu::draw(const DrawArgs& args, const GrVertices& vertices) {
     this->handleDirtyContext();
     GrVertices::Iterator iter;
     const GrNonInstancedVertices* verts = iter.init(vertices);
     do {
         this->onDraw(args, *verts);
     } while ((verts = iter.next()));
 }