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

Unified 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 side-by-side diff with in-line comments
Download patch
Index: content/common/cc_messages.cc
diff --git a/content/common/cc_messages.cc b/content/common/cc_messages.cc
index 7b14aeb19c35e872a4f4811347294f29080118a3..9d0df06b237d52fb489f721242004b36866069b8 100644
--- a/content/common/cc_messages.cc
+++ b/content/common/cc_messages.cc
@@ -680,7 +680,7 @@ void ParamTraits<cc::DelegatedFrameData>::Write(Message* m,
size_t to_reserve = sizeof(p.device_scale_factor);
to_reserve += p.resource_list.size() * sizeof(cc::TransferableResource);
for (size_t i = 0; i < p.render_pass_list.size(); ++i) {
- const cc::RenderPass* pass = p.render_pass_list[i];
+ 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.
to_reserve += sizeof(size_t) * 2;
to_reserve += ReserveSizeForRenderPassWrite(*pass);
}
@@ -689,7 +689,7 @@ void ParamTraits<cc::DelegatedFrameData>::Write(Message* m,
WriteParam(m, p.device_scale_factor);
WriteParam(m, p.resource_list);
WriteParam(m, p.render_pass_list.size());
- for (const auto* pass : p.render_pass_list) {
+ for (const auto& pass : p.render_pass_list) {
WriteParam(m, pass->quad_list.size());
WriteParam(m, pass->shared_quad_state_list.size());
WriteParam(m, *pass);
« 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