WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
36175
[Qt] Make QtLaucher able to select the ViewportUpdateMode
https://bugs.webkit.org/show_bug.cgi?id=36175
Summary
[Qt] Make QtLaucher able to select the ViewportUpdateMode
Diego Gonzalez
Reported
2010-03-16 11:06:44 PDT
Provide to QtLauncher a way to change the ViewPortUpdateMode when it's in graphics based mode.
Attachments
Proposed patch
(5.07 KB, patch)
2010-03-16 11:14 PDT
,
Diego Gonzalez
kenneth
: review-
Details
Formatted Diff
Diff
(committed in r56117) Proposed patch
(5.07 KB, patch)
2010-03-16 12:53 PDT
,
Diego Gonzalez
no flags
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Diego Gonzalez
Comment 1
2010-03-16 11:14:41 PDT
Created
attachment 50811
[details]
Proposed patch
Kenneth Rohde Christiansen
Comment 2
2010-03-16 11:28:08 PDT
Comment on
attachment 50811
[details]
Proposed patch
> +void LauncherWindow::selectViewportUpdateMode(int mode)
select is not a really good name for this. Consider using "change" or "set"
> +{ > + if (!isGraphicsBased()) > + return;
I guess you should update the gViewportUpdateMode anyway, so you are bailing out a bit too early
> + > + gViewportUpdateMode = QGraphicsView::ViewportUpdateMode(mode); > + WebViewGraphicsBased* view = static_cast<WebViewGraphicsBased*>(m_view); > + view->setViewportUpdateMode(gViewportUpdateMode); > +}
Diego Gonzalez
Comment 3
2010-03-16 12:53:36 PDT
Created
attachment 50827
[details]
(committed in
r56117
) Proposed patch Changed according Kenneth's suggestions
Benjamin Poulain
Comment 4
2010-03-17 03:03:58 PDT
Setting the status of the task so it does not appear unassigned. I might misunderstand the purpose of the patch but for me this should not go in QtLauncher. If the objective is to compare performance, you should create a reduction in the webkit benchmarks. Definitely not update the launcher. If the objective is to find bugs, you should add autotests in graphicsview. Why do you need that in the launcher?
Jesus Sanchez-Palencia
Comment 5
2010-03-17 07:16:37 PDT
(In reply to
comment #4
)
> Why do you need that in the launcher?
QtLauncher is becoming a more complete test tool of QtWebKit. And we are focusing on mobile tests (Symbian and Maemo). We are following
https://bugs.webkit.org/show_bug.cgi?id=35303
requirements, some ideas from Simon and others, to have all command line options as menu options on the Launcher. With this it will become easier to test QtWebKit on Symbian devices.
Benjamin Poulain
Comment 6
2010-03-17 07:35:31 PDT
(In reply to
comment #5
)
> We are following
https://bugs.webkit.org/show_bug.cgi?id=35303
requirements,
This bug does not say why such options would be required, neither why it is specific to Symbian.
Jesus Sanchez-Palencia
Comment 7
2010-03-17 07:52:45 PDT
(In reply to
comment #6
)
> This bug does not say why such options would be required, neither why it is > specific to Symbian.
Yes, that is true. My guess is that No'am is mainly working with Symbian debugging and that is why he opened the bug. But I don't think that those wanted features are Symbian specific, that is why we are leaving them available for any platform/OS.
Antonio Gomes
Comment 8
2010-03-17 10:38:26 PDT
Comment on
attachment 50827
[details]
(committed in
r56117
) Proposed patch the ChangeLog was wrong =/ Fixed, and landed manually in
r56117
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug