StreamDeviceInfo has only two fields: |device| and |session_id|.
We can move |session_id| to MediaStreamDevice and remove StreamDeviceInfo altogether.
guidou@chromium.org, can we do this?
Sorry, I didn't really understand what you meant by "your plan" in your above comment. If you could elaborate a bit more on that?
As I see, StreamDeviceInfo is referenced at quite a few places. Also, I'll be doing this along with my regular work. So, if you are expecting the timeline, then this task might take some time to complete.
I meant that if/when you choose to do it, it would be good to discuss the design/approach you are going to take before actually submitting CLs since that class is referenced in many places, as you have already noticed.
No need for timelines and no hurry.
Sure. Thanks!
At the initial look, I think the approach in this case would be to simply replace StreamDeviceInfo with MediaStreamDevice wherever applicable. Since, there's not much difference between the two structs, the replacement should be pretty straight forward.
Comment 1 by guidou@chromium.org
, Aug 30 2017