Delete GpuVideoService and move GpuVideoDecodeAccelerator ownership to stubs

content/common/gpu/gpu_channel.h

Index: content/common/gpu/gpu_channel.h
diff --git a/content/common/gpu/gpu_channel.h b/content/common/gpu/gpu_channel.h
index 8477b9204494545c30f0e72241f5e224a21cfdd5..e137aa8b2069d403ad36f6ddf9f1605ea6faf16f 100644
--- a/content/common/gpu/gpu_channel.h
+++ b/content/common/gpu/gpu_channel.h
@@ -178,6 +178,8 @@ class GpuChannel : public IPC::Channel::Listener,
   typedef IDMap<GpuCommandBufferStub, IDMapOwnPointer> StubMap;
   StubMap stubs_;
 
+  std::map<int32, GpuCommandBufferStub*> stubs_by_video_decoder_id_;

[Ami GONE FROM CHROMIUM, 2011/06/30 22:20:39]

Does it work to use an IDMap<GpuCommandBufferStub, IDMapExternalPointer> instead
here to make explicit the relationship between this and stubs_, and get the
side-effect of single-thread-checking?

[vrk (LEFT CHROMIUM), 2011/07/01 18:12:14]

No need for this map anymore! Create/Destroy messages directly routed to stub.

+
   typedef IDMap<GpuSurfaceStub, IDMapOwnPointer> SurfaceMap;
   SurfaceMap surfaces_;