)]}'
{
  "commit": "94de9a4e0693cd573be006e384dd7a41d1ac6c24",
  "tree": "0522032a3cc1c541b5ea24d8e9901dbbabebf449",
  "parents": [
    "cf52339fa73bbb7aece4e82cf59d9f5453d94185"
  ],
  "author": {
    "name": "Andres Calderon Jaramillo",
    "email": "andrescj@chromium.org",
    "time": "Tue Apr 25 18:06:08 2023"
  },
  "committer": {
    "name": "Chromium LUCI CQ",
    "email": "chromium-scoped@luci-project-accounts.iam.gserviceaccount.com",
    "time": "Tue Apr 25 18:06:08 2023"
  },
  "message": "media: Remove stable.mojom.VideoFrame traits.\n\nThis CL removes the mojo traits which map stable.mojom.VideoFrame to\nscoped_refptr\u003c::media::VideoFrame\u003e. This is in preparation to CL:4457504\nwhich allows the OOPVideoDecoder to re-use frames previously received\nfrom the remote decoder. Removing the traits is useful because with that\nCL, not every incoming stable.mojom.VideoFrame received by the\nOOPVideoDecoder will result in a media::VideoFrame, and creating a\nmedia::VideoFrame from said stable.mojom.VideoFrame incurs in some\noverhead such as consuming a strictly incrementing counter [1] and\nvalidating the underlying GpuMemoryBufferHandle [2].\n\nThis CL moves the validation of the incoming stable.mojom.VideoFrame\nfrom the mojo traits to the OOPVideoDecoder which is the only component\nthat receives them.\n\nFunctionally, this CL doesn\u0027t change much: we still create a\nmedia::VideoFrame for every incoming stable.mojom.VideoFrame. The actual\nre-use of frames is implemented in CL:4457504.\n\n[1] https://source.chromium.org/chromium/chromium/src/+/main:media/base/video_frame.cc;l\u003d1422;drc\u003d269a0caf81464f39a3d54942a891652ba1f2ba40\n[2] https://source.chromium.org/chromium/chromium/src/+/main:media/mojo/mojom/stable/stable_video_decoder_types_mojom_traits.cc;l\u003d1679-1682;drc\u003dbd1cbddfe3cbecc083674c0c9de89b2e67b2cc04\n\nBug: b:277832201, b:195769334\nTest: video.*oopvd* on volteer\nChange-Id: I50e71d84188f99ce630a75a5bc3002f75b787de5\nReviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4450245\nReviewed-by: Dale Curtis \u003cdalecurtis@chromium.org\u003e\nCommit-Queue: Andres Calderon Jaramillo \u003candrescj@chromium.org\u003e\nReviewed-by: Pilar Molina Lopez \u003cpmolinalopez@chromium.org\u003e\nReviewed-by: Alex Gough \u003cajgo@chromium.org\u003e\nCode-Coverage: Findit \u003cfindit-for-me@appspot.gserviceaccount.com\u003e\nCr-Commit-Position: refs/heads/main@{#1135387}\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "53cb5a1da1be8c32277e406865d3e25d6b446cad",
      "old_mode": 33188,
      "old_path": "media/gpu/chromeos/BUILD.gn",
      "new_id": "5a471a6ac2bc038bc2a3acefeced76f0d03170be",
      "new_mode": 33188,
      "new_path": "media/gpu/chromeos/BUILD.gn"
    },
    {
      "type": "modify",
      "old_id": "0b2048034f3c9f3433c9b0e253e5150ad5f093a3",
      "old_mode": 33188,
      "old_path": "media/gpu/chromeos/oop_video_decoder.cc",
      "new_id": "f1cf3513b27237d52f3001fb1fd5ef54c7cc43f8",
      "new_mode": 33188,
      "new_path": "media/gpu/chromeos/oop_video_decoder.cc"
    },
    {
      "type": "modify",
      "old_id": "045bc7106452c96e1ff12c03f41fc4cde37f1665",
      "old_mode": 33188,
      "old_path": "media/gpu/chromeos/oop_video_decoder.h",
      "new_id": "a3c686833295a9ae07218ab7a235c7d56f52587e",
      "new_mode": 33188,
      "new_path": "media/gpu/chromeos/oop_video_decoder.h"
    },
    {
      "type": "modify",
      "old_id": "80dbb1a51ccae677f1042343a8976daa293d077c",
      "old_mode": 33188,
      "old_path": "media/mojo/mojom/stable/BUILD.gn",
      "new_id": "3da893d37806144c78a865aeedecdb9287908417",
      "new_mode": 33188,
      "new_path": "media/mojo/mojom/stable/BUILD.gn"
    },
    {
      "type": "modify",
      "old_id": "9e49e4b747cad2899b42880b7711d542fdc63724",
      "old_mode": 33188,
      "old_path": "media/mojo/mojom/stable/stable_video_decoder_types_mojom_traits.cc",
      "new_id": "0802fa22e3a3e910e96f884fabe06dd97caafbfa",
      "new_mode": 33188,
      "new_path": "media/mojo/mojom/stable/stable_video_decoder_types_mojom_traits.cc"
    },
    {
      "type": "modify",
      "old_id": "ce018c2ed3f335e4890d0f6d7549bf8fbc5a5516",
      "old_mode": 33188,
      "old_path": "media/mojo/mojom/stable/stable_video_decoder_types_mojom_traits.h",
      "new_id": "2f9ea9762162e398a18f432c0bb94e8e0f84511b",
      "new_mode": 33188,
      "new_path": "media/mojo/mojom/stable/stable_video_decoder_types_mojom_traits.h"
    },
    {
      "type": "modify",
      "old_id": "3811f55386501055a3d23e4e91c3c4656442125f",
      "old_mode": 33188,
      "old_path": "media/mojo/services/stable_video_decoder_service.cc",
      "new_id": "f873ab350f077e404f1d263b6e6d62021a0b80e3",
      "new_mode": 33188,
      "new_path": "media/mojo/services/stable_video_decoder_service.cc"
    },
    {
      "type": "modify",
      "old_id": "a65dcc51c625a64180d1387b1bd7eea1cae392a9",
      "old_mode": 33188,
      "old_path": "media/mojo/services/stable_video_decoder_service_unittest.cc",
      "new_id": "fba17108c04e59925500c0c5f583b8a96a4b7313",
      "new_mode": 33188,
      "new_path": "media/mojo/services/stable_video_decoder_service_unittest.cc"
    }
  ]
}