[Webkit-unassigned] [Bug 44430] [Qt] WebCore crashes from the web process.

[bugzilla-daemon at webkit.org]

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





--- Comment #4 from Balazs Kelemen <kb at inf.u-szeged.hu>  2010-08-24 04:42:05 PST ---
(In reply to comment #3)
> (From update of attachment 65236 [details])
> First of all, each of these should be separate patches!
> 
> Secondly, there is another patch from Jesus and Diego fixing some of these issues, but it is not 100% ready yet.

That patch does not covers the PluginData problem. Do you think we should introduce a PluginDataContext there?

> 
> Thirdly, this is too hacky and it is better to spend the time on fixing this properly by removing the layering violations or helping Jesus and Diego finish their patch.

I can degrease the hackiness via using dynamic_cast but as I know rtti should be avoided in WebKit (and also in Qt maybe).
You should take in account that even if we could solve the layering violations (that is materialized now in the form of casting and abstract client class to the concrete qt one), we won't have the fancy API-s of WebKit in WebKit2 so we should provide a default behavior.
I would ask you to help me by discussing what design we should follow to solve these problems. Creating *Context abstractions? Using PlatformStrategies somehow? Merging API of WebKit and WebKit2?

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