tree 4ead89796350c05758d9bfc4d98752c2b41891df
parent a28780e994692f2494d8d54fb4539003a212dd6b
author aleloi <aleloi@webrtc.org> 1479463384 -0800
committer Commit bot <commit-bot@chromium.org> 1479463388 +0000

Changed the interface AudioMixer::RemoveSource to have a void return type.

In the AudioMixerImpl implementation, removing a source never fails
and the return value is always true (see audio_mixer/audio_mixer_impl.cc).

A return value of |false| signaled that removing a source failed for
some reason. We have come to the conclusion that
   * we don't know how to handle a return value of |false|
   * we can't think of why an alternative implementation would need to
     signal failure when removing a stream.

To avoid having a status code that is never read, never acted upon and
probably never set to anything but |true|, we change ::RemoveSource to
not have a return value.

NOTRY=True
BUG=webrtc:6346

Review-Url: https://codereview.webrtc.org/2506173003
Cr-Commit-Position: refs/heads/master@{#15150}