diff options
author | Daniel Cheng <dcheng@chromium.org> | 2018-01-12 05:03:35 +0900 |
---|---|---|
committer | Qijiang Fan <fqj@google.com> | 2020-06-05 08:13:36 +0900 |
commit | 8d9abd67d4f2d10fa71d4c20251535b7dc5e6bb4 (patch) | |
tree | 9f4b32232c1cc6ee2a5c2384b7d5f17f307bb59b /mojo/public/tools/bindings/chromium_bindings_configuration.gni | |
parent | e19895470228b2b4d91451f8c408685d952cb0a8 (diff) | |
download | libchrome-8d9abd67d4f2d10fa71d4c20251535b7dc5e6bb4.tar.gz |
Revert "Basic Audio Service hosting mojom::SystemInfo"
This reverts commit a334d5f07d2fbf48dbcf6933d00027ce16b128a5.
Reason for revert: accidentally passed CQ without security review
Original change's description:
> Basic Audio Service hosting mojom::SystemInfo
>
> In this CL:
> * //services/audio infrastructure
> * AudioDeviceDescription type mapping - may need to be moved to //media? (Someday we want it all
> to be in //audio though.)
> * Build files/dependencies - please review vigorously, I'm not sure everything is fine there.
> * mojom::SystemInfo interface and implementation on top of AudioManager,
> * Service implementation.
> * AudioSystemToMojoAdapter: client-side media::AudioSystem implementation on top of
> mojom::SystemInfo
> * Unit tests.
> * A fix for Connector::BindInterface() when there are binder overrides available (see discussion
> in PS17)
>
> Upcoming CLs for AudioService infrastructure:
> - Run in-process Audio service and
> instantiate BrowserMainLoop::audio_system_ with AudioSystemToMojoAdapter, so
> that audio system information is served through the service.
> - UMA stats for service usage.
> - Tear down the service when not in use (service side, client side)
> - Out-of-process AudioService implementation + unit tests
>
> Design doc (being updated now):
> https://docs.google.com/document/d/1s_Fd1WRDdpb5n6C2MSJjeC3fis6hULZwfKMeDd4K5tI/edit?usp=sharing
>
> Bug: 740943,792441
> Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
> Change-Id: Iae7f69386a6d2138c69d9566cf41655b57158b7c
> Reviewed-on: https://chromium-review.googlesource.com/799791
> Commit-Queue: Olga Sharonova <olka@chromium.org>
> Reviewed-by: Ken Rockot <rockot@chromium.org>
> Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
> Reviewed-by: Max Morin <maxmorin@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#528714}
TBR=dalecurtis@chromium.org,rockot@chromium.org,olka@chromium.org,maxmorin@chromium.org
Change-Id: Ib5c26464a3dacf0d2a5b3255295ebb268bd6fe20
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 740943, 792441
Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Reviewed-on: https://chromium-review.googlesource.com/862107
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Commit-Queue: Daniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#528721}
CrOS-Libchrome-Original-Commit: 8cd4afa14015d6d8cf2cd90e649f36c1458a4fbf
Diffstat (limited to 'mojo/public/tools/bindings/chromium_bindings_configuration.gni')
-rw-r--r-- | mojo/public/tools/bindings/chromium_bindings_configuration.gni | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/mojo/public/tools/bindings/chromium_bindings_configuration.gni b/mojo/public/tools/bindings/chromium_bindings_configuration.gni index 34e8095643..fa7ce7efe0 100644 --- a/mojo/public/tools/bindings/chromium_bindings_configuration.gni +++ b/mojo/public/tools/bindings/chromium_bindings_configuration.gni @@ -27,7 +27,6 @@ _typemap_imports = [ "//mojo/public/cpp/bindings/tests/chromium_typemaps.gni", "//net/interfaces/typemaps.gni", "//sandbox/mac/mojom/typemaps.gni", - "//services/audio/public/cpp/typemaps.gni", "//services/device/public/interfaces/typemaps.gni", "//services/identity/public/cpp/typemaps.gni", "//services/network/public/cpp/typemaps.gni", |