Bug 86960 - [EFL] Clients should be able to set page cache capacity.
Summary: [EFL] Clients should be able to set page cache capacity.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit EFL (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Mikhail Pozdnyakov
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-05-20 03:43 PDT by Mikhail Pozdnyakov
Modified: 2012-05-22 11:19 PDT (History)
4 users (show)

See Also:


Attachments
patch (2.88 KB, patch)
2012-05-20 05:38 PDT, Mikhail Pozdnyakov
no flags Details | Formatted Diff | Diff
patch v2 (fixed api description, typo in changelog, efl wrapper is used in _ewk_init_body) (3.77 KB, patch)
2012-05-21 06:37 PDT, Mikhail Pozdnyakov
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Mikhail Pozdnyakov 2012-05-20 03:43:06 PDT
There should be an api setting page cache capacity, so that a client can improve document load speed substantially
by caching a large number previously viewed pages or reduce memory usage decreasing the size of page cache.

This setting is exported in other WebKit ports (Qt, GTK, MAC) either as a separate setting or as a part of caching model.
Comment 1 Mikhail Pozdnyakov 2012-05-20 05:38:05 PDT
Created attachment 142901 [details]
patch
Comment 2 Raphael Kubo da Costa (:rakuco) 2012-05-20 14:26:41 PDT
Comment on attachment 142901 [details]
patch

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

The addition itself looks fine, but you should mention what the default is. BTW, _ewk_init_body() calls setCapacity() directly, you might want to route that call via ewk_settings now.

> Source/WebKit/efl/ChangeLog:9
> +        Those will be used both by clients ant DRT Layout Test Controller.

Nitpick: s/ant/and/.

> Source/WebKit/efl/ewk/ewk_settings.h:302
> + * The page cache improves user experience when navigating forth or back
> + * to pages in the forward/back history.

It'd be good to describe how it achieves this improvement: what does it cache, for example?

> Source/WebKit/efl/ewk/ewk_settings.h:306
> +EAPI unsigned         ewk_settings_page_cache_capacity_get();

This is a public header used by C applications, so you need to use foo(void) instead of foo().
Comment 3 Mikhail Pozdnyakov 2012-05-21 06:37:35 PDT
Created attachment 143021 [details]
patch v2 (fixed api description, typo in changelog, efl wrapper is used in _ewk_init_body)
Comment 4 Mikhail Pozdnyakov 2012-05-21 06:39:12 PDT
(In reply to comment #2)
> (From update of attachment 142901 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=142901&action=review
> 
> The addition itself looks fine, but you should mention what the default is. BTW, _ewk_init_body() calls setCapacity() directly, you might want to route that call via ewk_settings now.
> 
> > Source/WebKit/efl/ChangeLog:9
> > +        Those will be used both by clients ant DRT Layout Test Controller.
> 
> Nitpick: s/ant/and/.
> 
> > Source/WebKit/efl/ewk/ewk_settings.h:302
> > + * The page cache improves user experience when navigating forth or back
> > + * to pages in the forward/back history.
> 
> It'd be good to describe how it achieves this improvement: what does it cache, for example?
> 
> > Source/WebKit/efl/ewk/ewk_settings.h:306
> > +EAPI unsigned         ewk_settings_page_cache_capacity_get();
> 
> This is a public header used by C applications, so you need to use foo(void) instead of foo().
Thanks for review. Fixed in https://bugs.webkit.org/attachment.cgi?id=143021.
Comment 5 Raphael Kubo da Costa (:rakuco) 2012-05-21 08:06:22 PDT
Comment on attachment 143021 [details]
patch v2 (fixed api description, typo in changelog, efl wrapper is used in _ewk_init_body)

Looks good, thank you.
Comment 6 Csaba Osztrogonác 2012-05-21 08:34:08 PDT
Comment on attachment 143021 [details]
patch v2 (fixed api description, typo in changelog, efl wrapper is used in _ewk_init_body)

rs=me.
Comment 7 WebKit Review Bot 2012-05-21 08:54:19 PDT
Comment on attachment 143021 [details]
patch v2 (fixed api description, typo in changelog, efl wrapper is used in _ewk_init_body)

Clearing flags on attachment: 143021

Committed r117785: <http://trac.webkit.org/changeset/117785>
Comment 8 WebKit Review Bot 2012-05-21 08:54:25 PDT
All reviewed patches have been landed.  Closing bug.