Cleanup: Remove double semicolons.

cloud_print/service/win/chrome_launcher.cc

Index: cloud_print/service/win/chrome_launcher.cc
diff --git a/cloud_print/service/win/chrome_launcher.cc b/cloud_print/service/win/chrome_launcher.cc
index 8966a200cf5cd6097eb28902731d411c1e662004..5c9013aa9a098bf5d73a89cc32e1a15dcff36a53 100644
--- a/cloud_print/service/win/chrome_launcher.cc
+++ b/cloud_print/service/win/chrome_launcher.cc
@@ -13,7 +13,6 @@
 #include "base/json/json_reader.h"
 #include "base/json/json_writer.h"
 #include "base/process/process.h"
-#include "base/process/process.h"
 #include "base/values.h"
 #include "base/win/registry.h"
 #include "base/win/scoped_handle.h"
@@ -201,8 +200,8 @@ void ChromeLauncher::Run() {
   const base::TimeDelta default_time_out = base::TimeDelta::FromSeconds(1);
   const base::TimeDelta max_time_out = base::TimeDelta::FromHours(1);
 
-  for (base::TimeDelta time_out = default_time_out;;

[tfarina, 2015/12/30 20:32:59]

really? Maybe just add a space after the first ';'? To indicate that the
'condition' is empty? Because otherwise, this foor loop is perfectly valid code
and saves a couple of lines.

[Lei Zhang, 2015/12/30 20:59:59]

Meh. For-loops without a condition are harder to read, since most of the time
readers expect a condition. Readers may need to do a double-take to make sure
they read it right. "while (1)" OTOH clearly implies the reader should look for
returns/breaks inside.

-       time_out = std::min(time_out * 2, max_time_out)) {
+  base::TimeDelta time_out = default_time_out;
+  while (1) {
     base::FilePath chrome_path =
         chrome_launcher_support::GetAnyChromePath(false /* is_sxs */);
 
@@ -254,6 +253,8 @@ void ChromeLauncher::Run() {
     }
     if (stop_event_.TimedWait(time_out))
       break;
+
+    time_out = std::min(time_out * 2, max_time_out);
   }
 }