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

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

Issue 3534010: Mac: Fix crash when destroying RWHVMacs that use accelerated drawing. (Closed)
Patch Set: '' Created 10 years, 2 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/render_widget_host_view_mac.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/renderer_host/render_widget_host_view_mac.h
diff --git a/chrome/browser/renderer_host/render_widget_host_view_mac.h b/chrome/browser/renderer_host/render_widget_host_view_mac.h
index 41b43167bdbbe88be437865f111daab21838b76c..aeac96f60d8d78fc2dee76de2be8c57f0c78c9c5 100644
--- a/chrome/browser/renderer_host/render_widget_host_view_mac.h
+++ b/chrome/browser/renderer_host/render_widget_host_view_mac.h
@@ -224,6 +224,13 @@ class RenderWidgetHostViewMac : public RenderWidgetHostView {
virtual gfx::PluginWindowHandle AllocateFakePluginWindowHandle(bool opaque,
bool root);
virtual void DestroyFakePluginWindowHandle(gfx::PluginWindowHandle window);
+
+ // Helper to do the actual cleanup after a plugin handle has been destroyed.
+ // Required because DestroyFakePluginWindowHandle() isn't always called for
+ // all handles (it's e.g. not called on navigation, when the RWHVMac gets
+ // destroyed anyway).
+ void DeallocFakePluginWindowHandle(gfx::PluginWindowHandle window);
+
virtual void AcceleratedSurfaceSetIOSurface(gfx::PluginWindowHandle window,
int32 width,
int32 height,
« no previous file with comments | « no previous file | chrome/browser/renderer_host/render_widget_host_view_mac.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698