Remove FrameGenerator::root_window_

services/ui/ws/frame_generator.h

Index: services/ui/ws/frame_generator.h
diff --git a/services/ui/ws/frame_generator.h b/services/ui/ws/frame_generator.h
index ea5887eaab1f8165024120430198ec5d2023a993..18214583c89b2f92004940fcda4a83adb57e3313 100644
--- a/services/ui/ws/frame_generator.h
+++ b/services/ui/ws/frame_generator.h
@@ -8,15 +8,11 @@
 #include <memory>
 
 #include "base/macros.h"
-#include "base/timer/timer.h"
-#include "cc/ipc/display_compositor.mojom.h"
+#include "cc/output/compositor_frame.h"
 #include "cc/output/compositor_frame_sink_client.h"
 #include "cc/scheduler/begin_frame_source.h"
-#include "cc/surfaces/frame_sink_id.h"
-#include "cc/surfaces/local_surface_id_allocator.h"
 #include "cc/surfaces/surface_id.h"
-#include "cc/surfaces/surface_reference.h"
-#include "services/ui/public/interfaces/window_tree_constants.mojom.h"
+#include "cc/surfaces/surface_info.h"
 #include "ui/gfx/geometry/rect.h"
 #include "ui/gfx/native_widget_types.h"

[Fady Samuel, 2017/03/22 19:02:19]

nit: You probably don't need this either.

[Alex Z., 2017/03/22 19:19:45]

Done.

 
@@ -28,15 +24,12 @@ class RenderPass;
 namespace ui {
 namespace ws {
 
-class ServerWindow;
-
 // Responsible for redrawing the display in response to the redraw requests by
 // submitting CompositorFrames to the owned CompositorFrameSink.
 class FrameGenerator : public cc::CompositorFrameSinkClient,
                        public cc::BeginFrameObserver {
  public:
   FrameGenerator(

[Fady Samuel, 2017/03/22 19:02:19]

nit explicit

[Alex Z., 2017/03/22 19:19:45]

Done.

-      ServerWindow* root_window,
       std::unique_ptr<cc::CompositorFrameSink> compositor_frame_sink);
   ~FrameGenerator() override;
 
@@ -47,6 +40,8 @@ class FrameGenerator : public cc::CompositorFrameSinkClient,
   void OnSurfaceCreated(const cc::SurfaceInfo& surface_info);
 
   void OnWindowDamaged();
+  void OnWindowVisibilityChanged(bool visible);
+  void OnWindowBoundsChanged(const gfx::Rect& bounds);
 
  private:
   // cc::CompositorFrameSinkClient implementation:
@@ -69,7 +64,7 @@ class FrameGenerator : public cc::CompositorFrameSinkClient,
   void OnBeginFrameSourcePausedChanged(bool paused) override;
 
   // Generates the CompositorFrame.
-  cc::CompositorFrame GenerateCompositorFrame(const gfx::Rect& output_rect);
+  cc::CompositorFrame GenerateCompositorFrame();
 
   // DrawWindow creates SurfaceDrawQuad for the window manager and appends it to
   // the provided cc::RenderPass.
@@ -79,8 +74,10 @@ class FrameGenerator : public cc::CompositorFrameSinkClient,
   // FrameGenerator as an observer to/from begin_frame_source_ accordingly.
   void SetNeedsBeginFrame(bool needs_begin_frame);
 
-  ServerWindow* const root_window_;
   float device_scale_factor_ = 1.f;
+  bool is_window_visible_ = false;
+  ;

[Fady Samuel, 2017/03/22 19:02:19]

remove this.

[Alex Z., 2017/03/22 19:19:45]

Done.

+  gfx::Rect bounds_;
 
   std::unique_ptr<cc::CompositorFrameSink> compositor_frame_sink_;
   cc::BeginFrameArgs last_begin_frame_args_;