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

Unified Diff: ui/compositor/compositor_vsync_manager.h

Issue 2511273002: Decouple BrowserCompositorOutputSurface from BeginFrameSource. (Closed)
Patch Set: Another attempt to fix the crash Created 4 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
Index: ui/compositor/compositor_vsync_manager.h
diff --git a/ui/compositor/compositor_vsync_manager.h b/ui/compositor/compositor_vsync_manager.h
index 54bcc1a1d295f4d7bc7c68dd9bb1511b8180b78c..4d2a8244d7e1f70ff6e92ab42bc4e537b45f8567 100644
--- a/ui/compositor/compositor_vsync_manager.h
+++ b/ui/compositor/compositor_vsync_manager.h
@@ -10,6 +10,7 @@
#include "base/observer_list.h"
#include "base/synchronization/lock.h"
#include "base/time/time.h"
+#include "cc/scheduler/vsync_observer.h"
#include "ui/compositor/compositor_export.h"
namespace ui {
@@ -20,12 +21,6 @@ namespace ui {
class COMPOSITOR_EXPORT CompositorVSyncManager
: public base::RefCounted<CompositorVSyncManager> {
public:
- class Observer {
- public:
- virtual void OnUpdateVSyncParameters(base::TimeTicks timebase,
- base::TimeDelta interval) = 0;
- };
-
CompositorVSyncManager();
// The "authoritative" vsync interval, if provided, will override |interval|
@@ -42,8 +37,8 @@ class COMPOSITOR_EXPORT CompositorVSyncManager
void UpdateVSyncParameters(base::TimeTicks timebase,
base::TimeDelta interval);
- void AddObserver(Observer* observer);
- void RemoveObserver(Observer* observer);
+ void AddObserver(cc::VSyncObserver* observer);
+ void RemoveObserver(cc::VSyncObserver* observer);
private:
friend class base::RefCounted<CompositorVSyncManager>;
@@ -52,7 +47,7 @@ class COMPOSITOR_EXPORT CompositorVSyncManager
void NotifyObservers(base::TimeTicks timebase, base::TimeDelta interval);
- base::ObserverList<Observer> observer_list_;
+ base::ObserverList<cc::VSyncObserver> observer_list_;
base::TimeTicks last_timebase_;
base::TimeDelta last_interval_;

Powered by Google App Engine
This is Rietveld 408576698