Take advantage of XDAMAGE to support Screen Capture under ChromeOS.

Take advantage of XDAMAGE to support Screen Capture under ChromeOS.

Screen Capture polls under Linux/X11, to avoid issues with poor XDAMAGE
support in some X hardware drivers for things like 3D rendering.

This CL enables XDAMAGE support under ChromeOS, since on that platform we
can make sure that the drivers support it properly.

BUG=177977
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=190032

[Wez, 2013-03-17 01:56:37]

PTAL

[Sergey Ulanov, 2013-03-17 05:36:32]

lgtm

https://chromiumcodereview.appspot.com/12408035/diff/1/media/video/capture/sc...
File media/video/capture/screen/screen_capture_device.cc (right):

https://chromiumcodereview.appspot.com/12408035/diff/1/media/video/capture/sc...
media/video/capture/screen/screen_capture_device.cc:277: if (!screen_capturer_)
nit: Maybe pull this line out of the #ifdef - it's common between the two cases.

[Wez, 2013-03-19 18:25:44]

https://chromiumcodereview.appspot.com/12408035/diff/1/media/video/capture/sc...
File media/video/capture/screen/screen_capture_device.cc (right):

https://chromiumcodereview.appspot.com/12408035/diff/1/media/video/capture/sc...
media/video/capture/screen/screen_capture_device.cc:277: if (!screen_capturer_)
On 2013/03/17 05:36:32, sergeyu wrote:
> nit: Maybe pull this line out of the #ifdef - it's common between the two
cases.

Since it's only duplicating one line, I chose to avoid splitting the if from the
code it executes.

[commit-bot: I haz the power, 2013-03-21 20:26:35]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/wez@chromium.org/12408035/1

[commit-bot: I haz the power, 2013-03-22 18:14:48]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/wez@chromium.org/12408035/1

[commit-bot: I haz the power, 2013-03-22 20:07:24]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/wez@chromium.org/12408035/1

[commit-bot: I haz the power, 2013-03-23 14:46:19]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/wez@chromium.org/12408035/1

[commit-bot: I haz the power, 2013-03-23 16:39:49]

Change committed as 190032