Reland of Makes ScreenMus not check that displays are non-empty

Reland of Makes ScreenMus not check that displays are non-empty (patchset #1 id:1 of https://codereview.chromium.org/1905893002/ )

Reason for revert:
Relanding since the revert did not green up the test.

Original issue's description:
> Revert of Makes ScreenMus not check that displays are non-empty (patchset #1 id:1 of https://codereview.chromium.org/1912523002/ )
>
> Reason for revert:
> Speculative revert for failing interactive_ui_tests on Linux Chromium OS ASan LSan Tests (1), starting with
> https://build.chromium.org/p/chromium.memory/builders/Linux%20Chromium%20OS%20ASan%20LSan%20Tests%20%281%29/builds/11887
>
> Failure:
> [ RUN      ] DragDropTest.DragDropAcrossMultiDisplay
> Xlib:  extension "RANDR" missing on display ":9".
> [9809:9809:0420/230942:21040728698:INFO:lock_state_controller.cc(95)] Constructing LockStateController instance 0x61600006c980
> Xlib:  extension "RANDR" missing on display ":9".
> ../../ash/drag_drop/drag_drop_interactive_uitest.cc:153: Failure
> Value of: target_view->dropped()
>   Actual: false
> Expected: true
> [9809:9809:0420/230942:21040854605:INFO:lock_state_controller.cc(101)] Destroying LockStateController instance 0x61600006c980
> [  FAILED  ] DragDropTest.DragDropAcrossMultiDisplay (231 ms)
>
>
> Original issue's description:
> > Makes ScreenMus not check that displays are non-empty
> >
> > WaitForIncomingMethodCall can return in the case of the pipe being
> > broken (say mus is going down). Adds code to handle that case.
> >
> > BUG=594600
> > TEST=none
> > R=ben@chromium.org
>
> TBR=ben@chromium.org,sky@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=594600
TBR=ben@chromium.org,sky@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=594600
Committed: https://crrev.com/d0549569cece757249c77ebee22bb995d8ffb985
Cr-Commit-Position: refs/heads/master@{#388735}

[kjellander_chromium, 2016-04-21 10:37:51]

Created Reland of Makes ScreenMus not check that displays are non-empty

[commit-bot: I haz the power, 2016-04-21 10:38:01]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1905873003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1905873003/1

[commit-bot: I haz the power, 2016-04-21 10:38:31]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-22 19:33:10]

Description was changed from

==========
Reland of Makes ScreenMus not check that displays are non-empty (patchset #1
id:1 of https://codereview.chromium.org/1905893002/ )

Reason for revert:
Relanding since the revert did not green up the test.

Original issue's description:
> Revert of Makes ScreenMus not check that displays are non-empty (patchset #1
id:1 of https://codereview.chromium.org/1912523002/ )
> 
> Reason for revert:
> Speculative revert for failing interactive_ui_tests on Linux Chromium OS ASan
LSan Tests (1), starting with
>
https://build.chromium.org/p/chromium.memory/builders/Linux%20Chromium%20OS%2...
> 
> Failure:
> [ RUN      ] DragDropTest.DragDropAcrossMultiDisplay
> Xlib:  extension "RANDR" missing on display ":9".
> [9809:9809:0420/230942:21040728698:INFO:lock_state_controller.cc(95)]
Constructing LockStateController instance 0x61600006c980
> Xlib:  extension "RANDR" missing on display ":9".
> ../../ash/drag_drop/drag_drop_interactive_uitest.cc:153: Failure
> Value of: target_view->dropped()
>   Actual: false
> Expected: true
> [9809:9809:0420/230942:21040854605:INFO:lock_state_controller.cc(101)]
Destroying LockStateController instance 0x61600006c980
> [  FAILED  ] DragDropTest.DragDropAcrossMultiDisplay (231 ms)
> 
> 
> Original issue's description:
> > Makes ScreenMus not check that displays are non-empty
> > 
> > WaitForIncomingMethodCall can return in the case of the pipe being
> > broken (say mus is going down). Adds code to handle that case.
> > 
> > BUG=594600
> > TEST=none
> > R=ben@chromium.org
> 
> TBR=ben@chromium.org,sky@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=594600

TBR=ben@chromium.org,sky@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=594600
==========

to

==========
Reland of Makes ScreenMus not check that displays are non-empty (patchset #1
id:1 of https://codereview.chromium.org/1905893002/ )

Reason for revert:
Relanding since the revert did not green up the test.

Original issue's description:
> Revert of Makes ScreenMus not check that displays are non-empty (patchset #1
id:1 of https://codereview.chromium.org/1912523002/ )
>
> Reason for revert:
> Speculative revert for failing interactive_ui_tests on Linux Chromium OS ASan
LSan Tests (1), starting with
>
https://build.chromium.org/p/chromium.memory/builders/Linux%20Chromium%20OS%2...
>
> Failure:
> [ RUN      ] DragDropTest.DragDropAcrossMultiDisplay
> Xlib:  extension "RANDR" missing on display ":9".
> [9809:9809:0420/230942:21040728698:INFO:lock_state_controller.cc(95)]
Constructing LockStateController instance 0x61600006c980
> Xlib:  extension "RANDR" missing on display ":9".
> ../../ash/drag_drop/drag_drop_interactive_uitest.cc:153: Failure
> Value of: target_view->dropped()
>   Actual: false
> Expected: true
> [9809:9809:0420/230942:21040854605:INFO:lock_state_controller.cc(101)]
Destroying LockStateController instance 0x61600006c980
> [  FAILED  ] DragDropTest.DragDropAcrossMultiDisplay (231 ms)
>
>
> Original issue's description:
> > Makes ScreenMus not check that displays are non-empty
> >
> > WaitForIncomingMethodCall can return in the case of the pipe being
> > broken (say mus is going down). Adds code to handle that case.
> >
> > BUG=594600
> > TEST=none
> > R=ben@chromium.org
>
> TBR=ben@chromium.org,sky@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=594600

TBR=ben@chromium.org,sky@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=594600

Committed: https://crrev.com/d0549569cece757249c77ebee22bb995d8ffb985
Cr-Commit-Position: refs/heads/master@{#388735}
==========

[commit-bot: I haz the power, 2016-04-22 19:33:11]

Patchset 1 (id:??) landed as
https://crrev.com/d0549569cece757249c77ebee22bb995d8ffb985
Cr-Commit-Position: refs/heads/master@{#388735}