Don't destroy GpuChannelHost on error in CommandBufferProxyImpl.

This allows CreateTransferBuffer to succeed still when the context
is lost. That means context loss doesn't look like other fatal errors
with shared memory when setting up a context, allowing us to tell if
we should retry making a context. If CreateTransferBuffer fails, then
we should not retry unambiguously.

R=piman@chromium.org

Bug: 772574
Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: Ia4c9f75ed40aa0483fed94552d6689b84efc379b
Reviewed-on: https://chromium-review.googlesource.com/720269
Commit-Queue: danakj <danakj@chromium.org>
Reviewed-by: Antoine Labour <piman@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#509660}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 1c7840a1b003ec921ffd756381c8f1c9b6be3c56
3 files changed