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

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

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.cc
diff --git a/components/mus/surfaces/top_level_display_client.cc b/components/mus/surfaces/top_level_display_client.cc
index a62ae8be5c60c54dab4a25b2014b04e93746b804..e4d3040d01feafd62a680781f8e374ef1ebb6651 100644
--- a/components/mus/surfaces/top_level_display_client.cc
+++ b/components/mus/surfaces/top_level_display_client.cc
@@ -38,24 +38,18 @@ TopLevelDisplayClient::TopLevelDisplayClient(
factory_.Create(cc_id_);
display_.reset(new cc::Display(this, surfaces_state_->manager(), nullptr,
- nullptr, cc::RendererSettings()));
+ nullptr, cc::RendererSettings(),
+ cc_id_.id_namespace()));
scoped_ptr<cc::OutputSurface> output_surface =
make_scoped_ptr(new DirectOutputSurface(
- new SurfacesContextProvider(this, widget, gpu_state)));
+ new SurfacesContextProvider(widget, gpu_state), task_runner_.get()));
int max_frames_pending = output_surface->capabilities().max_frames_pending;
DCHECK_GT(max_frames_pending, 0);
- synthetic_frame_source_.reset(new cc::SyntheticBeginFrameSource(
- task_runner_.get(), cc::BeginFrameArgs::DefaultInterval()));
-
- scheduler_.reset(
- new cc::DisplayScheduler(display_.get(), synthetic_frame_source_.get(),
- task_runner_.get(), max_frames_pending));
-
if (gpu_state->HardwareRenderingAvailable()) {
- display_->Initialize(std::move(output_surface), scheduler_.get());
+ display_->Initialize(std::move(output_surface), task_runner_.get());
} else {
// TODO(rjkroege): Implement software compositing.
}
@@ -87,9 +81,6 @@ void TopLevelDisplayClient::RequestCopyOfOutput(
factory_.RequestCopyOfSurface(cc_id_, std::move(output_request));
}
-void TopLevelDisplayClient::CommitVSyncParameters(base::TimeTicks timebase,
- base::TimeDelta interval) {}
-
void TopLevelDisplayClient::OutputSurfaceLost() {
if (!display_) // Shutdown case
return;
@@ -99,20 +90,6 @@ void TopLevelDisplayClient::OutputSurfaceLost() {
void TopLevelDisplayClient::SetMemoryPolicy(
const cc::ManagedMemoryPolicy& policy) {}
-void TopLevelDisplayClient::OnVSyncParametersUpdated(int64_t timebase,
- int64_t interval) {
- auto timebase_time_ticks = base::TimeTicks::FromInternalValue(timebase);
- auto interval_time_delta = base::TimeDelta::FromInternalValue(interval);
-
- if (interval_time_delta.is_zero()) {
- // TODO(brianderson): We should not be receiving 0 intervals.
- interval_time_delta = cc::BeginFrameArgs::DefaultInterval();
- }
-
- synthetic_frame_source_->OnUpdateVSyncParameters(timebase_time_ticks,
- interval_time_delta);
-}
-
void TopLevelDisplayClient::ReturnResources(
const cc::ReturnedResourceArray& resources) {
// TODO(fsamuel): Implement this.

Powered by Google App Engine
This is Rietveld 408576698