Bug 81250 - [chromium] Don't create a quad for video layers without a frame (avoid use of uninitialized m_format)
Summary: [chromium] Don't create a quad for video layers without a frame (avoid use of...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Dana Jansens
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-03-15 11:46 PDT by Dana Jansens
Modified: 2012-03-16 01:20 PDT (History)
5 users (show)

See Also:


Attachments
Patch (1.53 KB, patch)
2012-03-15 11:48 PDT, Dana Jansens
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dana Jansens 2012-03-15 11:46:54 PDT
[chromium] Don't create a quad for video layers without a frame (avoid use of uninitialized m_format)
Comment 1 Dana Jansens 2012-03-15 11:48:51 PDT
Created attachment 132093 [details]
Patch
Comment 2 James Robinson 2012-03-15 11:51:44 PDT
Comment on attachment 132093 [details]
Patch

R=me. Any way to get more direct test coverage of this?
Comment 3 Dana Jansens 2012-03-15 12:01:47 PDT
I don't see a nice place to insert a unit test, but I guess this code path is exercised by layout tests already since valgrind noticed it?
Comment 4 WebKit Review Bot 2012-03-16 01:20:21 PDT
Comment on attachment 132093 [details]
Patch

Clearing flags on attachment: 132093

Committed r110964: <http://trac.webkit.org/changeset/110964>
Comment 5 WebKit Review Bot 2012-03-16 01:20:25 PDT
All reviewed patches have been landed.  Closing bug.