Fix valgrind memory leak in session_manager_unittest....

remoting/host/session_manager_unittest.cc

 // 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.
 
 #include "base/message_loop.h"
 #include "base/task.h"
 #include "remoting/base/mock_objects.h"
 #include "remoting/host/mock_objects.h"
 #include "remoting/host/session_manager.h"
 #include "testing/gmock/include/gmock/gmock.h"
@@ skipping 15 matching lines @@
  public:
   SessionManagerTest() {
   }
 
  protected:
   void Init() {
     capturer_ = new MockCapturer();
     encoder_ = new MockEncoder();
     client_ = new MockClientConnection();
     record_ = new SessionManager(&message_loop_,
-                                &message_loop_,
-                                &message_loop_,
-                                capturer_,
-                                encoder_);
+                                 &message_loop_,
+                                 &message_loop_,
+                                 capturer_,
+                                 encoder_);
   }
 
   scoped_refptr<SessionManager> record_;
   scoped_refptr<MockClientConnection> client_;
   MockCapturer* capturer_;
   MockEncoder* encoder_;
   MessageLoop message_loop_;
  private:
   DISALLOW_COPY_AND_ASSIGN(SessionManagerTest);
 };
@@ skipping 11 matching lines @@
   dirty_rects.swap(temp_rects);
   arg0->Run(data);
   delete arg0;
 }
 
 ACTION_P(FinishEncode, msg) {
   Encoder::EncodingState state = (Encoder::EncodingStarting |
                                   Encoder::EncodingInProgress |
                                   Encoder::EncodingEnded);
   arg2->Run(msg, state);
-}
-
-ACTION_P(AssignCaptureData, data) {
-  arg0[0] = data[0];
-  arg0[1] = data[1];
-  arg0[2] = data[2];
-}
-
-ACTION_P(AssignDirtyRect, rects) {
-  *arg0 = *rects;
+  delete arg2;

[awong, 2010/08/26 20:39:06]

Stupid non-self-deleting callbacks.

 }
 
 TEST_F(SessionManagerTest, OneRecordCycle) {
   Init();
 
   InvalidRects update_rects;
   update_rects.insert(gfx::Rect(0, 0, 10, 10));
   DataPlanes planes;
   for (int i = 0; i < DataPlanes::kPlaneCount; ++i) {
     planes.data[i] = reinterpret_cast<uint8*>(i);
@@ skipping 32 matching lines @@
 
   // Make sure all tasks are completed.
   message_loop_.RunAllPending();
 }
 
 // TODO(hclam): Add test for double buffering.
 // TODO(hclam): Add test for multiple captures.
 // TODO(hclam): Add test for interruption.
 
 }  // namespace remoting