)]}'
{
  "commit": "6e4fcac31312f2dda5b60d33874ff0cd62f94321",
  "tree": "43b836deb2a03b9a8be123935681db5aba875aad",
  "parents": [
    "82ce7e5515c86c68392dee56f2883812b869185b"
  ],
  "author": {
    "name": "Guido Urdaneta",
    "email": "guidou@webrtc.org",
    "time": "Tue Feb 09 12:26:13 2021"
  },
  "committer": {
    "name": "Commit Bot",
    "email": "commit-bot@chromium.org",
    "time": "Tue Feb 09 12:26:26 2021"
  },
  "message": "Revert \"Remove thread hops from events provided by JsepTransportController.\"\n\nThis reverts commit f554b3c577f69fa9ffad5c07155898c2d985ac76.\n\nReason for revert: Parent CL breaks FYI bots.\nSee https://webrtc-review.googlesource.com/c/src/+/206466\n\nOriginal change\u0027s description:\n\u003e Remove thread hops from events provided by JsepTransportController.\n\u003e\n\u003e Events associated with Subscribe* methods in JTC had trampolines that\n\u003e would use an async invoker to fire the events on the signaling thread.\n\u003e This was being done for the purposes of PeerConnection but the concept\n\u003e of a signaling thread is otherwise not applicable to JTC and use of\n\u003e JTC from PC is inconsistent across threads (as has been flagged in\n\u003e webrtc:9987).\n\u003e\n\u003e This change makes all CallbackList members only accessible from the\n\u003e network thread and moves the signaling thread related work over to\n\u003e PeerConnection, which makes hops there more visible as well as making\n\u003e that class easier to refactor for thread efficiency.\n\u003e\n\u003e This CL removes the AsyncInvoker from JTC (webrtc:12339)\n\u003e\n\u003e The signaling_thread_ variable is also removed from JTC and more thread\n\u003e checks added to catch errors.\n\u003e\n\u003e Bug: webrtc:12427, webrtc:11988, webrtc:12339\n\u003e Change-Id: Id232aedd00dfd5403b2ba0ca147d3eca7c12c7c5\n\u003e Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/206062\n\u003e Commit-Queue: Tommi \u003ctommi@webrtc.org\u003e\n\u003e Reviewed-by: Niels Moller \u003cnisse@webrtc.org\u003e\n\u003e Cr-Commit-Position: refs/heads/master@{#33195}\n\nTBR\u003dnisse@webrtc.org,tommi@webrtc.org\n\nChange-Id: I6134b71b74a9408854b79d44506d513519e9cf4d\nNo-Presubmit: true\nNo-Tree-Checks: true\nNo-Try: true\nBug: webrtc:12427\nBug: webrtc:11988\nBug: webrtc:12339\nReviewed-on: https://webrtc-review.googlesource.com/c/src/+/206467\nReviewed-by: Guido Urdaneta \u003cguidou@webrtc.org\u003e\nCommit-Queue: Guido Urdaneta \u003cguidou@webrtc.org\u003e\nCr-Commit-Position: refs/heads/master@{#33203}\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "71e2f1eeae7eecf046b7ff12edd8740619619e2a",
      "old_mode": 33188,
      "old_path": "pc/connection_context.h",
      "new_id": "02d08a191ec7b4925b9c9a3b335f8ec48948adcd",
      "new_mode": 33188,
      "new_path": "pc/connection_context.h"
    },
    {
      "type": "modify",
      "old_id": "28ba899cb3b015729b33d4a5511af52292785a9c",
      "old_mode": 33188,
      "old_path": "pc/jsep_transport_controller.cc",
      "new_id": "0ded1de84f0213cdb4c56d627e5f2c6b65ff408e",
      "new_mode": 33188,
      "new_path": "pc/jsep_transport_controller.cc"
    },
    {
      "type": "modify",
      "old_id": "949c9ad1dc1898ca77b64adc801f2db77643fba3",
      "old_mode": 33188,
      "old_path": "pc/jsep_transport_controller.h",
      "new_id": "59d66a24f263fe5bdbdd31d64f65091382015ae2",
      "new_mode": 33188,
      "new_path": "pc/jsep_transport_controller.h"
    },
    {
      "type": "modify",
      "old_id": "0424afe876d44834c4e166196953c95ef39521fc",
      "old_mode": 33188,
      "old_path": "pc/jsep_transport_controller_unittest.cc",
      "new_id": "9efa205368ff4163837016a1e678ed04a0f38d13",
      "new_mode": 33188,
      "new_path": "pc/jsep_transport_controller_unittest.cc"
    },
    {
      "type": "modify",
      "old_id": "b4e42467668354568904d67fb34b99c456fa9d94",
      "old_mode": 33188,
      "old_path": "pc/peer_connection.cc",
      "new_id": "f82fe35c6d42be449fb59633a7b69e58abc35bc4",
      "new_mode": 33188,
      "new_path": "pc/peer_connection.cc"
    },
    {
      "type": "modify",
      "old_id": "75af0ae1708677c61c2b975089b2a74566e082d8",
      "old_mode": 33188,
      "old_path": "pc/peer_connection.h",
      "new_id": "92e33d28588f522953cc83c79e44b5bf3482f993",
      "new_mode": 33188,
      "new_path": "pc/peer_connection.h"
    },
    {
      "type": "modify",
      "old_id": "8588ca8dbfe771062b9eef538a2eb895a86e445b",
      "old_mode": 33188,
      "old_path": "pc/sdp_offer_answer.cc",
      "new_id": "9fa4188e10af43ba94c0686830efefd6e76e2620",
      "new_mode": 33188,
      "new_path": "pc/sdp_offer_answer.cc"
    },
    {
      "type": "modify",
      "old_id": "fefaa00c723d5f4d83f6403426d88ccc8420c261",
      "old_mode": 33188,
      "old_path": "test/peer_scenario/scenario_connection.cc",
      "new_id": "8e5b3162cb0f1d96f6186a441c52aae2a7929872",
      "new_mode": 33188,
      "new_path": "test/peer_scenario/scenario_connection.cc"
    }
  ]
}