Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(511)

Side by Side Diff: cc/quads/draw_quad_unittest.cc

Issue 2324723002: Speed up quad deserialization. (Closed)
Patch Set: comment added Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « cc/ipc/cc_param_traits.cc ('k') | cc/quads/largest_draw_quad.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2012 The Chromium Authors. All rights reserved. 1 // Copyright 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "cc/quads/draw_quad.h" 5 #include "cc/quads/draw_quad.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include <algorithm> 9 #include <algorithm>
10 10
(...skipping 928 matching lines...) Expand 10 before | Expand all | Expand 10 after
939 case DrawQuad::SURFACE_CONTENT: 939 case DrawQuad::SURFACE_CONTENT:
940 largest = std::max(largest, sizeof(SurfaceDrawQuad)); 940 largest = std::max(largest, sizeof(SurfaceDrawQuad));
941 break; 941 break;
942 case DrawQuad::TILED_CONTENT: 942 case DrawQuad::TILED_CONTENT:
943 largest = std::max(largest, sizeof(TileDrawQuad)); 943 largest = std::max(largest, sizeof(TileDrawQuad));
944 break; 944 break;
945 case DrawQuad::STREAM_VIDEO_CONTENT: 945 case DrawQuad::STREAM_VIDEO_CONTENT:
946 largest = std::max(largest, sizeof(StreamVideoDrawQuad)); 946 largest = std::max(largest, sizeof(StreamVideoDrawQuad));
947 break; 947 break;
948 case DrawQuad::YUV_VIDEO_CONTENT: 948 case DrawQuad::YUV_VIDEO_CONTENT:
949 largest = std::max(largest, 949 largest = std::max(largest, sizeof(YUVVideoDrawQuad));
950 sizeof(YUVVideoDrawQuad) + sizeof(gfx::ColorSpace));
951 break; 950 break;
952 case DrawQuad::INVALID: 951 case DrawQuad::INVALID:
953 break; 952 break;
954 } 953 }
955 } 954 }
956 EXPECT_EQ(LargestDrawQuadSize(), largest); 955 EXPECT_EQ(LargestDrawQuadSubclassSize(), largest);
957 956
958 if (!HasFailure()) 957 if (!HasFailure())
959 return; 958 return;
960 959
961 // On failure, output the size of all quads for debugging. 960 // On failure, output the size of all quads for debugging.
962 LOG(ERROR) << "largest " << largest; 961 LOG(ERROR) << "largest " << largest;
963 LOG(ERROR) << "kLargestDrawQuad " << LargestDrawQuadSize(); 962 LOG(ERROR) << "kLargestDrawQuad " << LargestDrawQuadSubclassSize();
964 for (int i = 0; i <= DrawQuad::MATERIAL_LAST; ++i) { 963 for (int i = 0; i <= DrawQuad::MATERIAL_LAST; ++i) {
965 switch (static_cast<DrawQuad::Material>(i)) { 964 switch (static_cast<DrawQuad::Material>(i)) {
966 case DrawQuad::DEBUG_BORDER: 965 case DrawQuad::DEBUG_BORDER:
967 LOG(ERROR) << "DebugBorderDrawQuad " << sizeof(DebugBorderDrawQuad); 966 LOG(ERROR) << "DebugBorderDrawQuad " << sizeof(DebugBorderDrawQuad);
968 break; 967 break;
969 case DrawQuad::PICTURE_CONTENT: 968 case DrawQuad::PICTURE_CONTENT:
970 LOG(ERROR) << "PictureDrawQuad " << sizeof(PictureDrawQuad); 969 LOG(ERROR) << "PictureDrawQuad " << sizeof(PictureDrawQuad);
971 break; 970 break;
972 case DrawQuad::TEXTURE_CONTENT: 971 case DrawQuad::TEXTURE_CONTENT:
973 LOG(ERROR) << "TextureDrawQuad " << sizeof(TextureDrawQuad); 972 LOG(ERROR) << "TextureDrawQuad " << sizeof(TextureDrawQuad);
(...skipping 17 matching lines...) Expand all
991 LOG(ERROR) << "YUVVideoDrawQuad " << sizeof(YUVVideoDrawQuad); 990 LOG(ERROR) << "YUVVideoDrawQuad " << sizeof(YUVVideoDrawQuad);
992 break; 991 break;
993 case DrawQuad::INVALID: 992 case DrawQuad::INVALID:
994 break; 993 break;
995 } 994 }
996 } 995 }
997 } 996 }
998 997
999 } // namespace 998 } // namespace
1000 } // namespace cc 999 } // namespace cc
OLDNEW
« no previous file with comments | « cc/ipc/cc_param_traits.cc ('k') | cc/quads/largest_draw_quad.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698