Delete GpuVideoService and move GpuVideoDecodeAccelerator ownership to stubs

content/common/gpu/gpu_channel.h

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_COMMON_GPU_GPU_CHANNEL_H_
 #define CONTENT_COMMON_GPU_GPU_CHANNEL_H_
 #pragma once
 
 #include <set>
 #include <string>
@@ skipping 160 matching lines @@
   MessageRouter router_;
 
   // The share group that all contexts associated with a particular renderer
   // process use.
   scoped_refptr<gfx::GLShareGroup> share_group_;
 
 #if defined(ENABLE_GPU)
   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_;
 
   std::set<int32> latched_routes_;
 #endif  // defined (ENABLE_GPU)
 
   // A collection of transport textures created.
   typedef IDMap<TransportTexture, IDMapOwnPointer> TransportTextureMap;
   TransportTextureMap transport_textures_;
 
   bool log_messages_;  // True if we should log sent and received messages.
   gpu::gles2::DisallowedExtensions disallowed_extensions_;
   GpuWatchdog* watchdog_;
 
   DISALLOW_COPY_AND_ASSIGN(GpuChannel);
 };
 
 #endif  // CONTENT_COMMON_GPU_GPU_CHANNEL_H_