Refactored MediaStreamImpl to be a RenderViewObserver as the other delegates in RenderViewImpl.

content/renderer/media/peer_connection_handler_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 <string>
 
 #include "base/memory/scoped_ptr.h"
 #include "base/message_loop.h"
 #include "base/utf_string_conversions.h"
 #include "content/renderer/media/mock_media_stream_dependency_factory.h"
@@ skipping 40 matching lines @@
   {
     WebKit::WebMediaStreamDescriptor copy_2(copy_1);
   }
 }
 
 TEST(PeerConnectionHandlerTest, Basic) {
   MessageLoop loop;
 
   scoped_ptr<WebKit::MockWebPeerConnectionHandlerClient> mock_client(
       new WebKit::MockWebPeerConnectionHandlerClient());
-  scoped_refptr<MockMediaStreamImpl> mock_ms_impl(new MockMediaStreamImpl());
+  scoped_ptr<MockMediaStreamImpl> mock_ms_impl(new MockMediaStreamImpl());
   scoped_ptr<MockMediaStreamDependencyFactory> mock_dependency_factory(
       new MockMediaStreamDependencyFactory());
   mock_dependency_factory->CreatePeerConnectionFactory(NULL,
                                                        NULL,
                                                        NULL,
                                                        NULL,
                                                        NULL);
   scoped_ptr<PeerConnectionHandler> pc_handler(
       new PeerConnectionHandler(mock_client.get(),
                                 mock_ms_impl.get(),
@@ skipping 75 matching lines @@
   EXPECT_EQ(stream_label, mock_peer_connection->stream_label());
   EXPECT_TRUE(mock_peer_connection->stream_changes_committed());
 
   pc_handler->stop();
   EXPECT_FALSE(pc_handler->native_peer_connection_.get());
   // PC handler is expected to be deleted when stop calls
   // MediaStreamImpl::ClosePeerConnection. We own and delete it here instead of
   // in the mock.
   pc_handler.reset();
 }