Network traffic annotation added to OAuth2ApiCallFlow and its subclasses.

components/cryptauth/cryptauth_device_manager.cc

Index: components/cryptauth/cryptauth_device_manager.cc
diff --git a/components/cryptauth/cryptauth_device_manager.cc b/components/cryptauth/cryptauth_device_manager.cc
index 0fa6a825e5ffe02728812f02d36dd642d6b22ef3..9576eed220197335aae3a5fa50fae7959cf355fb 100644
--- a/components/cryptauth/cryptauth_device_manager.cc
+++ b/components/cryptauth/cryptauth_device_manager.cc
@@ -18,6 +18,7 @@
 #include "components/prefs/pref_service.h"
 #include "components/prefs/scoped_user_pref_update.h"
 #include "components/proximity_auth/logging/logging.h"
+#include "net/traffic_annotation/network_traffic_annotation.h"
 
 namespace cryptauth {
 
@@ -541,11 +542,33 @@ void CryptAuthDeviceManager::OnSyncRequested(
   GetMyDevicesRequest request;
   request.set_invocation_reason(invocation_reason);
   request.set_allow_stale_read(is_sync_speculative);
+  net::PartialNetworkTrafficAnnotationTag partial_traffic_annotation =
+      net::DefinePartialNetworkTrafficAnnotation("...", "oauth2_api_call_flow",

[sacomoto, 2017/06/20 09:48:18]

cryptauth_get_my_devices

[Ramin Halavati, 2017/06/20 10:46:16]

Done.

+                                                 R"(
+      semantics {
+        sender: "..."

[sacomoto, 2017/06/20 09:48:18]

CryptAuth Device Manager

[Ramin Halavati, 2017/06/20 10:46:16]

Done.

+        description: "..."

[sacomoto, 2017/06/20 09:48:18]

Gets a list of the devices registered (for the same user) on CryptAuth

[Ramin Halavati, 2017/06/20 10:46:16]

Done.

+        trigger: "..."

[sacomoto, 2017/06/20 09:48:18]

Periodically every 24h, or by API request

[Ramin Halavati, 2017/06/20 10:46:16]

Done.

+        data: "..."

[sacomoto, 2017/06/20 09:48:18]

None

[Ramin Halavati, 2017/06/20 10:46:15]

If sign-in is required, shouldn't this be user authentication token?
Also for other requests that are dependent on sign-in.

+        destination: GOOGLE_OWNED_SERVICE
+      }
+      policy {
+        setting: "..."

[sacomoto, 2017/06/20 09:48:18]

This feature cannot be disabled in settings.

[Ramin Halavati, 2017/06/20 10:46:15]

Done.

+        chrome_policy {
+          [POLICY_NAME] {

[sacomoto, 2017/06/20 09:48:18]

SigninAllowed

[Ramin Halavati, 2017/06/20 10:46:16]

Done.

+            policy_options {mode: MANDATORY/RECOMMENDED/UNSET}
+            [POLICY_NAME]: ... //(value to disable it)

[sacomoto, 2017/06/20 09:48:18]

SigninAllowed: false

[Ramin Halavati, 2017/06/20 10:46:15]

Done.

+          }
+        }
+        policy_exception_justification: "..."
+      })");
   cryptauth_client_->GetMyDevices(
-      request, base::Bind(&CryptAuthDeviceManager::OnGetMyDevicesSuccess,
-                          weak_ptr_factory_.GetWeakPtr()),
+      request,
+      base::Bind(&CryptAuthDeviceManager::OnGetMyDevicesSuccess,
+                 weak_ptr_factory_.GetWeakPtr()),
       base::Bind(&CryptAuthDeviceManager::OnGetMyDevicesFailure,
-                 weak_ptr_factory_.GetWeakPtr()));
+                 weak_ptr_factory_.GetWeakPtr()),
+      partial_traffic_annotation);
 }
 
 }  // namespace cryptauth