Bug 88814

Summary: [chromium] Use TextureDrawQuad for video layer output when it produces a single GL texture
Product: WebKit Reporter: Dana Jansens <danakj>
Component: New BugsAssignee: Dana Jansens <danakj>
Status: RESOLVED FIXED    
Severity: Normal CC: cc-bugs, enne, eric.carlson, feature-media-reviews, fischman, jamesr, piman, webkit.review.bot, wjmaclean
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 88363    
Bug Blocks: 88826    
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Patch enne: review+

Dana Jansens
Reported 2012-06-11 15:35:01 PDT
[chromium] Use TextureDrawQuad for video layer output when it produces a single GL texture
Attachments
Patch (7.00 KB, patch)
2012-06-11 15:44 PDT, Dana Jansens 		no flags
DetailsFormatted DiffDiff
Patch (7.06 KB, patch)
2012-06-11 16:12 PDT, Dana Jansens 		no flags
DetailsFormatted DiffDiff
Patch (9.08 KB, patch)
2012-06-11 17:17 PDT, Dana Jansens 		no flags
DetailsFormatted DiffDiff
Patch (9.34 KB, patch)
2012-06-11 17:30 PDT, Dana Jansens 		enne: review+
DetailsFormatted DiffDiff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.
Dana Jansens
Comment 1 2012-06-11 15:44:25 PDT
Created attachment 146937 [details] Patch
Dana Jansens
Comment 2 2012-06-11 16:12:51 PDT
Created attachment 146949 [details] Patch rebase
Adrienne Walker
Comment 3 2012-06-11 16:31:04 PDT
Comment on attachment 146949 [details] Patch R=me.
Dana Jansens
Comment 4 2012-06-11 16:31:47 PDT
Thanks, gonna let EWS chew on this still.
Dana Jansens
Comment 5 2012-06-11 17:17:48 PDT
Created attachment 146971 [details] Patch
Dana Jansens
Comment 6 2012-06-11 17:18:10 PDT
The diff http://pastebin.com/1PvfSQvL
Dana Jansens
Comment 7 2012-06-11 17:30:42 PDT
Created attachment 146975 [details] Patch regenerated changelog and remove all removed methods from LRC header. enne can you please double check the drawStreamTexture() code for me?
Adrienne Walker
Comment 8 2012-06-11 17:42:40 PDT
Comment on attachment 146975 [details] Patch R=me. Still looks good.
Dana Jansens
Comment 9 2012-06-12 07:39:42 PDT
Committed r120073: <http://trac.webkit.org/changeset/120073>
Note You need to log in before you can comment on or make changes to this bug.