Bug 36607

Summary: [Qt] Package build on Maemo 5 is broken
Product: WebKit Reporter: Simon Hausmann <hausmann>
Component: New BugsAssignee: Simon Hausmann <hausmann>
Status: CLOSED FIXED    
Severity: Blocker CC: casper.vandonderen, commit-queue, luiz
Priority: P1 Keywords: Qt
Version: 528+ (Nightly build)   
Hardware: Other   
OS: Linux   
Bug Depends on:    
Bug Blocks: 35784    
Attachments:
Description Flags
Patch none

Description Simon Hausmann 2010-03-25 09:44:43 PDT
RenderThemeQt.cpp depends on themeQtNoListboxesUserAgentStyleSheet when compiling for maemo5.

The corresponding stylesheet is only embedded in WebCore.pri when the corresponding listbox feature is enabled:

contains(DEFINES, ENABLE_NO_LISTBOX_RENDERING=1) {
    STYLESHEETS_EMBED += $$PWD/css/themeQtNoListboxes.css
}

For the packaging we need to generate all files. Therefore it's the easiest to always embed the stylesheet, but enable the #define for the feature only when compiling for Maemo5.

The error message when compiling the package on Maemo5 is:

platform/qt/RenderThemeQt.cpp:187: error: 'themeQtNoListboxesUserAgentStyleSheet' was not declared in this scope
Comment 1 Simon Hausmann 2010-03-25 09:46:17 PDT
Created attachment 51649 [details]
Patch
Comment 2 WebKit Commit Bot 2010-03-25 16:12:14 PDT
Comment on attachment 51649 [details]
Patch

Clearing flags on attachment: 51649

Committed r56576: <http://trac.webkit.org/changeset/56576>
Comment 3 WebKit Commit Bot 2010-03-25 16:12:19 PDT
All reviewed patches have been landed.  Closing bug.
Comment 4 Simon Hausmann 2010-03-26 03:47:45 PDT
Revision r56576 cherry-picked into qtwebkit-2.0 with commit d362c2e1c4559bf5ab4b4efaf3b8b09613f7c3fd
Comment 5 Luiz Agostini 2010-03-31 11:10:21 PDT
*** Bug 36880 has been marked as a duplicate of this bug. ***