From 845054d28b8554f73cfe4aa380c3c98fe63923d4 Mon Sep 17 00:00:00 2001
From: Dmitry Gozman <dgozman@gmail.com>
Date: Mon, 10 May 2021 08:41:18 -0700
Subject: [PATCH] feat(firefox): bump to 1257 and 1247 (stable) (#6476)

This fixes addBinding on pages with CSP.
---
 browsers.json                                | 4 ++--
 tests/browsercontext-expose-function.spec.ts | 9 +++++++++
 2 files changed, 11 insertions(+), 2 deletions(-)

diff --git a/browsers.json b/browsers.json
index 3d56552591..3936dda0e7 100644
--- a/browsers.json
+++ b/browsers.json
@@ -8,12 +8,12 @@
     },
     {
       "name": "firefox",
-      "revision": "1256",
+      "revision": "1257",
       "installByDefault": true
     },
     {
       "name": "firefox-stable",
-      "revision": "1246",
+      "revision": "1247",
       "installByDefault": false
     },
     {
diff --git a/tests/browsercontext-expose-function.spec.ts b/tests/browsercontext-expose-function.spec.ts
index 0823a2fb9f..bf2e2fc80e 100644
--- a/tests/browsercontext-expose-function.spec.ts
+++ b/tests/browsercontext-expose-function.spec.ts
@@ -85,3 +85,12 @@ it('exposeBindingHandle should work', async ({context}) => {
   expect(await target.evaluate(x => x.foo)).toBe(42);
   expect(result).toEqual(17);
 });
+
+it('should work with CSP', async ({ page, context, server }) => {
+  server.setCSP('/empty.html', 'default-src "self"');
+  await page.goto(server.EMPTY_PAGE);
+  let called = false;
+  await context.exposeBinding('hi', () => called = true);
+  await page.evaluate(() => (window as any).hi());
+  expect(called).toBe(true);
+});