 Chromium Code Reviews
 Chromium Code Reviews Issue 3010054:
  Mac: Well-behaved accelerated plugins, preparation  (Closed)
    
  
    Issue 3010054:
  Mac: Well-behaved accelerated plugins, preparation  (Closed) 
  | Index: chrome/browser/renderer_host/accelerated_surface_container_manager_mac.h | 
| diff --git a/chrome/browser/renderer_host/accelerated_surface_container_manager_mac.h b/chrome/browser/renderer_host/accelerated_surface_container_manager_mac.h | 
| index 7b49ccc0e8a38a2c200ff42c2895149cd67bf8bb..cdd2f6b883e575f1dec891475d0e89c16e8debc5 100644 | 
| --- a/chrome/browser/renderer_host/accelerated_surface_container_manager_mac.h | 
| +++ b/chrome/browser/renderer_host/accelerated_surface_container_manager_mac.h | 
| @@ -37,7 +37,7 @@ class AcceleratedSurfaceContainerManagerMac { | 
| // Indicates whether a "root" PluginWindowHandle has been allocated, | 
| // which means that we are using accelerated compositing and should | 
| // short-circuit the normal drawing process. | 
| 
stuartmorgan
2010/08/11 16:40:44
This comment pretty clearly doesn't match the func
 
Ken Russell (switch to Gerrit)
2010/08/14 02:12:11
Will fix.
 | 
| - bool HasRootContainer(); | 
| + bool IsRootContainer(gfx::PluginWindowHandle id); | 
| // Sets the size and backing store of the plugin instance. There are two | 
| // versions: the IOSurface version is used on systems where the IOSurface | 
| @@ -59,6 +59,7 @@ class AcceleratedSurfaceContainerManagerMac { | 
| // Draws all of the managed plugin containers into the given OpenGL | 
| // context, which must already be current. | 
| 
stuartmorgan
2010/08/11 16:40:44
Again, the comment is now completely wrong.
 
Ken Russell (switch to Gerrit)
2010/08/14 02:12:11
Will fix.
 | 
| void Draw(CGLContextObj context, | 
| + gfx::PluginWindowHandle id, | 
| bool draw_root_container); | 
| // Causes the next Draw call on each container to trigger a texture upload. |