From 9ab82127b5c20f0af93bc751651c6771d2d2f402 Mon Sep 17 00:00:00 2001
From: polykernel <81340136+polykernel@users.noreply.github.com>
Date: Fri, 3 Sep 2021 09:41:52 -0400
Subject: [PATCH] brave: add support for command line arguments

Add formal argument `commandLineArgs` to the brave derivation and append it
to the `gappsWrapperArgs` expected by the `wrapGAppsHook`. This adds some
flexibility to the wrapper creation of brave, and aligns with customization
avalible in other chromium derivation.
---
 pkgs/applications/networking/browsers/brave/default.nix | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/pkgs/applications/networking/browsers/brave/default.nix b/pkgs/applications/networking/browsers/brave/default.nix
index fe7cfb6c7b36..2f66076a5a6d 100644
--- a/pkgs/applications/networking/browsers/brave/default.nix
+++ b/pkgs/applications/networking/browsers/brave/default.nix
@@ -41,6 +41,7 @@
 , zlib
 , xdg-utils
 , wrapGAppsHook
+, commandLineArgs ? ""
 }:
 
 let
@@ -156,6 +157,11 @@ stdenv.mkDerivation rec {
       runHook postInstall
   '';
 
+  preFixup = ''
+    # Add command line args to wrapGApp.
+    gappsWrapperArgs+=(--add-flags ${lib.escapeShellArg commandLineArgs})
+  '';
+
   installCheckPhase = ''
     # Bypass upstream wrapper which suppresses errors
     $out/opt/brave.com/brave/brave --version