Removed the use of WebFrame::frameForCurrentContext() in MediaStreamCenter::didStopLocalMediaStream

content/renderer/media/media_stream_extra_data.h

 // 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.
 
 #ifndef CONTENT_RENDERER_MEDIA_MEDIA_STREAM_EXTRA_DATA_H_
 #define CONTENT_RENDERER_MEDIA_MEDIA_STREAM_EXTRA_DATA_H_
 
+#include "base/callback.h"
 #include "base/compiler_specific.h"
 #include "base/memory/ref_counted.h"
 #include "content/common/content_export.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/platform/WebMediaStreamDescriptor.h"
 
 namespace webrtc {
 class MediaStreamInterface;
 class LocalMediaStreamInterface;
 }  // namespace webrtc
 
 class CONTENT_EXPORT MediaStreamExtraData
     : NON_EXPORTED_BASE(public WebKit::WebMediaStreamDescriptor::ExtraData) {
  public:
+  typedef base::Callback<void(const WebKit::WebMediaStreamDescriptor&)>
+      StreamStopCallback;
+
   explicit MediaStreamExtraData(webrtc::MediaStreamInterface* remote_stream);
   explicit MediaStreamExtraData(
       webrtc::LocalMediaStreamInterface* local_stream);
   virtual ~MediaStreamExtraData();
 
+  void SetLocalStreamStopCallback(
+      const StreamStopCallback& stop_callback);
+  const StreamStopCallback& stream_stop_callback() {

[tommi (sloooow) - chröme, 2012/09/21 11:23:53]

Sorry, but this is more confusing.
Let's go back to the other approach you had but instead of naming the function
RunLocalStreamStopCallback, call it something that indicates that an event has
occurred like OnLocalStreamStopped().
The difference is that the getter approach and the 'Run' method, both expose an
implementation detail of this class.  An event on the other hand does not
promise anything about how the event will be handled so the implementation is
opaque as far as the caller is concerned.

[perkj_chrome, 2012/09/21 11:56:52]

Done.

+    return stream_stop_callback_;
+  }
+
   webrtc::MediaStreamInterface* remote_stream() { return remote_stream_.get(); }
   webrtc::LocalMediaStreamInterface* local_stream() {
     return local_stream_.get();
   }
 
  private:
+  StreamStopCallback stream_stop_callback_;
   scoped_refptr<webrtc::MediaStreamInterface> remote_stream_;
   scoped_refptr<webrtc::LocalMediaStreamInterface> local_stream_;
 
   DISALLOW_COPY_AND_ASSIGN(MediaStreamExtraData);
 };
 
 #endif  // CONTENT_RENDERER_MEDIA_MEDIA_STREAM_EXTRA_DATA_H_