Index: components/cryptauth/cryptauth_client_impl.cc |
diff --git a/components/cryptauth/cryptauth_client_impl.cc b/components/cryptauth/cryptauth_client_impl.cc |
index 259838598ef08a9ce1856025ef9d3a0dba08081f..97033684324c188361c68f10c6403407e038bb9b 100644 |
--- a/components/cryptauth/cryptauth_client_impl.cc |
+++ b/components/cryptauth/cryptauth_client_impl.cc |
@@ -67,43 +67,168 @@ void CryptAuthClientImpl::GetMyDevices( |
const GetMyDevicesRequest& request, |
const GetMyDevicesCallback& callback, |
const ErrorCallback& error_callback) { |
- MakeApiCall(kGetMyDevicesPath, request, callback, error_callback); |
+ net::PartialNetworkTrafficAnnotationTag partial_traffic_annotation = |
+ net::DefinePartialNetworkTrafficAnnotation("...", "oauth2_api_call_flow", |
+ R"( |
+ semantics { |
+ sender: "..." |
+ description: "..." |
+ trigger: "..." |
+ data: "..." |
+ destination: WEBSITE/GOOGLE_OWNED_SERVICE/OTHER/LOCAL |
+ } |
+ policy { |
+ setting: "..." |
+ chrome_policy { |
+ [POLICY_NAME] { |
+ policy_options {mode: MANDATORY/RECOMMENDED/UNSET} |
+ [POLICY_NAME]: ... //(value to disable it) |
+ } |
+ } |
+ policy_exception_justification: "..." |
+ })"); |
+ MakeApiCall(kGetMyDevicesPath, request, callback, error_callback, |
+ partial_traffic_annotation); |
} |
void CryptAuthClientImpl::FindEligibleUnlockDevices( |
const FindEligibleUnlockDevicesRequest& request, |
const FindEligibleUnlockDevicesCallback& callback, |
const ErrorCallback& error_callback) { |
- MakeApiCall(kFindEligibleUnlockDevicesPath, request, callback, |
- error_callback); |
+ net::PartialNetworkTrafficAnnotationTag partial_traffic_annotation = |
+ net::DefinePartialNetworkTrafficAnnotation("...", "oauth2_api_call_flow", |
+ R"( |
+ semantics { |
+ sender: "..." |
+ description: "..." |
+ trigger: "..." |
+ data: "..." |
+ destination: WEBSITE/GOOGLE_OWNED_SERVICE/OTHER/LOCAL |
+ } |
+ policy { |
+ setting: "..." |
+ chrome_policy { |
+ [POLICY_NAME] { |
+ policy_options {mode: MANDATORY/RECOMMENDED/UNSET} |
+ [POLICY_NAME]: ... //(value to disable it) |
+ } |
+ } |
+ policy_exception_justification: "..." |
+ })"); |
+ MakeApiCall(kFindEligibleUnlockDevicesPath, request, callback, error_callback, |
+ partial_traffic_annotation); |
} |
void CryptAuthClientImpl::SendDeviceSyncTickle( |
const SendDeviceSyncTickleRequest& request, |
const SendDeviceSyncTickleCallback& callback, |
const ErrorCallback& error_callback) { |
- MakeApiCall(kSendDeviceSyncTicklePath, request, callback, error_callback); |
+ net::PartialNetworkTrafficAnnotationTag partial_traffic_annotation = |
+ net::DefinePartialNetworkTrafficAnnotation("...", "oauth2_api_call_flow", |
+ R"( |
+ semantics { |
+ sender: "..." |
+ description: "..." |
+ trigger: "..." |
+ data: "..." |
+ destination: WEBSITE/GOOGLE_OWNED_SERVICE/OTHER/LOCAL |
+ } |
+ policy { |
+ setting: "..." |
+ chrome_policy { |
+ [POLICY_NAME] { |
+ policy_options {mode: MANDATORY/RECOMMENDED/UNSET} |
+ [POLICY_NAME]: ... //(value to disable it) |
+ } |
+ } |
+ policy_exception_justification: "..." |
+ })"); |
+ MakeApiCall(kSendDeviceSyncTicklePath, request, callback, error_callback, |
+ partial_traffic_annotation); |
} |
void CryptAuthClientImpl::ToggleEasyUnlock( |
const ToggleEasyUnlockRequest& request, |
const ToggleEasyUnlockCallback& callback, |
const ErrorCallback& error_callback) { |
- MakeApiCall(kToggleEasyUnlockPath, request, callback, error_callback); |
+ net::PartialNetworkTrafficAnnotationTag partial_traffic_annotation = |
+ net::DefinePartialNetworkTrafficAnnotation("...", "oauth2_api_call_flow", |
+ R"( |
+ semantics { |
+ sender: "..." |
+ description: "..." |
+ trigger: "..." |
+ data: "..." |
+ destination: WEBSITE/GOOGLE_OWNED_SERVICE/OTHER/LOCAL |
+ } |
+ policy { |
+ setting: "..." |
+ chrome_policy { |
+ [POLICY_NAME] { |
+ policy_options {mode: MANDATORY/RECOMMENDED/UNSET} |
+ [POLICY_NAME]: ... //(value to disable it) |
+ } |
+ } |
+ policy_exception_justification: "..." |
+ })"); |
+ MakeApiCall(kToggleEasyUnlockPath, request, callback, error_callback, |
+ partial_traffic_annotation); |
} |
void CryptAuthClientImpl::SetupEnrollment( |
const SetupEnrollmentRequest& request, |
const SetupEnrollmentCallback& callback, |
const ErrorCallback& error_callback) { |
- MakeApiCall(kSetupEnrollmentPath, request, callback, error_callback); |
+ net::PartialNetworkTrafficAnnotationTag partial_traffic_annotation = |
+ net::DefinePartialNetworkTrafficAnnotation("...", "oauth2_api_call_flow", |
+ R"( |
+ semantics { |
+ sender: "..." |
+ description: "..." |
+ trigger: "..." |
+ data: "..." |
+ destination: WEBSITE/GOOGLE_OWNED_SERVICE/OTHER/LOCAL |
+ } |
+ policy { |
+ setting: "..." |
+ chrome_policy { |
+ [POLICY_NAME] { |
+ policy_options {mode: MANDATORY/RECOMMENDED/UNSET} |
+ [POLICY_NAME]: ... //(value to disable it) |
+ } |
+ } |
+ policy_exception_justification: "..." |
+ })"); |
+ MakeApiCall(kSetupEnrollmentPath, request, callback, error_callback, |
+ partial_traffic_annotation); |
} |
void CryptAuthClientImpl::FinishEnrollment( |
const FinishEnrollmentRequest& request, |
const FinishEnrollmentCallback& callback, |
const ErrorCallback& error_callback) { |
- MakeApiCall(kFinishEnrollmentPath, request, callback, error_callback); |
+ net::PartialNetworkTrafficAnnotationTag partial_traffic_annotation = |
+ net::DefinePartialNetworkTrafficAnnotation("...", "oauth2_api_call_flow", |
+ R"( |
+ semantics { |
+ sender: "..." |
+ description: "..." |
+ trigger: "..." |
+ data: "..." |
+ destination: WEBSITE/GOOGLE_OWNED_SERVICE/OTHER/LOCAL |
+ } |
+ policy { |
+ setting: "..." |
+ chrome_policy { |
+ [POLICY_NAME] { |
+ policy_options {mode: MANDATORY/RECOMMENDED/UNSET} |
+ [POLICY_NAME]: ... //(value to disable it) |
+ } |
+ } |
+ policy_exception_justification: "..." |
+ })"); |
+ MakeApiCall(kFinishEnrollmentPath, request, callback, error_callback, |
+ partial_traffic_annotation); |
} |
std::string CryptAuthClientImpl::GetAccessTokenUsed() { |
@@ -115,7 +240,8 @@ void CryptAuthClientImpl::MakeApiCall( |
const std::string& request_path, |
const RequestProto& request_proto, |
const base::Callback<void(const ResponseProto&)>& response_callback, |
- const ErrorCallback& error_callback) { |
+ const ErrorCallback& error_callback, |
+ const net::PartialNetworkTrafficAnnotationTag& partial_traffic_annotation) { |
if (has_call_started_) { |
error_callback.Run( |
"Client has been used for another request. Do not reuse."); |
@@ -123,6 +249,9 @@ void CryptAuthClientImpl::MakeApiCall( |
} |
has_call_started_ = true; |
+ api_call_flow_->SetPartialNetworkTrafficAnnotation( |
+ partial_traffic_annotation); |
+ |
// The |device_classifier| field must be present for all CryptAuth requests. |
RequestProto request_copy(request_proto); |
request_copy.mutable_device_classifier()->CopyFrom(device_classifier_); |