feat(browserContext): add BrowserContext.browser() (#3849)

docs/api.md

@@ -309,6 +309,7 @@ await context.close();
 - [event: 'page'](#event-page)
 - [browserContext.addCookies(cookies)](#browsercontextaddcookiescookies)
 - [browserContext.addInitScript(script[, arg])](#browsercontextaddinitscriptscript-arg)
+- [browserContext.browser()](#browsercontextbrowser)
 - [browserContext.clearCookies()](#browsercontextclearcookies)
 - [browserContext.clearPermissions()](#browsercontextclearpermissions)
 - [browserContext.close()](#browsercontextclose)
@@ -398,6 +399,10 @@ await browserContext.addInitScript({
 ```
 
 > **NOTE** The order of evaluation of multiple scripts installed via [browserContext.addInitScript(script[, arg])](#browsercontextaddinitscriptscript-arg) and [page.addInitScript(script[, arg])](#pageaddinitscriptscript-arg) is not defined.
+
+#### browserContext.browser()
+- returns: <[null]|[Browser]> Returns the browser instance of the context. If it was launched as a persistent context null gets returned.
+
 #### browserContext.clearCookies()
 - returns: <[Promise]>
 
@@ -4430,6 +4435,7 @@ const backgroundPage = await context.waitForEvent('backgroundpage');
 - [event: 'page'](#event-page)
 - [browserContext.addCookies(cookies)](#browsercontextaddcookiescookies)
 - [browserContext.addInitScript(script[, arg])](#browsercontextaddinitscriptscript-arg)
+- [browserContext.browser()](#browsercontextbrowser)
 - [browserContext.clearCookies()](#browsercontextclearcookies)
 - [browserContext.clearPermissions()](#browsercontextclearpermissions)
 - [browserContext.close()](#browsercontextclose)

src/client/browserContext.ts

@@ -31,7 +31,7 @@ import { isUnderTest, headersObjectToArray } from '../utils/utils';
 export class BrowserContext extends ChannelOwner<channels.BrowserContextChannel, channels.BrowserContextInitializer> {
   _pages = new Set<Page>();
   private _routes: { url: URLMatch, handler: network.RouteHandler }[] = [];
-  readonly _browser: Browser | undefined;
+  readonly _browser: Browser | null = null;
   readonly _browserName: string;
   readonly _bindings = new Map<string, frames.FunctionWithSource>();
   _timeoutSettings = new TimeoutSettings();
@@ -92,6 +92,10 @@ export class BrowserContext extends ChannelOwner<channels.BrowserContextChannel,
     this._channel.setDefaultTimeoutNoReply({ timeout });
   }
 
+  browser(): Browser | null {
+    return this._browser;
+  }
+
   pages(): Page[] {
     return [...this._pages];
   }

src/client/download.ts

@@ -24,7 +24,7 @@ import * as fs from 'fs';
 import { mkdirIfNeeded } from '../utils/utils';
 
 export class Download extends ChannelOwner<channels.DownloadChannel, channels.DownloadInitializer> {
-  private _browser: Browser | undefined;
+  private _browser: Browser | null;
 
   static from(download: channels.DownloadChannel): Download {
     return (download as any)._object;

src/client/video.ts

@@ -20,7 +20,7 @@ import { BrowserContext } from './browserContext';
 import { ChannelOwner } from './channelOwner';
 
 export class Video extends ChannelOwner<channels.VideoChannel, channels.VideoInitializer> {
-  private _browser: Browser | undefined;
+  private _browser: Browser | null;
 
   static from(channel: channels.VideoChannel): Video {
     return (channel as any)._object;

test/browsercontext-basic.spec.ts

@@ -24,8 +24,10 @@ it('should create new context', async function({browser}) {
   const context = await browser.newContext();
   expect(browser.contexts().length).toBe(1);
   expect(browser.contexts().indexOf(context) !== -1).toBe(true);
+  expect(browser).toBe(context.browser());
   await context.close();
   expect(browser.contexts().length).toBe(0);
+  expect(browser).toBe(context.browser());
 });
 
 it('window.open should use parent tab context', async function({browser, server}) {