[webkit-reviews] review granted: [Bug 195169] [WPE] Enable web process sandbox : [Attachment 363324] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sun Mar 3 14:14:25 PST 2019

Daniel Bates <dbates at webkit.org> has granted Michael Catanzaro
<mcatanzaro at igalia.com>'s request for review:
Bug 195169: [WPE] Enable web process sandbox

Attachment 363324: Patch


--- Comment #12 from Daniel Bates <dbates at webkit.org> ---
Comment on attachment 363324
  --> https://bugs.webkit.org/attachment.cgi?id=363324

View in context: https://bugs.webkit.org/attachment.cgi?id=363324&action=review

I don’t know a lot about GTK or WPE, but felt this patch was sane. I do know
about bubble wrap (the packaging stuff not the bwrap executable).

> Source/WebKit/PlatformWPE.cmake:19
> +add_definitions(-DLIBDIR="${LIB_INSTALL_DIR}")

A “why”? for the ignorant in the change log would be nice :)

> Source/WebKit/UIProcess/Launcher/glib/BubblewrapLauncher.cpp:671

There’s something about this launcher’s name I really like... could it be
BUBBLEWRAP! :D Yes, that’s what it is.

> Source/cmake/BubblewrapSandboxChecks.cmake:4
> +	   message(FATAL_ERROR "bwrap executable is needed for

Bubblewrap sandbox <— I like it

More information about the webkit-reviews mailing list