commit | abab65e6dfb58a9a3024d045bebd472bd44d0014 | [log] [tgz] |
---|---|---|
author | Harvey Yang <chenghaoyang@google.com> | Thu Aug 11 12:11:05 2022 |
committer | Copybara-Service <copybara-worker@google.com> | Wed Aug 17 16:55:01 2022 |
tree | 86789a56fe50c1c2e898ab284d859381ea21c810 | |
parent | aa799daeb5b4ddd7d0d3a7ba9b11d9ca794b4da3 [diff] |
system_api: Add IioSensor service name |kIioSensor| is needed to migrate iioservice from being bootstrapped by ash-chrome to CrOS Mojo Service Manager, and let sensor clients connect to iioservice with the service. BUG=b:242164419 TEST=run on coachz Change-Id: I61fbbd78256870e9dfc2d99986ea78bb6fbb0278 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform2/+/3826403 Commit-Queue: Cheng-Hao Yang <chenghaoyang@chromium.org> Reviewed-by: Mike Frysinger <vapier@chromium.org> Tested-by: Cheng-Hao Yang <chenghaoyang@chromium.org> Reviewed-by: Chung-Sheng Wu <chungsheng@google.com> Auto-Submit: Cheng-Hao Yang <chenghaoyang@chromium.org> NOKEYCHECK=True GitOrigin-RevId: e07be67dc4e807e46803373513c65fc180b0f147
This directory (platform2/system_api
) contains constants and definitions like D-Bus service names that are shared between Chromium and Chromium OS.
This directory is only for things like headers and .proto files. No implementation should be added.
When writting a .proto file make sure to use:
option optimize_for = LITE_RUNTIME;
This will force usage of a lite protobuf instead of a full/heavy weight protobuf. The browser only links against the light version, so you will get cryptic link errors about missing parts of Message if you define a protobuf here and then try to use it in Chrome. Currently CrOS links against the full protobuffer library, but that might change in the future.
When declaring a protobuf, avoid use of required unless it is exactly what you mean. “Required is Forever” and very rarely should actually be used. Consult Protocol Buffer Basics: C++ for a detailed of this issue.