[webkit-reviews] review requested: [Bug 62108] [Qt] Failing test media/video-document-types.html : [Attachment 113300] patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Nov 2 03:55:11 PDT 2011


Deepak Sherveghar <bpwv64 at motorola.com> has asked  for review:
Bug 62108: [Qt] Failing test media/video-document-types.html
https://bugs.webkit.org/show_bug.cgi?id=62108

Attachment 113300: patch
https://bugs.webkit.org/attachment.cgi?id=113300&action=review

------- Additional Comments from Deepak Sherveghar <bpwv64 at motorola.com>
media/video-document-types.html (when removed from the QT skipped list) runs
fine now with the latest code base.

But logically, we still need to cancel the main resource load, if the data is
handled by MediaDocument.
Webkit2 as well as all other ports (GTK/Chromium etc) do that.

In my patch I didn't un-skip the media/video-document-types.html because the
media folder itself is in 
skipped list.


More information about the webkit-reviews mailing list