Code review comment for lp:~abreu-alexandre/webbrowser-app/support-oxide-and-qtwebkit

Olivier Tilloy (osomon) wrote :

In WebappContainer::withOxide(), can you make the comment extra explicit by explaining that this works under confinement because version 1.1 of the ubuntu apparmor policy allows access to this file, whereas version 1.0 knows only of QtWebKit.

I just want to make sure when I read this code again in 6 months from now I’ll understand right away why we wrote it and how it works.

« Back to merge proposal