Bug 52571 - [Qt] Fixed compilation with QT_NO_GRAPHICSVIEW
Summary: [Qt] Fixed compilation with QT_NO_GRAPHICSVIEW
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Qt (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC All
: P3 Enhancement
Assignee: Nobody
URL:
Keywords: Qt, QtTriaged
Depends on:
Blocks:
 
Reported: 2011-01-17 06:20 PST by Konstantin Tokarev
Modified: 2011-10-24 01:03 PDT (History)
4 users (show)

See Also:

Attachments
Fixed compilation with QT_NO_GRAPHICSVIEW (1.74 KB, patch)
2011-01-17 06:24 PST, Konstantin Tokarev 		benjamin: review-
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Konstantin Tokarev 2011-01-17 06:20:59 PST 
This patch fixes compilation of QtWebKit when QGraphicsView is disabled in Qt configuration
Comment 1 Konstantin Tokarev 2011-01-17 06:24:55 PST 
Created attachment 79161 [details]
Fixed compilation with QT_NO_GRAPHICSVIEW
Comment 2 Laszlo Gombos 2011-01-17 14:57:47 PST 
This seems like a DUP of bug 49750. 

Konstantin, do you have ACCELERATED_COMPOSITING enabled ?
Comment 3 Konstantin Tokarev 2011-01-18 07:29:09 PST 
>This seems like a DUP of bug 49750

Maybe. 

BTW, trunk still does not compile without this patch. Should I relocate it?

>Konstantin, do you have ACCELERATED_COMPOSITING enabled ?

Yes, it's enabled. However, I don't understand how can I manage it :)
Comment 4 Laszlo Gombos 2011-01-18 07:47:56 PST 
(In reply to comment #3)
> >This seems like a DUP of bug 49750
> 
> Maybe. 
> 
> BTW, trunk still does not compile without this patch. Should I relocate it?
> 
> >Konstantin, do you have ACCELERATED_COMPOSITING enabled ?
> 
> Yes, it's enabled. However, I don't understand how can I manage it :)

Perhaps ACCELERATED_COMPOSITING should not be enabled (and than the stub in GraphicsLayerQt.cpp is not needed).

Can you try the last patch attached to bug 4975 (https://bug-49750-attachments.webkit.org/attachment.cgi?id=79124). If it works I think we think we should mark this bug a DUP of 49750.

The path at bug 49750 disables ACCELERATED_COMPOSITING (unless Texture Mapper is turned on).
Comment 5 Konstantin Tokarev 2011-01-18 09:20:36 PST 
(In reply to comment #4)
> (In reply to comment #3)
> > >This seems like a DUP of bug 49750
> > 
> > Maybe. 
> > 
> > BTW, trunk still does not compile without this patch. Should I relocate it?
> > 
> > >Konstantin, do you have ACCELERATED_COMPOSITING enabled ?
> > 
> > Yes, it's enabled. However, I don't understand how can I manage it :)
> 
> Perhaps ACCELERATED_COMPOSITING should not be enabled (and than the stub in GraphicsLayerQt.cpp is not needed).
> 
> Can you try the last patch attached to bug 4975 (https://bug-49750-attachments.webkit.org/attachment.cgi?id=79124). If it works I think we think we should mark this bug a DUP of 49750.
> 
> The path at bug 49750 disables ACCELERATED_COMPOSITING (unless Texture Mapper is turned on).

I've applied that path. Nevertheless, I get ACCELERATED_COMPOSITING enabled in Platform.h

PLATFORM(QT) && (!defined(QT_NO_GRAPHICSVIEW) || USE(TEXTURE_MAPPER))

In my case, QT_NO_GRAPHICSVIEW is not defined here. Seems like no Qt headers were included
Comment 6 Eric Seidel (no email) 2011-01-24 15:39:49 PST 
Comment on attachment 79161 [details]
Fixed compilation with QT_NO_GRAPHICSVIEW

Seems kinda hacky.  Does this work?
Comment 7 Benjamin Poulain 2011-01-28 18:26:55 PST 
Comment on attachment 79161 [details]
Fixed compilation with QT_NO_GRAPHICSVIEW

Graphic layer is a concept of accelerated compositing. Accelerated compositing rely on QGraphicsView if you do not use texture mapper.

Having accelerated compositing without QGraphicsView and without texture mapper should not be supported IMHO.
Comment 8 Csaba Osztrogonác 2011-10-24 01:03:00 PDT 
Closing bug, based on Benjamin's comment.