Android: Don't try to access the native Screen instance when loading resources.

Android: Don't try to access the native Screen instance when loading resources.

On Aura Android the Screen instance may not be set yet when
ResourceBundle::InitSharedInstance is executed. Instead of using the Screen
instance to determine the scale factor, just use the available static methods
in gfx::Display and gfx::DeviceDisplayInfo.

BUG=507792
Committed: https://crrev.com/264cf401157f9b5ef4bad72020a001c1d0bb6d45
Cr-Commit-Position: refs/heads/master@{#363042}

[mfomitchev, 2015-11-24 22:06:06]

mfomitchev@chromium.org changed reviewers:
+ sadrul@chromium.org, sievers@chromium.org

[mfomitchev, 2015-11-24 22:10:06]

Note that we are using screen_aura.cc, not screen_android.cc on Android Aura. We
wouldn't have this problem with screen_android.cc, however if we'd use
screen_android.cc, we'd loose the ability to set the screen instance, meaning we
also wouldn't be able to use aura::Env::GetInstance()->last_mouse_location() in
GetCursorScreenPoint() for example.

[no sievers, 2015-11-24 23:09:39]

lgtm

[sadrul, 2015-11-25 21:00:34]

stamp lgtm

[mfomitchev, 2015-11-25 22:05:32]

The CQ bit was checked by mfomitchev@chromium.org

[commit-bot: I haz the power, 2015-11-25 22:09:25]

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

[commit-bot: I haz the power, 2015-11-25 22:09:26]

This CL has an open dependency (Issue 1455793005 Patch 80001). Please resolve
the dependency and try again.

[mfomitchev, 2015-12-01 22:07:56]

The CQ bit was checked by mfomitchev@chromium.org

[commit-bot: I haz the power, 2015-12-01 22:09:04]

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

[commit-bot: I haz the power, 2015-12-01 22:09:04]

This CL has an open dependency (Issue 1455793005 Patch 80001). Please resolve
the dependency and try again.

[mfomitchev, 2015-12-03 19:05:19]

The CQ bit was checked by mfomitchev@chromium.org

[mfomitchev, 2015-12-03 19:05:19]

The patchset sent to the CQ was uploaded after l-g-t-m from sadrul@chromium.org,
sievers@chromium.org
Link to the patchset: https://codereview.chromium.org/1474873002/#ps20001
(title: "Trying to get rid of the dependency...")

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

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

[commit-bot: I haz the power, 2015-12-03 20:14:31]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-12-03 20:15:33]

Description was changed from

==========
Android: Don't try to access the native Screen instance when loading resources.

On Aura Android the Screen instance may not be set yet when
ResourceBundle::InitSharedInstance is executed. Instead of using the Screen
instance to determine the scale factor, just use the available static methods
in gfx::Display and gfx::DeviceDisplayInfo.

BUG=507792
==========

to

==========
Android: Don't try to access the native Screen instance when loading resources.

On Aura Android the Screen instance may not be set yet when
ResourceBundle::InitSharedInstance is executed. Instead of using the Screen
instance to determine the scale factor, just use the available static methods
in gfx::Display and gfx::DeviceDisplayInfo.

BUG=507792

Committed: https://crrev.com/264cf401157f9b5ef4bad72020a001c1d0bb6d45
Cr-Commit-Position: refs/heads/master@{#363042}
==========

[commit-bot: I haz the power, 2015-12-03 20:15:34]

Patchset 2 (id:??) landed as
https://crrev.com/264cf401157f9b5ef4bad72020a001c1d0bb6d45
Cr-Commit-Position: refs/heads/master@{#363042}