Mus+Ash: Unified BeginFrame Skeleton

services/ui/ws/server_window.cc

Index: services/ui/ws/server_window.cc
diff --git a/services/ui/ws/server_window.cc b/services/ui/ws/server_window.cc
index f24bee147d8a3c9e2e40049d40f2cb65fdbe8ba1..95b84e7c6121f88aacac3f6c3d55b89369183794 100644
--- a/services/ui/ws/server_window.cc
+++ b/services/ui/ws/server_window.cc
@@ -288,7 +288,6 @@ void ServerWindow::SetOpacity(float value) {
     return;
   float old_opacity = opacity_;
   opacity_ = value;
-  delegate_->OnScheduleWindowPaint(this);

[sky, 2016/11/07 16:34:08]

How does this and other functions in this file end up triggering what was
OnScheduleWindowPaint?

[Fady Samuel, 2016/11/07 16:42:01]

In the short term, FrameGenerator now polls the state on ServerWindows once
every 16 ms. Once the window manager embeds surface IDs, the window manager will
tick whenever it changes or something it embeds gets a new surface ID.

   for (auto& observer : observers_)
     observer.OnWindowOpacityChanged(this, old_opacity, opacity_);
 }
@@ -314,7 +313,6 @@ void ServerWindow::SetTransform(const gfx::Transform& transform) {
     return;
 
   transform_ = transform;
-  delegate_->OnScheduleWindowPaint(this);
 }
 
 void ServerWindow::SetProperty(const std::string& name,
@@ -381,7 +379,6 @@ void ServerWindow::SetUnderlayOffset(const gfx::Vector2d& offset) {
     return;
 
   underlay_offset_ = offset;
-  delegate_->OnScheduleWindowPaint(this);
 }
 
 void ServerWindow::OnEmbeddedAppDisconnected() {