android: Remove DeviceDisplayInfo usage from ComopsitorImpl

android: Remove DeviceDisplayInfo usage from ComopsitorImpl

Use the window to get the correct display instead.

BUG=625089
Review-Url: https://codereview.chromium.org/2615793003
Cr-Commit-Position: refs/heads/master@{#442648}
Committed: https://chromium.googlesource.com/chromium/src/+/4868d4131b01a3e3520fcb2d65d9c5a1058a606a

[boliu, 2017-01-06 00:37:53]

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

[commit-bot: I haz the power, 2017-01-06 00:38:35]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[boliu, 2017-01-06 00:45:19]

boliu@chromium.org changed reviewers:
+ timav@chromium.org

[boliu, 2017-01-06 00:45:19]

ptal

[Tima Vaisburd, 2017-01-06 01:07:42]

lgtm

https://codereview.chromium.org/2615793003/diff/1/content/browser/renderer_ho...
File content/browser/renderer_host/compositor_impl_android.cc (right):

https://codereview.chromium.org/2615793003/diff/1/content/browser/renderer_ho...
content/browser/renderer_host/compositor_impl_android.cc:86: gfx::Size size =
display::Screen::GetScreen()
nit: this |size| can also be const

[boliu, 2017-01-06 01:23:55]

https://codereview.chromium.org/2615793003/diff/1/content/browser/renderer_ho...
File content/browser/renderer_host/compositor_impl_android.cc (right):

https://codereview.chromium.org/2615793003/diff/1/content/browser/renderer_ho...
content/browser/renderer_host/compositor_impl_android.cc:86: gfx::Size size =
display::Screen::GetScreen()
On 2017/01/06 01:07:42, Tima Vaisburd wrote:
> nit: this |size| can also be const

Done.

[boliu, 2017-01-06 01:24:21]

The CQ bit was checked by boliu@chromium.org

[boliu, 2017-01-06 01:24:21]

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

[commit-bot: I haz the power, 2017-01-06 01:25:38]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-06 02:57:26]

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

[commit-bot: I haz the power, 2017-01-06 02:57:26]

Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[boliu, 2017-01-10 18:29:45]

The CQ bit was checked by boliu@chromium.org

[commit-bot: I haz the power, 2017-01-10 18:30:24]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-10 19:13:14]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1484072985342840,
"parent_rev": "f380647990c63b5a258efba319f2aa1368b309f4", "commit_rev":
"4868d4131b01a3e3520fcb2d65d9c5a1058a606a"}

[commit-bot: I haz the power, 2017-01-10 19:13:43]

Description was changed from

==========
android: Remove DeviceDisplayInfo usage from ComopsitorImpl

Use the window to get the correct display instead.

BUG=625089
==========

to

==========
android: Remove DeviceDisplayInfo usage from ComopsitorImpl

Use the window to get the correct display instead.

BUG=625089

Review-Url: https://codereview.chromium.org/2615793003
Cr-Commit-Position: refs/heads/master@{#442648}
Committed:
https://chromium.googlesource.com/chromium/src/+/4868d4131b01a3e3520fcb2d65d9...
==========

[commit-bot: I haz the power, 2017-01-10 19:13:46]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/4868d4131b01a3e3520fcb2d65d9...

[mthiesse, 2017-01-12 21:12:13]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/2629683002/ by mthiesse@chromium.org.

The reason for reverting is: Crashes VrShell, can be re-landed after
crbug.com/680150 is fixed..