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

Unified Diff: content/renderer/render_widget.cc

Issue 15851006: Move synchronous compositor into content/browser (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 7 years, 7 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 | « content/renderer/render_widget.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/render_widget.cc
diff --git a/content/renderer/render_widget.cc b/content/renderer/render_widget.cc
index d5af8af1f85d6e3c43f0ba519a75d8081a89273a..81dc9171962004c25f860bca3d4d25d7d8510aa2 100644
--- a/content/renderer/render_widget.cc
+++ b/content/renderer/render_widget.cc
@@ -62,8 +62,7 @@
#include "webkit/renderer/compositor_bindings/web_rendering_stats_impl.h"
#if defined(OS_ANDROID)
-#include "content/renderer/android/synchronous_compositor_impl.h"
-#include "content/renderer/android/synchronous_compositor_output_surface.h"
+#include "content/renderer/android/synchronous_compositor_factory.h"
#endif
#if defined(OS_POSIX)
@@ -306,12 +305,10 @@ bool RenderWidget::AllowPartialSwap() const {
bool RenderWidget::UsingSynchronousRendererCompositor() const {
#if defined(OS_ANDROID)
- if (CommandLine::ForCurrentProcess()->HasSwitch(
- switches::kEnableSynchronousRendererCompositor)) {
- return true;
- }
-#endif
+ return SynchronousCompositorFactory::GetInstance() != NULL;
+#else
return false;
+#endif
}
bool RenderWidget::OnMessageReceived(const IPC::Message& message) {
@@ -586,8 +583,9 @@ scoped_ptr<cc::OutputSurface> RenderWidget::CreateOutputSurface() {
const CommandLine& command_line = *CommandLine::ForCurrentProcess();
#if defined(OS_ANDROID)
- if (GetSynchronousCompositor()) {
- return GetSynchronousCompositor()->CreateOutputSurface();
+ if (SynchronousCompositorFactory* factory =
+ SynchronousCompositorFactory::GetInstance()) {
+ return factory->CreateOutputSurface(routing_id());
}
#endif
@@ -630,17 +628,6 @@ scoped_ptr<cc::OutputSurface> RenderWidget::CreateOutputSurface() {
new CompositorOutputSurface(routing_id(), context, NULL));
}
-#if defined(OS_ANDROID)
-SynchronousCompositorImpl* RenderWidget::GetSynchronousCompositor() {
- const CommandLine& command_line = *CommandLine::ForCurrentProcess();
- if (!synchronous_compositor_ &&
- command_line.HasSwitch(switches::kEnableSynchronousRendererCompositor)) {
- synchronous_compositor_.reset(new SynchronousCompositorImpl(routing_id()));
- }
- return synchronous_compositor_.get();
-}
-#endif
-
void RenderWidget::OnViewContextSwapBuffersAborted() {
TRACE_EVENT0("renderer", "RenderWidget::OnSwapBuffersAborted");
while (!updates_pending_swap_.empty()) {
« no previous file with comments | « content/renderer/render_widget.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698