Revert of Tab/Desktop Capture: Use requests instead of timer-based refreshing.

Revert of Tab/Desktop Capture: Use requests instead of timer-based refreshing. (patchset #3 id:40001 of https://codereview.chromium.org/1864813002/ )

Reason for revert:
This has caused > 50 tryjobs to flake so far

BUG=601308
Original issue's description:
> Tab/Desktop Capture: Use requests instead of timer-based refreshing.
> 
> Remove the long-standing, hacky timer-based frame capture mechanisms
> from the tab (and Desktop-on-Aura) capture implementations.  Instead,
> downstream consumers will now make refresh frame requests when needed,
> and at the desired frequency for their specific use cases.
> 
> This change depends on https://codereview.chromium.org/1849003002/.
> 
> Also: 1) Deleted support implementation for the timer mechanisms from
> SmoothEventSampler, 2) updated WCVCD unit tests to only expect frames
> when explicitly requested, and 3) the NotificationCenter mechanism was
> removed from the WCVCD unit tests because it was removed from the impl
> long ago (but the unit tests were erroneously passing due to the timer
> refreshes).
> 
> BUG=486274
> 
> Committed: https://crrev.com/e3d466694d3a14958b478408577158c4c6804fe4
> Cr-Commit-Position: refs/heads/master@{#385668}

TBR=xjz@chromium.org,isheriff@chromium.org,mcasas@chromium.org,miu@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=486274

[jam, 2016-04-07 21:46:36]

Created Revert of Tab/Desktop Capture: Use requests instead of timer-based
refreshing.

[jam, 2016-04-07 21:47:06]

The CQ bit was unchecked by jam@chromium.org

[commit-bot: I haz the power, 2016-04-07 21:47:13]

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

[commit-bot: I haz the power, 2016-04-07 21:47:31]

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

[jam, 2016-04-07 21:47:43]

On 2016/04/07 21:46:36, jam wrote:
> Created Revert of Tab/Desktop Capture: Use requests instead of timer-based
> refreshing.

nvm, I see a fix just landed 15 minutes ago.

[miu, 2016-04-07 21:48:29]

Nononono!  You just broke the build again.  This was fixed in:
https://codereview.chromium.org/1871663002/

[miu, 2016-04-07 21:49:03]

On 2016/04/07 21:48:29, miu wrote:
> Nononono!  You just broke the build again.  This was fixed in:
> https://codereview.chromium.org/1871663002/

Ah, ok.  We're good!  :)

[miu, 2016-04-07 21:49:25]

Description was changed from

==========
Revert of Tab/Desktop Capture: Use requests instead of timer-based refreshing.
(patchset #3 id:40001 of https://codereview.chromium.org/1864813002/ )

Reason for revert:
This has caused > 50 tryjobs to flake so far

BUG=601308

Original issue's description:
> Tab/Desktop Capture: Use requests instead of timer-based refreshing.
> 
> Remove the long-standing, hacky timer-based frame capture mechanisms
> from the tab (and Desktop-on-Aura) capture implementations.  Instead,
> downstream consumers will now make refresh frame requests when needed,
> and at the desired frequency for their specific use cases.
> 
> This change depends on https://codereview.chromium.org/1849003002/.
> 
> Also: 1) Deleted support implementation for the timer mechanisms from
> SmoothEventSampler, 2) updated WCVCD unit tests to only expect frames
> when explicitly requested, and 3) the NotificationCenter mechanism was
> removed from the WCVCD unit tests because it was removed from the impl
> long ago (but the unit tests were erroneously passing due to the timer
> refreshes).
> 
> BUG=486274
> 
> Committed: https://crrev.com/e3d466694d3a14958b478408577158c4c6804fe4
> Cr-Commit-Position: refs/heads/master@{#385668}

TBR=xjz@chromium.org,isheriff@chromium.org,mcasas@chromium.org,miu@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=486274
==========

to

==========
Revert of Tab/Desktop Capture: Use requests instead of timer-based refreshing.
(patchset #3 id:40001 of https://codereview.chromium.org/1864813002/ )

Reason for revert:
This has caused > 50 tryjobs to flake so far

BUG=601308

Original issue's description:
> Tab/Desktop Capture: Use requests instead of timer-based refreshing.
> 
> Remove the long-standing, hacky timer-based frame capture mechanisms
> from the tab (and Desktop-on-Aura) capture implementations.  Instead,
> downstream consumers will now make refresh frame requests when needed,
> and at the desired frequency for their specific use cases.
> 
> This change depends on https://codereview.chromium.org/1849003002/.
> 
> Also: 1) Deleted support implementation for the timer mechanisms from
> SmoothEventSampler, 2) updated WCVCD unit tests to only expect frames
> when explicitly requested, and 3) the NotificationCenter mechanism was
> removed from the WCVCD unit tests because it was removed from the impl
> long ago (but the unit tests were erroneously passing due to the timer
> refreshes).
> 
> BUG=486274
> 
> Committed: https://crrev.com/e3d466694d3a14958b478408577158c4c6804fe4
> Cr-Commit-Position: refs/heads/master@{#385668}

TBR=xjz@chromium.org,isheriff@chromium.org,mcasas@chromium.org,miu@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=486274
==========