Commit aafe62af authored by Pier Angelo Vendrame's avatar Pier Angelo Vendrame 🎃
Browse files

BB 9173: Change the default Firefox profile directory to be relative. 

This commit makes Firefox look for the default profile directory in a
directory relative to the binary path.
The directory can be specified through the --with-relative-data-dir.
This is relative to the same directory as the firefox main binary for
Linux and Windows.

On macOS, we remove Contents/MacOS from it.
Or, in other words, the directory is relative to the application
bundle.

This behavior can be overriden at runtime, by placing a file called
system-install adjacent to the firefox main binary (also on macOS).
parent a22353bc

browser/config/mozconfigs/base-browser

+2 −0
Original line number Diff line number Diff line
@@ -48,3 +48,5 @@ ac_add_options --disable-legacy-profile-creation 
if test -z "$WASI_SYSROOT"; then

    ac_add_options --without-wasm-sandboxed-libraries

fi



ac_add_options --with-relative-data-dir=BaseBrowser/Data/Browser

moz.configure

+19 −0
Original line number Diff line number Diff line
@@ -963,6 +963,25 @@ with only_when(cross_compiling): 
    )

    set_config("JS_BINARY", depends_if("JS_BINARY")(lambda value: value[0]))



option(

    "--with-relative-data-dir",

    nargs=1,

    help="Sets the data directories to be relative to the application directory",

)





@depends("--with-relative-data-dir", target)

@imports("json")

def relative_data_dir(value, target):

    if value and target.os == "Android":

        die("--with-relative-data-dir is not supported on Android")

    if value:

        return json.dumps(value[0])





set_define("RELATIVE_DATA_DIR", relative_data_dir)





option(

    "--with-base-browser-version",

    nargs=1,

mozconfig-macos

+3 −0
Original line number Diff line number Diff line
@@ -10,3 +10,6 @@ ac_add_options --disable-update-agent 


# Let's make sure no preference is enabling either Adobe's or Google's CDM.

ac_add_options --disable-eme



# For base-browser we do not enable portable mode on macOS.

ac_add_options --without-relative-data-dir

python/mozbuild/mozbuild/mach_commands.py

+4 −4
Original line number Diff line number Diff line
@@ -2635,22 +2635,22 @@ def _run_desktop( 
        if appdata is True:

            appdata = tmpdir



        extra_env["MOZ_APP_DATA"] = os.path.normpath(

        extra_env["BB_APP_DATA"] = os.path.normpath(

            os.path.join(appdata, "AppData", "Roaming")

        )

        command_context.log(

            logging.INFO,

            "run",

            {"app_data": extra_env["MOZ_APP_DATA"]},

            {"app_data": extra_env["BB_APP_DATA"]},

            "Overriding application data directory to {app_data}",

        )

        extra_env["MOZ_LOCAL_APP_DATA"] = os.path.normpath(

        extra_env["BB_LOCAL_APP_DATA"] = os.path.normpath(

            os.path.join(appdata, "Local")

        )

        command_context.log(

            logging.INFO,

            "run",

            {"local_app_data": extra_env["MOZ_LOCAL_APP_DATA"]},

            {"local_app_data": extra_env["BB_LOCAL_APP_DATA"]},

            "Overriding local application data directory to {local_app_data}",

        )

toolkit/xre/LauncherRegistryInfo.cpp

+47 −3
Original line number Diff line number Diff line
@@ -14,6 +14,9 @@ 
#include <string>

#include <type_traits>



// tor-browser#42163

#include <filesystem>



// This is how long to wait after the last crash to try using the launcher

// process again. (bug 1846830) 45 days in 100-nanosecond intervals (FILETIME

// units)
@@ -677,6 +680,45 @@ LauncherResult<bool> LauncherRegistryInfo::ClearLauncherCrashTimestamp() { 


LauncherResult<std::wstring>

LauncherRegistryInfo::BuildDefaultBlocklistFilename() {

  // tor-browser#42163: Make the DLL blocklist obey portable mode

  {

    std::filesystem::path appDir;

    {

      mozilla::UniquePtr<wchar_t[]> appDirStr = GetFullBinaryPath();

      if (!appDirStr) {

        return LAUNCHER_ERROR_FROM_WIN32(ERROR_NOT_ENOUGH_MEMORY);

      }

      appDir = std::filesystem::path(appDirStr.get()).parent_path();

    }

    std::error_code ec;

    const bool isPortable =

        !std::filesystem::exists(appDir / L"system-install", ec);

    if (ec) {

      // exists is supposed not to set an error when a file does not exist

      // (whereas other functions such as is_regular_file sets it).

      // The standard is quite opaque about the meaning of the numeric codes.

      // Moreover, we use libcxx on Windows, and it seems they created a sort of

      // POSIX compatibility layer (e.g., for stat), see

      // libcxx/src/filesystem/posix_compat.h.

      // std::error_code has a message function, but all the various macro are

      // specific to handle Windows errors, so we have to use the generic error.

      // At least, at the moment the error is dropped eventually.

      return LAUNCHER_ERROR_GENERIC();

    }

    if (isPortable) {

      // RELATIVE_DATA_DIR must have forward slashes, but weakly_canonical

      // already changes them to backslashes.

      const std::filesystem::path blocklistPath =

          std::filesystem::weakly_canonical(

              appDir / L"" RELATIVE_DATA_DIR / L"blocklist", ec);

      if (ec) {

        return LAUNCHER_ERROR_GENERIC();

      }

      return blocklistPath.wstring();

    }

    // Normal installation, continue on Mozilla's path

  }



  // These flags are chosen to avoid I/O, see bug 1363398.

  const DWORD flags =

      KF_FLAG_SIMPLE_IDLIST | KF_FLAG_DONT_VERIFY | KF_FLAG_NO_ALIAS;
@@ -709,6 +751,8 @@ LauncherRegistryInfo::BuildDefaultBlocklistFilename() { 
}



LauncherResult<std::wstring> LauncherRegistryInfo::GetBlocklistFileName() {

  // tor-browser#42163: Make the DLL blocklist obey portable mode

#ifndef BASE_BROWSER_VERSION

  LauncherResult<Disposition> disposition = Open();

  if (disposition.isErr()) {

    return disposition.propagateErr();
@@ -724,19 +768,19 @@ LauncherResult<std::wstring> LauncherRegistryInfo::GetBlocklistFileName() { 
    UniquePtr<wchar_t[]> buf = readResult.unwrap();

    return std::wstring(buf.get());

  }



#endif

  LauncherResult<std::wstring> defaultBlocklistPath =

      BuildDefaultBlocklistFilename();

  if (defaultBlocklistPath.isErr()) {

    return defaultBlocklistPath.propagateErr();

  }



#ifndef BASE_BROWSER_VERSION

  LauncherVoidResult writeResult = WriteRegistryValueString(

      mRegKey, ResolveBlocklistValueName(), defaultBlocklistPath.inspect());

  if (writeResult.isErr()) {

    return writeResult.propagateErr();

  }



#endif

  return defaultBlocklistPath;

}