Ensure that NativeViewport destructs correctly on Linux.

Ensure that NativeViewport destructs correctly on Linux.

First, PlatformEventSource is a singleton so it shouldn't be created by PlatformViewportX11. Once we have multiple windows, this will be very clear. Until then, we also have UAFs on destruction of PlatformViewportX11 because PlatformEventSource could be lower in the stack calling a PlatformViewportX11 method which cause both of these objects to be deleted.

Second, ensure that child windows of the PlatformViewport aren't used after the parent window is destructed, otherwise we get X errors. This is done by ensuring that NativeViewportImpl, which owns the PlatformViewport, destructs OnscreenContextProvider first. OnscreenContextProvider will in turn destroy all windows in the CommandBufferDriver that it created.

Lastly, make NativeViewportImpl destroy itself when its platform window is closed (i.e. because the user closed the window). The old implementation in that method didn't do anything.

This is split off https://codereview.chromium.org/1139673003

BUG=484234
Committed: https://crrev.com/6345ee9d16605252a476a845cd40cb101dd5449d
Cr-Commit-Position: refs/heads/master@{#330394}

[jam, 2015-05-18 18:16:30]

jam@chromium.org changed reviewers:
+ erg@chromium.org

[Elliot Glaysher, 2015-05-18 18:20:43]

rubber stamp lgtm, since I've already reviewed this in the other patch.

[jam, 2015-05-18 18:21:00]

The CQ bit was checked by jam@chromium.org

[commit-bot: I haz the power, 2015-05-18 18:22:12]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1130353008/1

[commit-bot: I haz the power, 2015-05-18 20:18:35]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-05-18 22:41:43]

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