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

Unified Diff: content/browser/gpu/gpu_process_host_ui_shim.cc

Issue 8758017: aura: fix fullscreen pepper flash (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 1 month 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 | content/renderer/render_widget_fullscreen_pepper.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/gpu/gpu_process_host_ui_shim.cc
diff --git a/content/browser/gpu/gpu_process_host_ui_shim.cc b/content/browser/gpu/gpu_process_host_ui_shim.cc
index 5c992e2e9f5ed66d5d05beeb19d35406c7f87c62..fab719d5fc647a66acbdbcb209efd4945621dd6e 100644
--- a/content/browser/gpu/gpu_process_host_ui_shim.cc
+++ b/content/browser/gpu/gpu_process_host_ui_shim.cc
@@ -81,6 +81,18 @@ class ScopedSendOnIOThread {
bool cancelled_;
};
+RenderWidgetHostView* GetRenderWidgetHostViewFromID(int render_process_id,
+ int render_widget_id) {
+ content::RenderProcessHost* process =
+ content::RenderProcessHost::FromID(render_process_id);
+ if (!process)
+ return NULL;
+
+ RenderWidgetHost* host = static_cast<RenderWidgetHost*>(
+ process->GetListenerByID(render_widget_id));
+ return host ? host->view() : NULL;
+}
+
} // namespace
RouteToGpuProcessHostUIShimTask::RouteToGpuProcessHostUIShimTask(
@@ -230,11 +242,8 @@ void GpuProcessHostUIShim::OnResizeView(int32 renderer_id,
host_id_,
new AcceleratedSurfaceMsg_ResizeViewACK(route_id));
- RenderViewHost* host = RenderViewHost::FromID(renderer_id, render_view_id);
- if (!host)
- return;
-
- RenderWidgetHostView* view = host->view();
+ RenderWidgetHostView* view = GetRenderWidgetHostViewFromID(renderer_id,
+ render_view_id);
if (!view)
return;
@@ -277,12 +286,8 @@ void GpuProcessHostUIShim::OnAcceleratedSurfaceNew(
params.surface_id,
TransportDIB::DefaultHandleValue()));
- RenderViewHost* host = RenderViewHost::FromID(params.renderer_id,
- params.render_view_id);
- if (!host)
- return;
-
- RenderWidgetHostView* view = host->view();
+ RenderWidgetHostView* view = GetRenderWidgetHostViewFromID(
+ params.renderer_id, params.render_view_id);
if (!view)
return;
@@ -336,12 +341,8 @@ void GpuProcessHostUIShim::OnAcceleratedSurfaceBuffersSwapped(
host_id_,
new AcceleratedSurfaceMsg_BuffersSwappedACK(params.route_id));
- RenderViewHost* host = RenderViewHost::FromID(params.renderer_id,
- params.render_view_id);
- if (!host)
- return;
-
- RenderWidgetHostView* view = host->view();
+ RenderWidgetHostView* view = GetRenderWidgetHostViewFromID(
+ params.renderer_id, params.render_view_id);
if (!view)
return;
@@ -360,12 +361,8 @@ void GpuProcessHostUIShim::OnAcceleratedSurfacePostSubBuffer(
host_id_,
new AcceleratedSurfaceMsg_PostSubBufferACK(params.route_id));
- RenderViewHost* host = RenderViewHost::FromID(params.renderer_id,
- params.render_view_id);
- if (!host)
- return;
-
- RenderWidgetHostView* view = host->view();
+ RenderWidgetHostView* view = GetRenderWidgetHostViewFromID(
+ params.renderer_id, params.render_view_id);
if (!view)
return;
@@ -379,11 +376,8 @@ void GpuProcessHostUIShim::OnAcceleratedSurfacePostSubBuffer(
void GpuProcessHostUIShim::OnAcceleratedSurfaceRelease(
const GpuHostMsg_AcceleratedSurfaceRelease_Params& params) {
- RenderViewHost* host = RenderViewHost::FromID(params.renderer_id,
- params.render_view_id);
- if (!host)
- return;
- RenderWidgetHostView* view = host->view();
+ RenderWidgetHostView* view = GetRenderWidgetHostViewFromID(
+ params.renderer_id, params.render_view_id);
if (!view)
return;
view->AcceleratedSurfaceRelease(params.identifier);
« no previous file with comments | « no previous file | content/renderer/render_widget_fullscreen_pepper.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698