[Webkit-unassigned] [Bug 37880] [Qt] Patch for checking the content-type supported by the QWebFrame...

[bugzilla-daemon at webkit.org]

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





--- Comment #4 from adawit at kde.org  2010-05-10 09:17:08 PST ---
(In reply to comment #3)
> (From update of attachment 53863 [details])
> I suggest to send an email to webkit-qt at lists.webkit.org to see what the others thing. But I'm okay with having this in the API.
> 
> r- because it should be static and probably in a different class. See the 
> implementaion in FrameLoaderClientQt, it only calls static functions :)

Which class ? QWebSettings or QWebPage ? There is no other place to put it other than that. Also if the function is made static, it would mean duplicating the code and hence all the necessary headers from FrameLoaderClientQt::canShowMIMEType. Is that acceptable thing to do ?

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