)]}'
{
  "commit": "118d01ac35adcf4d1efb5d225f9d85d857115c0b",
  "tree": "3e698f41f9c6abfaf422f714528bef1dea6cb210",
  "parents": [
    "980cadd02c7384397a41c0e334e9f329f3cc5c65"
  ],
  "author": {
    "name": "Erik Språng",
    "email": "sprang@webrtc.org",
    "time": "Thu Jun 25 09:58:41 2020"
  },
  "committer": {
    "name": "Commit Bot",
    "email": "commit-bot@chromium.org",
    "time": "Thu Jun 25 12:41:48 2020"
  },
  "message": "Revert \"Reland \"Removes lock release in PacedSender callback.\"\"\n\nThis reverts commit b46df3da44c42f6e5055c69a8247a344887108ea.\n\nReason for revert: May cause deadlock.\n\nOriginal change\u0027s description:\n\u003e Reland \"Removes lock release in PacedSender callback.\"\n\u003e \n\u003e This is a reland of 6b9c60b06d04bc519195fca1f621b10accfeb46b\n\u003e \n\u003e Original change\u0027s description:\n\u003e \u003e Removes lock release in PacedSender callback.\n\u003e \u003e \n\u003e \u003e The PacedSender currently has logic to temporarily release its internal\n\u003e \u003e lock while sending or asking for padding.\n\u003e \u003e This creates some tricky situations in the pacing controller where we\n\u003e \u003e need to consider if some thread can enter while we the process thread is\n\u003e \u003e actually processing, just temporarily busy sending.\n\u003e \u003e \n\u003e \u003e Since the pacing call stack is no longer cyclic, we can actually remove\n\u003e \u003e this lock-release now.\n\u003e \u003e \n\u003e \u003e Bug: webrtc:10809\n\u003e \u003e Change-Id: Ic59c605252bed1f96a03406c908a30cd1012f995\n\u003e \u003e Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/173592\n\u003e \u003e Reviewed-by: Sebastian Jansson \u003csrte@webrtc.org\u003e\n\u003e \u003e Commit-Queue: Erik Språng \u003csprang@webrtc.org\u003e\n\u003e \u003e Cr-Commit-Position: refs/heads/master@{#31206}\n\u003e \n\u003e Bug: webrtc:10809\n\u003e Change-Id: Id39fc49b0a038e7ae3a0d9818fb0806c33ae0ae0\n\u003e Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/175656\n\u003e Reviewed-by: Sebastian Jansson \u003csrte@webrtc.org\u003e\n\u003e Commit-Queue: Erik Språng \u003csprang@webrtc.org\u003e\n\u003e Cr-Commit-Position: refs/heads/master@{#31332}\n\nTBR\u003dsprang@webrtc.org,srte@webrtc.org\n\n# Not skipping CQ checks because original CL landed \u003e 1 day ago.\n\nBug: webrtc:10809\nChange-Id: I6b06bafad8cd9eeb22107d04b953fd14b8131afa\nReviewed-on: https://webrtc-review.googlesource.com/c/src/+/178100\nReviewed-by: Erik Språng \u003csprang@webrtc.org\u003e\nCommit-Queue: Erik Språng \u003csprang@webrtc.org\u003e\nCr-Commit-Position: refs/heads/master@{#31564}\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "8d9c0b39fe6db2e5d9fcb4f8f17fbe44fe6b940a",
      "old_mode": 33188,
      "old_path": "modules/pacing/paced_sender.cc",
      "new_id": "e38863031b3db163d0a04059c6f88f1e424e00db",
      "new_mode": 33188,
      "new_path": "modules/pacing/paced_sender.cc"
    },
    {
      "type": "modify",
      "old_id": "f4ae047870d5e74d3a56b1a310f3a883842e0e77",
      "old_mode": 33188,
      "old_path": "test/direct_transport.cc",
      "new_id": "818480b18b5ec1d10805b870bd9d76ae7382cdf6",
      "new_mode": 33188,
      "new_path": "test/direct_transport.cc"
    }
  ]
}