Bug 65339 - [Qt] [WK2] Expose web view classes to QML
Summary: [Qt] [WK2] Expose web view classes to QML
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Qt (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Caio Marcelo de Oliveira Filho
URL:
Keywords: Qt, QtTriaged
Depends on:
Blocks:
 
Reported: 2011-07-28 13:16 PDT by Caio Marcelo de Oliveira Filho
Modified: 2011-08-03 07:25 PDT (History)
3 users (show)

See Also:


Attachments
Patch (7.15 KB, patch)
2011-07-28 13:27 PDT, Caio Marcelo de Oliveira Filho
no flags Details | Formatted Diff | Diff
Patch (12.36 KB, patch)
2011-08-02 16:47 PDT, Caio Marcelo de Oliveira Filho
benjamin: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Caio Marcelo de Oliveira Filho 2011-07-28 13:16:38 PDT
Create a plugin that expose our web view classes for WK2 as QML elements.
Comment 1 Caio Marcelo de Oliveira Filho 2011-07-28 13:27:47 PDT
Created attachment 102292 [details]
Patch
Comment 2 Caio Marcelo de Oliveira Filho 2011-07-29 10:08:49 PDT
Comment on attachment 102292 [details]
Patch

After talking in #qtwebkit with Benjamin and others, I'm working on adding tests for the QML plugin as well.
Comment 3 Caio Marcelo de Oliveira Filho 2011-08-02 16:47:08 PDT
Created attachment 102713 [details]
Patch
Comment 4 Caio Marcelo de Oliveira Filho 2011-08-02 16:50:17 PDT
Benjamin, could you take a look?

This commit depends on https://qt.gitorious.org/qt/qtdeclarative/merge_requests/1, I sent an email for QML devs that might want to merge, but feel welcome to take a look there too :-)
Comment 5 Benjamin Poulain 2011-08-02 18:58:01 PDT
Comment on attachment 102713 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=102713&action=review

Looks great

> Source/WebKit2/UIProcess/API/qt/qmlplugin/plugin.cpp:37
> +        qmlRegisterUncreatableType<QTouchWebPage>(uri, 5, 0, "TouchWebPage", QString::fromLatin1("Cannot create separate instance of TouchWebPage, use TouchWebView"));

QString::fromLatin1() -> QObject::tr().
Comment 6 Alexis Menard (darktears) 2011-08-03 04:35:01 PDT
Comment on attachment 102713 [details]
Patch

yeahhhh good stuff
Comment 7 Caio Marcelo de Oliveira Filho 2011-08-03 05:23:43 PDT
(In reply to comment #4)
> This commit depends on https://qt.gitorious.org/qt/qtdeclarative/merge_requests/1, I sent an email for QML devs that might want to merge, but feel welcome to take a look there too :-)

For the record, the fix was already merged in qt-declarative:master https://qt.gitorious.org/qt/qtdeclarative/commit/73aa113181fa2295729d24317c187661f740b898
Comment 8 Caio Marcelo de Oliveira Filho 2011-08-03 07:25:52 PDT
Committed r92277: <http://trac.webkit.org/changeset/92277>