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

Unified Diff: cc/resources/rasterizer_delegate.h

Issue 228173002: cc: Separate RasterWorkerPool interface from implementation details. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Name interface Rasterizer instead and keep RasterWorkerPool name for the implementation Created 6 years, 8 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
Index: cc/resources/rasterizer_delegate.h
diff --git a/cc/resources/rasterizer_delegate.h b/cc/resources/rasterizer_delegate.h
new file mode 100644
index 0000000000000000000000000000000000000000..5623ad89a2b1e1f93a1d4e366df9a9108f1c770c
--- /dev/null
+++ b/cc/resources/rasterizer_delegate.h
@@ -0,0 +1,46 @@
+// Copyright 2014 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CC_RESOURCES_RASTERIZER_DELEGATE_H_
+#define CC_RESOURCES_RASTERIZER_DELEGATE_H_
+
+#include <vector>
+
+#include "cc/resources/rasterizer.h"
+
+namespace cc {
+
+class RasterizerDelegate : public RasterizerClient {
vmpstr 2014/04/10 18:21:18 Does this implement all the functions for it to qu
reveman 2014/04/10 19:38:15 Not yet. It sort of does after https://codereview.
vmpstr 2014/04/10 19:40:53 Ah good point.
+ public:
+ virtual ~RasterizerDelegate();
+
+ static scoped_ptr<RasterizerDelegate> Create(RasterizerClient* client,
+ Rasterizer** rasterizers,
+ size_t num_rasterizers);
+
+ void SetClient(RasterizerClient* client);
+ void Shutdown();
+ void ScheduleTasks(RasterTaskQueue* raster_queue);
+ void CheckForCompletedTasks();
+
+ // Overriden from RasterizerClient:
+ virtual bool ShouldForceTasksRequiredForActivationToComplete() const OVERRIDE;
+ virtual void DidFinishRunningTasks() OVERRIDE;
+ virtual void DidFinishRunningTasksRequiredForActivation() OVERRIDE;
+
+ private:
+ RasterizerDelegate(RasterizerClient* client,
+ Rasterizer** rasterizers,
+ size_t num_rasterizers);
+
+ RasterizerClient* client_;
+ typedef std::vector<Rasterizer*> RasterizerVector;
+ RasterizerVector rasterizers_;
+ size_t did_finish_running_tasks_pending_count_;
+ size_t did_finish_running_tasks_required_for_activation_pending_count_;
+};
+
+} // namespace cc
+
+#endif // CC_RESOURCES_RASTERIZER_DELEGATE_H_

Powered by Google App Engine
This is Rietveld 408576698