Fix use-after-move in //chrome/browser/net/network_context_configuration_browsertest.cc

Fix use-after-move (potential) bugs found by the
"bugprone-use-after-move" clang-tidy check.

Bug: 1122844
Change-Id: I30ef2ad1444a2d21e59da4a590c8da1df4952668
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2382071Reviewed-by: Eric Roman <eroman@chromium.org>
Commit-Queue: Maksim Ivanov <emaxx@chromium.org>
Cr-Commit-Position: refs/heads/master@{#803657}

chrome/browser/net/network_context_configuration_browsertest.cc

@@ -167,11 +167,13 @@ class ConnectionTypeWaiter
 
   void Wait(network::mojom::ConnectionType expected_type) {
     auto current_type = network::mojom::ConnectionType::CONNECTION_UNKNOWN;
-    network::NetworkConnectionTracker::ConnectionTypeCallback callback =
-        base::BindOnce(&ConnectionTypeWaiter::OnConnectionChanged,
-                       base::Unretained(this));
-    while (!tracker_->GetConnectionType(&current_type, std::move(callback)) ||
-           current_type != expected_type) {
+    for (;;) {
+      network::NetworkConnectionTracker::ConnectionTypeCallback callback =
+          base::BindOnce(&ConnectionTypeWaiter::OnConnectionChanged,
+                         base::Unretained(this));
+      if (tracker_->GetConnectionType(&current_type, std::move(callback)) &&
+          current_type == expected_type)
+        break;
       run_loop_ = std::make_unique<base::RunLoop>(
           base::RunLoop::Type::kNestableTasksAllowed);
       run_loop_->Run();