Merge pull request #2734 from hardkoded/rename-product-to-browser

Rename product to browser

Author: kblok

.github/workflows/dotnet.yml

@@ -91,7 +91,7 @@ jobs:
     - name: Test (Linux)
       if: matrix.os == 'ubuntu-latest'
       env:
-        PRODUCT: ${{ matrix.browser }}
+        BROWSER: ${{ matrix.browser }}
         HEADLESS_MODE: ${{ matrix.mode }}
       run: |
           Xvfb :1 -screen 5 1024x768x8 &
@@ -101,7 +101,7 @@ jobs:
     - name: Test (Windows)
       if: matrix.os == 'windows-latest'
       env:
-        PRODUCT: ${{ matrix.browser }}
+        BROWSER: ${{ matrix.browser }}
         HEADLESS_MODE: ${{ matrix.mode }}
       run: |
         cd .\lib\PuppeteerSharp.Tests

README.md

@@ -60,7 +60,7 @@ await using var page = await browser.NewPageAsync();
 await page.GoToAsync("http://www.google.com");
 await page.ScreenshotAsync(outputFile);
 ```
-<sup><a href='https://github.com/hardkoded/puppeteer-sharp/blob/master/lib/PuppeteerSharp.Tests/ScreenshotTests/PageScreenshotTests.cs#L53-L61' title='Snippet source file'>snippet source</a> | <a href='#snippet-ScreenshotAsync' title='Start of snippet'>anchor</a></sup>
+<sup><a href='https://github.com/hardkoded/puppeteer-sharp/blob/master/lib/PuppeteerSharp.Tests/ScreenshotTests/PageScreenshotTests.cs#L54-L62' title='Snippet source file'>snippet source</a> | <a href='#snippet-ScreenshotAsync' title='Start of snippet'>anchor</a></sup>
 <!-- endSnippet -->
 
 You can also change the view port before generating the screenshot

lib/PuppeteerSharp.Nunit/PuppeteerTestAttribute.cs

@@ -20,7 +20,7 @@ public class PuppeteerTestAttribute : NUnitAttribute, IApplyToTest
     {
         private static TestExpectation[] _localExpectations;
         private static TestExpectation[] _upstreamExpectations;
-        public static readonly bool IsChrome = Environment.GetEnvironmentVariable("PRODUCT") != "FIREFOX";
+        public static readonly bool IsChrome = Environment.GetEnvironmentVariable("BROWSER") != "FIREFOX";
         // TODO: Change implementation when we implement Webdriver Bidi
         public static readonly bool IsCdp = true;
         public static readonly HeadlessMode Headless =

lib/PuppeteerSharp/BrowserFetcher.cs

@@ -206,8 +206,8 @@ internal static string GetBrowsersLocation()
             return assemblyDirectory.FullName;
         }
 
-        private static string GetDownloadURL(SupportedBrowser product, Platform platform, string baseUrl, string buildId)
-            => _downloadsUrl[product](platform, buildId, baseUrl);
+        private static string GetDownloadURL(SupportedBrowser browser, Platform platform, string baseUrl, string buildId)
+            => _downloadsUrl[browser](platform, buildId, baseUrl);
 
         private static void ExtractTar(string zipPath, string folderPath)
         {

lib/PuppeteerSharp/BrowserFetcherOptions.cs

@@ -17,7 +17,7 @@ public class BrowserFetcherOptions
         public delegate Task CustomFileDownloadAction(string address, string fileName);
 
         /// <summary>
-        /// Product. Defaults to Chrome.
+        /// Browser. Defaults to Chrome.
         /// </summary>
         public SupportedBrowser Browser { get; set; } = SupportedBrowser.Chrome;
 

lib/PuppeteerSharp/LaunchOptions.cs

@@ -53,7 +53,7 @@ public bool Headless
         /// <summary>
         /// Additional arguments to pass to the browser instance. List of Chromium flags can be found <a href="http://peter.sh/experiments/chromium-command-line-switches/">here</a>.
         /// </summary>
-        public string[] Args { get; set; } = Array.Empty<string>();
+        public string[] Args { get; set; } = [];
 
         /// <summary>
         /// Maximum time in milliseconds to wait for the browser instance to start. Defaults to 30000 (30 seconds). Pass 0 to disable timeout.

lib/PuppeteerSharp/Launcher.cs

@@ -63,7 +63,7 @@ public async Task<IBrowser> LaunchAsync(LaunchOptions options)
             {
                 SupportedBrowser.Chrome or SupportedBrowser.Chromium => new ChromeLauncher(executable, options),
                 SupportedBrowser.Firefox => new FirefoxLauncher(executable, options),
-                _ => throw new ArgumentException("Invalid product"),
+                _ => throw new ArgumentException("Invalid browser"),
             };
 
             try
@@ -135,14 +135,14 @@ public async Task<IBrowser> ConnectAsync(ConnectOptions options)
 
                 var version = await connection.SendAsync<BrowserGetVersionResponse>("Browser.getVersion").ConfigureAwait(false);
 
-                var product = version.Product.ToLower(CultureInfo.CurrentCulture).Contains("firefox")
+                var browser = version.Product.ToLower(CultureInfo.CurrentCulture).Contains("firefox")
                   ? SupportedBrowser.Firefox
                   : SupportedBrowser.Chromium;
 
                 var response = await connection.SendAsync<GetBrowserContextsResponse>("Target.getBrowserContexts").ConfigureAwait(false);
                 return await CdpBrowser
                     .CreateAsync(
-                        product,
+                        browser,
                         connection,
                         response.BrowserContextIds,
                         options.IgnoreHTTPSErrors,