)]}'
{
  "commit": "fa8a9465d50d516745fc353ba2ebb80f34b622fb",
  "tree": "884f0386e0ae0f3fc8af12c09f8bc617f85c681f",
  "parents": [
    "17490b53d230f5286af92a2be5638ecad438711b"
  ],
  "author": {
    "name": "Henrik Boström",
    "email": "hbos@webrtc.org",
    "time": "Thu Apr 15 08:44:00 2021"
  },
  "committer": {
    "name": "Commit Bot",
    "email": "commit-bot@chromium.org",
    "time": "Thu Apr 15 10:18:40 2021"
  },
  "message": "Remove obsolete DCHECK in remote_audio_source.cc.\n\nWhen fixing so that RemoteAudioSource does not end the track just\nbecause the audio channel is gone in Unified Plan[1], this made it\npossible for ~PeerConnection to delete all objects, including deleting\nthe MediaStreamTrack and its RemoteAudioSource, when all tracks are not\nin an ended state.\n\nIn a real application or Chromium, the PeerConnection would not be\ndestroyed prior to closing and not hit this DCHECK. But in upstream\ndependent projects\u0027 unit tests, it would be possible for ref counted\ntracks to be destroyed when the track are still kLive, and as a\nside-effect hit this DCHECK.\n\nsinks_ is just a list of raw pointers, and whether or not we have done\nsinks_.clear() prior to destruction is irrelevant going forward.\n\n[1] https://webrtc-review.googlesource.com/c/src/+/214136\n\nBug: chromium:1121454\nChange-Id: If6cf3dffcd3cb47d46694755b5dc45fa381285fc\nReviewed-on: https://webrtc-review.googlesource.com/c/src/+/215226\nCommit-Queue: Henrik Boström \u003chbos@webrtc.org\u003e\nReviewed-by: Harald Alvestrand \u003chta@webrtc.org\u003e\nCr-Commit-Position: refs/heads/master@{#33739}\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "848fba372b47de1ba5800dbcf65b6d4bb3ccfff2",
      "old_mode": 33188,
      "old_path": "pc/remote_audio_source.cc",
      "new_id": "9e65f6781c82335e8607a6e85128f3575d89b736",
      "new_mode": 33188,
      "new_path": "pc/remote_audio_source.cc"
    }
  ]
}