fix: make sure Playwright works in DevContainers

This patch fixes Invalid browser name exception when Playwright is used in a DevContainer context

Fixes #3174

Author: CurlyFire

src/Playwright.TestAdapter/PlaywrightSettingsProvider.cs

@@ -40,9 +40,8 @@ public static string BrowserName
         get
         {
             var browserFromEnv = Environment.GetEnvironmentVariable("BROWSER")?.ToLowerInvariant();
-            // GitHub Codespaces sets the BROWSER environment variable, ignore it if its bogus.
-            var ignoreValueFromEnv = Environment.GetEnvironmentVariable("CODESPACES") == "true" && browserFromEnv!.StartsWith("/vscode/");
-            if (!string.IsNullOrEmpty(browserFromEnv) && !ignoreValueFromEnv)
+            // GitHub Codespaces and DevContainers sets the BROWSER environment variable, ignore it if its bogus.
+            if (!string.IsNullOrEmpty(browserFromEnv) && !browserFromEnv!.StartsWith("/vscode/"))
             {
                 ValidateBrowserName(browserFromEnv!, "'BROWSER' environment variable", "\nTry to remove 'BROWSER' environment variable for using default browser");
                 return browserFromEnv!;