Bug 36270 - [Qt] QtLauncher needs an option to Enable/Disable a QGLWidget as Viewport
: [Qt] QtLauncher needs an option to Enable/Disable a QGLWidget as Viewport
Status: CLOSED FIXED
: WebKit
Tools / Tests
: 528+ (Nightly build)
: All All
: P2 Normal
Assigned To:
:
: Qt
:
: 35303 35784
  Show dependency treegraph
 
Reported: 2010-03-17 20:09 PST by
Modified: 2010-03-26 08:45 PST (History)


Attachments
Patch (4.65 KB, patch)
2010-03-17 20:17 PST, Jesus Sanchez-Palencia
no flags Review Patch | Details | Formatted Diff | Diff
Patch (4.98 KB, patch)
2010-03-18 08:16 PST, Jesus Sanchez-Palencia
no flags Review Patch | Details | Formatted Diff | Diff
Patch (5.19 KB, patch)
2010-03-23 11:40 PST, Jesus Sanchez-Palencia
no flags Review Patch | Details | Formatted Diff | Diff
Patch (4.84 KB, patch)
2010-03-23 13:16 PST, Jesus Sanchez-Palencia
hausmann: review+
Review Patch | Details | Formatted Diff | Diff


Note

You need to log in before you can comment on or make changes to this bug.


Description From 2010-03-17 20:09:08 PST
[Qt] QtLauncher needs an option to Enable/Disable a QGLWidget as Viewport
------- Comment #1 From 2010-03-17 20:17:07 PST -------
Created an attachment (id=50999) [details]
Patch
------- Comment #2 From 2010-03-18 00:24:53 PST -------
(From update of attachment 50999 [details])
No general objections, but this needs to be #ifdef, as it breaks the build with Qt configuration where there is no OpenGL support. Symbian is one of those platforms where OpenGL is not available.
------- Comment #3 From 2010-03-18 08:16:01 PST -------
Created an attachment (id=51024) [details]
Patch
------- Comment #4 From 2010-03-18 08:19:59 PST -------
(In reply to comment #3)
> Created an attachment (id=51024) [details] [details]
> Patch

Thanks for the review. :)

I didn't find any macro or global define on Qt's 4.7 documentation to handle this, so what I did was just to ensure that we are not building on Symbian.

Is that ok?
------- Comment #5 From 2010-03-19 00:53:46 PST -------
(From update of attachment 51024 [details])
I'm sorry, I should've been more specific :(. I didn't mean to disable OpenGl for S60 but when it's not enabled in Qt in general. S60 was just an example.

For example it's not unusual that Qt fo Embedded Linux is compiled without OpenGL support, too.

You can verify that OpenGL support is available in the .pro file using

contains(QT_CONFIG, opengl) {
    do opengl things
}

In addition there's a QT_NO_OPENGL define set when Qt is configured without OpenGL support.
------- Comment #6 From 2010-03-23 11:40:07 PST -------
Created an attachment (id=51442) [details]
Patch
------- Comment #7 From 2010-03-23 11:45:21 PST -------
(In reply to comment #5)
> (From update of attachment 51024 [details] [details])
> I'm sorry, I should've been more specific :(. I didn't mean to disable OpenGl
> for S60 but when it's not enabled in Qt in general. S60 was just an example.

I'm sorry that I didn't understand it correctly at first place. :)

> You can verify that OpenGL support is available in the .pro file using
> 
> contains(QT_CONFIG, opengl) {
>     do opengl things
> }
> 
> In addition there's a QT_NO_OPENGL define set when Qt is configured without
> OpenGL support.

Thanks for the tips! I didn't find this QT_NO_OPENGL define documented anywhere besides Qt src itself, so I sticked to the .pro approach.
------- Comment #8 From 2010-03-23 12:10:44 PST -------
Attachment 51442 [details] did not build on qt:
Build output: http://webkit-commit-queue.appspot.com/results/1087005
------- Comment #9 From 2010-03-23 13:16:20 PST -------
Created an attachment (id=51447) [details]
Patch

Trying to fix the build failure.
------- Comment #10 From 2010-03-26 06:00:50 PST -------
Rebased and Committed as r56618 by Antonio.
Thanks!
------- Comment #11 From 2010-03-26 08:42:46 PST -------
cherry-pick-for-backport: <r56618>
------- Comment #12 From 2010-03-26 08:45:38 PST -------
Revision r56618 cherry-picked into qtwebkit-2.0 with commit 8221def16a71a7ec51d582ce69f1bfdd6bab8a18