[go: up one dir, main page]
Google Git
Sign in
chromium / chromium / src / d517756f0f905014d84c684f0e306d7db65e7028
commitd517756f0f905014d84c684f0e306d7db65e7028[log] [tgz]
authorMatthew Cary <mattcary@chromium.org>Thu Jun 13 14:38:46 2019
committerMatthew Cary (CET) <mattcary@chromium.org>Thu Jun 13 14:38:46 2019
tree546f729b2228a6dadb5c064b14e8ef3e2a8c2424
parentf10c3148ed2f5a24673903b877c12067f05c3fc4 [diff]
ChromeOS media: remove unnecessary FD duplication.

The recent refactoring CL crrev.com/c/1613297 introduced
unnecessary duplication of file descriptors. Worse, those
descriptors could leak, eventually leading to resource
exhaustion. As an added bonus, several legacy uses of
SharedMemory and SharedMemoryHandle are updated to the new
API.

See crbug.com/965455 for a motivating example of leaky
file descriptors.

(cherry picked from commit 842e2a8cb87738c74ca6f95c7173961d0bf5bfe4)

Bug: 849207, 971575
Change-Id: I8b6c06689d99a12c97af19bd3eabdb0918b1ba22
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1645301
Commit-Queue: Matthew Cary (CET) <mattcary@chromium.org>
Reviewed-by: Wei Lee <wtlee@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#666626}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1657841
Reviewed-by: Matthew Cary (CET) <mattcary@chromium.org>
Cr-Commit-Position: refs/branch-heads/3809@{#287}
Cr-Branched-From: d82dec1a818f378c464ba307ddd9c92133eac355-refs/heads/master@{#665002}
4 files changed
tree: 546f729b2228a6dadb5c064b14e8ef3e2a8c2424
  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. cloud_print/
  13. components/
  14. content/
  15. courgette/
  16. crypto/
  17. dbus/
  18. device/
  19. docs/
  20. extensions/
  21. fuchsia/
  22. gin/
  23. google_apis/
  24. google_update/
  25. gpu/
  26. headless/
  27. infra/
  28. ios/
  29. ipc/
  30. jingle/
  31. media/
  32. mojo/
  33. native_client_sdk/
  34. net/
  35. pdf/
  36. ppapi/
  37. printing/
  38. remoting/
  39. rlz/
  40. sandbox/
  41. services/
  42. skia/
  43. sql/
  44. storage/
  45. styleguide/
  46. testing/
  47. third_party/
  48. tools/
  49. ui/
  50. url/
  51. .clang-format
  52. .eslintrc.js
  53. .git-blame-ignore-revs
  54. .gitattributes
  55. .gitignore
  56. .gn
  57. .vpython
  58. AUTHORS
  59. BUILD.gn
  60. CODE_OF_CONDUCT.md
  61. codereview.settings
  62. DEPS
  63. ENG_REVIEW_OWNERS
  64. LICENSE
  65. LICENSE.chromium_os
  66. OWNERS
  67. PRESUBMIT.py
  68. PRESUBMIT_test.py
  69. PRESUBMIT_test_mocks.py
  70. README.md
  71. 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.

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

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