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

Side by Side Diff: cc/ipc/cc_param_traits.cc

Issue 2102833002: cc: Fixup cc to not use auto to deduce a raw pointer. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 4 years, 5 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 | « no previous file | cc/layers/layer_proto_converter.cc » ('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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 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/ipc/cc_param_traits.h" 5 #include "cc/ipc/cc_param_traits.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 #include <utility> 8 #include <utility>
9 9
10 #include "base/numerics/safe_conversions.h" 10 #include "base/numerics/safe_conversions.h"
(...skipping 314 matching lines...) Expand 10 before | Expand all | Expand 10 after
325 WriteParam(m, p.output_rect); 325 WriteParam(m, p.output_rect);
326 WriteParam(m, p.damage_rect); 326 WriteParam(m, p.damage_rect);
327 WriteParam(m, p.transform_to_root_target); 327 WriteParam(m, p.transform_to_root_target);
328 WriteParam(m, p.has_transparent_background); 328 WriteParam(m, p.has_transparent_background);
329 WriteParam(m, base::checked_cast<uint32_t>(p.quad_list.size())); 329 WriteParam(m, base::checked_cast<uint32_t>(p.quad_list.size()));
330 330
331 cc::SharedQuadStateList::ConstIterator shared_quad_state_iter = 331 cc::SharedQuadStateList::ConstIterator shared_quad_state_iter =
332 p.shared_quad_state_list.begin(); 332 p.shared_quad_state_list.begin();
333 cc::SharedQuadStateList::ConstIterator last_shared_quad_state_iter = 333 cc::SharedQuadStateList::ConstIterator last_shared_quad_state_iter =
334 p.shared_quad_state_list.end(); 334 p.shared_quad_state_list.end();
335 for (const auto& quad : p.quad_list) { 335 for (auto* quad : p.quad_list) {
336 DCHECK(quad->rect.Contains(quad->visible_rect)) 336 DCHECK(quad->rect.Contains(quad->visible_rect))
337 << quad->material << " rect: " << quad->rect.ToString() 337 << quad->material << " rect: " << quad->rect.ToString()
338 << " visible_rect: " << quad->visible_rect.ToString(); 338 << " visible_rect: " << quad->visible_rect.ToString();
339 DCHECK(quad->opaque_rect.IsEmpty() || 339 DCHECK(quad->opaque_rect.IsEmpty() ||
340 quad->rect.Contains(quad->opaque_rect)) 340 quad->rect.Contains(quad->opaque_rect))
341 << quad->material << " rect: " << quad->rect.ToString() 341 << quad->material << " rect: " << quad->rect.ToString()
342 << " opaque_rect: " << quad->opaque_rect.ToString(); 342 << " opaque_rect: " << quad->opaque_rect.ToString();
343 343
344 switch (quad->material) { 344 switch (quad->material) {
345 case cc::DrawQuad::DEBUG_BORDER: 345 case cc::DrawQuad::DEBUG_BORDER:
(...skipping 171 matching lines...) Expand 10 before | Expand all | Expand 10 after
517 LogParam(p.output_rect, l); 517 LogParam(p.output_rect, l);
518 l->append(", "); 518 l->append(", ");
519 LogParam(p.damage_rect, l); 519 LogParam(p.damage_rect, l);
520 l->append(", "); 520 l->append(", ");
521 LogParam(p.transform_to_root_target, l); 521 LogParam(p.transform_to_root_target, l);
522 l->append(", "); 522 l->append(", ");
523 LogParam(p.has_transparent_background, l); 523 LogParam(p.has_transparent_background, l);
524 l->append(", "); 524 l->append(", ");
525 525
526 l->append("["); 526 l->append("[");
527 for (const auto& shared_quad_state : p.shared_quad_state_list) { 527 for (auto* shared_quad_state : p.shared_quad_state_list) {
528 if (shared_quad_state != p.shared_quad_state_list.front()) 528 if (shared_quad_state != p.shared_quad_state_list.front())
529 l->append(", "); 529 l->append(", ");
530 LogParam(*shared_quad_state, l); 530 LogParam(*shared_quad_state, l);
531 } 531 }
532 l->append("], ["); 532 l->append("], [");
533 for (const auto& quad : p.quad_list) { 533 for (auto* quad : p.quad_list) {
534 if (quad != p.quad_list.front()) 534 if (quad != p.quad_list.front())
535 l->append(", "); 535 l->append(", ");
536 switch (quad->material) { 536 switch (quad->material) {
537 case cc::DrawQuad::DEBUG_BORDER: 537 case cc::DrawQuad::DEBUG_BORDER:
538 LogParam(*cc::DebugBorderDrawQuad::MaterialCast(quad), l); 538 LogParam(*cc::DebugBorderDrawQuad::MaterialCast(quad), l);
539 break; 539 break;
540 case cc::DrawQuad::PICTURE_CONTENT: 540 case cc::DrawQuad::PICTURE_CONTENT:
541 NOTREACHED(); 541 NOTREACHED();
542 break; 542 break;
543 case cc::DrawQuad::TEXTURE_CONTENT: 543 case cc::DrawQuad::TEXTURE_CONTENT:
(...skipping 355 matching lines...) Expand 10 before | Expand all | Expand 10 after
899 #undef CC_IPC_CC_PARAM_TRAITS_MACROS_H_ 899 #undef CC_IPC_CC_PARAM_TRAITS_MACROS_H_
900 #include "cc/ipc/cc_param_traits_macros.h" 900 #include "cc/ipc/cc_param_traits_macros.h"
901 } // namespace IPC 901 } // namespace IPC
902 902
903 // Generate param traits log methods. 903 // Generate param traits log methods.
904 #include "ipc/param_traits_log_macros.h" 904 #include "ipc/param_traits_log_macros.h"
905 namespace IPC { 905 namespace IPC {
906 #undef CC_IPC_CC_PARAM_TRAITS_MACROS_H_ 906 #undef CC_IPC_CC_PARAM_TRAITS_MACROS_H_
907 #include "cc/ipc/cc_param_traits_macros.h" 907 #include "cc/ipc/cc_param_traits_macros.h"
908 } // namespace IPC 908 } // namespace IPC
OLDNEW
« no previous file with comments | « no previous file | cc/layers/layer_proto_converter.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698