Hide mouse cursor when screenshot is taking

chrome/browser/ui/ash/chrome_screenshot_grabber.cc

Index: chrome/browser/ui/ash/chrome_screenshot_grabber.cc
diff --git a/chrome/browser/ui/ash/chrome_screenshot_grabber.cc b/chrome/browser/ui/ash/chrome_screenshot_grabber.cc
index a1078e6bd7bcee53a5e70b2aca17ef42d9c570e5..9979dc17b675993334bfadceb724af8724f42843 100644
--- a/chrome/browser/ui/ash/chrome_screenshot_grabber.cc
+++ b/chrome/browser/ui/ash/chrome_screenshot_grabber.cc
@@ -377,6 +377,10 @@ void ChromeScreenshotGrabber::PrepareFileAndRunOnBlockingPool(
 void ChromeScreenshotGrabber::OnScreenshotCompleted(
     ui::ScreenshotGrabberObserver::Result result,
     const base::FilePath& screenshot_path) {
+  // Cursor is hidden during the screenshot. Show it when screenshot completed.
+  ash::Shell* shell = ash::Shell::GetInstance();
+  if (shell->cursor_manager())
+    shell->cursor_manager()->ShowCursor();

[sky, 2016/07/21 19:40:26]

Why is the show here and not in ScreenshotGrabber? I would expect the place
doing the hide to also do the show. To do otherwise is error prone. Also please
put the calls to show/hide in the same object that is easy to reason about
lifetime. See ScopedCursorHider in aura/window for what I mean.

[Qiang(Joe) Xu, 2016/07/21 23:02:49]

Thanks, based on the suggestion I got a updated version.

   // Do not show a notification that a screenshot was taken while no user is
   // logged in, since it is confusing for the user to get a message about it
   // after he logs in (crbug.com/235217).