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

Unified Diff: content/browser/file_descriptor_info_impl_unittest.cc

Issue 2594203004: Unifying ChildProcessLauncher across platforms. (Closed)
Patch Set: Clean-up. Created 3 years, 11 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 side-by-side diff with in-line comments
Download patch
Index: content/browser/file_descriptor_info_impl_unittest.cc
diff --git a/content/browser/file_descriptor_info_impl_unittest.cc b/content/browser/file_descriptor_info_impl_unittest.cc
index e14dc3120d4a57cdc5ff5388efac02f3c7f9181c..1d442f796820f283f47f83ac9c134c68f0e14bbe 100644
--- a/content/browser/file_descriptor_info_impl_unittest.cc
+++ b/content/browser/file_descriptor_info_impl_unittest.cc
@@ -77,10 +77,10 @@ TEST_F(FileDescriptorInfoTest, GetMappingWithIDAdjustment) {
target->Transfer(testingId1, base::ScopedFD(GetSafeFd()));
target->Transfer(testingId2, base::ScopedFD(GetSafeFd()));
- base::FileHandleMappingVector mapping =
+ std::unique_ptr<base::FileHandleMappingVector> mapping =
target->GetMappingWithIDAdjustment(100);
- ASSERT_EQ(mapping[0].second, 142);
- ASSERT_EQ(mapping[1].second, 143);
+ ASSERT_EQ((*mapping)[0].second, 142);
+ ASSERT_EQ((*mapping)[1].second, 143);
}
} // namespace content
« no previous file with comments | « content/browser/file_descriptor_info_impl.cc ('k') | content/browser/renderer_host/render_process_host_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698