Comments updated as result of infinite timeout for state keys generated

chrome/browser/chromeos/policy/server_backed_state_keys_broker.h

Index: chrome/browser/chromeos/policy/server_backed_state_keys_broker.h
diff --git a/chrome/browser/chromeos/policy/server_backed_state_keys_broker.h b/chrome/browser/chromeos/policy/server_backed_state_keys_broker.h
index 6fb80e94038eab2c9fdc8ef00746af18800ea6ca..e24052747b43b6e868f0638ff9db454e187dac39 100644
--- a/chrome/browser/chromeos/policy/server_backed_state_keys_broker.h
+++ b/chrome/browser/chromeos/policy/server_backed_state_keys_broker.h
@@ -46,10 +46,12 @@ class ServerBackedStateKeysBroker {
   // requested yet, calling this will also trigger their initial fetch.
   Subscription RegisterUpdateCallback(const base::Closure& callback);
 
-  // Requests state keys asynchronously. Invokes the passed callback exactly
-  // once (unless |this| gets destroyed before the callback happens), with the
-  // current state keys passed as a parameter to the callback. If there's a
-  // problem determining the state keys, the passed vector will be empty.
+  // Requests state keys asynchronously. Invokes the passed callback at most
+  // once, with the current state keys passed as a parameter to the callback. If
+  // there's a problem determining the state keys, the passed vector will be
+  // empty. If |this| gets destroyed before the callback happens or if the time
+  // sync fails / the network is not established, then the |callback| is never
+  // invoked.

[stevenjb, 2016/12/22 19:14:05]

Let's reference the issue here to give developers more context.

[igorcov, 2016/12/23 15:39:21]

Done.

   void RequestStateKeys(const StateKeysCallback& callback);
 
   // Get the set of current state keys. Empty if state keys are unavailable