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

Unified Diff: cc/resources/raster_worker_pool.cc

Issue 362073002: cc: Remove all traces of SkPicture cloning. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 6 years, 4 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 | « cc/resources/raster_worker_pool.h ('k') | cc/resources/tile_manager.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/resources/raster_worker_pool.cc
diff --git a/cc/resources/raster_worker_pool.cc b/cc/resources/raster_worker_pool.cc
index d01bdc80cf48cdf8835544f7283b892b9ff3f6eb..255d5af5f71e16c5614a4dc6e85fda3e92e29610 100644
--- a/cc/resources/raster_worker_pool.cc
+++ b/cc/resources/raster_worker_pool.cc
@@ -6,12 +6,10 @@
#include <algorithm>
-#include "base/atomic_sequence_num.h"
#include "base/debug/trace_event_synthetic_delay.h"
#include "base/lazy_instance.h"
#include "base/strings/stringprintf.h"
#include "base/threading/simple_thread.h"
-#include "base/threading/thread_local.h"
#include "cc/base/scoped_ptr_deque.h"
namespace cc {
@@ -50,34 +48,13 @@ class RasterTaskGraphRunner : public TaskGraphRunner,
virtual ~RasterTaskGraphRunner() { NOTREACHED(); }
- size_t GetPictureCloneIndexForCurrentThread() {
- // Use index 0 if called on non-raster thread.
- ThreadLocalState* thread_local_state = current_tls_.Get();
- return thread_local_state ? current_tls_.Get()->picture_clone_index : 0;
- }
-
private:
- struct ThreadLocalState {
- explicit ThreadLocalState(size_t picture_clone_index)
- : picture_clone_index(picture_clone_index) {}
-
- size_t picture_clone_index;
- };
-
// Overridden from base::DelegateSimpleThread::Delegate:
virtual void Run() OVERRIDE {
- // Use picture clone index 0..num_threads.
- int picture_clone_index = picture_clone_index_sequence_.GetNext();
- DCHECK_LE(0, picture_clone_index);
- DCHECK_GT(RasterWorkerPool::GetNumRasterThreads(), picture_clone_index);
- current_tls_.Set(new ThreadLocalState(picture_clone_index));
-
TaskGraphRunner::Run();
}
ScopedPtrDeque<base::DelegateSimpleThread> workers_;
- base::AtomicSequenceNumber picture_clone_index_sequence_;
- base::ThreadLocalPointer<ThreadLocalState> current_tls_;
};
base::LazyInstance<RasterTaskGraphRunner>::Leaky g_task_graph_runner =
@@ -198,11 +175,6 @@ TaskGraphRunner* RasterWorkerPool::GetTaskGraphRunner() {
}
// static
-size_t RasterWorkerPool::GetPictureCloneIndexForCurrentThread() {
- return g_task_graph_runner.Pointer()->GetPictureCloneIndexForCurrentThread();
-}
-
-// static
scoped_refptr<RasterizerTask> RasterWorkerPool::CreateRasterFinishedTask(
base::SequencedTaskRunner* task_runner,
const base::Closure& on_raster_finished_callback) {
« no previous file with comments | « cc/resources/raster_worker_pool.h ('k') | cc/resources/tile_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698