[webkit-reviews] review denied: [Bug 43590] Implementing video frame sharing between WebKit and Chromium : [Attachment 63661] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Fri Aug 13 16:02:11 PDT 2010
Darin Fisher (:fishd, Google) <fishd at chromium.org> has denied Victoria
<vrk at google.com>'s request for review:
Bug 43590: Implementing video frame sharing between WebKit and Chromium
https://bugs.webkit.org/show_bug.cgi?id=43590
Attachment 63661: Patch
https://bugs.webkit.org/attachment.cgi?id=63661&action=review
------- Additional Comments from Darin Fisher (:fishd, Google)
<fishd at chromium.org>
WebKit/chromium/src/WebMediaPlayerClientImpl.cpp:435
+
m_webMediaPlayer->putCurrentFrame(VideoFrameChromiumImpl::toWebVideoFrame(frame
));
probably need a 'delete frame' here. should document the memory ownership
model.
i have further review comments about style, etc. will get to those ASAP.
see also AssertMatchingEnums.cpp. i think you'll want to extend that for your
new enums.
constants should not have a 'k' prefix.
More information about the webkit-reviews
mailing list