Remove ScreenCapturer and WindowCapturer

Remove ScreenCapturer and WindowCapturer

This change removes ScreenCapturer and WindowCapturer from WebRTC.

BUG=webrtc:6513
Committed: https://crrev.com/98903d2f5ef420adb343358824b94731b4a00b38
Cr-Commit-Position: refs/heads/master@{#15033}

[Hzj_jie, 2016-11-10 02:44:00]

The CQ bit was checked by zijiehe@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-10 02:44:03]

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

[Hzj_jie, 2016-11-10 02:46:05]

The CQ bit was checked by zijiehe@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-10 02:46:13]

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

[Hzj_jie, 2016-11-10 02:47:08]

Description was changed from

==========
cl format


Remove ScreenCapturer and DesktopCapturer


Build break on Mac


Build break on Windows


Remove references of ScreenCapturer and WindowCapturer

BUG=
==========

to

==========
Remove ScreenCapturer and WindowCapturer

This change removes ScreenCapturer and WindowCapturer from WebRTC. This change
is blocked by Chromium change https://codereview.chromium.org/2495453002.

BUG=webrtc:6513
==========

[Hzj_jie, 2016-11-10 02:47:51]

Patchset #1 (id:1) has been deleted

[commit-bot: I haz the power, 2016-11-10 02:49:23]

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

[commit-bot: I haz the power, 2016-11-10 02:49:24]

Dry run: Try jobs failed on following builders:
  win_x64_dbg on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/win_x64_dbg/builds/2997)

[Hzj_jie, 2016-11-10 02:51:49]

The CQ bit was checked by zijiehe@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-10 02:51:54]

Dry run: 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-11-10 02:54:44]

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

[commit-bot: I haz the power, 2016-11-10 02:54:44]

Dry run: Try jobs failed on following builders:
  android_clang_dbg on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/android_clang_dbg/build...)
  android_compile_mips_dbg on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/android_compile_mips_db...)
  android_compile_x86_dbg on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/android_compile_x86_dbg...)

[Hzj_jie, 2016-11-10 03:24:37]

Patchset #1 (id:20001) has been deleted

[Hzj_jie, 2016-11-10 04:07:58]

The CQ bit was checked by zijiehe@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-10 04:08:01]

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

[Hzj_jie, 2016-11-10 04:08:31]

Patchset #1 (id:40001) has been deleted

[commit-bot: I haz the power, 2016-11-10 04:10:30]

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

[commit-bot: I haz the power, 2016-11-10 04:10:32]

Dry run: Try jobs failed on following builders:
  win_clang_dbg on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/win_clang_dbg/builds/8122)
  win_clang_rel on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/win_clang_rel/builds/8088)
  win_x64_clang_dbg on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/win_x64_clang_dbg/build...)

[Hzj_jie, 2016-11-10 04:14:24]

The CQ bit was checked by zijiehe@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-10 04:14:31]

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

[Hzj_jie, 2016-11-10 04:14:42]

Patchset #1 (id:60001) has been deleted

[commit-bot: I haz the power, 2016-11-10 04:52:29]

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

[commit-bot: I haz the power, 2016-11-10 04:52:30]

Dry run: This issue passed the CQ dry run.

[Hzj_jie, 2016-11-10 05:27:57]

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

[Sergey Ulanov, 2016-11-10 20:30:45]

Can you please run this change on chromium trybots to make sure it doesn't break
chromium? See instructions on https://webrtc.org/contributing/#running-tryjobs

[Hzj_jie, 2016-11-10 21:02:00]

Description was changed from

==========
Remove ScreenCapturer and WindowCapturer

This change removes ScreenCapturer and WindowCapturer from WebRTC. This change
is blocked by Chromium change https://codereview.chromium.org/2495453002.

BUG=webrtc:6513
==========

to

==========
Remove ScreenCapturer and WindowCapturer

This change removes ScreenCapturer and WindowCapturer from WebRTC. This change
is blocked by Chromium change https://codereview.chromium.org/2495453002.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:bot1,bot2;master.tryserver.chromium.mac:bot3

BUG=webrtc:6513
==========

[Hzj_jie, 2016-11-10 21:10:22]

Description was changed from

==========
Remove ScreenCapturer and WindowCapturer

This change removes ScreenCapturer and WindowCapturer from WebRTC. This change
is blocked by Chromium change https://codereview.chromium.org/2495453002.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:bot1,bot2;master.tryserver.chromium.mac:bot3

BUG=webrtc:6513
==========

to

==========
Remove ScreenCapturer and WindowCapturer

This change removes ScreenCapturer and WindowCapturer from WebRTC. This change
is blocked by Chromium change https://codereview.chromium.org/2495453002.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:bot1,bot2;master.tryserver.chromium.mac:bot3;master.tryserver.chromium.win:bot4

BUG=webrtc:6513
==========

[Hzj_jie, 2016-11-10 21:10:49]

On 2016/11/10 20:30:45, Sergey Ulanov wrote:
> Can you please run this change on chromium trybots to make sure it doesn't
break
> chromium? See instructions on https://webrtc.org/contributing/#running-tryjobs

Sure, I will try it after change https://codereview.chromium.org/2495453002/.

[Sergey Ulanov, 2016-11-10 23:05:57]

lgtm

[Hzj_jie, 2016-11-11 01:58:02]

On 2016/11/10 23:05:57, Sergey Ulanov wrote:
> lgtm

It looks like the patch cannot work well with removed files,
Failed to apply patch for
webrtc/modules/desktop_capture/fake_screen_capturer.cc:
While running git rm webrtc/modules/desktop_capture/fake_screen_capturer.cc;
  fatal: pathspec 'webrtc/modules/desktop_capture/fake_screen_capturer.cc' did
not match any files

I will try the change on my local machine to ensure there is no build break, at
least on Linux and Windows.

[Hzj_jie, 2016-11-11 03:31:21]

Description was changed from

==========
Remove ScreenCapturer and WindowCapturer

This change removes ScreenCapturer and WindowCapturer from WebRTC. This change
is blocked by Chromium change https://codereview.chromium.org/2495453002.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:bot1,bot2;master.tryserver.chromium.mac:bot3;master.tryserver.chromium.win:bot4

BUG=webrtc:6513
==========

to

==========
Remove ScreenCapturer and WindowCapturer

This change removes ScreenCapturer and WindowCapturer from WebRTC.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:bot1,bot2;master.tryserver.chromium.mac:bot3;master.tryserver.chromium.win:bot4

BUG=webrtc:6513
==========

[Hzj_jie, 2016-11-11 05:25:50]

On 2016/11/11 01:58:02, Hzj_jie wrote:
> On 2016/11/10 23:05:57, Sergey Ulanov wrote:
> > lgtm
> 
> It looks like the patch cannot work well with removed files,
> Failed to apply patch for
> webrtc/modules/desktop_capture/fake_screen_capturer.cc:
> While running git rm webrtc/modules/desktop_capture/fake_screen_capturer.cc;
>   fatal: pathspec 'webrtc/modules/desktop_capture/fake_screen_capturer.cc' did
> not match any files
> 
> I will try the change on my local machine to ensure there is no build break,
at
> least on Linux and Windows.

I have built remoting, chrome, content folders on both Linux and Windows, no
errors found. Hope it won't break Mac. I will go ahead to submit this change
first.

[Hzj_jie, 2016-11-11 05:27:16]

The CQ bit was checked by zijiehe@chromium.org

[commit-bot: I haz the power, 2016-11-11 05:27:21]

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-11-11 05:27:49]

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

[commit-bot: I haz the power, 2016-11-11 05:27:49]

Try jobs failed on following builders:
  bot1 on master.tryserver.chromium.linux (JOB_FAILED, no build URL)
  bot2 on master.tryserver.chromium.linux (JOB_FAILED, no build URL)
  bot3 on master.tryserver.chromium.mac (JOB_FAILED, no build URL)
  bot4 on master.tryserver.chromium.win (JOB_FAILED, no build URL)

[Hzj_jie, 2016-11-11 05:48:01]

Description was changed from

==========
Remove ScreenCapturer and WindowCapturer

This change removes ScreenCapturer and WindowCapturer from WebRTC.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:bot1,bot2;master.tryserver.chromium.mac:bot3;master.tryserver.chromium.win:bot4

BUG=webrtc:6513
==========

to

==========
Remove ScreenCapturer and WindowCapturer

This change removes ScreenCapturer and WindowCapturer from WebRTC.

BUG=webrtc:6513
==========

[Hzj_jie, 2016-11-11 05:48:08]

The CQ bit was checked by zijiehe@chromium.org

[commit-bot: I haz the power, 2016-11-11 05:48:16]

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-11-11 05:57:14]

Description was changed from

==========
Remove ScreenCapturer and WindowCapturer

This change removes ScreenCapturer and WindowCapturer from WebRTC.

BUG=webrtc:6513
==========

to

==========
Remove ScreenCapturer and WindowCapturer

This change removes ScreenCapturer and WindowCapturer from WebRTC.

BUG=webrtc:6513
==========

[commit-bot: I haz the power, 2016-11-11 05:57:15]

Committed patchset #1 (id:80001)

[commit-bot: I haz the power, 2016-11-11 05:57:25]

Description was changed from

==========
Remove ScreenCapturer and WindowCapturer

This change removes ScreenCapturer and WindowCapturer from WebRTC.

BUG=webrtc:6513
==========

to

==========
Remove ScreenCapturer and WindowCapturer

This change removes ScreenCapturer and WindowCapturer from WebRTC.

BUG=webrtc:6513

Committed: https://crrev.com/98903d2f5ef420adb343358824b94731b4a00b38
Cr-Commit-Position: refs/heads/master@{#15033}
==========

[commit-bot: I haz the power, 2016-11-11 05:57:26]

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