Deleted VideoCapturer::screencast_max_pixels and related unused code.

Deleted VideoCapturer::screencast_max_pixels, together with
VideoChannel::GetScreencastMaxPixels and VideoChannel::GetScreencastFps.

Unused in webrtc, also unused in everything indexed by google and chromium code search. With the exception of the magicflute plugin, which I'm told doesn't matter.

Committed: https://crrev.com/e6bf587259da23e96a8de0957b172fd74c36c3c6
Cr-Commit-Position: refs/heads/master@{#11108}

[nisse-webrtc, 2015-12-18 13:08:12]

The CQ bit was checked by nisse@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2015-12-18 13:08:23]

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

[nisse-webrtc, 2015-12-18 13:18:44]

Description was changed from

==========
Deleted VideoCapturer::screencast_max_pixels.

Unused in webrtc, also unused in everything indexed by google code
search.

In contrast, there's one match for GetScreencastFps, in
//depot/googleclient/talk/magicflute/flutemediasession.cc&q=GetScreencastFps&ct=rc&cd=5&dr=CSs
(not entirely sure if that's related though), so keep
VideoChannel::GetScreencastFps for now.

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

to

==========
Deleted VideoCapturer::screencast_max_pixels, together with 
VideoChannel::GetScreencastMaxPixels and VideoChannel::GetScreencastFps.

Unused in webrtc, also unused in everything indexed by google and chromium code
search. With the exception of the magicflute plugin, which I'm told doesn't
matter.
==========

[nisse-webrtc, 2015-12-18 13:18:44]

nisse@webrtc.org changed reviewers:
+ magjed@webrtc.org, perkj@webrtc.org, pthatcher@webrtc.org

[nisse-webrtc, 2015-12-18 13:20:38]

The CQ bit was checked by nisse@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2015-12-18 13:20:47]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1532133002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1532133002/40001

[commit-bot: I haz the power, 2015-12-18 13:26:23]

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

[commit-bot: I haz the power, 2015-12-18 13:26:25]

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

[nisse-webrtc, 2015-12-18 14:25:41]

Please have a look, I'd like to remove this apparently unused feature. (I'm
going to be at work on Monday, Dec 21, but I don't really expect to do anything
with this cl until after the holidays).

[nisse-webrtc, 2015-12-18 14:31:07]

The CQ bit was checked by nisse@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2015-12-18 14:31:24]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1532133002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1532133002/60001

[commit-bot: I haz the power, 2015-12-18 16:32:19]

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

[commit-bot: I haz the power, 2015-12-18 16:32:19]

Dry run: Try jobs failed on following builders:
  android_arm64_rel on tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  linux_baremetal on tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  mac_baremetal on tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  win_baremetal on tryserver.webrtc (JOB_TIMED_OUT, no build URL)

[pthatcher1, 2015-12-18 21:16:24]

lgtm

[magjed_webrtc, 2015-12-21 11:22:00]

lgtm

[nisse-webrtc, 2015-12-21 12:09:56]

Thanks. 

I don't want to land it just before going on vacation. If any of you are in
office and ready to monitor any damage, feel free to commit, otherwise, I'll do
that after I'm back January 7.

/Niels

[pthatcher, 2015-12-21 20:21:20]

The CQ bit was checked by pthatcher@google.com

[commit-bot: I haz the power, 2015-12-21 20:21:29]

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

[commit-bot: I haz the power, 2015-12-21 21:18:17]

Description was changed from

==========
Deleted VideoCapturer::screencast_max_pixels, together with 
VideoChannel::GetScreencastMaxPixels and VideoChannel::GetScreencastFps.

Unused in webrtc, also unused in everything indexed by google and chromium code
search. With the exception of the magicflute plugin, which I'm told doesn't
matter.
==========

to

==========
Deleted VideoCapturer::screencast_max_pixels, together with 
VideoChannel::GetScreencastMaxPixels and VideoChannel::GetScreencastFps.

Unused in webrtc, also unused in everything indexed by google and chromium code
search. With the exception of the magicflute plugin, which I'm told doesn't
matter.
==========

[commit-bot: I haz the power, 2015-12-21 21:18:17]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-12-21 21:18:19]

Description was changed from

==========
Deleted VideoCapturer::screencast_max_pixels, together with 
VideoChannel::GetScreencastMaxPixels and VideoChannel::GetScreencastFps.

Unused in webrtc, also unused in everything indexed by google and chromium code
search. With the exception of the magicflute plugin, which I'm told doesn't
matter.
==========

to

==========
Deleted VideoCapturer::screencast_max_pixels, together with
VideoChannel::GetScreencastMaxPixels and VideoChannel::GetScreencastFps.

Unused in webrtc, also unused in everything indexed by google and chromium code
search. With the exception of the magicflute plugin, which I'm told doesn't
matter.

Committed: https://crrev.com/e6bf587259da23e96a8de0957b172fd74c36c3c6
Cr-Commit-Position: refs/heads/master@{#11108}
==========

[commit-bot: I haz the power, 2015-12-21 21:18:20]

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