Re: [Vimprobable-users] [PATCH] Move apply_settings to webview_load_committed_cb.

[Hannes S. <ha...@yl...>]

Jason Ryan <jas...@gm...> wrote:
> On 17/04/12 at 08:56am, Hannes Sch__ller wrote:
> > This is quite a complex subject, so let me first describe thec
> > current behaviour of the patch.
> 
> > <snip />
> 
> > Opinions welcome!
> 
> FWIW: this level of complexity leaves me cold. It is not
> functionality that I would use and my only view is that I hope it
> does not impact negatively on the performance or ease of use of what
> is an excellent browser.

It's quite simple: If you don't define any site-specific overrides,
the overhead is one function call, one if and one return. This could be
further optimised to just one if. I don't think you will feel any
performance impact from this.

Hannes