[Webkit-unassigned] [Bug 42234] Implementing VideoLayerChromium for gpu acceleration

Tue Jul 13 23:25:05 PDT 2010

https://bugs.webkit.org/show_bug.cgi?id=42234

David Levin <levin at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |ASSIGNED
         AssignedTo|webkit-unassigned at lists.web |vrk at google.com
                   |kit.org                     |
                 CC|                            |levin at chromium.org
     Ever Confirmed|0                           |1

--- Comment #2 from David Levin <levin at chromium.org>  2010-07-13 23:25:05 PST ---
Seems fine.

<Very optional>
imo, it would be nice to include a few more header files in places where things are used (like RefPtr, PassRefPtr, or adoptRef being used without including the corresponding header file (as this seems to be using the fact that some other header file happens to include it ), but this isn't critical (and I may even be recommending something that other WK reviewers would recommend against).
</Very optional>

I'd like someone else from the video team to give an LGTM before I do the final r+. (If you want this to be committed automatically for you, please mark this as cq? in addition to r?.)

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.