Remove dependency of Blimp on X11/Xlib.h header to avoid namespace polution

Remove dependency of Blimp on X11/Xlib.h header to avoid namespace polution
by #define-s

Xlib.h has #define Status int which conflicts with Status class in protobuf

BUG=597321
Committed: https://crrev.com/f4a91b9728184d060805126d604cb7c8733624ee
Cr-Commit-Position: refs/heads/master@{#382989}

[xyzzyz, 2016-03-23 23:32:02]

xyzzyz@chromium.org changed reviewers:
+ wez@chromium.org

[Wez, 2016-03-23 23:33:21]

lgtm

https://codereview.chromium.org/1826073002/diff/1/blimp/client/app/linux/blim...
File blimp/client/app/linux/blimp_main.cc (right):

https://codereview.chromium.org/1826073002/diff/1/blimp/client/app/linux/blim...
blimp/client/app/linux/blimp_main.cc:29: gfx::InitializeThreadedX11();
nit: CHECK(...) so that we'll crash here if X11 fails to initialize, or connect?

[xyzzyz, 2016-03-23 23:49:50]

On 2016/03/23 23:33:21, Wez wrote:
> lgtm
> 
>
https://codereview.chromium.org/1826073002/diff/1/blimp/client/app/linux/blim...
> File blimp/client/app/linux/blimp_main.cc (right):
> 
>
https://codereview.chromium.org/1826073002/diff/1/blimp/client/app/linux/blim...
> blimp/client/app/linux/blimp_main.cc:29: gfx::InitializeThreadedX11();
> nit: CHECK(...) so that we'll crash here if X11 fails to initialize, or
connect?

Done

[xyzzyz, 2016-03-23 23:49:55]

The CQ bit was checked by xyzzyz@chromium.org

[xyzzyz, 2016-03-23 23:49:55]

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

[commit-bot: I haz the power, 2016-03-23 23:50:06]

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

[commit-bot: I haz the power, 2016-03-24 00:32:15]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-24 00:38:50]

Description was changed from

==========
Remove dependency of Blimp on X11/Xlib.h header to avoid namespace polution
by #define-s

Xlib.h has #define Status int which conflicts with Status class in protobuf

BUG=597321
==========

to

==========
Remove dependency of Blimp on X11/Xlib.h header to avoid namespace polution
by #define-s

Xlib.h has #define Status int which conflicts with Status class in protobuf

BUG=597321

Committed: https://crrev.com/f4a91b9728184d060805126d604cb7c8733624ee
Cr-Commit-Position: refs/heads/master@{#382989}
==========

[commit-bot: I haz the power, 2016-03-24 00:38:51]

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