Retry 114494 - Remove BindStateHolder and have Bind() return a Callback<> object directly."

chrome/browser/chromeos/process_proxy/process_output_watcher.h

 // 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.
 
 #ifndef CHROME_BROWSER_CHROMEOS_PROCESS_PROXY_PROCESS_OUTPUT_WATCHER_H_
 #define CHROME_BROWSER_CHROMEOS_PROCESS_PROXY_PROCESS_OUTPUT_WATCHER_H_
 #pragma once
 
 #include <string>
 
 #include "base/callback.h"
 
 namespace {
 
 const int kReadBufferSize = 256;
 
 }  // namespace
 
 enum ProcessOutputType {
   PROCESS_OUTPUT_TYPE_OUT,
   PROCESS_OUTPUT_TYPE_ERR
 };
 
-typedef base::Callback<void(ProcessOutputType, std::string)>
+typedef base::Callback<void(ProcessOutputType, const std::string&)>
       ProcessOutputCallback;
 
 // This class should live on its own thread because running class makes
 // underlying thread block. It deletes itself when watching is stopped.
 class ProcessOutputWatcher {
  public:
   ProcessOutputWatcher(int out_fd, int err_fd, int stop_fd,
                        const ProcessOutputCallback& callback);
 
   // This will block current thread!!!!
@@ skipping 24 matching lines @@
   int err_fd_;
   int stop_fd_;
   int max_fd_;
 
   // Callback that will be invoked when some output is detected.
   ProcessOutputCallback on_read_callback_;
 
   DISALLOW_COPY_AND_ASSIGN(ProcessOutputWatcher);
 };
 #endif  // CHROME_BROWSER_CHROMEOS_PROCESS_PROXY_PROCESS_OUTPUT_WATCHER_H_