Bug 30327

Summary: [Qt] Fix build break for Qt 4.4
Product: WebKit Reporter: Fumitoshi Ukai <ukai>
Component: WebCore Misc.Assignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, eric, kenneth, laszlo.gombos, webkit.review.bot
Priority: P2 Keywords: Qt
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Linux   
Attachments:
Description Flags
proposed patch none

Description Fumitoshi Ukai 2009-10-13 01:19:44 PDT
WebKit/Qt build fails with Qt 4.4, at least the following two errors.

../../../WebCore/platform/graphics/qt/GraphicsContextQt.cpp: In member function ‘void WebCore::GraphicsContext::clipOut(const WebCore::Path&)’:
../../../WebCore/platform/graphics/qt/GraphicsContextQt.cpp:1071: error: no match for ‘operator&’ in ‘clippedOut & newClip’

operator& for QPainterPath was introduced in Qt 4.5.
I believe it is equivalent to intersected() that exists in Qt 4.4

../../../WebCore/plugins/qt/PluginViewQt.cpp: In member function ‘virtual void WebCore::PluginView::paint(WebCore::GraphicsContext*, const WebCore::IntRect&)’:
../../../WebCore/plugins/qt/PluginViewQt.cpp:182: error: ‘fromX11Pixmap’ is not a member of ‘QPixmap’
../../../WebCore/plugins/qt/PluginViewQt.cpp:182: error: ‘ExplicitlyShared’ is not a member of ‘QPixmap’

QPixmap::fromX11Pixmap was introduced in Qt 4.5, and I couldn't find any fallback in Qt 4.4..
Comment 1 Laszlo Gombos 2009-12-28 09:49:10 PST
Created attachment 45560 [details]
proposed patch

Enables building (on Linux requires to turn off NPAPI support) with Qt 4.4.

I think we should keep the 4.4 support around in the trunk for a while. 4.4.3 was released in 28 September 2008, which was not that long ago and at the moment it does not seems like a big effort to maintain most QtWebKit features for Qt 4.4 as well.
Comment 2 WebKit Review Bot 2009-12-28 09:51:12 PST
style-queue ran check-webkit-style on attachment 45560 [details] without any errors.
Comment 3 Adam Barth 2009-12-28 10:22:05 PST
Comment on attachment 45560 [details]
proposed patch

In general, you can land build fixes without review.
Comment 4 WebKit Commit Bot 2009-12-28 11:15:29 PST
Comment on attachment 45560 [details]
proposed patch

Clearing flags on attachment: 45560

Committed r52601: <http://trac.webkit.org/changeset/52601>
Comment 5 WebKit Commit Bot 2009-12-28 11:15:34 PST
All reviewed patches have been landed.  Closing bug.
Comment 6 Kenneth Rohde Christiansen 2009-12-29 06:45:26 PST
Why are we doing patches for supporting 4.4 which we decided to not support anymore?
Comment 7 Eric Seidel (no email) 2009-12-29 09:23:00 PST
I didn't realize (and I would assume  Fumitoshi didn't realize) that we were not supporting GCC 4.4.  Do we have a webkit-dev thread on the subject I should (re-)read?
Comment 8 Eric Seidel (no email) 2009-12-29 09:24:14 PST
Oh, sorry, I was confused.  This is Qt 4.4.  Not GCC 4.4.   My retract my previous comment.