Close the screenshot flash effect before the screenshot task is actually executed.

chrome/browser/ui/views/ash/screenshot_taker.cc

Index: chrome/browser/ui/views/ash/screenshot_taker.cc
diff --git a/chrome/browser/ui/views/ash/screenshot_taker.cc b/chrome/browser/ui/views/ash/screenshot_taker.cc
index 1f874f1be47b8e454461b54dbe81d69f0ef75a33..5a9c13669b6ef1ed854532f31176442654e494ed 100644
--- a/chrome/browser/ui/views/ash/screenshot_taker.cc
+++ b/chrome/browser/ui/views/ash/screenshot_taker.cc
@@ -117,7 +117,9 @@ void ScreenshotTaker::HandleTakeScreenshot(aura::Window* window) {
 
 void ScreenshotTaker::CloseVisualFeedbackLayer(const base::Closure& task) {
   visual_feedback_layer_.reset();
-  task.Run();
+  // We post |task| to the event loop rather than running directly,
+  // to make sure the view is closed before the task.
+  MessageLoopForUI::current()->PostTask(FROM_HERE, task);

[Daniel Erat, 2012/03/26 13:15:32]

I don't know if this will be guaranteed to work in the future.  I'd double-check
with someone working on the compositor.

[Jun Mukai, 2012/03/26 14:27:12]

Who should we ask for the double-check?  oshima?
The comment would be misleading by the way.  Here it's not necessary to
guarantee the task runs before the view visually disappears.  Here I just want
the visual_feedback_layer_ to disappear without waiting the end of task running.
Updated the comment.  Sorry for the confusing comment.

[Daniel Erat, 2012/03/26 14:49:15]

piman@ would know what the plans (if any) are for moving compositing off of the
UI thread.

 }
 
 void ScreenshotTaker::DisplayVisualFeedback(const gfx::Rect& rect,