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

Side by Side Diff: content/common/cc_messages.cc

Issue 1437413002: cc: Remove ScopedPtrVector and cc::remove_if. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase + fix Created 5 years, 1 month 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 (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 "content/common/cc_messages.h" 5 #include "content/common/cc_messages.h"
6 6
7 #include "cc/output/compositor_frame.h" 7 #include "cc/output/compositor_frame.h"
8 #include "cc/output/filter_operations.h" 8 #include "cc/output/filter_operations.h"
9 #include "cc/quads/draw_quad.h" 9 #include "cc/quads/draw_quad.h"
10 #include "cc/quads/largest_draw_quad.h" 10 #include "cc/quads/largest_draw_quad.h"
(...skipping 662 matching lines...) Expand 10 before | Expand all | Expand 10 after
673 l->append(")"); 673 l->append(")");
674 } 674 }
675 675
676 void ParamTraits<cc::DelegatedFrameData>::Write(Message* m, 676 void ParamTraits<cc::DelegatedFrameData>::Write(Message* m,
677 const param_type& p) { 677 const param_type& p) {
678 DCHECK_NE(0u, p.render_pass_list.size()); 678 DCHECK_NE(0u, p.render_pass_list.size());
679 679
680 size_t to_reserve = sizeof(p.device_scale_factor); 680 size_t to_reserve = sizeof(p.device_scale_factor);
681 to_reserve += p.resource_list.size() * sizeof(cc::TransferableResource); 681 to_reserve += p.resource_list.size() * sizeof(cc::TransferableResource);
682 for (size_t i = 0; i < p.render_pass_list.size(); ++i) { 682 for (size_t i = 0; i < p.render_pass_list.size(); ++i) {
683 const cc::RenderPass* pass = p.render_pass_list[i]; 683 const cc::RenderPass* pass = p.render_pass_list[i].get();
dcheng 2015/11/18 00:47:10 Would this work instead? for (const auto& pass :
vmpstr 2015/11/18 00:53:44 Yep. Done.
684 to_reserve += sizeof(size_t) * 2; 684 to_reserve += sizeof(size_t) * 2;
685 to_reserve += ReserveSizeForRenderPassWrite(*pass); 685 to_reserve += ReserveSizeForRenderPassWrite(*pass);
686 } 686 }
687 m->Reserve(to_reserve); 687 m->Reserve(to_reserve);
688 688
689 WriteParam(m, p.device_scale_factor); 689 WriteParam(m, p.device_scale_factor);
690 WriteParam(m, p.resource_list); 690 WriteParam(m, p.resource_list);
691 WriteParam(m, p.render_pass_list.size()); 691 WriteParam(m, p.render_pass_list.size());
692 for (const auto* pass : p.render_pass_list) { 692 for (const auto& pass : p.render_pass_list) {
693 WriteParam(m, pass->quad_list.size()); 693 WriteParam(m, pass->quad_list.size());
694 WriteParam(m, pass->shared_quad_state_list.size()); 694 WriteParam(m, pass->shared_quad_state_list.size());
695 WriteParam(m, *pass); 695 WriteParam(m, *pass);
696 } 696 }
697 } 697 }
698 698
699 bool ParamTraits<cc::DelegatedFrameData>::Read(const Message* m, 699 bool ParamTraits<cc::DelegatedFrameData>::Read(const Message* m,
700 base::PickleIterator* iter, 700 base::PickleIterator* iter,
701 param_type* p) { 701 param_type* p) {
702 if (!ReadParam(m, iter, &p->device_scale_factor)) 702 if (!ReadParam(m, iter, &p->device_scale_factor))
(...skipping 148 matching lines...) Expand 10 before | Expand all | Expand 10 after
851 l->append("TextureDrawQuad::OverlayResources(["); 851 l->append("TextureDrawQuad::OverlayResources([");
852 for (size_t i = 0; i < cc::DrawQuad::Resources::kMaxResourceIdCount; ++i) { 852 for (size_t i = 0; i < cc::DrawQuad::Resources::kMaxResourceIdCount; ++i) {
853 LogParam(p.size_in_pixels[i], l); 853 LogParam(p.size_in_pixels[i], l);
854 if (i < (cc::DrawQuad::Resources::kMaxResourceIdCount - 1)) 854 if (i < (cc::DrawQuad::Resources::kMaxResourceIdCount - 1))
855 l->append(", "); 855 l->append(", ");
856 } 856 }
857 l->append("])"); 857 l->append("])");
858 } 858 }
859 859
860 } // namespace IPC 860 } // namespace IPC
OLDNEW
« no previous file with comments | « content/browser/renderer_host/render_widget_host_view_mac.mm ('k') | content/common/cc_messages_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698