Cleanup: Pass std::string as const reference if possible

sync/test/fake_server/fake_server_entity.h

 // Copyright 2014 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.
 
 #ifndef SYNC_TEST_FAKE_SERVER_FAKE_SERVER_ENTITY_H_
 #define SYNC_TEST_FAKE_SERVER_FAKE_SERVER_ENTITY_H_
 
 #include <map>
 #include <string>
 
@@ skipping 17 matching lines @@
 
   // Returns the ID string of the top level node for the specified type.
   static std::string GetTopLevelId(const syncer::ModelType& model_type);
 
   virtual ~FakeServerEntity();
   const std::string& GetId() const;
   syncer::ModelType GetModelType() const;
   int64 GetVersion() const;
   void SetVersion(int64 version);
   const std::string& GetName() const;
-  void SetName(std::string name);
+  void SetName(const std::string& name);
 
   // Replaces |specifics_| with |updated_specifics|. This method is meant to be
   // used to mimic a client commit.
   void SetSpecifics(const sync_pb::EntitySpecifics& updated_specifics);
 
   // Common data items needed by server
   virtual std::string GetParentId() const = 0;
   virtual void SerializeAsProto(sync_pb::SyncEntity* proto) const = 0;
   virtual bool IsDeleted() const;
   virtual bool IsFolder() const;
@@ skipping 24 matching lines @@
   // The name of the entity.
   std::string name_;
 
   // The EntitySpecifics for the entity.
   sync_pb::EntitySpecifics specifics_;
 };
 
 }  // namespace fake_server
 
 #endif  // SYNC_TEST_FAKE_SERVER_FAKE_SERVER_ENTITY_H_