Bug 74933 – [Qt][WK2] Add test for application URL schemes.

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKit2

[Zeno Albisser]

ssia

[Zeno Albisser]

Created attachment 120038 [details]
patch for review.

[Kenneth Rohde Christiansen]

Comment on attachment 120038 [details]
patch for review.

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

Test looks fine.

> Source/WebKit2/UIProcess/API/qt/tests/qmltests/WebView/tst_applicationScheme.qml:13
> +        schemeDelegates: [
> +            UrlScheme {

Are there other types of scheme delegates than url schemes? just wondering

[Zeno Albisser]

Comment on attachment 120038 [details]
patch for review.

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

>> Source/WebKit2/UIProcess/API/qt/tests/qmltests/WebView/tst_applicationScheme.qml:13
>> +            UrlScheme {
> 
> Are there other types of scheme delegates than url schemes? just wondering

Nope. I already mentioned that in https://bugs.webkit.org/show_bug.cgi?id=74931.
I think the API contains a bit too much redundancy schemeDelegates [ UrlScheme { scheme: ... } ]
Actually may be we should rename "schemeDelegates" to "urlSchemeDelegates". But that would even add more redundancy. :-S
It would be nice to get rid of the "UrlScheme" keyword and make it implicit. But i'm not sure that is possible in QML.

[Zeno Albisser]

Created attachment 121376 [details]
patch for review. - update to fit the latest QML WebViewExperimental API.

[Zeno Albisser]

Comment on attachment 121376 [details]
patch for review. - update to fit the latest QML WebViewExperimental API.

Clearing flags on attachment: 121376

Committed r104280: <http://trac.webkit.org/changeset/104280>

[Zeno Albisser]

All reviewed patches have been landed.  Closing bug.

[Rafael Brandao]

I think you should add a new test case to show how the requested url could be handled. In that test case, for example, you could check if you've got "something" requested.

Another test could be to have multiple schemes, and see if you've got the right delegate for each urlscheme as you would expect. :-)

[Zeno Albisser]

(In reply to comment #7)
> I think you should add a new test case to show how the requested url could be handled. In that test case, for example, you could check if you've got "something" requested.
> 
> Another test could be to have multiple schemes, and see if you've got the right delegate for each urlscheme as you would expect. :-)

Sure... i'll add some more stuff soon. :)

[Zeno Albisser]

Created attachment 121751 [details]
patch for review. - add more test cases.

as suggested by Rafael Brandao.

[Kenneth Rohde Christiansen]

Comment on attachment 121751 [details]
patch for review. - add more test cases.

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

> Source/WebKit2/UIProcess/API/qt/tests/qmltests/WebView/tst_applicationScheme.qml:37
> +                    if ( request.url.toLowerCase() == "scheme3://url1")

why space before request?

[Zeno Albisser]

Committed r105728: <http://trac.webkit.org/changeset/105728>