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

Unified Diff: content/browser/renderer_host/compositor_impl_android.cc

Issue 1449133002: TaskGraphRunner refactor (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: feedback Created 5 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/compositor/gpu_process_transport_factory.cc ('k') | content/renderer/raster_worker_pool.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/renderer_host/compositor_impl_android.cc
diff --git a/content/browser/renderer_host/compositor_impl_android.cc b/content/browser/renderer_host/compositor_impl_android.cc
index 1e2abf706c01e3941398fefad8966ef5b7d93d60..90d3241cd838b483274e61161a3bb237b8b33e18 100644
--- a/content/browser/renderer_host/compositor_impl_android.cc
+++ b/content/browser/renderer_host/compositor_impl_android.cc
@@ -30,7 +30,7 @@
#include "cc/output/context_provider.h"
#include "cc/output/output_surface.h"
#include "cc/output/output_surface_client.h"
-#include "cc/raster/task_graph_runner.h"
+#include "cc/raster/single_thread_task_graph_runner.h"
#include "cc/scheduler/begin_frame_source.h"
#include "cc/surfaces/onscreen_display_client.h"
#include "cc/surfaces/surface_display_output_surface.h"
@@ -157,37 +157,24 @@ bool g_use_surface_manager = false;
base::LazyInstance<cc::SurfaceManager> g_surface_manager =
LAZY_INSTANCE_INITIALIZER;
-
int g_surface_id_namespace = 0;
-class SingleThreadTaskGraphRunner
- : public cc::TaskGraphRunner,
- public base::DelegateSimpleThread::Delegate {
+class SingleThreadTaskGraphRunner : public cc::SingleThreadTaskGraphRunner {
public:
- SingleThreadTaskGraphRunner()
- : worker_thread_(
- this,
- "CompositorTileWorker1",
- base::SimpleThread::Options(base::ThreadPriority::BACKGROUND)) {
- worker_thread_.Start();
+ SingleThreadTaskGraphRunner() {
+ Start("CompositorTileWorker1",
+ base::SimpleThread::Options(base::ThreadPriority::BACKGROUND));
}
~SingleThreadTaskGraphRunner() override {
Shutdown();
- worker_thread_.Join();
}
-
- private:
- // Overridden from base::DelegateSimpleThread::Delegate:
- void Run() override { cc::TaskGraphRunner::Run(); }
-
- base::DelegateSimpleThread worker_thread_;
};
base::LazyInstance<SingleThreadTaskGraphRunner> g_task_graph_runner =
LAZY_INSTANCE_INITIALIZER;
-} // anonymous namespace
+} // anonymous namespace
// static
Compositor* Compositor::Create(CompositorClient* client,
« no previous file with comments | « content/browser/compositor/gpu_process_transport_factory.cc ('k') | content/renderer/raster_worker_pool.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698