Fix error handling in X11 screen capturer

Fix error handling in X11 screen capturer

Previusly errors from XServerPixelBuffer::CaptureRect() were not always
handled, which results in a black frame returned from the capturer
instead of an error.

BUG=webrtc:7305
Review-Url: https://codereview.webrtc.org/2738513005
Cr-Commit-Position: refs/heads/master@{#17101}
Committed: https://chromium.googlesource.com/external/webrtc/+/9fa7e4a0ac20b89f6c147f27a7e04a762378aaa4

[Sergey Ulanov, 2017-03-06 22:39:52]

sergeyu@chromium.org changed reviewers:
+ zijiehe@chromium.org

[Hzj_jie, 2017-03-07 20:36:33]

On 2017/03/06 22:39:52, Sergey Ulanov wrote:

lgtm.

[Sergey Ulanov, 2017-03-08 00:32:21]

The CQ bit was checked by sergeyu@chromium.org

[commit-bot: I haz the power, 2017-03-08 00:32:25]

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

[commit-bot: I haz the power, 2017-03-08 00:36:54]

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

[commit-bot: I haz the power, 2017-03-08 00:36:55]

Try jobs failed on following builders:
  presubmit on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/presubmit/builds/14593)

[Sergey Ulanov, 2017-03-08 00:49:33]

Description was changed from

==========
Fix error handling in X11 screen capturer

Previusly errors from XServerPixelBuffer::CaptureRect() were not always
handled, which results in a black frame returned from the capturer
instead of an error.
==========

to

==========
Fix error handling in X11 screen capturer

Previusly errors from XServerPixelBuffer::CaptureRect() were not always
handled, which results in a black frame returned from the capturer
instead of an error.

BUG=webrtc:7305
==========

[Sergey Ulanov, 2017-03-08 00:49:37]

The CQ bit was checked by sergeyu@chromium.org

[commit-bot: I haz the power, 2017-03-08 00:49:46]

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

[commit-bot: I haz the power, 2017-03-08 01:02:21]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1488934177851750, "parent_rev":
"b09b3f9a6202e8bcbd8f3e12af1257a1d00cc5a0", "commit_rev":
"9fa7e4a0ac20b89f6c147f27a7e04a762378aaa4"}

[commit-bot: I haz the power, 2017-03-08 01:02:24]

Description was changed from

==========
Fix error handling in X11 screen capturer

Previusly errors from XServerPixelBuffer::CaptureRect() were not always
handled, which results in a black frame returned from the capturer
instead of an error.

BUG=webrtc:7305
==========

to

==========
Fix error handling in X11 screen capturer

Previusly errors from XServerPixelBuffer::CaptureRect() were not always
handled, which results in a black frame returned from the capturer
instead of an error.

BUG=webrtc:7305

Review-Url: https://codereview.webrtc.org/2738513005
Cr-Commit-Position: refs/heads/master@{#17101}
Committed:
https://chromium.googlesource.com/external/webrtc/+/9fa7e4a0ac20b89f6c147f27a...
==========

[commit-bot: I haz the power, 2017-03-08 01:02:25]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/external/webrtc/+/9fa7e4a0ac20b89f6c147f27a...