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

Unified Diff: cc/trees/proxy.h

Issue 1419283002: cc: Split Proxy and TaskRunnerProvider for the LayerTreeHost (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address comments. 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 | « cc/trees/occlusion_tracker_perftest.cc ('k') | cc/trees/proxy.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/trees/proxy.h
diff --git a/cc/trees/proxy.h b/cc/trees/proxy.h
index 8780281384bd3851de2cbb020829302033d41b89..132a4ff62d5fda0d0a0660ee769b13b4e26f235a 100644
--- a/cc/trees/proxy.h
+++ b/cc/trees/proxy.h
@@ -28,11 +28,11 @@ class LayerTreeDebugState;
class OutputSurface;
struct RendererCapabilities;
-// Abstract class responsible for proxying commands from the main-thread side of
-// the compositor over to the compositor implementation.
-class CC_EXPORT Proxy : public TaskRunnerProvider {
+// Abstract interface responsible for proxying commands from the main-thread
+// side of the compositor over to the compositor implementation.
+class CC_EXPORT Proxy {
public:
- ~Proxy() override;
+ virtual ~Proxy() {}
virtual void FinishAllRendering() = 0;
@@ -85,13 +85,6 @@ class CC_EXPORT Proxy : public TaskRunnerProvider {
// Testing hooks
virtual bool MainFrameWillHappenForTesting() = 0;
-
- protected:
- Proxy(scoped_refptr<base::SingleThreadTaskRunner> main_task_runner,
- scoped_refptr<base::SingleThreadTaskRunner> impl_task_runner);
-
- private:
- DISALLOW_COPY_AND_ASSIGN(Proxy);
};
} // namespace cc
« no previous file with comments | « cc/trees/occlusion_tracker_perftest.cc ('k') | cc/trees/proxy.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698