Removing ServerWindowCompositorFrameSinkManager::ShouldDraw()

Removing ServerWindowCompositorFrameSinkManager::ShouldDraw()

With underlay surfaces unused/going away, ServerWindowCompositorFrameSinkManager
should always draw.

BUG=NONE

[mfomitchev, 2017-01-04 21:16:32]

The CQ bit was checked by mfomitchev@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-04 21:16:58]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[mfomitchev, 2017-01-04 22:16:11]

mfomitchev@chromium.org changed reviewers:
+ sky@chromium.org

[sky, 2017-01-04 22:20:00]

LGTM

[sky, 2017-01-04 22:20:31]

Description was changed from

==========
Removing ServerWindowCompositorFrameSinkManager::ShouldDraw()

With underlay surfaces unused/going away, ServerWindowCompositorFrameSinkManager
should always draw.

BUG=NONE
==========

to

==========
Removing ServerWindowCompositorFrameSinkManager::ShouldDraw()

With underlay surfaces unused/going away, ServerWindowCompositorFrameSinkManager
should always draw.

BUG=NONE
==========

[sky, 2017-01-04 22:20:31]

sky@chromium.org changed reviewers:
+ tsepez@chromium.org

[sky, 2017-01-04 22:20:40]

+tsepez for mojom

[Tom Sepez, 2017-01-04 22:48:53]

RS LGTM on deleting code.

[commit-bot: I haz the power, 2017-01-04 23:11:46]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-04 23:11:47]

Dry run: This issue passed the CQ dry run.

[sadrul, 2017-01-06 05:30:42]

The CQ bit was checked by sadrul@chromium.org

[commit-bot: I haz the power, 2017-01-06 05:31:02]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-06 06:24:05]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-06 06:24:07]

Failed to apply patch for services/ui/ws/frame_generator.cc:
While running git apply --index -p1;
  error: patch failed: services/ui/ws/frame_generator.cc:159
  error: services/ui/ws/frame_generator.cc: patch does not apply

Patch:       services/ui/ws/frame_generator.cc
Index: services/ui/ws/frame_generator.cc
diff --git a/services/ui/ws/frame_generator.cc
b/services/ui/ws/frame_generator.cc
index
b72d0783b10dbaac146aad6fa77f883720cb6ca5..dad25f51bfe50cb59811352e344e82664c68238b
100644
--- a/services/ui/ws/frame_generator.cc
+++ b/services/ui/ws/frame_generator.cc
@@ -159,8 +159,7 @@ void FrameGenerator::DrawWindow(cc::RenderPass* pass,
ServerWindow* window) {
   if (!window->visible())
     return;
 
-  if (!window->compositor_frame_sink_manager() ||
-      !window->compositor_frame_sink_manager()->ShouldDraw())
+  if (!window->compositor_frame_sink_manager())
     return;
 
   cc::SurfaceId default_surface_id =

[sadrul, 2017-01-10 06:34:32]

This CL landed in https://codereview.chromium.org/2621843003/