Revert of Add framerate to VideoSinkWants and ability to signal on overuse

Revert of Add framerate to VideoSinkWants and ability to signal on overuse (patchset #8 id:410001 of https://codereview.webrtc.org/2781433002/ )

Reason for revert:
This has resulted in failure of CallPerfTest.ReceivesCpuOveruseAndUnderuse test on the Win7 build bot https://build.chromium.org/p/client.webrtc.perf/builders/Win7/builds/1780

Original issue's description:
> Reland of Add framerate to VideoSinkWants and ability to signal on overuse (patchset #1 id:1 of https://codereview.webrtc.org/2764133002/ )
> 
> Reason for revert:
> Found issue with test case, will add fix to reland cl.
> 
> Original issue's description:
> > Revert of Add framerate to VideoSinkWants and ability to signal on overuse (patchset #14 id:250001 of https://codereview.webrtc.org/2716643002/ )
> > 
> > Reason for revert:
> > Breaks perf tests:
> > https://build.chromium.org/p/client.webrtc.perf/builders/Win7/builds/1679
> > https://build.chromium.org/p/client.webrtc.perf/builders/Android32%20Tests%20%28L%20Nexus5%29/builds/2325
> > 
> > Original issue's description:
> > > Add framerate to VideoSinkWants and ability to signal on overuse
> > > 
> > > In ViEEncoder, try to reduce framerate instead of resolution if the
> > > current degradation preference is maintain-resolution rather than
> > > balanced.
> > > 
> > > BUG=webrtc:4172
> > > 
> > > Review-Url: https://codereview.webrtc.org/2716643002
> > > Cr-Commit-Position: refs/heads/master@{#17327}
> > > Committed: https://chromium.googlesource.com/external/webrtc/+/72acf2526177bb4dbb5103cd6e165eb4361a5ae6
> > 
> > TBR=nisse@webrtc.org,magjed@webrtc.org,kthelgason@webrtc.org,ilnik@webrtc.org,stefan@webrtc.org,sprang@webrtc.org
> > # Skipping CQ checks because original CL landed less than 1 days ago.
> > NOPRESUBMIT=true
> > NOTREECHECKS=true
> > NOTRY=true
> > BUG=webrtc:4172
> > 
> > Review-Url: https://codereview.webrtc.org/2764133002
> > Cr-Commit-Position: refs/heads/master@{#17331}
> > Committed: https://chromium.googlesource.com/external/webrtc/+/8b45b11144c968b4173215c76f78c710c9a2ed0b
> 
> TBR=nisse@webrtc.org,magjed@webrtc.org,kthelgason@webrtc.org,ilnik@webrtc.org,stefan@webrtc.org,skvlad@webrtc.org
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> BUG=webrtc:4172
> 
> Review-Url: https://codereview.webrtc.org/2781433002
> Cr-Commit-Position: refs/heads/master@{#17474}
> Committed: https://chromium.googlesource.com/external/webrtc/+/3ea3c77e93121b1ab9d5e46641e6764f2cca0d51

TBR=ilnik@webrtc.org,stefan@webrtc.org,asapersson@webrtc.org,sprang@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:4172
Review-Url: https://codereview.webrtc.org/2783183003
Cr-Commit-Position: refs/heads/master@{#17477}
Committed: https://chromium.googlesource.com/external/webrtc/+/f9ed235c9b7248694edcb46feb1f29ce7456ab59

[lliuu, 2017-03-30 17:44:26]

The CQ bit was checked by lliuu@webrtc.org

[lliuu, 2017-03-30 17:44:26]

Created Revert of Add framerate to VideoSinkWants and ability to signal on
overuse

[commit-bot: I haz the power, 2017-03-30 17:44:31]

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-30 17:44:39]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490895866041000, "parent_rev":
"7a3615b6b394bfe12a0ab528dee0dd33620e0e63", "commit_rev":
"f9ed235c9b7248694edcb46feb1f29ce7456ab59"}

[commit-bot: I haz the power, 2017-03-30 17:44:43]

Description was changed from

==========
Revert of Add framerate to VideoSinkWants and ability to signal on overuse
(patchset #8 id:410001 of https://codereview.webrtc.org/2781433002/ )

Reason for revert:
This has resulted in failure of CallPerfTest.ReceivesCpuOveruseAndUnderuse test
on the Win7 build bot
https://build.chromium.org/p/client.webrtc.perf/builders/Win7/builds/1780

Original issue's description:
> Reland of Add framerate to VideoSinkWants and ability to signal on overuse
(patchset #1 id:1 of https://codereview.webrtc.org/2764133002/ )
> 
> Reason for revert:
> Found issue with test case, will add fix to reland cl.
> 
> Original issue's description:
> > Revert of Add framerate to VideoSinkWants and ability to signal on overuse
(patchset #14 id:250001 of https://codereview.webrtc.org/2716643002/ )
> > 
> > Reason for revert:
> > Breaks perf tests:
> > https://build.chromium.org/p/client.webrtc.perf/builders/Win7/builds/1679
> >
https://build.chromium.org/p/client.webrtc.perf/builders/Android32%20Tests%20...
> > 
> > Original issue's description:
> > > Add framerate to VideoSinkWants and ability to signal on overuse
> > > 
> > > In ViEEncoder, try to reduce framerate instead of resolution if the
> > > current degradation preference is maintain-resolution rather than
> > > balanced.
> > > 
> > > BUG=webrtc:4172
> > > 
> > > Review-Url: https://codereview.webrtc.org/2716643002
> > > Cr-Commit-Position: refs/heads/master@{#17327}
> > > Committed:
https://chromium.googlesource.com/external/webrtc/+/72acf2526177bb4dbb5103cd6...
> > 
> >
TBR=nisse@webrtc.org,magjed@webrtc.org,kthelgason@webrtc.org,ilnik@webrtc.org...
> > # Skipping CQ checks because original CL landed less than 1 days ago.
> > NOPRESUBMIT=true
> > NOTREECHECKS=true
> > NOTRY=true
> > BUG=webrtc:4172
> > 
> > Review-Url: https://codereview.webrtc.org/2764133002
> > Cr-Commit-Position: refs/heads/master@{#17331}
> > Committed:
https://chromium.googlesource.com/external/webrtc/+/8b45b11144c968b4173215c76...
> 
>
TBR=nisse@webrtc.org,magjed@webrtc.org,kthelgason@webrtc.org,ilnik@webrtc.org...
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> BUG=webrtc:4172
> 
> Review-Url: https://codereview.webrtc.org/2781433002
> Cr-Commit-Position: refs/heads/master@{#17474}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/3ea3c77e93121b1ab9d5e4664...

TBR=ilnik@webrtc.org,stefan@webrtc.org,asapersson@webrtc.org,sprang@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:4172
==========

to

==========
Revert of Add framerate to VideoSinkWants and ability to signal on overuse
(patchset #8 id:410001 of https://codereview.webrtc.org/2781433002/ )

Reason for revert:
This has resulted in failure of CallPerfTest.ReceivesCpuOveruseAndUnderuse test
on the Win7 build bot
https://build.chromium.org/p/client.webrtc.perf/builders/Win7/builds/1780

Original issue's description:
> Reland of Add framerate to VideoSinkWants and ability to signal on overuse
(patchset #1 id:1 of https://codereview.webrtc.org/2764133002/ )
> 
> Reason for revert:
> Found issue with test case, will add fix to reland cl.
> 
> Original issue's description:
> > Revert of Add framerate to VideoSinkWants and ability to signal on overuse
(patchset #14 id:250001 of https://codereview.webrtc.org/2716643002/ )
> > 
> > Reason for revert:
> > Breaks perf tests:
> > https://build.chromium.org/p/client.webrtc.perf/builders/Win7/builds/1679
> >
https://build.chromium.org/p/client.webrtc.perf/builders/Android32%20Tests%20...
> > 
> > Original issue's description:
> > > Add framerate to VideoSinkWants and ability to signal on overuse
> > > 
> > > In ViEEncoder, try to reduce framerate instead of resolution if the
> > > current degradation preference is maintain-resolution rather than
> > > balanced.
> > > 
> > > BUG=webrtc:4172
> > > 
> > > Review-Url: https://codereview.webrtc.org/2716643002
> > > Cr-Commit-Position: refs/heads/master@{#17327}
> > > Committed:
https://chromium.googlesource.com/external/webrtc/+/72acf2526177bb4dbb5103cd6...
> > 
> >
TBR=nisse@webrtc.org,magjed@webrtc.org,kthelgason@webrtc.org,ilnik@webrtc.org...
> > # Skipping CQ checks because original CL landed less than 1 days ago.
> > NOPRESUBMIT=true
> > NOTREECHECKS=true
> > NOTRY=true
> > BUG=webrtc:4172
> > 
> > Review-Url: https://codereview.webrtc.org/2764133002
> > Cr-Commit-Position: refs/heads/master@{#17331}
> > Committed:
https://chromium.googlesource.com/external/webrtc/+/8b45b11144c968b4173215c76...
> 
>
TBR=nisse@webrtc.org,magjed@webrtc.org,kthelgason@webrtc.org,ilnik@webrtc.org...
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> BUG=webrtc:4172
> 
> Review-Url: https://codereview.webrtc.org/2781433002
> Cr-Commit-Position: refs/heads/master@{#17474}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/3ea3c77e93121b1ab9d5e4664...

TBR=ilnik@webrtc.org,stefan@webrtc.org,asapersson@webrtc.org,sprang@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:4172

Review-Url: https://codereview.webrtc.org/2783183003
Cr-Commit-Position: refs/heads/master@{#17477}
Committed:
https://chromium.googlesource.com/external/webrtc/+/f9ed235c9b7248694edcb46fe...
==========

[commit-bot: I haz the power, 2017-03-30 17:44:44]

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

[sprang_webrtc, 2017-03-31 08:50:29]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.webrtc.org/2789823002/ by sprang@webrtc.org.

The reason for reverting is: Seem to be a flaky test rather than an issue with
this cl. Creating reland, will add code to reduce flakiness to that test..