In a discussion with Patrick after bug #201486 we agreed that it may be a good thing to check a little bit more thoroughly whether we are running inside a Snap sandbox: instead of looking for a “SNAP” environment variable only, we can check also for the “SNAP_NAME” and “SNAP_REVISION” variables. The rationale is that “SNAP” is kind of a short variable and not completely unlikely to be set by an user without being inside a Snap container. Checking a couple of environment variables should be fine (it is very unlikely that they would be all set accidentally at the same time), and we don't want anyway to make complex checks anyway.
Created attachment 378790 [details] Patch
Added Patrick in CC, as he is our sandboxing expert, in case he wants to comment on this :)
Comment on attachment 378790 [details] Patch Looks good.
Comment on attachment 378790 [details] Patch Clearing flags on attachment: 378790 Committed r249882: <https://trac.webkit.org/changeset/249882>
All reviewed patches have been landed. Closing bug.