Bug 100483 - [Qt][WK2] Add QML API to clear memory caches and local storage
Summary: [Qt][WK2] Add QML API to clear memory caches and local storage
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Qt (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P3 Minor
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-10-26 02:20 PDT by Tero Tarvainen
Modified: 2022-02-28 03:56 PST (History)
10 users (show)

See Also:


Attachments
[Qt][WK2] Add QML API to clear memory caches and local storage (2.77 KB, patch)
2012-10-30 01:19 PDT, Tero Tarvainen
jturcotte: review-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Tero Tarvainen 2012-10-26 02:20:06 PDT
Qt WebKit2 doesn't contain API for QML apps to clear memory or application caches nor local storage. 
The functionality is already implemented to WebKit2, but need to be exposed to QML apps also.
Comment 1 Tero Tarvainen 2012-10-30 01:19:38 PDT
Created attachment 171390 [details]
[Qt][WK2] Add QML API to clear memory caches and local storage
Comment 2 Andras Becsi 2012-10-30 04:37:20 PDT
Comment on attachment 171390 [details]
[Qt][WK2] Add QML API to clear memory caches and local storage

You need to set r? and cq? if you'd like your patch to be reviewed and landed. LGTM otherwise.
Comment 3 Tero Tarvainen 2012-10-30 05:31:55 PDT
(In reply to comment #2)
> (From update of attachment 171390 [details])
> You need to set r? and cq? if you'd like your patch to be reviewed and landed. LGTM otherwise.

Thanks
Comment 4 Jocelyn Turcotte 2012-10-31 05:45:43 PDT
Comment on attachment 171390 [details]
[Qt][WK2] Add QML API to clear memory caches and local storage

This is going to clear the caches on all the pages, which might be misleading since it's called on a page instance. I'm not too sure how we should address this as we don't have any all-page API yet, we should be looking at it once we released Qt 5.0.
If you don't need this API now somehow I think it would be better to wait until we have a proper solution.
I'm setting it r- for now because I think that this should not be the final solution, tell me if you have further ideas.
Comment 5 Sergio Villar Senin 2012-10-31 15:28:40 PDT
(In reply to comment #4)
> (From update of attachment 171390 [details])
> This is going to clear the caches on all the pages, which might be misleading since it's called on a page instance. I'm not too sure how we should address this as we don't have any all-page API yet, we should be looking at it once we released Qt 5.0.

So should this be part of the WebContext for example?
Comment 6 Tero Tarvainen 2012-10-31 22:49:35 PDT
(In reply to comment #4)
> (From update of attachment 171390 [details] [details])
> This is going to clear the caches on all the pages, which might be misleading since it's called on a page instance. I'm not too sure how we should address this as we don't have any all-page API yet, we should be looking at it once we released Qt 5.0.

Good point. 

One approach would be to create QWebCacheManager class which would handle the cleanup and expose access to it through QQuickWebViewExperimental. And then later on change the access to be part of the all-page API once that's available?
Comment 7 Jocelyn Turcotte 2012-11-01 03:29:23 PDT
We had plans to expose the WebContext in QML at some point but it wasn't easy to decide if we would require any WebView to be created within a WebContext, if a WebView could create its own default WebContext, etc. so we decided not to expose it. Most QML items should either affect the display or input events.

Though we would need a similar mechanism to support static settings in QWebSettings in WebKit2 like storage paths.

One quick idea is to have a QtWebKit global object, similar to the Qt object ( http://doc.qt.digia.com/qt/qdeclarativeglobalobject.html ) and within there expose context/global functions. This assumes that we support only one WebContext at a time which is something we didn't agree to yet.
This is just an example and I haven't tried it myself, maybe it's not even possible to let the QtWebKit import add anything to the global object.

But all this is a bit out of the focus of the team at the moment and any such change shouldn't be done before we branched for 5.0.
Comment 8 Jocelyn Turcotte 2014-02-03 03:23:03 PST
=== Bulk closing of Qt bugs ===

If you believe that this bug report is still relevant for a non-Qt port of webkit.org, please re-open it and remove [Qt] from the summary.

If you believe that this is still an important QtWebKit bug, please fill a new report at https://bugreports.qt-project.org and add a link to this issue. See http://qt-project.org/wiki/ReportingBugsInQt for additional guidelines.