From 03b0f911d99c2e59e14de29eaa1af7eec1bdf986 Mon Sep 17 00:00:00 2001
From: Dmitry Gozman <dgozman@gmail.com>
Date: Fri, 29 Jul 2022 09:23:02 -0700
Subject: [PATCH] fix(screenshot): avoid unnecessary work (#16035)

Do not evaluate when not hiding caret nor disabling animations.
---
 packages/playwright-core/src/server/screenshotter.ts | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/packages/playwright-core/src/server/screenshotter.ts b/packages/playwright-core/src/server/screenshotter.ts
index eebf4ab8c1..66eebc3d82 100644
--- a/packages/playwright-core/src/server/screenshotter.ts
+++ b/packages/playwright-core/src/server/screenshotter.ts
@@ -140,6 +140,9 @@ export class Screenshotter {
   }
 
   async _preparePageForScreenshot(progress: Progress, hideCaret: boolean, disableAnimations: boolean) {
+    if (!hideCaret && !disableAnimations)
+      return;
+
     if (disableAnimations)
       progress.log('  disabled all CSS animations');
     await Promise.all(this._page.frames().map(async frame => {