[webkit-reviews] review granted: [Bug 213899] [Flatpak SDK][WPE] Launching the remote inspector kills MB : [Attachment 403766] Patch

[bugzilla-daemon at webkit.org]

Adrian Perez <aperez at igalia.com> has granted Philippe Normand
<pnormand at igalia.com>'s request for review:
Bug 213899: [Flatpak SDK][WPE] Launching the remote inspector kills MB
https://bugs.webkit.org/show_bug.cgi?id=213899

Attachment 403766: Patch

https://bugs.webkit.org/attachment.cgi?id=403766&action=review




--- Comment #6 from Adrian Perez <aperez at igalia.com> ---
Comment on attachment 403766
  --> https://bugs.webkit.org/attachment.cgi?id=403766
Patch

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

>> Source/JavaScriptCore/inspector/remote/glib/RemoteInspectorUtils.cpp:49
>> +	    const char* path = g_getenv("WEBKIT_INJECTED_BUNDLE_PATH");
> 
> I was lazy here. I can a new variable if the reviewer kindly asks :)

As this is going to be used only for developer builds, I think it is okay
to reuse this variable that is already being set by all our scripts instead
of introducing a new one which would need handling in many places.