Commit e79a745b authored by Paul Moy's avatar Paul Moy Committed by Chromium LUCI CQ

chromebox_for_meetings: make PolicyInfo nullable

When a client requests PolicyInfo, but Chrome's service hasn't collected
any policy info from the device yet, a nullptr is returned. This
currently causes an unexpected null validation error in Mojo on the
client. Fix this by marking PolicyInfo optional and checking for nullptr
in the client before using the returned PolicyInfoPtr.

Bug: b:175792552
Change-Id: I409a83364434201c55039078ad9d183bd160cb56
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2597905
Commit-Queue: Kyle Williams <kdgwill@chromium.org>
Reviewed-by: default avatarKyle Williams <kdgwill@chromium.org>
Cr-Commit-Position: refs/heads/master@{#839480}
parent 29e48cde
...@@ -14,7 +14,7 @@ interface MeetDevicesInfo { ...@@ -14,7 +14,7 @@ interface MeetDevicesInfo {
// Request device policy information // Request device policy information
// Note: will return nullptr on failure to query policy information // Note: will return nullptr on failure to query policy information
GetPolicyInfo@1()=>(PolicyInfo info); GetPolicyInfo@1()=>(PolicyInfo? info);
// Request device system information // Request device system information
GetSysInfo@2()=>(SysInfo info); GetSysInfo@2()=>(SysInfo info);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment