Revert of [WebRTC] A real ScreenCapturer test

Revert of [WebRTC] A real ScreenCapturer test (patchset #8 id:240001 of https://codereview.webrtc.org/2268093002/ )

Reason for revert:
ScreenCapturerTest.CaptureUpdatedRegion fails on Win DrMemory Full.

Original issue's description:
> [WebRTC] A real ScreenCapturer test
>
> We do not have a real ScreenCapturer test before. And after CL 2210443002, a new
> ScreenDrawer interface is added to the code base to draw various shapes on the
> screen. This change is to use ScreenDrawer to test ScreenCapturer. Besides test
> cases, some other changes are included,
>
> 1. A WaitForPendingPaintings() function in ScreenDrawer, to wait for a
> ScreenDrawer to finish all the pending draws. This function now only sleeps 50
> milliseconds on X11 and 100 milliseconds on Windows.
>
> 2. A Color structure to help handle a big-endian or little-endian safe color and
> provide functions to compare with DesktopFrame::data(). Both ScreenDrawer and
> DesktopFrameGenerator (in change 2202443002) can use this class to create colors
> and compare with or paint to a DesktopFrame.
>
> 3. ScreenDrawer now uses Color structure instead of uint32_t.
>
> BUG=314516
>
> TBR=kjellander@chromium.org
>
> Committed: https://crrev.com/9d1c54ace0dc9f68da0152aa1ded2a8dba0a43ae
> Cr-Commit-Position: refs/heads/master@{#14058}

TBR=sergeyu@chromium.org,jamiewalch@chromium.org,kjellander@chromium.org,zijiehe@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=314516
Committed: https://crrev.com/4c44202dc348613695a4b529bbd7c9bdab6195ec
Cr-Commit-Position: refs/heads/master@{#14071}

[åsapersson, 2016-09-05 12:45:23]

The CQ bit was checked by asapersson@webrtc.org

[åsapersson, 2016-09-05 12:45:23]

Created Revert of [WebRTC] A real ScreenCapturer test

[commit-bot: I haz the power, 2016-09-05 12:45:34]

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

[kjellander_webrtc, 2016-09-05 13:02:36]

On 2016/09/05 12:45:34, commit-bot: I haz the power wrote:
> CQ is trying da patch. Follow status at
>  
>
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

For next time: please provide a breakage link in the revert CL
(https://build.chromium.org/p/client.webrtc/builders/Win%20DrMemory%20Full/bui...).

You could also save some trybot resources by not checking "revert and submit
this CL" and add NOTRY=True (but I agree that our limited trybot resources is an
infrastructure problem that should be fixed separately)

zijiehe: not that Dr Memory Full is a pretty slow test environment, which may
explain why your test is failing there. If you don't know how to resolve the
problem it's OK to exclude the test for Dr memory by altering
https://chromium.googlesource.com/external/webrtc/+/master/tools/valgrind-web...
in the re-land CL.

[commit-bot: I haz the power, 2016-09-05 13:55:22]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-05 13:55:32]

Description was changed from

==========
Revert of [WebRTC] A real ScreenCapturer test (patchset #8 id:240001 of
https://codereview.webrtc.org/2268093002/ )

Reason for revert:
ScreenCapturerTest.CaptureUpdatedRegion fails on Win DrMemory Full.

Original issue's description:
> [WebRTC] A real ScreenCapturer test
> 
> We do not have a real ScreenCapturer test before. And after CL 2210443002, a
new
> ScreenDrawer interface is added to the code base to draw various shapes on the
> screen. This change is to use ScreenDrawer to test ScreenCapturer. Besides
test
> cases, some other changes are included,
> 
> 1. A WaitForPendingPaintings() function in ScreenDrawer, to wait for a
> ScreenDrawer to finish all the pending draws. This function now only sleeps 50
> milliseconds on X11 and 100 milliseconds on Windows.
> 
> 2. A Color structure to help handle a big-endian or little-endian safe color
and
> provide functions to compare with DesktopFrame::data(). Both ScreenDrawer and
> DesktopFrameGenerator (in change 2202443002) can use this class to create
colors
> and compare with or paint to a DesktopFrame.
> 
> 3. ScreenDrawer now uses Color structure instead of uint32_t.
> 
> BUG=314516
> 
> TBR=kjellander@chromium.org
> 
> Committed: https://crrev.com/9d1c54ace0dc9f68da0152aa1ded2a8dba0a43ae
> Cr-Commit-Position: refs/heads/master@{#14058}

TBR=sergeyu@chromium.org,jamiewalch@chromium.org,kjellander@chromium.org,ziji...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=314516
==========

to

==========
Revert of [WebRTC] A real ScreenCapturer test (patchset #8 id:240001 of
https://codereview.webrtc.org/2268093002/ )

Reason for revert:
ScreenCapturerTest.CaptureUpdatedRegion fails on Win DrMemory Full.

Original issue's description:
> [WebRTC] A real ScreenCapturer test
>
> We do not have a real ScreenCapturer test before. And after CL 2210443002, a
new
> ScreenDrawer interface is added to the code base to draw various shapes on the
> screen. This change is to use ScreenDrawer to test ScreenCapturer. Besides
test
> cases, some other changes are included,
>
> 1. A WaitForPendingPaintings() function in ScreenDrawer, to wait for a
> ScreenDrawer to finish all the pending draws. This function now only sleeps 50
> milliseconds on X11 and 100 milliseconds on Windows.
>
> 2. A Color structure to help handle a big-endian or little-endian safe color
and
> provide functions to compare with DesktopFrame::data(). Both ScreenDrawer and
> DesktopFrameGenerator (in change 2202443002) can use this class to create
colors
> and compare with or paint to a DesktopFrame.
>
> 3. ScreenDrawer now uses Color structure instead of uint32_t.
>
> BUG=314516
>
> TBR=kjellander@chromium.org
>
> Committed: https://crrev.com/9d1c54ace0dc9f68da0152aa1ded2a8dba0a43ae
> Cr-Commit-Position: refs/heads/master@{#14058}

TBR=sergeyu@chromium.org,jamiewalch@chromium.org,kjellander@chromium.org,ziji...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=314516

Committed: https://crrev.com/4c44202dc348613695a4b529bbd7c9bdab6195ec
Cr-Commit-Position: refs/heads/master@{#14071}
==========

[commit-bot: I haz the power, 2016-09-05 13:55:33]

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

[Hzj_jie, 2016-09-05 21:25:25]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.webrtc.org/2313653003/ by zijiehe@chromium.org.

The reason for reverting is: Resubmit capturer tests.