Pass resourceless software mode in BeginFrameArgs

cc/output/software_renderer.cc

 // Copyright 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "cc/output/software_renderer.h"
 
 #include "base/debug/trace_event.h"
 #include "cc/base/math_util.h"
 #include "cc/output/compositor_frame.h"
 #include "cc/output/compositor_frame_ack.h"
@@ skipping 454 matching lines @@
                                           quad_rect,
                                           &current_paint_);
   }
 
   if (needs_layer)
     current_canvas_->restore();
 }
 
 void SoftwareRenderer::DrawTileQuad(const DrawingFrame* frame,
                                     const TileDrawQuad* quad) {
-  DCHECK(!output_surface_->ForcedDrawToSoftwareDevice());
+  DCHECK(resource_provider_);

[danakj, 2014/07/10 18:33:59]

is this adding some value that i don't see? i'd just remove this line

[boliu, 2014/07/10 18:40:36]

This was supposed to check resourceless software draw never generates tile
quads. I could have added a method to RendererClient just to keep this DCHECK,
but I thought that was overkill, so added the somewhat equivalent
resource_provider_ check.

Umm, maybe just need a comment?

[danakj, 2014/07/10 18:43:21]

Oh, I see this is NULL in resourceless mode.

   DCHECK(IsSoftwareResource(quad->resource_id));
 
   ResourceProvider::ScopedReadLockSoftware lock(resource_provider_,
                                                 quad->resource_id);
   if (!lock.valid())
     return;
   DCHECK_EQ(GL_CLAMP_TO_EDGE, lock.wrap_mode());
 
   gfx::RectF visible_tex_coord_rect = MathUtil::ScaleRectProportional(
       quad->tex_coord_rect, quad->rect, quad->visible_rect);
@@ skipping 158 matching lines @@
 }
 
 void SoftwareRenderer::DidChangeVisibility() {
   if (visible())
     EnsureBackbuffer();
   else
     DiscardBackbuffer();
 }
 
 }  // namespace cc