| commit | ed44935144f119d21fd03f139629464f72678e9c | [log] [tgz] |
|---|---|---|
| author | Jason Iman <[email protected]> | Mon Oct 27 07:47:59 2025 |
| committer | Copybara-Service <[email protected]> | Fri Oct 31 04:11:49 2025 |
| tree | 52d35f3f38d6275e900dfcff946411c32e217fd1 | |
| parent | f4c725f3a06b89f3636333979f9f5c32491e607a [diff] |
system_api: Add ConnectNamespace as patchpanel virtual guest This is necessary for DNS proxy to be able to listen on ConnectNamespace users. BUG=b:436153507 TEST=CQ Change-Id: I3be03e63a91d22e4fb1f63a3365a73978a5fe9e3 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform2/+/7080712 Reviewed-by: Hugo Benichi <[email protected]> Commit-Queue: Jason Iman <[email protected]> Tested-by: Jason Iman <[email protected]> NOKEYCHECK=True GitOrigin-RevId: 30126e735eefaab2aec35834c904f4c1d50468e0
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.