Implement RenderPass StructTraits

components/bitmap_uploader/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 "components/bitmap_uploader/bitmap_uploader.h"
 
 #include <stddef.h>
 #include <utility>
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "cc/ipc/compositor_frame.mojom.h"
+#include "cc/quads/render_pass.h"
+#include "cc/quads/solid_color_draw_quad.h"
+#include "cc/quads/texture_draw_quad.h"
 #include "components/mus/public/cpp/gles2_context.h"
 #include "components/mus/public/cpp/surfaces/surfaces_type_converters.h"
-#include "components/mus/public/cpp/surfaces/surfaces_utils.h"
 #include "components/mus/public/cpp/window.h"
 #include "components/mus/public/cpp/window_surface.h"
 
 namespace bitmap_uploader {
 namespace {
 
 const uint32_t g_transparent_color = 0x00000000;
 
 }  // namespace
 
@@ skipping 38 matching lines @@
                                Format format) {
   width_ = width;
   height_ = height;
   bitmap_ = std::move(data);
   format_ = format;
   if (surface_)
     Upload();
 }
 
 void BitmapUploader::Upload() {
-  const gfx::Rect bounds(window_->bounds());
-  cc::mojom::RenderPassPtr pass = mojo::CreateDefaultPass(1, bounds);
+  const gfx::Rect bounds(window_->bounds().size());

[danakj, 2016/06/22 20:51:35]

is there a test for this behaviour change?

[Fady Samuel, 2016/06/22 21:41:19]

Unfortunately bitmap_uploader lacks tests. I'll write one in a separate CL. I
have tested Mus+Ash manually with this CL.

+
   cc::mojom::CompositorFramePtr frame = cc::mojom::CompositorFrame::New();
-
-  // TODO(rjkroege): Support device scale factor in PDF viewer
+  // TODO(rjkroege): Support device scale factors other than 1.
   frame->metadata.device_scale_factor = 1.0f;
-
   frame->resources.resize(0u);
 
-  pass->quads.resize(0u);
-  // The SharedQuadState is owned by the SharedQuadStateList shared_quad_states.
-  cc::SharedQuadState* sqs =
-      pass->shared_quad_states.AllocateAndConstruct<cc::SharedQuadState>();
-  mojo::ConfigureSharedQuadState(bounds.size(), sqs);
+  cc::RenderPassId render_pass_id(1, 1);
+  std::unique_ptr<cc::RenderPass> pass = cc::RenderPass::Create();
+  pass->SetAll(cc::RenderPassId(1, 1), bounds, bounds, gfx::Transform(),
+               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 */, SkXfermode::kSrc_Mode,
+              0 /* sorting_context_id */);
 
   if (bitmap_.get()) {
     gpu::gles2::GLES2Interface* gl = gles2_context_->interface();
     gfx::Size bitmap_size(width_, height_);
     GLuint texture_id = BindTextureForSize(bitmap_size);
     gl->TexSubImage2D(GL_TEXTURE_2D, 0, 0, 0, bitmap_size.width(),
                       bitmap_size.height(), TextureFormat(), GL_UNSIGNED_BYTE,
                       &((*bitmap_)[0]));
 
     gpu::Mailbox mailbox;
@@ skipping 10 matching lines @@
     resource.id = next_resource_id_++;
     resource_to_texture_id_map_[resource.id] = texture_id;
     resource.format = cc::ResourceFormat::RGBA_8888;
     resource.filter = GL_LINEAR;
     resource.size = bitmap_size;
     resource.mailbox_holder =
         gpu::MailboxHolder(mailbox, sync_token, GL_TEXTURE_2D);
     resource.read_lock_fences_enabled = false;
     resource.is_software = false;
     resource.is_overlay_candidate = false;
+    frame->resources.push_back(std::move(resource));
 
-    cc::mojom::DrawQuadPtr quad = cc::mojom::DrawQuad::New();
-    quad->material = cc::mojom::Material::TEXTURE_CONTENT;
+    cc::TextureDrawQuad* quad =
+        pass->CreateAndAppendDrawQuad<cc::TextureDrawQuad>();
 
     gfx::Size rect_size;
     if (width_ <= bounds.width() && height_ <= bounds.height()) {
       rect_size.SetSize(width_, height_);
     } else {
       // The source bitmap is larger than the viewport. Resize it while
       // maintaining the aspect ratio.
       float width_ratio = static_cast<float>(width_) / bounds.width();
       float height_ratio = static_cast<float>(height_) / bounds.height();
       if (width_ratio > height_ratio) {
         rect_size.SetSize(bounds.width(), height_ / width_ratio);
       } else {
         rect_size.SetSize(width_ / height_ratio, bounds.height());
       }
     }
     gfx::Rect rect(rect_size);
-    quad->rect = rect;
-    quad->opaque_rect = rect;
-    quad->visible_rect = rect;
-    quad->needs_blending = true;
-    quad->shared_quad_state_index = 0u;
-
-    cc::mojom::TextureQuadStatePtr texture_state =
-        cc::mojom::TextureQuadState::New();
-    texture_state->resource_id = resource.id;
-    texture_state->premultiplied_alpha = true;
-    texture_state->uv_top_left.SetPoint(0.f, 0.f);
-    texture_state->uv_bottom_right.SetPoint(1.f, 1.f);
-    texture_state->background_color = g_transparent_color;
-    for (int i = 0; i < 4; ++i)
-      texture_state->vertex_opacity.push_back(1.f);
-    texture_state->y_flipped = false;
-
-    frame->resources.push_back(std::move(resource));
-    quad->texture_quad_state = std::move(texture_state);
-    pass->quads.push_back(std::move(quad));
+    const bool needs_blending = true;
+    const bool premultiplied_alpha = true;
+    const gfx::PointF uv_top_left(0.f, 0.f);
+    const gfx::PointF uv_bottom_right(1.f, 1.f);
+    float vertex_opacity[4] = {1.f, 1.f, 1.f, 1.f};
+    const bool y_flipped = false;
+    const bool nearest_neighbor = false;
+    quad->SetAll(sqs, rect, rect, rect, needs_blending, resource.id,
+                 gfx::Size(), premultiplied_alpha, uv_top_left, uv_bottom_right,
+                 g_transparent_color, vertex_opacity, y_flipped,
+                 nearest_neighbor, false);
   }
 
   if (color_ != g_transparent_color) {
-    cc::mojom::DrawQuadPtr quad = cc::mojom::DrawQuad::New();
-    quad->material = cc::mojom::Material::SOLID_COLOR;
-    quad->rect = bounds;
-    quad->visible_rect = bounds;
-    quad->needs_blending = true;
-    quad->shared_quad_state_index = 0u;
-
-    cc::mojom::SolidColorQuadStatePtr color_state =
-        cc::mojom::SolidColorQuadState::New();
-    color_state->color = color_;
-    color_state->force_anti_aliasing_off = false;
-
-    quad->solid_color_quad_state = std::move(color_state);
-    pass->quads.push_back(std::move(quad));
+    cc::SolidColorDrawQuad* quad =
+        pass->CreateAndAppendDrawQuad<cc::SolidColorDrawQuad>();
+    const bool force_antialiasing_off = false;
+    const gfx::Rect opaque_rect(0, 0, 0, 0);
+    const bool needs_blending = true;
+    quad->SetAll(sqs, bounds, opaque_rect, bounds, needs_blending, color_,
+                 force_antialiasing_off);
   }
 
   frame->passes.push_back(std::move(pass));
 
   // TODO(rjkroege, fsamuel): We should throttle frames.
   surface_->SubmitCompositorFrame(std::move(frame), base::Closure());
 }
 
 uint32_t BitmapUploader::BindTextureForSize(const gfx::Size& size) {
   gpu::gles2::GLES2Interface* gl = gles2_context_->interface();
@@ skipping 23 matching lines @@
     DCHECK_EQ(1, resource.count);
     gl->WaitSyncTokenCHROMIUM(resource.sync_token.GetConstData());
     uint32_t texture_id = resource_to_texture_id_map_[resource.id];
     DCHECK_NE(0u, texture_id);
     resource_to_texture_id_map_.erase(resource.id);
     gl->DeleteTextures(1, &texture_id);
   }
 }
 
 }  // namespace bitmap_uploader