Remove the IPC to request DemuxerConfigs when config changes

media/base/android/demuxer_stream_player_params.h

Index: media/base/android/demuxer_stream_player_params.h
diff --git a/media/base/android/demuxer_stream_player_params.h b/media/base/android/demuxer_stream_player_params.h
index e77d5d294a506302f15230845e9fdcc3fbcc6b13..0b8886eb3b1403eee70bc83928d398d3932b680f 100644
--- a/media/base/android/demuxer_stream_player_params.h
+++ b/media/base/android/demuxer_stream_player_params.h
@@ -54,6 +54,12 @@ struct MEDIA_EXPORT DemuxerData {
 
   DemuxerStream::Type type;
   std::vector<AccessUnit> access_units;
+  // If the last entry in |access_units| has a status equal to |kConfigChanged|,
+  // a corresponding DemuxerConfigs is added into this vector. The
+  // DemuxerConfigs should only contain information of the stream that is
+  // specified by |type|. This solves the issue that we need multiple IPCs when
+  // demuxer configs change.
+  std::vector<DemuxerConfigs> demuxer_configs;

[jschuh, 2014/05/08 16:52:53]

I'm a bit confused. Are you using the vector to represent an optional field or
do you anticipate sending mutliple configs at some point?

[qinmin, 2014/05/08 17:20:04]

This represents an optional field, there should be at most one DemuxerConfigs in
this IPC

On 2014/05/08 16:52:53, Justin Schuh wrote:

[qinmin, 2014/05/08 19:33:20]

Justin, do you prefer a vector or a separate ParamTraits with defined read/write
behavior for an optional field? Not sure what is the general rule for this.
Thanks

 };
 
 };  // namespace media