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

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

Issue 650393002: Modify ALLOW_UNUSED to allow enabling unused local warnings on MSVC. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Revert review comment Created 6 years, 2 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
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 <algorithm> 7 #include <algorithm>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/compiler_specific.h" 10 #include "base/compiler_specific.h"
(...skipping 87 matching lines...) Expand 10 before | Expand all | Expand 10 after
98 #define CREATE_SHARED_STATE() \ 98 #define CREATE_SHARED_STATE() \
99 scoped_ptr<RenderPass> render_pass = RenderPass::Create(); \ 99 scoped_ptr<RenderPass> render_pass = RenderPass::Create(); \
100 SharedQuadState* shared_state(CreateSharedQuadState(render_pass.get())); \ 100 SharedQuadState* shared_state(CreateSharedQuadState(render_pass.get())); \
101 SharedQuadState* copy_shared_state = \ 101 SharedQuadState* copy_shared_state = \
102 render_pass->CreateAndAppendSharedQuadState(); \ 102 render_pass->CreateAndAppendSharedQuadState(); \
103 copy_shared_state->CopyFrom(shared_state); 103 copy_shared_state->CopyFrom(shared_state);
104 104
105 #define QUAD_DATA \ 105 #define QUAD_DATA \
106 gfx::Rect quad_rect(30, 40, 50, 60); \ 106 gfx::Rect quad_rect(30, 40, 50, 60); \
107 gfx::Rect quad_visible_rect(40, 50, 30, 20); \ 107 gfx::Rect quad_visible_rect(40, 50, 30, 20); \
108 gfx::Rect ALLOW_UNUSED quad_opaque_rect(60, 55, 10, 10); \ 108 gfx::Rect quad_opaque_rect( 60, 55, 10, 10); \
109 bool ALLOW_UNUSED needs_blending = true; 109 ALLOW_UNUSED_LOCAL(quad_opaque_rect); \
110 bool needs_blending = true; \
111 ALLOW_UNUSED_LOCAL(needs_blending);
110 112
111 #define SETUP_AND_COPY_QUAD_NEW(Type, quad) \ 113 #define SETUP_AND_COPY_QUAD_NEW(Type, quad) \
112 DrawQuad* copy_new = \ 114 DrawQuad* copy_new = \
113 render_pass->CopyFromAndAppendDrawQuad(quad_new, copy_shared_state); \ 115 render_pass->CopyFromAndAppendDrawQuad(quad_new, copy_shared_state); \
114 CompareDrawQuad(quad_new, copy_new, copy_shared_state); \ 116 CompareDrawQuad(quad_new, copy_new, copy_shared_state); \
115 const Type* ALLOW_UNUSED copy_quad = Type::MaterialCast(copy_new); 117 const Type* copy_quad = Type::MaterialCast(copy_new); \
118 ALLOW_UNUSED_LOCAL(copy_quad);
116 119
117 #define SETUP_AND_COPY_QUAD_ALL(Type, quad) \ 120 #define SETUP_AND_COPY_QUAD_ALL(Type, quad) \
118 DrawQuad* copy_all = \ 121 DrawQuad* copy_all = \
119 render_pass->CopyFromAndAppendDrawQuad(quad_all, copy_shared_state); \ 122 render_pass->CopyFromAndAppendDrawQuad(quad_all, copy_shared_state); \
120 CompareDrawQuad(quad_all, copy_all, copy_shared_state); \ 123 CompareDrawQuad(quad_all, copy_all, copy_shared_state); \
121 copy_quad = Type::MaterialCast(copy_all); 124 copy_quad = Type::MaterialCast(copy_all);
122 125
123 #define SETUP_AND_COPY_QUAD_NEW_RP(Type, quad, a) \ 126 #define SETUP_AND_COPY_QUAD_NEW_RP(Type, quad, a) \
124 DrawQuad* copy_new = render_pass->CopyFromAndAppendRenderPassDrawQuad( \ 127 DrawQuad* copy_new = render_pass->CopyFromAndAppendRenderPassDrawQuad( \
125 quad_new, copy_shared_state, a); \ 128 quad_new, copy_shared_state, a); \
126 CompareDrawQuad(quad_new, copy_new, copy_shared_state); \ 129 CompareDrawQuad(quad_new, copy_new, copy_shared_state); \
127 const Type* ALLOW_UNUSED copy_quad = Type::MaterialCast(copy_new); 130 const Type* copy_quad = Type::MaterialCast(copy_new); \
131 ALLOW_UNUSED_LOCAL(copy_quad);
128 132
129 #define SETUP_AND_COPY_QUAD_ALL_RP(Type, quad, a) \ 133 #define SETUP_AND_COPY_QUAD_ALL_RP(Type, quad, a) \
130 DrawQuad* copy_all = render_pass->CopyFromAndAppendRenderPassDrawQuad( \ 134 DrawQuad* copy_all = render_pass->CopyFromAndAppendRenderPassDrawQuad( \
131 quad_all, copy_shared_state, a); \ 135 quad_all, copy_shared_state, a); \
132 CompareDrawQuad(quad_all, copy_all, copy_shared_state); \ 136 CompareDrawQuad(quad_all, copy_all, copy_shared_state); \
133 copy_quad = Type::MaterialCast(copy_all); 137 copy_quad = Type::MaterialCast(copy_all);
134 138
135 #define CREATE_QUAD_1_NEW(Type, a) \ 139 #define CREATE_QUAD_1_NEW(Type, a) \
136 Type* quad_new = render_pass->CreateAndAppendDrawQuad<Type>(); \ 140 Type* quad_new = render_pass->CreateAndAppendDrawQuad<Type>(); \
137 { QUAD_DATA quad_new->SetNew(shared_state, quad_rect, a); } \ 141 { QUAD_DATA quad_new->SetNew(shared_state, quad_rect, a); } \
(...skipping 870 matching lines...) Expand 10 before | Expand all | Expand 10 after
1008 LOG(ERROR) << "YUVVideoDrawQuad " << sizeof(YUVVideoDrawQuad); 1012 LOG(ERROR) << "YUVVideoDrawQuad " << sizeof(YUVVideoDrawQuad);
1009 break; 1013 break;
1010 case DrawQuad::INVALID: 1014 case DrawQuad::INVALID:
1011 break; 1015 break;
1012 } 1016 }
1013 } 1017 }
1014 } 1018 }
1015 1019
1016 } // namespace 1020 } // namespace
1017 } // namespace cc 1021 } // namespace cc
OLDNEW
« base/compiler_specific.h ('K') | « base/tuple_unittest.cc ('k') | ipc/ipc_message_macros.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698