)]}'
{
  "commit": "31957526b2008c7bcefc12f8d7c394251acfc5e4",
  "tree": "6153fc4f0550780be216d4cce6b23cb279ca43ea",
  "parents": [
    "3225248425fe2b3cafe389465b94ebc24968d9ba"
  ],
  "author": {
    "name": "Erik Språng",
    "email": "sprang@webrtc.org",
    "time": "Mon Sep 28 14:54:05 2026"
  },
  "committer": {
    "name": "webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com",
    "email": "webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com",
    "time": "Mon Sep 28 14:55:16 2026"
  },
  "message": "Revert \"Report parsed key id on SFrame unknown-key decrypt failures\"\n\nThis reverts commit 43f5a1c963381978d29a2f2b85633bb9ad7303f5.\n\nReason for revert: Breaks downstream build\n\n\nOriginal change\u0027s description:\n\u003e Report parsed key id on SFrame unknown-key decrypt failures\n\u003e\n\u003e `ToSframeDecryptFailure` previously mapped `invalid_parameter_error` to\n\u003e a `kKeyId` failure with an empty key id, because the Cisco Sframe\n\u003e library did not surface the key id parsed from the SFrame header.\n\u003e\n\u003e The library now returns a dedicated\n\u003e `SFrameErrorType::unknown_key_id_error` carrying the parsed key id, so\n\u003e the mapping is switched over to it and the key id is propagated into\n\u003e `SframeDecryptFailure::KeyId`. Applications listening for\n\u003e `SframeDecryptErrorCallback` can now tell *which* key is missing,\n\u003e matching\n\u003e [SFrameTransformErrorEvent.keyID](https://w3c.github.io/webrtc-encoded-transform/#dom-sframetransformerrorevent-keyid).\n\u003e\n\u003e `invalid_parameter_error` now falls through to the `kSyntax` default,\n\u003e which is the correct classification for a malformed header.\n\u003e\n\u003e Additional changes:\n\u003e - Re-enabled `SframeEncryptorDecryptorTest.DISABLED_DecryptReportsKeyIdErrorForUnknownKey`\n\u003e   and tightened it to assert the reported key id equals the encryption key id.\n\u003e\n\u003e Bug: webrtc:479862368\n\u003e Change-Id: I5af6c6b42f1fe46acbdeba6168d4a20f1f8a0869\n\u003e Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/506020\n\u003e Commit-Queue: Kacper Wasniowski \u003ckwasniow@cisco.com\u003e\n\u003e Reviewed-by: Danil Chapovalov \u003cdanilchap@webrtc.org\u003e\n\u003e Reviewed-by: Tomas Gunnarsson \u003ctommi@webrtc.org\u003e\n\u003e Cr-Commit-Position: refs/heads/main@{#48732}\n\nBug: webrtc:479862368\nNo-Presubmit: true\nNo-Tree-Checks: true\nNo-Try: true\nChange-Id: I8d7a55bf4b80a8027853178664e88732ca3f2915\nReviewed-on: https://webrtc-review.googlesource.com/c/src/+/506120\nBot-Commit: rubber-stamper@appspot.gserviceaccount.com \u003crubber-stamper@appspot.gserviceaccount.com\u003e\nCommit-Queue: rubber-stamper@appspot.gserviceaccount.com \u003crubber-stamper@appspot.gserviceaccount.com\u003e\nReviewed-by: Danil Chapovalov \u003cdanilchap@webrtc.org\u003e\nAuto-Submit: Erik Språng \u003csprang@webrtc.org\u003e\nCr-Commit-Position: refs/heads/main@{#48738}\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "55945e130674458ec03196ed0e5e9852d27406be",
      "old_mode": 33188,
      "old_path": "modules/sframe/sframe_decryptor.cc",
      "new_id": "a47772c32f5dd0a36613cb6c0805cd942ab82ea0",
      "new_mode": 33188,
      "new_path": "modules/sframe/sframe_decryptor.cc"
    },
    {
      "type": "modify",
      "old_id": "c510ebc1484083288b18090f047f0e557fb8ba8e",
      "old_mode": 33188,
      "old_path": "modules/sframe/sframe_encryptor_decryptor_unittest.cc",
      "new_id": "a6c1dcc080c151c800d9c55c773e52d6160d147d",
      "new_mode": 33188,
      "new_path": "modules/sframe/sframe_encryptor_decryptor_unittest.cc"
    }
  ]
}