Bug 131488 - [GTK] Regression: video is gone after r166889
Summary: [GTK] Regression: video is gone after r166889
Status: RESOLVED WORKSFORME
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-04-10 02:44 PDT by Xabier Rodríguez Calvar
Modified: 2014-04-10 18:29 PDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Xabier Rodríguez Calvar 2014-04-10 02:44:40 PDT
I bisected the problem to that revision.
Comment 1 Xabier Rodríguez Calvar 2014-04-10 04:18:13 PDT

*** This bug has been marked as a duplicate of bug 131471 ***
Comment 2 Sergio Villar Senin 2014-04-10 04:31:49 PDT
I don't think this is a dup at all. Video should be working in the software path as it was before martin's patch.

Also moving to std::unique_ptr shouldn't cause this kind of issues, we need to review the changes in r166889.
Comment 3 Sergio Villar Senin 2014-04-10 04:34:19 PDT
Reopening it for further investigation
Comment 4 Sergio Villar Senin 2014-04-10 04:35:51 PDT
Video rendering was broken after r166889
Comment 5 Xabier Rodríguez Calvar 2014-04-10 09:14:50 PDT
It looks like r166889 forced the sofware AC path which is currently not working. Bug 131471 turns the hardware AC path back on and fixes the issue, hence closing.
Comment 6 Hyowon Kim 2014-04-10 18:29:13 PDT
I'm sorry about the inconvenience caused by my patch.