gpu: Use a single container for GLInProcessContext share group

gpu: Use a single container for GLInProcessContext share group

Instead of having two global containers for GLInProcessContext
and InProcessCommandBuffer, use a single one for the former
and pass in the service-side share group while we are holding the
lock anyways.

This makes it simpler to support multiple service threads.

BUG=332146
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=250765

[no sievers, 2014-02-12 01:24:32]

https://codereview.chromium.org/158133004/diff/1/gpu/command_buffer/service/i...
File gpu/command_buffer/service/in_process_command_buffer.cc (left):

https://codereview.chromium.org/158133004/diff/1/gpu/command_buffer/service/i...
gpu/command_buffer/service/in_process_command_buffer.cc:546:
(*it)->context_lost_ = true;
glLoseContextCHROMIUM() already propagates the lost context from the decoder to
other decoders in the ContextGroup.

Maybe we eventually want to unify the 'loses all contexts on context lost'
behavior that is implemented in GpuCommandBufferStub and the channel. But I
assume as far as this is concerned it's good enough to make tests happy for now?

[piman, 2014-02-12 01:41:12]

lgtm

[no sievers, 2014-02-12 17:15:33]

The CQ bit was checked by sievers@chromium.org

[commit-bot: I haz the power, 2014-02-12 17:15:56]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sievers@chromium.org/158133004/1

[commit-bot: I haz the power, 2014-02-12 18:51:47]

Change committed as 250765