Remove stl_util's deletion functions from media/.

media/filters/media_source_state.h

Index: media/filters/media_source_state.h
diff --git a/media/filters/media_source_state.h b/media/filters/media_source_state.h
index 588cc90d94a4b1b5eeaf673335d6de86a335e887..0e1012f18835cdd039342f24cd0f02ab112fd1de 100644
--- a/media/filters/media_source_state.h
+++ b/media/filters/media_source_state.h
@@ -194,9 +194,7 @@ class MEDIA_EXPORT MediaSourceState {
   using DemuxerStreamMap = std::map<StreamParser::TrackId, ChunkDemuxerStream*>;
   DemuxerStreamMap audio_streams_;
   DemuxerStreamMap video_streams_;
-
-  typedef std::map<StreamParser::TrackId, ChunkDemuxerStream*> TextStreamMap;
-  TextStreamMap text_stream_map_;  // |this| owns the map's stream pointers.
+  DemuxerStreamMap text_streams_;

[Avi (use Gerrit), 2016/09/26 21:07:26]

The old ownership here, where the audio and video streams weren't owned but the
text streams were, was hard to reason about. What made it worse was that the
CreateDemuxerStreamCB sometimes returned owned objects and sometimes didn't.

So I made all streams the same.

When reviewing, please carefully review and verify the main ownership change
here, which is removing ownership of the text streams from the MediaSourceState
and having the ChunkDemuxer own them.

[wolenetz, 2016/09/27 22:01:11]

Acknowledged.

 
   std::unique_ptr<FrameProcessor> frame_processor_;
   scoped_refptr<MediaLog> media_log_;