[Webkit-unassigned] [Bug 28865] [Qt] Make cursor set cleaner in QtWebKit Api: eliminate SetCursorEvent hack.

[bugzilla-daemon at webkit.org]

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





--- Comment #3 from Antonio Gomes (tonikitoo) <tonikitoo at gmail.com>  2009-09-01 05:42:02 PDT ---

> QCoreApplication::postEvent(widget, new SetCursorEvent(cursor.impl()));

of course if does not, but if you do not assigneda QWidget as parent of it will
fail in a static_cast and QWebPage hostWindow()->plaftormWidget() will be null
... so method will bail our and no cursor will be set at all.

> The above call doesn't require a QWidget but merely a QObject (QWidget,
> QGraphicsWidget, ... are all QObjects)!

that is a bug by itself.

> secondly: You do not need to duplicate the SetCursorEvent class when
> implementing this inside WebKit, and this definately is the right place to
> implement a QGraphicsItem based widget. You will see that my current patch
> doesn't duplicate the class.

it does not duplicate if it is inside Webkit, of course. but i do not care only
about making things to work inside webkit. image you are an embedder ... and
your code would not be in webkit ...

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