exo: Avoid crash at startup when XDG_RUNTIME_DIR is not set.

exo: Avoid crash at startup when XDG_RUNTIME_DIR is not set.

Without this check we'd crash when creating the watcher and
asking for the server file descriptor from a null server instance.

TBR=sky@chromium.org
BUG=
TEST=chrome --enable-wayland-server (without XDG_RUNTIME_DIR set)

Committed: https://crrev.com/45971325317d8e5a019cfa45adc19265ab6e9a0a
Cr-Commit-Position: refs/heads/master@{#437795}

[reveman, 2016-12-11 04:49:06]

The CQ bit was checked by reveman@chromium.org

[commit-bot: I haz the power, 2016-12-11 04:49:10]

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-12-11 17:42:07]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481431746159760, "parent_rev":
"143788decce5590c3e46e704a3ec17df007e000f", "commit_rev":
"5b24df7735b18f434dd272da9fa5567b65047b38"}

[commit-bot: I haz the power, 2016-12-11 17:42:30]

Description was changed from

==========
exo: Avoid crash at startup when XDG_RUNTIME_DIR is not set.

Without this check we'd crash when creating the watcher and
asking for the server file descriptor from a null server instance.

TBR=sky@chromium.org
BUG=
TEST=chrome --enable-wayland-server (without XDG_RUNTIME_DIR set)
==========

to

==========
exo: Avoid crash at startup when XDG_RUNTIME_DIR is not set.

Without this check we'd crash when creating the watcher and
asking for the server file descriptor from a null server instance.

TBR=sky@chromium.org
BUG=
TEST=chrome --enable-wayland-server (without XDG_RUNTIME_DIR set)

Review-Url: https://codereview.chromium.org/2569633002
==========

[commit-bot: I haz the power, 2016-12-11 17:42:31]

Committed patchset #1 (id:1)

[sky, 2016-12-12 02:18:38]

LGTM

https://codereview.chromium.org/2569633002/diff/1/chrome/browser/chrome_brows...
File chrome/browser/chrome_browser_main_extra_parts_exo.cc (right):

https://codereview.chromium.org/2569633002/diff/1/chrome/browser/chrome_brows...
chrome/browser/chrome_browser_main_extra_parts_exo.cc:134: if (wayland_server_)
Maybe add a comment as to why the null check?

[reveman, 2016-12-12 11:37:36]

https://codereview.chromium.org/2569633002/diff/1/chrome/browser/chrome_brows...
File chrome/browser/chrome_browser_main_extra_parts_exo.cc (right):

https://codereview.chromium.org/2569633002/diff/1/chrome/browser/chrome_brows...
chrome/browser/chrome_browser_main_extra_parts_exo.cc:134: if (wayland_server_)
On 2016/12/12 at 02:18:38, sky wrote:
> Maybe add a comment as to why the null check?

Done: https://codereview.chromium.org/2562363002

[commit-bot: I haz the power, 2016-12-12 15:09:50]

Description was changed from

==========
exo: Avoid crash at startup when XDG_RUNTIME_DIR is not set.

Without this check we'd crash when creating the watcher and
asking for the server file descriptor from a null server instance.

TBR=sky@chromium.org
BUG=
TEST=chrome --enable-wayland-server (without XDG_RUNTIME_DIR set)

Review-Url: https://codereview.chromium.org/2569633002
==========

to

==========
exo: Avoid crash at startup when XDG_RUNTIME_DIR is not set.

Without this check we'd crash when creating the watcher and
asking for the server file descriptor from a null server instance.

TBR=sky@chromium.org
BUG=
TEST=chrome --enable-wayland-server (without XDG_RUNTIME_DIR set)

Committed: https://crrev.com/45971325317d8e5a019cfa45adc19265ab6e9a0a
Cr-Commit-Position: refs/heads/master@{#437795}
==========

[commit-bot: I haz the power, 2016-12-12 15:09:51]

Patchset 1 (id:??) landed as
https://crrev.com/45971325317d8e5a019cfa45adc19265ab6e9a0a
Cr-Commit-Position: refs/heads/master@{#437795}