[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

[tor-commits] [Git][tpo/applications/mullvad-browser][mullvad-browser-153.4.0esr-16.0-1] 2 commits: fixup! BB 43535: Enable tests

Title: GitLab

Pier Angelo Vendrame pushed to branch mullvad-browser-153.4.0esr-16.0-1 at The Tor Project / Applications / Mullvad Browser

Commits:

  • 0390b15b
    by Beatriz Rizental at 2026-10-07T16:31:47+02:00 
    fixup! BB 43535: Enable tests

This reverts commit e42d7f1136aa55573c01c91a87dec69bbfbdb529.
  • 7eeecda8
    by Beatriz Rizental at 2026-10-07T16:31:48+02:00 
    fixup! BB 45120: Add tests for Base Browser

Bug 44820: The mochitest harness built the testing-common resource
mapping as "file:///" + testingModulesDir. On Linux and macOS
testingModulesDir is already an absolute path, so this produced
"file:////home/...", which content processes fail to load from.

2 changed files:

Changes:

  • testing/mochitest/runtests.py
    ... ... @@ -1701,7 +1701,7 @@ class MochitestDesktop:
    1701 1701
    1702 1702 
                 if options.testingModulesDir is not None:
    1703 1703 
                     manifestFile.write(
    1704 
    -                    f"resource testing-common file:///{options.testingModulesDir}\n"
    1704 
    +                    f"resource testing-common {Path(options.testingModulesDir).as_uri()}/\n"
    1705 1705 
                     )
    1706 1706 
             if options.store_chrome_manifest:
    1707 1707 
                 shutil.copyfile(manifest, options.store_chrome_manifest)

  • testing/specialpowers/api.js
    ... ... @@ -40,18 +40,6 @@ this.specialpowers = class extends ExtensionAPI {
    40 40 
             uri,
    41 41 
             resProto.ALLOW_CONTENT_ACCESS
    42 42 
           );
    43 
    -    } else {
    44 
    -      // This is a hack!
    45 
    -      // For some reason, this specific substituion has an extra `/` in the path.
    46 
    -      // This is a workaround to fix it.
    47 
    -      //
    48 
    -      // TODO (#43545): Remove this once we have a proper fix.
    49 
    -      let uri = resProto.getSubstitution("testing-common");
    50 
    -      resProto.setSubstitution(
    51 
    -        "testing-common",
    52 
    -        Services.io.newURI(uri.spec.replace("file:////", "file:///")),
    53 
    -        resProto.ALLOW_CONTENT_ACCESS
    54 
    -      );
    55 43 
         }
    56 44
    57 45 
         SpecialPowersParent.registerActor();

    • —
    View it on GitLab.
    You're receiving this email because of your account on gitlab.torproject.org. Manage all notifications · Help Notification message regarding https://gitlab.torproject.org/tpo/applications/mullvad-browser/-/compare/0759d92f5b878377996a62065365044c1e4e6edb...7eeecda827af6724b9b8a16b6b81185980aafb34 at 1791383522 

    _______________________________________________
tor-commits mailing list -- tor-commits@xxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to tor-commits-leave@xxxxxxxxxxxxxxxxxxxx