Remove obsolete DCHECK in remote_audio_source.cc.

When fixing so that RemoteAudioSource does not end the track just
because the audio channel is gone in Unified Plan[1], this made it
possible for ~PeerConnection to delete all objects, including deleting
the MediaStreamTrack and its RemoteAudioSource, when all tracks are not
in an ended state.

In a real application or Chromium, the PeerConnection would not be
destroyed prior to closing and not hit this DCHECK. But in upstream
dependent projects' unit tests, it would be possible for ref counted
tracks to be destroyed when the track are still kLive, and as a
side-effect hit this DCHECK.

sinks_ is just a list of raw pointers, and whether or not we have done
sinks_.clear() prior to destruction is irrelevant going forward.

[1] https://webrtc-review.googlesource.com/c/src/+/214136

Bug: chromium:1121454
Change-Id: If6cf3dffcd3cb47d46694755b5dc45fa381285fc
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/215226
Commit-Queue: Henrik Boström <hbos@webrtc.org>
Reviewed-by: Harald Alvestrand <hta@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#33739}
1 file changed
tree: 884f0386e0ae0f3fc8af12c09f8bc617f85c681f
  1. .clang-format
  2. .git-blame-ignore-revs
  3. .gitignore
  4. .gn
  5. .vpython
  6. AUTHORS
  7. BUILD.gn
  8. CODE_OF_CONDUCT.md
  9. DEPS
  10. DIR_METADATA
  11. ENG_REVIEW_OWNERS
  12. LICENSE
  13. OWNERS
  14. PATENTS
  15. PRESUBMIT.py
  16. README.chromium
  17. README.md
  18. WATCHLISTS
  19. abseil-in-webrtc.md
  20. api/
  21. audio/
  22. build_overrides/
  23. call/
  24. codereview.settings
  25. common_audio/
  26. common_video/
  27. data/
  28. docs/
  29. examples/
  30. g3doc.lua
  31. g3doc/
  32. license_template.txt
  33. logging/
  34. media/
  35. modules/
  36. native-api.md
  37. net/
  38. p2p/
  39. pc/
  40. presubmit_test.py
  41. presubmit_test_mocks.py
  42. pylintrc
  43. resources/
  44. rtc_base/
  45. rtc_tools/
  46. sdk/
  47. stats/
  48. style-guide.md
  49. style-guide/
  50. system_wrappers/
  51. test/
  52. tools_webrtc/
  53. video/
  54. webrtc.gni
  55. webrtc_lib_link_test.cc
  56. whitespace.txt
README.md

WebRTC is a free, open software project that provides browsers and mobile applications with Real-Time Communications (RTC) capabilities via simple APIs. The WebRTC components have been optimized to best serve this purpose.

Our mission: To enable rich, high-quality RTC applications to be developed for the browser, mobile platforms, and IoT devices, and allow them all to communicate via a common set of protocols.

The WebRTC initiative is a project supported by Google, Mozilla and Opera, amongst others.

Development

See here for instructions on how to get started developing with the native code.

Authoritative list of directories that contain the native API header files.

More info