Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1013)

Unified Diff: chrome/browser/renderer_host/accelerated_surface_container_mac.h

Issue 1110011: Fixed missing code in EnqueueTextureForDeletion.... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 10 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/browser/renderer_host/accelerated_surface_container_mac.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/renderer_host/accelerated_surface_container_mac.h
===================================================================
--- chrome/browser/renderer_host/accelerated_surface_container_mac.h (revision 42374)
+++ chrome/browser/renderer_host/accelerated_surface_container_mac.h (working copy)
@@ -42,7 +42,8 @@
class AcceleratedSurfaceContainerMac {
public:
- AcceleratedSurfaceContainerMac();
+ AcceleratedSurfaceContainerMac(
+ AcceleratedSurfaceContainerManagerMac* manager);
virtual ~AcceleratedSurfaceContainerMac();
// Sets the backing store and size of this accelerated surface container.
@@ -51,12 +52,10 @@
// used on Mac OS X 10.5 and earlier.
void SetSizeAndIOSurface(int32 width,
int32 height,
- uint64 io_surface_identifier,
- AcceleratedSurfaceContainerManagerMac* manager);
+ uint64 io_surface_identifier);
void SetSizeAndTransportDIB(int32 width,
int32 height,
- TransportDIB::Handle transport_dib,
- AcceleratedSurfaceContainerManagerMac* manager);
+ TransportDIB::Handle transport_dib);
// Tells the accelerated surface container that it has moved relative to the
// origin of the window, for example because of a scroll event.
@@ -71,18 +70,14 @@
// time the drawing context has changed.
void ForceTextureReload() { texture_needs_upload_ = true; }
- // Enqueue our texture for later deletion. Call this before deleting
- // this object.
- void EnqueueTextureForDeletion(
- AcceleratedSurfaceContainerManagerMac* manager);
+ private:
+ // The manager of this accelerated surface container.
+ AcceleratedSurfaceContainerManagerMac* manager_;
- private:
// The x and y coordinates of the plugin window on the web page.
int x_;
int y_;
- void ReleaseIOSurface();
-
// The IOSurfaceRef, if any, that has been handed from the GPU
// plugin process back to the browser process for drawing.
// This is held as a CFTypeRef because we can't refer to the
@@ -111,6 +106,12 @@
// True if we need to upload the texture again during the next draw.
bool texture_needs_upload_;
+ // Releases the IOSurface reference, if any, retained by this object.
+ void ReleaseIOSurface();
+
+ // Enqueue our texture for later deletion.
+ void EnqueueTextureForDeletion();
+
DISALLOW_COPY_AND_ASSIGN(AcceleratedSurfaceContainerMac);
};
« no previous file with comments | « no previous file | chrome/browser/renderer_host/accelerated_surface_container_mac.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698