Revert 276344 "Add callback in VideoDecoder and AudioDecoder to ..."

Revert 276344 "Add callback in VideoDecoder and AudioDecoder to ..."

Speculatively reverting ALL changes made between 37.0.2043 and 2044 to the media module to see if major regression disappears.  Every change affecting this module will be reverted on branch only as an experiment over the weekend.

> Add callback in VideoDecoder and AudioDecoder to return decoded frames.
> 
> Previously decoded frames were returned using the DecodeCB callback,
> which is called in response to Decode(). This didn't work very well,
> particularly for GPU decoder, because decode buffers do not always map
> to decoded frames 1 to 1.
> 
> BUG=338529
> 
> Review URL: https://codereview.chromium.org/297553002

TBR=sergeyu@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=277176

[amineer_google, 2014-06-14 03:53:09]

Committed patchset #1 manually as r277176 (tree was closed).

[xhwang, 2014-06-14 15:38:44]

On 2014/06/14 03:53:09, amineer wrote:
> Committed patchset #1 manually as r277176 (tree was closed).

Is there a bug tracking the regression you mentioned?

[chromium-reviews, 2014-06-14 16:53:38]

https://code.google.com/p/chromium/issues/detail?id=383715

I already talked to Sergey about this, and while his explanation of why his
change is unrelated made sense to me, we still have some major challenges
(the bug above shows fixed, but only because of a blacklist, and we now
have other issues cropping up as a result) in the code.  I was trying a
blanket revert of each and every media module change in the CL range but
unfortunately it failed compile.  I'll be working this more tomorrow /
Monday but any assistance you might be able to provide would be greatly
appreciated.


On Sat, Jun 14, 2014 at 8:38 AM, <xhwang@chromium.org> wrote:

> On 2014/06/14 03:53:09, amineer wrote:
>
>> Committed patchset #1 manually as r277176 (tree was closed).
>>
>
> Is there a bug tracking the regression you mentioned?
>
> https://codereview.chromium.org/331863004/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.