Bug 61687

Summary: [Texmap][Qt] Disable accelerated plugins/media until they're working.
Product: WebKit Reporter: Noam Rosenthal <noam>
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: ademar, commit-queue, kenneth, laszlo.gombos
Priority: P2 Keywords: Qt
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Noam Rosenthal
Reported 2011-05-28 09:28:45 PDT
This will allow us to enable texture mapper without causing regressions.
Attachments
Patch (3.78 KB, patch)
2011-05-28 09:35 PDT, Noam Rosenthal 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Noam Rosenthal
Comment 1 2011-05-28 09:35:44 PDT
Created attachment 95263 [details] Patch
WebKit Commit Bot
Comment 2 2011-05-30 11:28:21 PDT
Comment on attachment 95263 [details] Patch Clearing flags on attachment: 95263 Committed r87694: <http://trac.webkit.org/changeset/87694>
WebKit Commit Bot
Comment 3 2011-05-30 11:28:26 PDT
All reviewed patches have been landed. Closing bug.
Ademar Reis
Comment 4 2011-05-30 12:04:14 PDT
The changelog reads: """ Until this works properly, we should indicate that our media implementation in texture mapper doesn't support accelerated compositing. This makes LayoutTests/compositing/video/* work again. """ Does it mean that the current implementation (as in qtwebkit-2.2, for example) is broken? I'm asking because if the plugins are broken with AC, then we better disable it on the branch as well (cherry-pick this commit).
Noam Rosenthal
Comment 5 2011-05-30 12:07:53 PDT
(In reply to comment #4) > The changelog reads: > > """ > Until this works properly, we should indicate that our media implementation in texture mapper doesn't support accelerated compositing. > > This makes LayoutTests/compositing/video/* work again. > """ > > Does it mean that the current implementation (as in qtwebkit-2.2, for example) is broken? I'm asking because if the plugins are broken with AC, then we better disable it on the branch as well (cherry-pick this commit). No, this is just for texture-mapper. Doesn't apply to the release branch :)
Note You need to log in before you can comment on or make changes to this bug.