Revert of Delete the class WebRtcVideoCapturerAdapter::MediaVideoFrameFactory

Revert of Delete the class WebRtcVideoCapturerAdapter::MediaVideoFrameFactory (patchset #5 id:80001 of https://codereview.chromium.org/2244213002/ )

Reason for revert:
Somehow breaks timestamping. Reverting for investigation. To reproduce, run

./browser_tests --gtest_filter='WebRtcVideoQualityBrowserTests/WebRtcVideoQualityBrowserTest*' --run-manual --ui-test-action-max-timeout=200000

This produces lots of warnings like

[1:14:0822/153830:WARNING:video_capture_input.cc(79)] Same/old NTP timestamp (3680861906110 <= 3680861906110) for incoming frame. Dropping.
[1:14:0822/153830:WARNING:video_capture_input.cc(79)] Same/old NTP timestamp (3680861906042 <= 3680861906042) for incoming frame. Dropping.

Original issue's description:
> Delete the class WebRtcVideoCapturerAdapter::MediaVideoFrameFactory
>
> Instead, use the AdaptFrame and OnFrame methods of the VideoCapturer
> base class.
>
> This cl also applies the new timestamp aligner logic, to translate
> Chrome's camera timestamps into the timescale of rtc::TimeMicros(). No
> other change in behavior is intended.
>
> BUG=webrtc:5682, webrtc:5740, 516700
>
> Committed: https://crrev.com/5b9a4b339447b033ca52244a02c0bef17067385f
> Cr-Commit-Position: refs/heads/master@{#413093}

TBR=perkj@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=webrtc:5682, webrtc:5740, 516700
Committed: https://crrev.com/e977fbdcd25a49046e23976dfe2f0a667c4ad037
Cr-Commit-Position: refs/heads/master@{#413444}

[nisse-chromium (ooo August 14), 2016-08-22 13:47:27]

The CQ bit was checked by nisse@chromium.org

[nisse-chromium (ooo August 14), 2016-08-22 13:47:28]

Created Revert of Delete the class
WebRtcVideoCapturerAdapter::MediaVideoFrameFactory

[commit-bot: I haz the power, 2016-08-22 13:47:40]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-22 13:47:40]

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

[commit-bot: I haz the power, 2016-08-22 13:47:41]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[kjellander_chromium, 2016-08-22 13:54:37]

kjellander@chromium.org changed reviewers:
+ kjellander@chromium.org

[kjellander_chromium, 2016-08-22 13:54:37]

The CQ bit was checked by kjellander@chromium.org

[kjellander_chromium, 2016-08-22 13:54:37]

lgtm

[commit-bot: I haz the power, 2016-08-22 13:54:49]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-22 15:15:52]

Description was changed from

==========
Revert of Delete the class WebRtcVideoCapturerAdapter::MediaVideoFrameFactory
(patchset #5 id:80001 of https://codereview.chromium.org/2244213002/ )

Reason for revert:
Somehow breaks timestamping. Reverting for investigation. To reproduce, run

./browser_tests
--gtest_filter='WebRtcVideoQualityBrowserTests/WebRtcVideoQualityBrowserTest*'
--run-manual --ui-test-action-max-timeout=200000

This produces lots of warnings like

[1:14:0822/153830:WARNING:video_capture_input.cc(79)] Same/old NTP timestamp
(3680861906110 <= 3680861906110) for incoming frame. Dropping.
[1:14:0822/153830:WARNING:video_capture_input.cc(79)] Same/old NTP timestamp
(3680861906042 <= 3680861906042) for incoming frame. Dropping.


Original issue's description:
> Delete the class WebRtcVideoCapturerAdapter::MediaVideoFrameFactory
> 
> Instead, use the AdaptFrame and OnFrame methods of the VideoCapturer
> base class.
> 
> This cl also applies the new timestamp aligner logic, to translate
> Chrome's camera timestamps into the timescale of rtc::TimeMicros(). No
> other change in behavior is intended.
> 
> BUG=webrtc:5682, webrtc:5740, 516700
> 
> Committed: https://crrev.com/5b9a4b339447b033ca52244a02c0bef17067385f
> Cr-Commit-Position: refs/heads/master@{#413093}

TBR=perkj@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=webrtc:5682, webrtc:5740, 516700
==========

to

==========
Revert of Delete the class WebRtcVideoCapturerAdapter::MediaVideoFrameFactory
(patchset #5 id:80001 of https://codereview.chromium.org/2244213002/ )

Reason for revert:
Somehow breaks timestamping. Reverting for investigation. To reproduce, run

./browser_tests
--gtest_filter='WebRtcVideoQualityBrowserTests/WebRtcVideoQualityBrowserTest*'
--run-manual --ui-test-action-max-timeout=200000

This produces lots of warnings like

[1:14:0822/153830:WARNING:video_capture_input.cc(79)] Same/old NTP timestamp
(3680861906110 <= 3680861906110) for incoming frame. Dropping.
[1:14:0822/153830:WARNING:video_capture_input.cc(79)] Same/old NTP timestamp
(3680861906042 <= 3680861906042) for incoming frame. Dropping.


Original issue's description:
> Delete the class WebRtcVideoCapturerAdapter::MediaVideoFrameFactory
> 
> Instead, use the AdaptFrame and OnFrame methods of the VideoCapturer
> base class.
> 
> This cl also applies the new timestamp aligner logic, to translate
> Chrome's camera timestamps into the timescale of rtc::TimeMicros(). No
> other change in behavior is intended.
> 
> BUG=webrtc:5682, webrtc:5740, 516700
> 
> Committed: https://crrev.com/5b9a4b339447b033ca52244a02c0bef17067385f
> Cr-Commit-Position: refs/heads/master@{#413093}

TBR=perkj@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=webrtc:5682, webrtc:5740, 516700
==========

[commit-bot: I haz the power, 2016-08-22 15:15:54]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-22 15:17:55]

Description was changed from

==========
Revert of Delete the class WebRtcVideoCapturerAdapter::MediaVideoFrameFactory
(patchset #5 id:80001 of https://codereview.chromium.org/2244213002/ )

Reason for revert:
Somehow breaks timestamping. Reverting for investigation. To reproduce, run

./browser_tests
--gtest_filter='WebRtcVideoQualityBrowserTests/WebRtcVideoQualityBrowserTest*'
--run-manual --ui-test-action-max-timeout=200000

This produces lots of warnings like

[1:14:0822/153830:WARNING:video_capture_input.cc(79)] Same/old NTP timestamp
(3680861906110 <= 3680861906110) for incoming frame. Dropping.
[1:14:0822/153830:WARNING:video_capture_input.cc(79)] Same/old NTP timestamp
(3680861906042 <= 3680861906042) for incoming frame. Dropping.


Original issue's description:
> Delete the class WebRtcVideoCapturerAdapter::MediaVideoFrameFactory
> 
> Instead, use the AdaptFrame and OnFrame methods of the VideoCapturer
> base class.
> 
> This cl also applies the new timestamp aligner logic, to translate
> Chrome's camera timestamps into the timescale of rtc::TimeMicros(). No
> other change in behavior is intended.
> 
> BUG=webrtc:5682, webrtc:5740, 516700
> 
> Committed: https://crrev.com/5b9a4b339447b033ca52244a02c0bef17067385f
> Cr-Commit-Position: refs/heads/master@{#413093}

TBR=perkj@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=webrtc:5682, webrtc:5740, 516700
==========

to

==========
Revert of Delete the class WebRtcVideoCapturerAdapter::MediaVideoFrameFactory
(patchset #5 id:80001 of https://codereview.chromium.org/2244213002/ )

Reason for revert:
Somehow breaks timestamping. Reverting for investigation. To reproduce, run

./browser_tests
--gtest_filter='WebRtcVideoQualityBrowserTests/WebRtcVideoQualityBrowserTest*'
--run-manual --ui-test-action-max-timeout=200000

This produces lots of warnings like

[1:14:0822/153830:WARNING:video_capture_input.cc(79)] Same/old NTP timestamp
(3680861906110 <= 3680861906110) for incoming frame. Dropping.
[1:14:0822/153830:WARNING:video_capture_input.cc(79)] Same/old NTP timestamp
(3680861906042 <= 3680861906042) for incoming frame. Dropping.

Original issue's description:
> Delete the class WebRtcVideoCapturerAdapter::MediaVideoFrameFactory
>
> Instead, use the AdaptFrame and OnFrame methods of the VideoCapturer
> base class.
>
> This cl also applies the new timestamp aligner logic, to translate
> Chrome's camera timestamps into the timescale of rtc::TimeMicros(). No
> other change in behavior is intended.
>
> BUG=webrtc:5682, webrtc:5740, 516700
>
> Committed: https://crrev.com/5b9a4b339447b033ca52244a02c0bef17067385f
> Cr-Commit-Position: refs/heads/master@{#413093}

TBR=perkj@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=webrtc:5682, webrtc:5740, 516700

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

[commit-bot: I haz the power, 2016-08-22 15:17:56]

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

[nisse-chromium (ooo August 14), 2016-08-23 10:03:27]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2270723002/ by nisse@chromium.org.

The reason for reverting is: Identified timestamp problem. Intending to fix and
reland..