[go: nahoru, domu]

Revert "[//media] Add RenderableGMBVideoFramePool MultiplanarSI flow on Mac"

This reverts commit 60007e376cb8cd470a067102553bcef92445b231.

Reason for revert: Gold failures in https://crbug.com/1451025

Original change's description:
> [//media] Add RenderableGMBVideoFramePool MultiplanarSI flow on Mac
>
> This CL adds a flow to RenderableGMBVideoFramePool wherein it creates
> Multiplanar SharedImages on Mac. A few notes:
>
> * I tested the flow with Google Meet visual effects, which I confirmed
> (repeatedly :) were broken if the flow wasn't correct
> * I enabled the flow only on Mac at this time as I wasn't able to
> test on any other platform, and I don't know if further changes might
> be required on other platforms
> * I added the change to
> media/renderers/video_frame_rgba_to_yuva_converter.cc in response to
> a gpu-side DCHECK [1] going off in local testing.  It seems like it
> might need to be generalized in the future.
> * The only implementation of the new Context::CreateSI() method that
> is invoked in this flow is WebGraphicsContext3DVideoFramePool, but I
> filled in the others for completeness
>
> This patch is based on a prototype by hitawala@.
>
> [1] https://source.chromium.org/chromium/chromium/src/+/main:gpu/command_buffer/service/copy_shared_image_helper.cc;l=642-644?q=copy_shared_image&ss=chromium
>
> Change-Id: I57d1ea8a0b2b5c35ddee7468f48a199b06f46f5d
> Bug: 1429004
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4488909
> Reviewed-by: Kentaro Hara <haraken@chromium.org>
> Commit-Queue: Colin Blundell <blundell@chromium.org>
> Reviewed-by: Saifuddin Hitawala <hitawala@chromium.org>
> Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
> Reviewed-by: Vasiliy Telezhnikov <vasilyt@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#1152495}

Bug: 1451025
Bug: 1429004
Change-Id: Ie61c35797a5102c315d9992fba98c2a932e5e1d0
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4584419
Commit-Queue: Kai Ninomiya <kainino@chromium.org>
Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
Owners-Override: Kai Ninomiya <kainino@google.com>
Cr-Commit-Position: refs/heads/main@{#1152827}
7 files changed
tree: 7959a83a71c17036ccb26234dd967cdca5019819
  1. android_webview/
  2. apps/
  3. ash/
  4. base/
  5. build/
  6. build_overrides/
  7. buildtools/
  8. cc/
  9. chrome/
  10. chromecast/
  11. chromeos/
  12. codelabs/
  13. components/
  14. content/
  15. courgette/
  16. crypto/
  17. dbus/
  18. device/
  19. docs/
  20. extensions/
  21. fuchsia_web/
  22. gin/
  23. google_apis/
  24. google_update/
  25. gpu/
  26. headless/
  27. infra/
  28. ios/
  29. ipc/
  30. media/
  31. mojo/
  32. native_client_sdk/
  33. net/
  34. pdf/
  35. ppapi/
  36. printing/
  37. remoting/
  38. rlz/
  39. sandbox/
  40. services/
  41. skia/
  42. sql/
  43. storage/
  44. styleguide/
  45. testing/
  46. third_party/
  47. tools/
  48. ui/
  49. url/
  50. weblayer/
  51. .clang-format
  52. .clang-tidy
  53. .eslintrc.js
  54. .git-blame-ignore-revs
  55. .gitattributes
  56. .gitignore
  57. .gn
  58. .mailmap
  59. .rustfmt.toml
  60. .vpython3
  61. .yapfignore
  62. ATL_OWNERS
  63. AUTHORS
  64. BUILD.gn
  65. CODE_OF_CONDUCT.md
  66. codereview.settings
  67. DEPS
  68. DIR_METADATA
  69. LICENSE
  70. LICENSE.chromium_os
  71. OWNERS
  72. PRESUBMIT.py
  73. PRESUBMIT_test.py
  74. PRESUBMIT_test_mocks.py
  75. README.md
  76. WATCHLISTS
README.md

Logo Chromium

Chromium is an open-source browser project that aims to build a safer, faster, and more stable way for all users to experience the web.

The project's web site is https://www.chromium.org.

To check out the source code locally, don't use git clone! Instead, follow the instructions on how to get the code.

Documentation in the source is rooted in docs/README.md.

Learn how to Get Around the Chromium Source Code Directory Structure .

For historical reasons, there are some small top level directories. Now the guidance is that new top level directories are for product (e.g. Chrome, Android WebView, Ash). Even if these products have multiple executables, the code should be in subdirectories of the product.

If you found a bug, please file it at https://crbug.com/new.