[Sugar-devel] [PATCH browse] Improve session store and restore

Simon Schampijer simon at schampijer.de
Wed Jan 4 04:59:48 EST 2012


On 04/01/12 03:57, Manuel Quiñones wrote:
> This is the review from Simon Schampijer of commit
> 14102bcd65f015dcbf12d1406df8ef7ffb64d13d .
>
> * the sessionstore code is moved into Browser, we do have
>    set_history_index/get_history_index already there, the TabbedView
>    does use that path for their requests about the history
>
> * Rename a few bits s/session/history and the return value from
>    self.get_back_forward_list() Use back_forward_list as variable
>
> * Fold _get_history and _set_history into their appropriate 'mother'
>    methods
>
> * Make set_history_index a bit cleaner, the API provided by webkitgtk
>    does not seem to cleanly allow our usage like:
>    get_current_item_index and go_to_back_forward_item(index) also the
>    API does seem to miss a back_forward_list.get_length() (see the code
>    we have to do in _items_history_as_list) there only exist a
>    back_forward_list.get_back_length() and
>    back_forward_list.get_forward_length().
>
> Signed-off-by: Manuel Quiñones<manuq at laptop.org>

Looks good, let's get it in!

Cheers,
    Simon


More information about the Sugar-devel mailing list