Remove DesktopRegion parameter from DesktopCapturer::Capture

Remove DesktopRegion parameter from DesktopCapturer::Capture

BUG=https://bugs.chromium.org/p/webrtc/issues/detail?id=6513

Committed: https://crrev.com/249beee124254cd175622a8b9261a2cfa24cad9b
Cr-Commit-Position: refs/heads/master@{#14676}

[Hzj_jie, 2016-10-18 21:48:37]

Patchset #1 (id:1) has been deleted

[Hzj_jie, 2016-10-18 23:01:04]

Description was changed from

==========
Remove DesktopRegion parameter from DesktopCapturer::Capture

BUG=https://bugs.chromium.org/p/webrtc/issues/detail?id=6513
==========

to

==========
Remove DesktopRegion parameter from DesktopCapturer::Capture

BUG=https://bugs.chromium.org/p/webrtc/issues/detail?id=6513
==========

[Hzj_jie, 2016-10-18 23:01:04]

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

[Sergey Ulanov, 2016-10-18 23:37:10]

https://codereview.chromium.org/2433503002/diff/20001/webrtc/modules/desktop_...
File webrtc/modules/desktop_capture/desktop_capturer.h (right):

https://codereview.chromium.org/2433503002/diff/20001/webrtc/modules/desktop_...
webrtc/modules/desktop_capture/desktop_capturer.h:71: virtual void Capture() {
CaptureFrame(); }
I think CaptureFrame() is a better name than Capture(), so maybe just remove the
old Capture()?

[Hzj_jie, 2016-10-19 00:08:57]

Patchset #2 (id:40001) has been deleted

[Hzj_jie, 2016-10-19 00:49:17]

https://codereview.chromium.org/2433503002/diff/20001/webrtc/modules/desktop_...
File webrtc/modules/desktop_capture/desktop_capturer.h (right):

https://codereview.chromium.org/2433503002/diff/20001/webrtc/modules/desktop_...
webrtc/modules/desktop_capture/desktop_capturer.h:71: virtual void Capture() {
CaptureFrame(); }
On 2016/10/18 23:37:10, Sergey Ulanov wrote:
> I think CaptureFrame() is a better name than Capture(), so maybe just remove
the
> old Capture()?

No opinion, yes, I can remove old Capture() function.

[Sergey Ulanov, 2016-10-19 05:23:05]

lgtm

[Hzj_jie, 2016-10-19 05:43:26]

The CQ bit was checked by zijiehe@chromium.org

[commit-bot: I haz the power, 2016-10-19 05:43:34]

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, 2016-10-19 06:13:31]

Description was changed from

==========
Remove DesktopRegion parameter from DesktopCapturer::Capture

BUG=https://bugs.chromium.org/p/webrtc/issues/detail?id=6513
==========

to

==========
Remove DesktopRegion parameter from DesktopCapturer::Capture

BUG=https://bugs.chromium.org/p/webrtc/issues/detail?id=6513
==========

[commit-bot: I haz the power, 2016-10-19 06:13:33]

Committed patchset #2 (id:60001)

[commit-bot: I haz the power, 2016-10-19 06:13:44]

Description was changed from

==========
Remove DesktopRegion parameter from DesktopCapturer::Capture

BUG=https://bugs.chromium.org/p/webrtc/issues/detail?id=6513
==========

to

==========
Remove DesktopRegion parameter from DesktopCapturer::Capture

BUG=https://bugs.chromium.org/p/webrtc/issues/detail?id=6513

Committed: https://crrev.com/249beee124254cd175622a8b9261a2cfa24cad9b
Cr-Commit-Position: refs/heads/master@{#14676}
==========

[commit-bot: I haz the power, 2016-10-19 06:13:44]

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