Add generic interfaces for the sinks of the media stream audio track

content/renderer/media/media_stream_dependency_factory.h

Index: content/renderer/media/media_stream_dependency_factory.h
diff --git a/content/renderer/media/media_stream_dependency_factory.h b/content/renderer/media/media_stream_dependency_factory.h
index 1e7b790a13d5990c346176f6e8f6c99a67f00046..d0759708b8fe7466fb3bccd2a87f2f3d99679486 100644
--- a/content/renderer/media/media_stream_dependency_factory.h
+++ b/content/renderer/media/media_stream_dependency_factory.h
@@ -148,7 +148,8 @@ class CONTENT_EXPORT MediaStreamDependencyFactory
 
   static void AddNativeTrackToBlinkTrack(
       webrtc::MediaStreamTrackInterface* native_track,
-      const blink::WebMediaStreamTrack& webkit_track);
+      const blink::WebMediaStreamTrack& webkit_track,
+      bool is_local_track);

[Alpha Left Google, 2013/12/02 19:11:38]

I suggest there are two methods to be clear; having a boolean is not very
readable.

e.g.

AddLocalTrackToBlinkTrack(...native_track, ...webrtc_track)  and
AddRemoteTrackToBlinkTrack(...native_track, ...webrtc_track)

[no longer working on chromium, 2013/12/03 11:45:45]

The reason why we have the is_local_track flag is to distinguish the local audio
track with remote audio track for now, since those AddToAudioTrack()
RemoveFromAudioTrack() methods only support local track.

Adding a flag is Tommi's opinion in order to help dchecking only local track is
supported on those methods.

This solution is deemed to be temporary and the flag will be removed once the
remote audio track can use the same interface as the local audio track.

Also, on the video side, they don't need this at all.

I talked to Per, and he thinks using a flag is the simplest solution and prefers
not adding more temporary method.

Hope it is OK to keep it this way.

 
   static webrtc::MediaStreamInterface* GetNativeMediaStream(
       const blink::WebMediaStream& stream);