Code review comment for lp:~abreu-alexandre/webbrowser-app/better-control-webengine-lib-loaded

Revision history for this message
Olivier Tilloy (osomon) wrote :

> totally, I can go into all sorts of ways to make it work, but
> I am a bit reluctant to go there & make it more convoluted, instead of fixing
> it the right way ..

Well, it’s not really about whether it’s convoluted or not. Alberto’s suggestion has the advantage of having the properties being set at construction time, which definitely avoids potential issues. Additionally, lazy loading UnityWebApps would mean a (potentially) reduced startup time (I haven’t measured anything though).

« Back to merge proposal