Show the cursor if it was hidden when starting partical screenshot.

Show the cursor if it was hidden when starting partical screenshot.

BUG=485649
TEST=PartialScreenShotTest.VisibilityTest
R=mukai@chromium.org

Committed: https://crrev.com/55dae7df996493abc81e6b9e435f69f1c23aa9f4
Cr-Commit-Position: refs/heads/master@{#330005}

[oshima, 2015-05-14 22:20:36]

The CQ bit was checked by oshima@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-05-14 22:22:30]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1128933009/1

[Jun Mukai, 2015-05-14 22:49:10]

https://codereview.chromium.org/1128933009/diff/1/ash/utility/partial_screens...
File ash/utility/partial_screenshot_controller.cc (right):

https://codereview.chromium.org/1128933009/diff/1/ash/utility/partial_screens...
ash/utility/partial_screenshot_controller.cc:106: cursor_manager_->HideCursor();
Does it have to hide the cursor again?
When cursor is invisible, and then partial screenshot is taken, then keeping it
visinble sounds natural to me, because user uses the mouse during the partial
screenshot session.

[oshima, 2015-05-14 23:15:55]

https://codereview.chromium.org/1128933009/diff/1/ash/utility/partial_screens...
File ash/utility/partial_screenshot_controller.cc (right):

https://codereview.chromium.org/1128933009/diff/1/ash/utility/partial_screens...
ash/utility/partial_screenshot_controller.cc:106: cursor_manager_->HideCursor();
On 2015/05/14 22:49:10, Jun Mukai wrote:
> Does it have to hide the cursor again?
> When cursor is invisible, and then partial screenshot is taken, then keeping
it
> visinble sounds natural to me, because user uses the mouse during the partial
> screenshot session.

I was thinking "shift-ctrl-f5 then esc" scenario, but it's probably not
important. simplified the code.

[Jun Mukai, 2015-05-14 23:17:33]

lgtm

[oshima, 2015-05-14 23:18:48]

The CQ bit was checked by oshima@chromium.org

[commit-bot: I haz the power, 2015-05-14 23:21:12]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1128933009/20001

[commit-bot: I haz the power, 2015-05-15 00:06:09]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-05-15 00:06:10]

Try jobs failed on following builders:
  win8_chromium_rel on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_rel...)

[oshima, 2015-05-15 00:38:54]

The CQ bit was checked by oshima@chromium.org

[oshima, 2015-05-15 00:38:55]

The patchset sent to the CQ was uploaded after l-g-t-m from mukai@chromium.org
Link to the patchset: https://codereview.chromium.org/1128933009/#ps40001
(title: " ")

[commit-bot: I haz the power, 2015-05-15 00:39:27]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1128933009/40001

[commit-bot: I haz the power, 2015-05-15 01:15:38]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-05-15 01:16:22]

Patchset 3 (id:??) landed as
https://crrev.com/55dae7df996493abc81e6b9e435f69f1c23aa9f4
Cr-Commit-Position: refs/heads/master@{#330005}