| 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_);
|
|
|