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

Unified Diff: content/browser/renderer_host/render_widget_host_view_mac.mm

Issue 8595002: Get rid of gpu_messages.h include in chrome by having the RenderWidgetHostView implementation ask... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: fix aura/views build 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 | « content/browser/renderer_host/render_widget_host_view_aura.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/renderer_host/render_widget_host_view_mac.mm
===================================================================
--- content/browser/renderer_host/render_widget_host_view_mac.mm (revision 110645)
+++ content/browser/renderer_host/render_widget_host_view_mac.mm (working copy)
@@ -19,8 +19,6 @@
#include "base/sys_string_conversions.h"
#include "base/utf_string_conversions.h"
#import "content/browser/accessibility/browser_accessibility_cocoa.h"
-#include "content/browser/gpu/gpu_process_host.h"
-#include "content/browser/gpu/gpu_process_host_ui_shim.h"
#include "content/browser/mac/closure_blocks_leopard_compat.h"
#include "content/browser/plugin_process_host.h"
#import "content/browser/renderer_host/accelerated_plugin_view_mac.h"
@@ -935,25 +933,9 @@
// process to the corresponding render widget in the renderer process, while
// the latter identifies the channel from the GpuCommandBufferStub in the
// GPU process to the corresponding command buffer client in the renderer.
+
+ render_widget_host_->AcknowledgeSwapBuffers(route_id, gpu_host_id);
}
-
- // TODO(apatrick): Send the acknowledgement via the UI thread when running in
- // single process or in process GPU mode for now. This is bad from a
- // performance point of view but the plan is to not use AcceleratedSurface at
- // all in these cases.
- if (gpu_host_id == 0) {
- BrowserThread::PostTask(
- BrowserThread::UI,
- FROM_HERE,
- base::Bind(&GpuProcessHostUIShim::SendToGpuHost,
- gpu_host_id,
- new AcceleratedSurfaceMsg_BuffersSwappedACK(route_id)));
- } else {
- GpuProcessHost::SendOnIO(
- gpu_host_id,
- content::CAUSE_FOR_GPU_LAUNCH_NO_LAUNCH,
- new AcceleratedSurfaceMsg_BuffersSwappedACK(route_id));
- }
}
void RenderWidgetHostViewMac::OnAcceleratedCompositingStateChange() {
« no previous file with comments | « content/browser/renderer_host/render_widget_host_view_aura.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698