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

Unified Diff: services/ui/ws/display.h

Issue 2400723003: Mus+Ash: Use standard cc mechanism for surface lifetime. (Closed)
Patch Set: Only AddObserver once Created 4 years, 2 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
« no previous file with comments | « no previous file | services/ui/ws/display.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: services/ui/ws/display.h
diff --git a/services/ui/ws/display.h b/services/ui/ws/display.h
index 027d12bfd0dcf3b8add38b1ce1f3b4aa03c211e3..415592d8430b41f4189360faeacdc1a4b3412cc8 100644
--- a/services/ui/ws/display.h
+++ b/services/ui/ws/display.h
@@ -55,7 +55,6 @@ class Display : public PlatformDisplayDelegate,
public mojom::WindowTreeHost,
public FocusControllerObserver,
public FocusControllerDelegate,
- public ServerWindowObserver,
public UserIdTrackerObserver,
public WindowManagerWindowTreeFactorySetObserver {
public:
@@ -83,11 +82,6 @@ class Display : public PlatformDisplayDelegate,
// Schedules a paint for the specified region in the coordinates of |window|.
void SchedulePaint(const ServerWindow* window, const gfx::Rect& bounds);
- // Schedules destruction of surfaces in |window|. If a frame has been
- // scheduled but not drawn surface destruction is delayed until the frame is
- // drawn, otherwise destruction is immediate.
- void ScheduleSurfaceDestruction(ServerWindow* window);
-
display::Display::Rotation GetRotation() const;
gfx::Size GetSize() const;
@@ -175,7 +169,6 @@ class Display : public PlatformDisplayDelegate,
void OnNativeCaptureLost() override;
void OnViewportMetricsChanged(const ViewportMetrics& old_metrics,
const ViewportMetrics& new_metrics) override;
- void OnCompositorFrameDrawn() override;
// FocusControllerDelegate:
bool CanHaveActiveChildren(ServerWindow* window) const override;
@@ -187,9 +180,6 @@ class Display : public PlatformDisplayDelegate,
ServerWindow* old_focused_window,
ServerWindow* new_focused_window) override;
- // ServerWindowObserver:
- void OnWindowDestroyed(ServerWindow* window) override;
-
// UserIdTrackerObserver:
void OnUserIdRemoved(const UserId& id) override;
@@ -210,10 +200,6 @@ class Display : public PlatformDisplayDelegate,
ServerWindowTracker activation_parents_;
- // Set of windows with surfaces that need to be destroyed once the frame
- // draws.
- std::set<ServerWindow*> windows_needing_frame_destruction_;
-
WindowManagerDisplayRootMap window_manager_display_root_map_;
DISALLOW_COPY_AND_ASSIGN(Display);
« no previous file with comments | « no previous file | services/ui/ws/display.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698