Commit b6c13fe8 authored by mukai@chromium.org's avatar mukai@chromium.org

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

The screenshot task takes time so the flash could be noticeably long.

BUG=120092
TEST=manually

Review URL: https://chromiumcodereview.appspot.com/9854012

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@128925 0039d316-1c4b-4281-b951-d872f2087c98
parent 95562961
...@@ -117,7 +117,9 @@ void ScreenshotTaker::HandleTakeScreenshot(aura::Window* window) { ...@@ -117,7 +117,9 @@ void ScreenshotTaker::HandleTakeScreenshot(aura::Window* window) {
void ScreenshotTaker::CloseVisualFeedbackLayer(const base::Closure& task) { void ScreenshotTaker::CloseVisualFeedbackLayer(const base::Closure& task) {
visual_feedback_layer_.reset(); visual_feedback_layer_.reset();
task.Run(); // Hide the visual feedback immediately because |task| may take a long time
// to finish.
MessageLoopForUI::current()->PostTask(FROM_HERE, task);
} }
void ScreenshotTaker::DisplayVisualFeedback(const gfx::Rect& rect, void ScreenshotTaker::DisplayVisualFeedback(const gfx::Rect& rect,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment