Cleanup: Pass std::string as const reference from testing/

Passing std::string by reference can prevent extra copying of object. BUG=367418 TEST= R=phajdan.jr@chromium.org Review-Url: https://codereview.chromium.org/2001093008 Cr-Commit-Position: refs/heads/master@{#396030}
marcosholgado · May 25, 2016 · a690281 · a690281
1 parent d1a23ec
commit a690281
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/testing/multiprocess_func_list.cc b/testing/multiprocess_func_list.cc
@@ -40,7 +40,7 @@ AppendMultiProcessTest::AppendMultiProcessTest(
       ProcessFunctions(main_func_ptr, setup_func_ptr);
 }
 
-int InvokeChildProcessTest(std::string test_name) {
+int InvokeChildProcessTest(const std::string& test_name) {
   MultiProcessTestMap& func_lookup_table = GetMultiprocessFuncMap();
   MultiProcessTestMap::iterator it = func_lookup_table.find(test_name);
   if (it != func_lookup_table.end()) {

diff --git a/testing/multiprocess_func_list.h b/testing/multiprocess_func_list.h
@@ -47,7 +47,7 @@ class AppendMultiProcessTest {
 
 // Invoke the main function of a test previously registered with
 // MULTIPROCESS_TEST_MAIN()
-int InvokeChildProcessTest(std::string test_name);
+int InvokeChildProcessTest(const std::string& test_name);
 
 // This macro creates a global MultiProcessTest::AppendMultiProcessTest object
 // whose constructor does the work of adding the global mapping.