Shelly Lin [:shelly] Assignee
	Description • 11 years ago

This is for landing Part 1 and Part 2 from bug 879669, for more details, please see bug 879669.

Carry r+ from roc.

Try result:
https://tbpl.mozilla.org/?tree=Try&rev=d38e4006114e

	Shelly Lin [:shelly] Assignee
	Comment 1 • 11 years ago

Carry r+ from bug 879669.

	Shelly Lin [:shelly] Assignee
	Comment 3 • 11 years ago

Comment on attachment 8340930 [details] [diff] [review]
Part2: Add VideoTrackEncoder

Review of attachment 8340930 [details] [diff] [review]:
-----------------------------------------------------------------

::: content/media/VideoSegment.cpp
@@ +31,5 @@
>  VideoFrame::TakeFrom(VideoFrame* aFrame)
>  {
>    mImage = aFrame->mImage.forget();
>    mIntrinsicSize = aFrame->mIntrinsicSize;
> +  mForceBlack = aFrame->GetForceBlack();

Added the fix where |TakeFrom(aFrame)| should also copy the value of mForceBlack from aFrame.

	Shelly Lin [:shelly] Assignee
	Comment 4 • 11 years ago

Try result for the second part:
https://tbpl.mozilla.org/?tree=Try&rev=c7a5a0456da3

	Ryan VanderMeulen [:RyanVM]
	Comment 5 • 11 years ago

https://hg.mozilla.org/integration/b2g-inbound/rev/72f2f506fe01
https://hg.mozilla.org/integration/b2g-inbound/rev/5dddd0b5ff51

	Ryan VanderMeulen [:RyanVM]
	Comment 6 • 11 years ago

https://hg.mozilla.org/mozilla-central/rev/72f2f506fe01
https://hg.mozilla.org/mozilla-central/rev/5dddd0b5ff51