Fixed window capute may end when window's full screen mode switchs on/off

If a desktop captured window switches on/off it full screen mode, the capture may be unexpectedly terminated. During the transition of full screen mode on/off, the window can be temporarily invisible.

BUG=498484
Committed: https://crrev.com/c94bd9bf86d39b2b17ffaad0775d28e875f24b08
Cr-Commit-Position: refs/heads/master@{#10583}

[GeorgeZ, 2015-11-04 23:41:30]

Description was changed from

==========
Fixed window capute may end when window's full screen mode on/off

BUG=498484
==========

to

==========
If a desktop captured window switches on/off it full screen mode, the capture
may be unexpectedly terminated. During the transition of full screen mode
on/off, the window can be temporarily invisible. 

BUG=498484
==========

[GeorgeZ, 2015-11-04 23:41:30]

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

[GeorgeZ, 2015-11-04 23:42:23]

Sergey,

Would you please have a review for me.

Thanks,

George

[GeorgeZ, 2015-11-09 20:57:19]

On 2015/11/04 23:42:23, GeorgeZ wrote:
> Sergey,
> 
> Would you please have a review for me.
> 
> Thanks,
> 
> George

Sergey,

Do you have time to do a review.

Thanks,

George

[Sergey Ulanov, 2015-11-09 21:27:40]

lgtm

https://codereview.webrtc.org/1426103005/diff/1/webrtc/modules/desktop_captur...
File webrtc/modules/desktop_capture/window_capturer_win.cc (right):

https://codereview.webrtc.org/1426103005/diff/1/webrtc/modules/desktop_captur...
webrtc/modules/desktop_capture/window_capturer_win.cc:166: // on Mac, or if
window is invisible. Window can be temporarily invisible
reword first sentence: 
// Return a 1x1 black frame if the window is minimized or invisible, to match
// behavior on mac

[GeorgeZ, 2015-11-09 21:59:06]

https://codereview.webrtc.org/1426103005/diff/1/webrtc/modules/desktop_captur...
File webrtc/modules/desktop_capture/window_capturer_win.cc (right):

https://codereview.webrtc.org/1426103005/diff/1/webrtc/modules/desktop_captur...
webrtc/modules/desktop_capture/window_capturer_win.cc:166: // on Mac, or if
window is invisible. Window can be temporarily invisible
On 2015/11/09 21:27:40, Sergey Ulanov wrote:
> reword first sentence: 
> // Return a 1x1 black frame if the window is minimized or invisible, to match
> // behavior on mac

Done.

[GeorgeZ, 2015-11-09 21:59:28]

The CQ bit was checked by gyzhou@chromium.org

[GeorgeZ, 2015-11-09 21:59:29]

The patchset sent to the CQ was uploaded after l-g-t-m from sergeyu@chromium.org
Link to the patchset: https://codereview.webrtc.org/1426103005/#ps20001 (title:
"change base on reviewer's comments")

[commit-bot: I haz the power, 2015-11-09 21:59:40]

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

[commit-bot: I haz the power, 2015-11-09 23:59:55]

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

[commit-bot: I haz the power, 2015-11-09 23:59:56]

Try jobs failed on following builders:
  android_arm64_rel on tryserver.webrtc (JOB_TIMED_OUT, no build URL)

[GeorgeZ, 2015-11-10 15:30:00]

The CQ bit was checked by gyzhou@chromium.org

[commit-bot: I haz the power, 2015-11-10 15:30:07]

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

[commit-bot: I haz the power, 2015-11-10 15:33:55]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-11-10 15:34:06]

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