Network Service: Update comments about the callback parameter for...

Network Service: Update comments about the callback parameter for NetworkConnectionTracker::GetConnectionType()

This CL updates the comments about the callback parameter for
NetworkConnectionTracker::GetConnectionType() to make it clear
that the |callback| will not be called when connection type
can be retrieved synchronously.

Bug: None
Change-Id: I39eefe13311d9e594dc3f2518fb543014ccf5487
Reviewed-on: https://chromium-review.googlesource.com/c/1336078Reviewed-by: Robbie McElrath <rmcelrath@chromium.org>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Commit-Queue: Jun Cai <juncai@chromium.org>
Cr-Commit-Position: refs/heads/master@{#608928}

services/network/public/cpp/network_connection_tracker.h

@@ -62,11 +62,11 @@ class COMPONENT_EXPORT(NETWORK_CPP) NetworkConnectionTracker
   ~NetworkConnectionTracker() override;
 
   // If connection type can be retrieved synchronously, returns true and |type|
-  // will contain the current connection type; Otherwise, returns false and
-  // does not modify |type|, in which case, |callback| will be called on the
-  // calling thread when connection type is ready. This method is thread safe.
-  // Please also refer to net::NetworkChangeNotifier::GetConnectionType() for
-  // documentation.
+  // will contain the current connection type, and |callback| will not be
+  // called; Otherwise, returns false and does not modify |type|, in which
+  // case, |callback| will be called on the calling thread when connection type
+  // is ready. This method is thread safe. Please also refer to
+  // net::NetworkChangeNotifier::GetConnectionType() for documentation.
   virtual bool GetConnectionType(network::mojom::ConnectionType* type,
                                  ConnectionTypeCallback callback);