Add a done task to ScreenRecorder::Stop()

remoting/protocol/buffered_socket_writer.h

 // Copyright (c) 2010 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 REMOTING_PROTOCOL_BUFFERED_SOCKET_WRITER_H_
 #define REMOTING_PROTOCOL_BUFFERED_SOCKET_WRITER_H_
 
 #include <list>
 
 #include "base/lock.h"
@@ skipping 72 matching lines @@
   void OnWritten(int result);
 
   // This method is called when an error is encountered.
   void HandleError(int result);
 
   // Must be locked when accessing |socket_|, |queue_| and |buffer_size_|;
   Lock lock_;
 
   net::Socket* socket_;
   MessageLoop* message_loop_;
-  scoped_ptr<WriteFailedCallback> write_failed_callback_;

[Sergey Ulanov, 2011/01/20 19:14:17]

Why do you need to remove this?

[Alpha Left Google, 2011/01/20 19:24:18]

Going through I found this is not used at all so I cleaned it up at the same
time.

 
   bool write_pending_;
 
   net::CompletionCallbackImpl<BufferedSocketWriterBase> written_callback_;
 
   bool closed_;
 };
 
 class BufferedSocketWriter : public BufferedSocketWriterBase {
  public:
@@ skipping 17 matching lines @@
  protected:
   virtual void GetNextPacket_Locked(net::IOBuffer** buffer, int* size);
   virtual void AdvanceBufferPosition_Locked(int written);
   virtual void OnError_Locked(int result);
 };
 
 }  // namespace protocol
 }  // namespace remoting
 
 #endif  // REMOTING_PROTOCOL_BUFFERED_SOCKET_WRITER_H_