From 1e6410ba6753ba7046f57d70110ca8a3a1ef8607 Mon Sep 17 00:00:00 2001
From: Max Schmitt <max@schmitt.mx>
Date: Fri, 21 Feb 2025 14:35:42 +0100
Subject: [PATCH] fixes 2

---
 packages/playwright-core/src/server/frames.ts | 3 +--
 tests/assets/drag-n-drop-manual.html          | 2 --
 tests/page/page-drag.spec.ts                  | 4 ++--
 3 files changed, 3 insertions(+), 6 deletions(-)

diff --git a/packages/playwright-core/src/server/frames.ts b/packages/playwright-core/src/server/frames.ts
index 74c05e32d0..87685050ec 100644
--- a/packages/playwright-core/src/server/frames.ts
+++ b/packages/playwright-core/src/server/frames.ts
@@ -1211,8 +1211,7 @@ export class Frame extends SdkObject {
       dom.assertDone(await this._retryWithProgressIfNotConnected(progress, target, options.strict, false /* performActionPreChecks */, async handle => {
         return handle._retryPointerAction(progress, 'move and up', false, async point => {
           // NOTE: Normal browsers emit usually a lot of dragover/mousemove events during drag'n
-          // drop operations. We want to emit minimal to make the ecosystem work. When using
-          // Native drag'n drop the browser does emit dropover events instead.
+          // drop operations. We want to emit minimal (2) to make Angular CDK work.
           await this._page.mouse.move(point.x, point.y);
           await this._page.mouse.move(point.x, point.y);
           await this._page.mouse.up();
diff --git a/tests/assets/drag-n-drop-manual.html b/tests/assets/drag-n-drop-manual.html
index 424639bf94..143b87ef9a 100644
--- a/tests/assets/drag-n-drop-manual.html
+++ b/tests/assets/drag-n-drop-manual.html
@@ -62,7 +62,6 @@ div:not(.mouse-helper) {
           sourceElement.style.border = 'dashed'
         });
         
-        // Mouse move handler - move the element
         document.addEventListener('mousemove', function(e) {
           if (!isDragging) return;
           
@@ -70,7 +69,6 @@ div:not(.mouse-helper) {
           sourceElement.style.top = (e.clientY - offsetY) + 'px';
         });
         
-        // Mouse up handler - stop dragging and check if dropped on target
         document.addEventListener('mouseup', function(e) {
           if (!isDragging) return;
           
diff --git a/tests/page/page-drag.spec.ts b/tests/page/page-drag.spec.ts
index 7b25b4e604..906e037d91 100644
--- a/tests/page/page-drag.spec.ts
+++ b/tests/page/page-drag.spec.ts
@@ -64,7 +64,7 @@ it.describe('Drag and drop', () => {
       browserName === 'firefox' ? 'mousemove at 240;350' : 'dragstart at 120;86',
       'dragenter at 240;350',
       // NOTE: Normal browsers emit usually a lot of dragover events during drag'n
-      // drop operations. We want to emit minimal (2) to make the ecosystem work.
+      // drop operations. We want to emit minimal (2) to make Angular CDK work.
       'dragover at 240;350',
       'dragover at 240;350',
       'drop at 240;350',
@@ -334,7 +334,7 @@ it.describe('Drag and drop', () => {
     expect(await events.jsonValue()).toEqual([
       { type: 'mousemove', x: 120, y: 86 },
       // NOTE: Normal browsers emit usually a lot of mousemove events during drag'n
-      // drop operations. We want to emit minimal(2) to make the ecosystem work.
+      // drop operations. We want to emit minimal (2) to make Angular CDK work.
       { type: 'mousemove', x: 240, y: 350 },
       { type: 'mousemove', x: 240, y: 350 },
     ]);