commit | 95ba8b36c447f6e45b35363b6a1c53972d178661 | [log] [tgz] |
---|---|---|
author | Ryo Hashimoto <hashimoto@google.com> | Fri Jun 03 09:36:44 2022 |
committer | Copybara-Service <copybara-worker@google.com> | Thu Jul 07 20:53:44 2022 |
tree | 4eb6e4a8a4227919b7f0eeb0026fb6b5298af579 | |
parent | 015f8e3a58f86567c1748d9d3698bd2939bd6157 [diff] |
cryptohome: Delete SetProjectId D-Bus method This D-Bus method was superseded by SetMediaRWDataFileProjectId. BUG=b:175764830 TEST=tast run arc.QuotaProjectID Change-Id: I6ed15fecdbd8ccb5039ffce837e59d32dfaf8cc7 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform2/+/3686980 Commit-Queue: Ryo Hashimoto <hashimoto@chromium.org> Tested-by: Ryo Hashimoto <hashimoto@chromium.org> Reviewed-by: John L Chen <zuan@chromium.org> NOKEYCHECK=True GitOrigin-RevId: 2c777cc2e498bdca477e72f81cb78ff07658b222
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.