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

Alexandre Abreu (abreu-alexandre) 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