[Webkit-unassigned] [Bug 20081] [Qt] Add support for windowless NPAPI plugins

[bugzilla-daemon at webkit.org]

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





--- Comment #40 from Girish Ramakrishnan <girish at forwardbias.in>  2009-10-04 21:40:39 PDT ---
Kenneth, thanks for reminding me about zooming :) I forgot to test it with
that.

Important history is being lost, so I put up the patch in
http://gitorious.org/~girish/qtwebkit/girishs-clone/commits/windowless. Changes
include:
1. Use X Pixmaps (93c3aaf)
2. Add PluginQuirkUsesScreenDefaultVisual for Flash (654645). Inspired by
Kenneth's suggestion on irc.
3. Make zoom work (06e79ab)

I think it's ready for landing. Here are the pending bugs:
1. Printing does not work (but no crash).
2. -graphicssystem opengl crashes. We cannot paint X11 Pixmap using QPainter. I
will disable this in a separate patch (since we need this history)
3. QGVLauncher crashes on exit - WebCore::Chrome::repaint uses the client. The
client is alreay gone by then.
4. QGraphicsWebView works, yay! But I need a way to detect painting on qgv and
turn off transparency. Any suggestions?
5. When using ARGB visuals on tlw, mouse over crashes. 

In the process of debugging 5, I think I have found a bug. It appears that svn
commit 48604 (0292c6) has introduced a bug - it calls the view's winId(). This
actually ends up creating a native window. I don't think is what we want. For
the windowless case, we want the window's winId (and not the views). Even in
the windowed case, only the container needs to have a winId and not the view
itself. Simon, Kenneth, is that correct?

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