Fix task concurrency in components/update_client

components/update_client/update_client_internal.h

Index: components/update_client/update_client_internal.h
diff --git a/components/update_client/update_client_internal.h b/components/update_client/update_client_internal.h
index 8adf5f83591aa312df8129d2d5987ca97933134a..de32294e1d4fa0a6ce2983f2bc8a6aed6743c7cd 100644
--- a/components/update_client/update_client_internal.h
+++ b/components/update_client/update_client_internal.h
@@ -65,10 +65,17 @@ class UpdateClientImpl : public UpdateClient {
 
   scoped_refptr<Configurator> config_;
 
-  // Contains the tasks that are queued up.
+  // Contains the tasks that are queued up. In the current implementation,

[waffles, 2015/10/29 22:59:33]

Maybe say "pending" instead of "queued up".

(It's obvious that a queue of task contains queued tasks.)

+  // only update tasks (background tasks) are queued up. These tasks are
+  // pending while they are in this queue. They are not being handled for
+  // the moment.
   std::queue<Task*> task_queue_;
 
-  // Contains all tasks in progress.
+  // Contains all tasks in progress. These are the tasks that the update engine
+  // is executing at one moment. Install tasks are run concurrently, update
+  // tasks are always serialized, and update tasks are queued up if install
+  // tasks are running. In addition, concurrent install tasks for the same id
+  // is not allowed.

[waffles, 2015/10/29 22:59:33]

is -> are

   std::set<Task*> tasks_;
 
   // TODO(sorin): try to make the ping manager an observer of the service.