[Webkit-unassigned] [Bug 44062] [Qt] Use LAZY_NATIVE_CURSOR

[bugzilla-daemon at webkit.org]

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





--- Comment #5 from Balazs Kelemen <kb at inf.u-szeged.hu>  2010-08-17 02:00:40 PST ---
> 
> WebCore/platform/qt/CursorQt.cpp:73
>  +  #endif
> I guess you need to return 0 after this

Good catch.

> 
> WebCore/platform/qt/CursorQt.cpp:83
>  +          m_platformCursor = new QCursor(Qt::ArrowCursor);
> wouldn't a lookup table be better?

There is not other way of mapping from cursor types to QCursor objects than hard coding it so the best lookup table is a switch :)

> 
> WebCore/platform/qt/CursorQt.cpp:86
>  +          m_platformCursor = new QCursor(Qt::CrossCursor);
> Where are these being deleted?

Right, it should be deleted in the destructor.

> 
> 
> Does the Qt QWidget::setCursor stuff still work after this change? Please test.

I did not see any regressions however I just did a quick browsing with the change. I would use the bots for testing. I will upload a new patch that fixes the problems you find (with the EFL and the spelling thing).

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