Correctly convert warp bounds from screen to native.

Correctly convert warp bounds from screen to native.

This is a regression in https://codereview.chromium.org/1114633002c

BUG=485832
TEST=covered by unittests

Committed: https://crrev.com/48060cf46055a412e7ce74438062815ba9a6659a
Cr-Commit-Position: refs/heads/master@{#329567}

[oshima, 2015-05-12 16:40:55]

Patchset #2 (id:20001) has been deleted

[oshima, 2015-05-12 16:40:58]

Patchset #1 (id:1) has been deleted

[oshima, 2015-05-12 16:44:01]

oshima@chromium.org changed reviewers:
+ mukai@chromium.org

[Jun Mukai, 2015-05-12 17:08:46]

lgtm with nits

https://codereview.chromium.org/1132303005/diff/40001/ash/display/display_uti...
File ash/display/display_util.cc (right):

https://codereview.chromium.org/1132303005/diff/40001/ash/display/display_uti...
ash/display/display_util.cc:13: #include "ash/wm/window_util.h"
Is this necessary?

https://codereview.chromium.org/1132303005/diff/40001/ash/display/display_uti...
ash/display/display_util.cc:169: gfx::Point
end_in_native(bounds_in_screen.right(), bounds_in_screen.bottom());
nit: end_in_native = bounds_in_screen.bottom_right();

[oshima, 2015-05-12 18:24:26]

https://codereview.chromium.org/1132303005/diff/40001/ash/display/display_uti...
File ash/display/display_util.cc (right):

https://codereview.chromium.org/1132303005/diff/40001/ash/display/display_uti...
ash/display/display_util.cc:13: #include "ash/wm/window_util.h"
On 2015/05/12 17:08:46, Jun Mukai wrote:
> Is this necessary?

Done.

https://codereview.chromium.org/1132303005/diff/40001/ash/display/display_uti...
ash/display/display_util.cc:169: gfx::Point
end_in_native(bounds_in_screen.right(), bounds_in_screen.bottom());
On 2015/05/12 17:08:46, Jun Mukai wrote:
> nit: end_in_native = bounds_in_screen.bottom_right();

Done.

[oshima, 2015-05-12 18:24:44]

The CQ bit was checked by oshima@chromium.org

[oshima, 2015-05-12 18:24:44]

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/1132303005/#ps60001
(title: " ")

[commit-bot: I haz the power, 2015-05-12 18:25:01]

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

[commit-bot: I haz the power, 2015-05-12 19:45:04]

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

[commit-bot: I haz the power, 2015-05-12 19:45:05]

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-13 00:48:11]

The CQ bit was checked by oshima@chromium.org

[oshima, 2015-05-13 00:48:11]

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/1132303005/#ps80001
(title: "added SupportsMultipleDisplays check")

[commit-bot: I haz the power, 2015-05-13 00:50:15]

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

[commit-bot: I haz the power, 2015-05-13 02:12:38]

Committed patchset #3 (id:80001)

[commit-bot: I haz the power, 2015-05-13 02:13:29]

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