Fix GPU service and display initialization race.

Fix GPU service and display initialization race.

When running mustash on device with Ozone, display initialization was
happening before GPU service initialization was finished. The DRM thread
wasn't ready when DrmNativeDisplayDelegate went to use it.

The problem was made much worse by crrev.com/2484223006 when display
initialization was moved so it started earlier. With this patch, we
start display initialization when it's known to be safe.

BUG=659804
Committed: https://crrev.com/3e7dcd6974ae2895ba6cc2f38141c90338d20fc7
Cr-Commit-Position: refs/heads/master@{#432388}

[kylechar, 2016-11-15 19:22:32]

kylechar@chromium.org changed reviewers:
+ sky@chromium.org

[sky, 2016-11-15 19:53:05]

LGTM

https://codereview.chromium.org/2498073003/diff/1/services/ui/service.cc
File services/ui/service.cc (right):

https://codereview.chromium.org/2498073003/diff/1/services/ui/service.cc#newc...
services/ui/service.cc:231: 
remove empty line.

[kylechar, 2016-11-15 20:00:12]

Thanks sky!

https://codereview.chromium.org/2498073003/diff/1/services/ui/service.cc
File services/ui/service.cc (right):

https://codereview.chromium.org/2498073003/diff/1/services/ui/service.cc#newc...
services/ui/service.cc:231: 
On 2016/11/15 19:53:05, sky wrote:
> remove empty line.

Done.

[kylechar, 2016-11-15 20:00:19]

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

[commit-bot: I haz the power, 2016-11-15 20:01:19]

Dry run: 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, 2016-11-15 20:13:15]

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

[commit-bot: I haz the power, 2016-11-15 20:13:16]

Dry run: Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)

[kylechar, 2016-11-15 20:21:49]

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

[commit-bot: I haz the power, 2016-11-15 20:22:42]

Dry run: 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, 2016-11-15 20:41:29]

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

[commit-bot: I haz the power, 2016-11-15 20:41:30]

Dry run: Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)

[kylechar, 2016-11-16 03:48:04]

The CQ bit was checked by kylechar@chromium.org

[kylechar, 2016-11-16 03:48:04]

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

[commit-bot: I haz the power, 2016-11-16 03:48:35]

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, 2016-11-16 06:24:51]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-11-16 06:29:47]

Description was changed from

==========
Fix GPU service and display initialization race.

When running mustash on device with Ozone, display initialization was
happening before GPU service initialization was finished. The DRM thread
wasn't ready when DrmNativeDisplayDelegate went to use it.

The problem was made much worse by crrev.com/2484223006 when display
initialization was moved so it started earlier. With this patch, we
start display initialization when it's known to be safe.

BUG=659804
==========

to

==========
Fix GPU service and display initialization race.

When running mustash on device with Ozone, display initialization was
happening before GPU service initialization was finished. The DRM thread
wasn't ready when DrmNativeDisplayDelegate went to use it.

The problem was made much worse by crrev.com/2484223006 when display
initialization was moved so it started earlier. With this patch, we
start display initialization when it's known to be safe.

BUG=659804

Committed: https://crrev.com/3e7dcd6974ae2895ba6cc2f38141c90338d20fc7
Cr-Commit-Position: refs/heads/master@{#432388}
==========

[commit-bot: I haz the power, 2016-11-16 06:29:47]

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