| commit | f71907ee5a49b35b73cd6839b68e26d24246f4a3 | [log] [tgz] |
|---|---|---|
| author | Michael Ershov <miersh@google.com> | Tue Nov 15 19:27:32 2022 +0100 |
| committer | Copybara-Service <copybara-worker@google.com> | Wed Nov 16 03:34:41 2022 -0800 |
| tree | cafbb9e1a105461b529f68b32dec1a921ca511fa | |
| parent | 249fd145fd9098095b56898ed2334bb2d84d3309 [diff] |
system_api: Add DBUS call for session_manager Create a new DBus method constant for StartSessionEx. It will allow switching between generating owner key in session_manager or in Chrome. BUG=b:244407123 TEST=None Change-Id: Id8d4d4f239ec9ba3e1ba3ec35edcb2136e7918b0 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform2/+/4028664 Reviewed-by: Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Michael Ershov <miersh@google.com> Tested-by: Michael Ershov <miersh@google.com> NOKEYCHECK=True GitOrigin-RevId: 7d7bf0ae00122405fb2ccbb30655b93b09454f01
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.