Fix kLargestDrawQuadSize

cc/quads/largest_draw_quad.cc

 // Copyright 2014 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/quads/largest_draw_quad.h"
 
 #include <stddef.h>
 
 #include <algorithm>
 
 #include "cc/quads/debug_border_draw_quad.h"
 #include "cc/quads/picture_draw_quad.h"
 #include "cc/quads/render_pass_draw_quad.h"
 #include "cc/quads/solid_color_draw_quad.h"
 #include "cc/quads/stream_video_draw_quad.h"
 #include "cc/quads/surface_draw_quad.h"
 #include "cc/quads/texture_draw_quad.h"
 #include "cc/quads/tile_draw_quad.h"
 #include "cc/quads/yuv_video_draw_quad.h"
 
 namespace {
-const size_t kLargestDrawQuadSize =
-    sizeof(cc::YUVVideoDrawQuad) + sizeof(gfx::ColorSpace);
+const size_t kLargestDrawQuadSize1 = sizeof(cc::StreamVideoDrawQuad);
+const size_t kLargestDrawQuadSize2 = sizeof(cc::RenderPassDrawQuad) >
+                                             kLargestDrawQuadSize1
+                                         ? sizeof(cc::RenderPassDrawQuad)
+                                         : kLargestDrawQuadSize1;
+const size_t kLargestDrawQuadSize = sizeof(cc::YUVVideoDrawQuad) >
+                                            kLargestDrawQuadSize2
+                                        ? sizeof(cc::YUVVideoDrawQuad)
+                                        : kLargestDrawQuadSize2;
+
 }  // namespace
 
 namespace cc {
 
 size_t LargestDrawQuadSize() {
-  // Currently the largest quad is either a RenderPassDrawQuad or a
-  // StreamVideoDrawQuad depends on hardware structure.
+  // Currently the largest quad is either a RenderPassDrawQuad,
+  // StreamVideoDrawQuad or YUVVideoDrawQuad depending on hardware structure.
 
-  // Use compile assert to make sure largest is actually larger than all other
+  // Use compile assert to make sure largest is actually larger than all
   // type of draw quads.
   static_assert(sizeof(DebugBorderDrawQuad) <= kLargestDrawQuadSize,
                 "Largest Draw Quad size needs update. DebugBorderDrawQuad is "
                 "currently largest.");
   static_assert(sizeof(PictureDrawQuad) <= kLargestDrawQuadSize,
                 "Largest Draw Quad size needs update. PictureDrawQuad is "
                 "currently largest.");
   static_assert(sizeof(TextureDrawQuad) <= kLargestDrawQuadSize,
                 "Largest Draw Quad size needs update. TextureDrawQuad is "
                 "currently largest.");
   static_assert(sizeof(SolidColorDrawQuad) <= kLargestDrawQuadSize,
                 "Largest Draw Quad size needs update. SolidColorDrawQuad is "
                 "currently largest.");
   static_assert(sizeof(SurfaceDrawQuad) <= kLargestDrawQuadSize,
                 "Largest Draw Quad size needs update. SurfaceDrawQuad is "
                 "currently largest.");
   static_assert(sizeof(TileDrawQuad) <= kLargestDrawQuadSize,
                 "Largest Draw Quad size needs update. TileDrawQuad is "
                 "currently largest.");
   static_assert(sizeof(YUVVideoDrawQuad) <= kLargestDrawQuadSize,
                 "Largest Draw Quad size needs update. YUVVideoDrawQuad is "
                 "currently largest.");
+  static_assert(sizeof(RenderPassDrawQuad) <= kLargestDrawQuadSize,
+                "Largest Draw Quad size needs update. RenderPassDrawQuad is "
+                "currently largest.");
+  static_assert(sizeof(StreamVideoDrawQuad) <= kLargestDrawQuadSize,
+                "Largest Draw Quad size needs update. StreamVideoDrawQuad is "
+                "currently largest.");
 
   return kLargestDrawQuadSize;
 }
 
 }  // namespace cc