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

courgette/ensemble_unittest.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "courgette/base_test_unittest.h"
 #include "courgette/courgette.h"
 #include "courgette/streams.h"
 
 #if defined(OS_WIN) && !defined(NDEBUG)
 // Ensemble tests still take too long on Debug Windows so disabling for now
 // TODO(dgarrett) http://code.google.com/p/chromium/issues/detail?id=101614
 #define MAYBE_PE DISABLED_PE
 #define MAYBE_PE64 DISABLED_PE64
 #define MAYBE_Elf32 DISABLED_Elf32
 #else
 #define MAYBE_PE PE
 #define MAYBE_PE64 PE64
 #define MAYBE_Elf32 Elf32
 #endif
 
 class EnsembleTest : public BaseTest {
  public:
-
-  void TestEnsemble(std::string src_bytes, std::string tgt_bytes) const;
+  void TestEnsemble(const std::string& src_bytes,
+                    const std::string& tgt_bytes) const;
 
   void PeEnsemble() const;
   void Pe64Ensemble() const;
   void Elf32Ensemble() const;
 };
 
-void EnsembleTest::TestEnsemble(std::string src_bytes,
-                                std::string tgt_bytes) const {
-
+void EnsembleTest::TestEnsemble(const std::string& src_bytes,
+                                const std::string& tgt_bytes) const {
   courgette::SourceStream source;
   courgette::SourceStream target;
 
   source.Init(src_bytes);
   target.Init(tgt_bytes);
 
   courgette::SinkStream patch_sink;
 
   courgette::Status status;
 
@@ skipping 75 matching lines @@
   PeEnsemble();
 }
 
 TEST_F(EnsembleTest, MAYBE_PE64) {
   Pe64Ensemble();
 }
 
 TEST_F(EnsembleTest, MAYBE_Elf32) {
   Elf32Ensemble();
 }