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

Unified Diff: components/mus/surfaces/top_level_display_client.h

Issue 1821863002: Hook up ui::Compositor to Display's BeginFrameSource (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase; refactor mus output surface Created 4 years, 9 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: components/mus/surfaces/top_level_display_client.h
diff --git a/components/mus/surfaces/top_level_display_client.h b/components/mus/surfaces/top_level_display_client.h
index de5ce2ac7a9c97343e4aff5f5b7ae10dd304c12b..350eeaf25c2b8da39357ca15d0d9f88ccc41bf91 100644
--- a/components/mus/surfaces/top_level_display_client.h
+++ b/components/mus/surfaces/top_level_display_client.h
@@ -14,8 +14,6 @@
#include "cc/surfaces/surface_factory_client.h"
#include "components/mus/gles2/gpu_state.h"
#include "components/mus/public/interfaces/window_manager.mojom.h"
-#include "components/mus/surfaces/surfaces_context_provider.h"
-#include "components/mus/surfaces/surfaces_context_provider_delegate.h"
#include "components/mus/surfaces/surfaces_state.h"
#include "mojo/public/cpp/bindings/strong_binding.h"
#include "ui/gfx/native_widget_types.h"
@@ -25,7 +23,6 @@ class CopyOutputResult;
class Display;
class DisplayScheduler;
class SurfaceFactory;
-class SyntheticBeginFrameSource;
}
namespace mus {
@@ -37,8 +34,7 @@ class SurfacesState;
// provided AcceleratedWidget. Frames are submitted here. New frames are
// scheduled to be generated here based on VSync.
class TopLevelDisplayClient : public cc::DisplayClient,
- public cc::SurfaceFactoryClient,
- public SurfacesContextProviderDelegate {
+ public cc::SurfaceFactoryClient {
public:
TopLevelDisplayClient(gfx::AcceleratedWidget widget,
const scoped_refptr<GpuState>& gpu_state,
@@ -53,15 +49,9 @@ class TopLevelDisplayClient : public cc::DisplayClient,
private:
// DisplayClient implementation.
- // TODO(rjkroege, fsamuel): This won't work correctly with multiple displays.
- void CommitVSyncParameters(base::TimeTicks timebase,
- base::TimeDelta interval) override;
void OutputSurfaceLost() override;
void SetMemoryPolicy(const cc::ManagedMemoryPolicy& policy) override;
- // SurfacesContextProviderDelegate:
- void OnVSyncParametersUpdated(int64_t timebase, int64_t interval) override;
-
// SurfaceFactoryClient implementation.
void ReturnResources(const cc::ReturnedResourceArray& resources) override;
void SetBeginFrameSource(cc::BeginFrameSource* begin_frame_source) override;
@@ -74,8 +64,6 @@ class TopLevelDisplayClient : public cc::DisplayClient,
gfx::Size last_submitted_frame_size_;
scoped_ptr<cc::CompositorFrame> pending_frame_;
- scoped_ptr<cc::SyntheticBeginFrameSource> synthetic_frame_source_;
- scoped_ptr<cc::DisplayScheduler> scheduler_;
scoped_ptr<cc::Display> display_;
DISALLOW_COPY_AND_ASSIGN(TopLevelDisplayClient);

Powered by Google App Engine
This is Rietveld 408576698