[Merge to M60] X11: Fix use-of-uninitialized-variable

[Merge to M60] X11: Fix use-of-uninitialized-variable

> Some failure paths in XGetVisualInfo don't set |nitems_return| to 0,
> and instead just return nullptr directly.  This CL initializes that
> variable to 0 manually to handle those cases.
>
> BUG=725884
> R=derat@chromium.org
>
> Review-Url: https://codereview.chromium.org/2910753002
> Cr-Commit-Position: refs/heads/master@{#475159}

BUG=725884
TBR=derat@chromium.org
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2912123002
Cr-Commit-Position: refs/branch-heads/3112@{#34}
Cr-Branched-From: b6460e24cf59f429d69de255538d0fc7a425ccf9-refs/heads/master@{#474897}
Committed: https://chromium.googlesource.com/chromium/src/+/00b8c5e102551cff803b67ca26678c0f1b0348e8

[Tom Anderson, 2017-05-30 18:19:21]

The CQ bit was checked by thomasanderson@chromium.org

[Tom Anderson, 2017-05-30 18:19:30]

Description was changed from

==========
[Merge to M60] X11: Fix use-of-uninitialized-variable

> Some failure paths in XGetVisualInfo don't set |nitems_return| to 0,
> and instead just return nullptr directly.  This CL initializes that
> variable to 0 manually to handle those cases.
>
> BUG=725884
> R=derat@chromium.org
>
> Review-Url: https://codereview.chromium.org/2910753002
> Cr-Commit-Position: refs/heads/master@{#475159}

BUG=725884
R=derat@chromium.org
NOTRY=true
NOPRESUBMIT=true
==========

to

==========
[Merge to M60] X11: Fix use-of-uninitialized-variable

> Some failure paths in XGetVisualInfo don't set |nitems_return| to 0,
> and instead just return nullptr directly.  This CL initializes that
> variable to 0 manually to handle those cases.
>
> BUG=725884
> R=derat@chromium.org
>
> Review-Url: https://codereview.chromium.org/2910753002
> Cr-Commit-Position: refs/heads/master@{#475159}

BUG=725884
TBR=derat@chromium.org
NOTRY=true
NOPRESUBMIT=true
==========

[commit-bot: I haz the power, 2017-05-30 18:20:09]

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-05-30 18:20:29]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1496168361933530, "parent_rev":
"00143b99de7f4a57fddfea523c4a392f57a974a3", "commit_rev":
"00b8c5e102551cff803b67ca26678c0f1b0348e8"}

[commit-bot: I haz the power, 2017-05-30 18:20:39]

Description was changed from

==========
[Merge to M60] X11: Fix use-of-uninitialized-variable

> Some failure paths in XGetVisualInfo don't set |nitems_return| to 0,
> and instead just return nullptr directly.  This CL initializes that
> variable to 0 manually to handle those cases.
>
> BUG=725884
> R=derat@chromium.org
>
> Review-Url: https://codereview.chromium.org/2910753002
> Cr-Commit-Position: refs/heads/master@{#475159}

BUG=725884
TBR=derat@chromium.org
NOTRY=true
NOPRESUBMIT=true
==========

to

==========
[Merge to M60] X11: Fix use-of-uninitialized-variable

> Some failure paths in XGetVisualInfo don't set |nitems_return| to 0,
> and instead just return nullptr directly.  This CL initializes that
> variable to 0 manually to handle those cases.
>
> BUG=725884
> R=derat@chromium.org
>
> Review-Url: https://codereview.chromium.org/2910753002
> Cr-Commit-Position: refs/heads/master@{#475159}

BUG=725884
TBR=derat@chromium.org
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2912123002
Cr-Commit-Position: refs/branch-heads/3112@{#34}
Cr-Branched-From:
b6460e24cf59f429d69de255538d0fc7a425ccf9-refs/heads/master@{#474897}
Committed:
https://chromium.googlesource.com/chromium/src/+/00b8c5e102551cff803b67ca2667...
==========

[commit-bot: I haz the power, 2017-05-30 18:20:41]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/00b8c5e102551cff803b67ca2667...