Reland: Update DisplayConfigurator to use the asynchronous tasks

ui/display/chromeos/configure_displays_task.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ui/display/chromeos/configure_displays_task.h"
 
 #include "base/auto_reset.h"
 #include "base/bind.h"
 #include "ui/display/types/display_snapshot.h"
 #include "ui/display/types/native_display_delegate.h"
@@ skipping 47 matching lines @@
 }
 
 void ConfigureDisplaysTask::Run() {
   // Synchronous configurators will recursively call Run(). In that case just
   // defer their call to the next iteration in the while-loop. This is done to
   // guard against stack overflows if the display has a large list of broken
   // modes.
   if (is_configuring_)
     return;
 
-  base::AutoReset<bool> recursivity_guard(&is_configuring_, true);
-  while (!pending_request_indexes_.empty()) {
-    size_t index = pending_request_indexes_.front();
-    DisplayConfigureRequest* request = &requests_[index];
-    pending_request_indexes_.pop();
-    delegate_->Configure(*request->display, request->mode, request->origin,
-                         base::Bind(&ConfigureDisplaysTask::OnConfigured,
-                                    weak_ptr_factory_.GetWeakPtr(), index));
+  {
+    base::AutoReset<bool> recursivity_guard(&is_configuring_, true);
+    while (!pending_request_indexes_.empty()) {
+      size_t index = pending_request_indexes_.front();
+      DisplayConfigureRequest* request = &requests_[index];
+      pending_request_indexes_.pop();
+      delegate_->Configure(*request->display, request->mode, request->origin,
+                           base::Bind(&ConfigureDisplaysTask::OnConfigured,
+                                      weak_ptr_factory_.GetWeakPtr(), index));
+    }
   }
 
-  // Nothing should be modified after the |callback_| is called since thie
+  // Nothing should be modified after the |callback_| is called since the
   // task may be deleted in the callback.
   if (num_displays_configured_ == requests_.size())
     callback_.Run(task_status_);
 }
 
 void ConfigureDisplaysTask::OnConfigured(size_t index, bool success) {
   DisplayConfigureRequest* request = &requests_[index];
   VLOG(2) << "Configured status=" << success
           << " display=" << request->display->display_id()
           << " origin=" << request->origin.ToString()
@@ skipping 14 matching lines @@
   }
 
   num_displays_configured_++;
   if (!success)
     task_status_ = ERROR;
 
   Run();
 }
 
 }  // namespace ui