refactor: remove no longer needed bringToFront workaround

Author: web-padawan

packages/combo-box/src/vaadin-combo-box-base-mixin.js

@@ -184,15 +184,6 @@ export const ComboBoxBaseMixin = (superClass) =>
         this.clearElement.addEventListener('mousedown', this._boundOnClearButtonMouseDown);
       }
 
-      const bringToFrontListener = () => {
-        requestAnimationFrame(() => {
-          this._overlayElement.bringToFront();
-        });
-      };
-
-      this.addEventListener('mousedown', bringToFrontListener);
-      this.addEventListener('touchstart', bringToFrontListener);
-
       this.addController(new VirtualKeyboardController(this));
     }
 

test/integration/dialog-combo-box.test.js

@@ -1,14 +1,6 @@
 import { expect } from '@vaadin/chai-plugins';
 import { sendKeys } from '@vaadin/test-runner-commands';
-import {
-  fixtureSync,
-  mousedown,
-  nextFrame,
-  nextUpdate,
-  oneEvent,
-  outsideClick,
-  touchstart,
-} from '@vaadin/testing-helpers';
+import { fixtureSync, nextUpdate, oneEvent, outsideClick } from '@vaadin/testing-helpers';
 import './not-animated-styles.js';
 import '@vaadin/combo-box';
 import '@vaadin/dialog';
@@ -78,26 +70,4 @@ describe('combo-box in dialog', () => {
       expect(dialog.opened).to.be.false;
     });
   });
-
-  describe('modeless', () => {
-    beforeEach(async () => {
-      dialog.modeless = true;
-      comboBox.open();
-      await nextFrame();
-    });
-
-    it('should not end up behind the dialog overlay on mousedown', async () => {
-      mousedown(comboBox);
-      await nextFrame();
-      expect(comboBox.$.overlay._last).to.be.true;
-    });
-
-    it('should not end up behind the dialog overlay on touchstart', async () => {
-      const { left: x, top: y } = comboBox.getBoundingClientRect();
-      touchstart(comboBox, { x, y });
-
-      await nextFrame();
-      expect(comboBox.$.overlay._last).to.be.true;
-    });
-  });
 });