Commit cde9497a authored by Fady Samuel's avatar Fady Samuel Committed by Commit Bot

viz; Cleanup SurfaceManager::RequestSurfaceResolution

SurfaceManager::RequestSurfaceResolution and
SurfaceDependencyTracker::RequestSurfaceResolution take in an unused
SurfaceDependencyDeadline parameter. This CL removes it.

Bug: none
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel
Change-Id: I914cd1f45423ef231f76a5b58a29a178a8bbb941
Reviewed-on: https://chromium-review.googlesource.com/580528Reviewed-by: default avatarJohn Bauman <jbauman@chromium.org>
Commit-Queue: Fady Samuel <fsamuel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488452}
parent 32717cbf
...@@ -150,7 +150,7 @@ bool Surface::QueueFrame(CompositorFrame frame, ...@@ -150,7 +150,7 @@ bool Surface::QueueFrame(CompositorFrame frame,
// Ask the surface manager to inform |this| when its dependencies are // Ask the surface manager to inform |this| when its dependencies are
// resolved. // resolved.
surface_manager_->RequestSurfaceResolution(this, &deadline_); surface_manager_->RequestSurfaceResolution(this);
} }
// Returns resources for the previous pending frame. // Returns resources for the previous pending frame.
......
...@@ -20,9 +20,7 @@ SurfaceDependencyTracker::SurfaceDependencyTracker( ...@@ -20,9 +20,7 @@ SurfaceDependencyTracker::SurfaceDependencyTracker(
SurfaceDependencyTracker::~SurfaceDependencyTracker() = default; SurfaceDependencyTracker::~SurfaceDependencyTracker() = default;
void SurfaceDependencyTracker::RequestSurfaceResolution( void SurfaceDependencyTracker::RequestSurfaceResolution(Surface* surface) {
Surface* surface,
SurfaceDependencyDeadline* deadline) {
DCHECK(surface->HasPendingFrame()); DCHECK(surface->HasPendingFrame());
const CompositorFrame& pending_frame = surface->GetPendingFrame(); const CompositorFrame& pending_frame = surface->GetPendingFrame();
......
...@@ -31,8 +31,7 @@ class CC_SURFACES_EXPORT SurfaceDependencyTracker { ...@@ -31,8 +31,7 @@ class CC_SURFACES_EXPORT SurfaceDependencyTracker {
// Called when |surface| has a pending CompositorFrame and it wishes to be // Called when |surface| has a pending CompositorFrame and it wishes to be
// informed when that surface's dependencies are resolved. // informed when that surface's dependencies are resolved.
void RequestSurfaceResolution(Surface* surface, void RequestSurfaceResolution(Surface* surface);
SurfaceDependencyDeadline* deadline);
void OnSurfaceActivated(Surface* surface); void OnSurfaceActivated(Surface* surface);
void OnSurfaceDependenciesChanged( void OnSurfaceDependenciesChanged(
......
...@@ -63,10 +63,8 @@ std::string SurfaceManager::SurfaceReferencesToString() { ...@@ -63,10 +63,8 @@ std::string SurfaceManager::SurfaceReferencesToString() {
} }
#endif #endif
void SurfaceManager::RequestSurfaceResolution( void SurfaceManager::RequestSurfaceResolution(Surface* surface) {
Surface* surface, dependency_tracker_.RequestSurfaceResolution(surface);
SurfaceDependencyDeadline* deadline) {
dependency_tracker_.RequestSurfaceResolution(surface, deadline);
} }
Surface* SurfaceManager::CreateSurface( Surface* SurfaceManager::CreateSurface(
......
...@@ -62,8 +62,7 @@ class CC_SURFACES_EXPORT SurfaceManager { ...@@ -62,8 +62,7 @@ class CC_SURFACES_EXPORT SurfaceManager {
std::string SurfaceReferencesToString(); std::string SurfaceReferencesToString();
#endif #endif
void RequestSurfaceResolution(Surface* surface, void RequestSurfaceResolution(Surface* surface);
SurfaceDependencyDeadline* deadline);
// Creates a Surface for the given SurfaceClient. The surface will be // Creates a Surface for the given SurfaceClient. The surface will be
// destroyed when DestroySurface is called, all of its destruction // destroyed when DestroySurface is called, all of its destruction
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment