Implement BrowserFetcher (#342)

Author: kblok

ISSUE_TEMPLATE.md

@@ -19,7 +19,7 @@ E.g.
 ```csharp
 // Arrange
 var options = new LaunchOptions { /*  */ };
-var chromiumRevision = Downloader.DefaultRevision;
+var chromiumRevision = BrowserFetcher.DefaultRevision;
 var browser = await Puppeteer.LaunchAsync(options, chromiumRevision);
 var page = browser.NewPageAsync();
 

README.md

@@ -25,11 +25,11 @@ Puppeteer Sharp is a .NET port of the official [Node.JS Puppeteer API](https://g
 ## Take screenshots
 
 ```cs
-await Downloader.CreateDefault().DownloadRevisionAsync(chromiumRevision);
+await Downloader.CreateDefault().DownloadAsync(chromiumRevision);
 var browser = await Puppeteer.LaunchAsync(new LaunchOptions
 {
     Headless = true
-}, chromiumRevision);
+});
 var page = await browser.NewPageAsync();
 await page.GoToAsync("http://www.google.com");
 await page.ScreenshotAsync(outputFile);
@@ -50,11 +50,11 @@ await page.SetViewport(new ViewPortOptions
 ## Generate PDF files
 
 ```cs
-await Downloader.CreateDefault().DownloadRevisionAsync(chromiumRevision);
+await new BrowserFetcher().DownloadAsync(BrowserFetcher.DefaultRevision);
 var browser = await Puppeteer.LaunchAsync(new LaunchOptions
 {
     Headless = true
-}, chromiumRevision);
+});
 var page = await browser.NewPageAsync();
 await page.GoToAsync("http://www.google.com");
 await page.PdfAsync(outputFile);

demos/PuppeteerSharpPdfDemo/Program.cs

@@ -16,10 +16,10 @@ public static async Task Main(string[] args)
             };
 
             Console.WriteLine("Downloading chromium");
-            await Downloader.CreateDefault().DownloadRevisionAsync(Downloader.DefaultRevision);
+            await new BrowserFetcher().DownloadAsync(BrowserFetcher.DefaultRevision);
 
             Console.WriteLine("Navigating google");
-            using (var browser = await Puppeteer.LaunchAsync(options, Downloader.DefaultRevision))
+            using (var browser = await Puppeteer.LaunchAsync(options))
             using (var page = await browser.NewPageAsync())
             {
                 await page.GoToAsync("http://www.google.com");

docfx_project/api/index.md

@@ -7,11 +7,11 @@ Puppeteer Sharp is a .NET port of the official [Node.JS Puppeteer API](https://g
 ## Take screenshots
 
 ```cs
-await Downloader.CreateDefault().DownloadRevisionAsync(chromiumRevision);
+await Downloader.CreateDefault().DownloadAsync(chromiumRevision);
 var browser = await Puppeteer.LaunchAsync(new LaunchOptions
 {
     Headless = true
-}, chromiumRevision);
+});
 var page = await browser.NewPageAsync();
 await page.GoToAsync("http://www.google.com");
 await page.ScreenshotAsync(outputFile);
@@ -32,11 +32,11 @@ await page.SetViewport(new ViewPortOptions
 ## Generate PDF files
 
 ```cs
-await Downloader.CreateDefault().DownloadRevisionAsync(chromiumRevision);
+await new BrowserFetcher().DownloadAsync(BrowserFetcher.DefaultRevision);
 var browser = await Puppeteer.LaunchAsync(new LaunchOptions
 {
     Headless = true
-}, chromiumRevision);
+});
 var page = await browser.NewPageAsync();
 await page.GoToAsync("http://www.google.com");
 await page.PdfAsync(outputFile);

docfx_project/index.md

@@ -7,11 +7,11 @@ Puppeteer Sharp is a .NET port of the official [Node.JS Puppeteer API](https://g
 ## Take screenshots
 
 ```cs
-await Downloader.CreateDefault().DownloadRevisionAsync(chromiumRevision);
+await Downloader.CreateDefault().DownloadAsync(chromiumRevision);
 var browser = await Puppeteer.LaunchAsync(new LaunchOptions
 {
     Headless = true
-}, chromiumRevision);
+});
 var page = await browser.NewPageAsync();
 await page.GoToAsync("http://www.google.com");
 await page.ScreenshotAsync(outputFile);
@@ -32,11 +32,11 @@ await page.SetViewport(new ViewPortOptions
 ## Generate PDF files
 
 ```cs
-await Downloader.CreateDefault().DownloadRevisionAsync(chromiumRevision);
+await new BrowserFetcher().DownloadAsync(BrowserFetcher.DefaultRevision);
 var browser = await Puppeteer.LaunchAsync(new LaunchOptions
 {
     Headless = true
-}, chromiumRevision);
+});
 var page = await browser.NewPageAsync();
 await page.GoToAsync("http://www.google.com");
 await page.PdfAsync(outputFile);

lib/PuppeteerSharp.TestServer/wwwroot/chromium-linux.zip

@@ -14,7 +14,7 @@ public DisconnectedTests(ITestOutputHelper output) : base(output)
         [Fact]
         public async Task ShouldEmittedWhenBrowserGetsClosedDisconnectedOrUnderlyingWebsocketGetsClosed()
         {
-            var originalBrowser = await Puppeteer.LaunchAsync(TestConstants.DefaultBrowserOptions(), TestConstants.ChromiumRevision, TestConstants.LoggerFactory);
+            var originalBrowser = await Puppeteer.LaunchAsync(TestConstants.DefaultBrowserOptions(), TestConstants.LoggerFactory);
             var connectOptions = new ConnectOptions { BrowserWSEndpoint = originalBrowser.WebSocketEndpoint };
             var remoteBrowser1 = await Puppeteer.ConnectAsync(connectOptions, TestConstants.LoggerFactory);
             var remoteBrowser2 = await Puppeteer.ConnectAsync(connectOptions, TestConstants.LoggerFactory);
@@ -37,4 +37,4 @@ public async Task ShouldEmittedWhenBrowserGetsClosedDisconnectedOrUnderlyingWebs
             Assert.Equal(1, disconnectedRemote2);
         }
     }
-}
+}

lib/PuppeteerSharp.Tests/BrowserTests/Events/DisconnectedTests.cs

@@ -15,8 +15,8 @@ await Assert.ThrowsAsync<ChromeProcessException>(async () =>
             {
                 var options = TestConstants.DefaultBrowserOptions();
                 options.Args = new[] { "-remote-debugging-port=-2" };
-                await Puppeteer.LaunchAsync(options, TestConstants.ChromiumRevision, TestConstants.LoggerFactory);
+                await Puppeteer.LaunchAsync(options, TestConstants.LoggerFactory);
             });
         }
     }
-}
+}

lib/PuppeteerSharp.Tests/Issues/Issue0128.cs

@@ -24,7 +24,7 @@ public PuppeteerBrowserBaseTest(ITestOutputHelper output) : base(output)
 
         protected virtual async Task InitializeAsync()
         {
-            Browser = await Puppeteer.LaunchAsync(TestConstants.DefaultBrowserOptions(), TestConstants.ChromiumRevision, TestConstants.LoggerFactory);
+            Browser = await Puppeteer.LaunchAsync(TestConstants.DefaultBrowserOptions(), TestConstants.LoggerFactory);
         }
 
         protected virtual async Task DisposeAsync() => await Browser.CloseAsync();

lib/PuppeteerSharp.Tests/PuppeteerBrowserBaseTest.cs

@@ -21,7 +21,7 @@ public void Dispose()
 
         private async Task SetupAsync()
         {
-            var downloaderTask = Downloader.CreateDefault().DownloadRevisionAsync(TestConstants.ChromiumRevision);
+            var downloaderTask = new BrowserFetcher().DownloadAsync(BrowserFetcher.DefaultRevision);
 
             Server = SimpleServer.Create(TestConstants.Port, TestUtils.FindParentDirectory("PuppeteerSharp.TestServer"));
             HttpsServer = SimpleServer.CreateHttps(TestConstants.HttpsPort, TestUtils.FindParentDirectory("PuppeteerSharp.TestServer"));
@@ -32,4 +32,4 @@ private async Task SetupAsync()
             await Task.WhenAll(downloaderTask, serverStart, httpsServerStart);
         }
     }
-}
+}