cc: Move filters from RenderPassDrawQuad to RenderPass

services/ui/demo/bitmap_uploader.cc

 // Copyright 2015 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 "services/ui/demo/bitmap_uploader.h"
 
 #include <stddef.h>
 #include <utility>
 
 #include "base/bind.h"
@@ skipping 66 matching lines @@
   const gfx::Rect bounds(window_->bounds().size());
 
   cc::CompositorFrame frame;
   // TODO(rjkroege): Support device scale factors other than 1.
   frame.metadata.device_scale_factor = 1.0f;
   frame.resource_list.resize(0u);
 
   const cc::RenderPassId render_pass_id(1, 1);
   std::unique_ptr<cc::RenderPass> pass = cc::RenderPass::Create();
   pass->SetAll(render_pass_id, bounds, bounds, gfx::Transform(),
+               cc::FilterOperations(), cc::FilterOperations(),

[danakj, 2016/12/01 01:50:46]

This should use SetNew for that reason. It can set has_transparent_background
directly.

[ajuma, 2016/12/13 15:08:12]

Done.

                true /* has_transparent_background */);
 
   // The SharedQuadState is owned by the SharedQuadStateList
   // shared_quad_state_list.
   cc::SharedQuadState* sqs = pass->CreateAndAppendSharedQuadState();
   sqs->SetAll(gfx::Transform(), bounds.size(), bounds, bounds,
               false /* is_clipped */, 1.f /* opacity */, SkBlendMode::kSrc,
               0 /* sorting_context_id */);
 
   if (bitmap_.get()) {
@@ skipping 137 matching lines @@
   // TODO(fsamuel): Implement this.
 }
 
 void BitmapUploader::SetExternalTilePriorityConstraints(
     const gfx::Rect& viewport_rect,
     const gfx::Transform& transform) {
   // TODO(fsamuel): Implement this.
 }
 
 }  // namespace ui