Refactor VideoStub interface to accept ownership of video packets.

remoting/host/screen_recorder_unittest.cc

 // Copyright (c) 2012 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 "remoting/host/screen_recorder.h"
 
 #include "base/bind.h"
 #include "base/message_loop.h"
 #include "remoting/base/base_mock_objects.h"
 #include "remoting/host/host_mock_objects.h"
@@ skipping 125 matching lines @@
 
   // Expect the encoder be called.
   EXPECT_CALL(*encoder_, Encode(data, false, _))
       .WillRepeatedly(FinishEncode());
 
   MockVideoStub video_stub;
   EXPECT_CALL(*connection_, video_stub())
       .WillRepeatedly(Return(&video_stub));
 
   // By default delete the arguments when ProcessVideoPacket is received.
-  EXPECT_CALL(video_stub, ProcessVideoPacket(_, _))
+  EXPECT_CALL(video_stub, ProcessVideoPacketPtr(_, _))
       .WillRepeatedly(FinishSend());
 
   // For the first time when ProcessVideoPacket is received we stop the
   // ScreenRecorder.
-  EXPECT_CALL(video_stub, ProcessVideoPacket(_, _))
+  EXPECT_CALL(video_stub, ProcessVideoPacketPtr(_, _))
       .WillOnce(DoAll(
           FinishSend(),
           StopScreenRecorder(record_,
                              base::Bind(&QuitMessageLoop, &message_loop_))))
       .RetiresOnSaturation();
 
   // Add the mock client connection to the session.
   record_->AddConnection(connection_.get());
 
   // Start the recording.
   record_->Start();
   message_loop_.Run();
 }
 
 TEST_F(ScreenRecorderTest, StopWithoutStart) {
   record_->Stop(base::Bind(&QuitMessageLoop, &message_loop_));
   message_loop_.Run();
 }
 
 }  // namespace remoting