Bug 77632 – [Qt] Split QQuickWebViewPrivate in two classes, for desktop and touch behavior.

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Jocelyn Turcotte]

[Qt] Split QQuickWebViewPrivate in two classes, for desktop and touch behavior.

[Jocelyn Turcotte]

Created attachment 125127 [details]
Patch

[Jocelyn Turcotte]

Created attachment 125128 [details]
Code moving patch (will squash before commit)

[Kenneth Rohde Christiansen]

Comment on attachment 125127 [details]
Patch

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

> Source/WebKit2/ChangeLog:11
> +        Reviewed by NOBODY (OOPS!).
> +
> +        Replace the runtime property setting the desktop behavior on each instance by a static boolean
> +        controlling the behavior used by WebView created in the future.
> +
> +        * UIProcess/API/qt/qquickwebpage.cpp:

a bit shorter lines and some added newlines could make this changelog readable

> Source/WebKit2/UIProcess/API/qt/qquickwebview.cpp:58
> +static bool sUseTraditionalDesktopBehaviour = false;

I believe s_ is what is commonly used

> Source/WebKit2/UIProcess/API/qt/qquickwebview.cpp:60
> +static QQuickWebViewPrivate* createPrivateObject(QQuickWebView* pub)

pub?

> Source/WebKit2/UIProcess/API/qt/qquickwebview.cpp:65
> +    if (sUseTraditionalDesktopBehaviour)
> +        return new QQuickWebViewDesktopPrivate(pub);
> +    return new QQuickWebViewTouchPrivate(pub);
> +}

im not convinced that Touch is the best way to describe the difference between the classes

> Source/WebKit2/UIProcess/API/qt/qquickwebview.cpp:684
> +void QQuickWebViewExperimental::setUseTraditionalDesktopBehaviour(bool enable)
> +{
> +    sUseTraditionalDesktopBehaviour = enable;
> +}
> +
> +bool QQuickWebViewExperimental::useTraditionalDesktopBehaviour()
> +{
> +    return sUseTraditionalDesktopBehaviour;
> +}
> +

So this cannot be changed at runtime?

[Kenneth Rohde Christiansen]

Comment on attachment 125127 [details]
Patch

Following ric discussions

[Kenneth Rohde Christiansen]

(In reply to comment #4)
> (From update of attachment 125127 [details])
> Following ric discussions

IRC that is

[Jocelyn Turcotte]

Created attachment 125133 [details]
Patch

[Kenneth Rohde Christiansen]

Comment on attachment 125133 [details]
Patch

rs=me

[Jocelyn Turcotte]

Committed r106658: <http://trac.webkit.org/changeset/106658>