Skip to content

ohos ci: Attempt to close USB mode popup#41695

Merged
yezhizhen merged 2 commits intoservo:mainfrom
jschwe:ohos-ci-popup
Jan 6, 2026
Merged

ohos ci: Attempt to close USB mode popup#41695
yezhizhen merged 2 commits intoservo:mainfrom
jschwe:ohos-ci-popup

Conversation

@jschwe
Copy link
Copy Markdown
Member

@jschwe jschwe commented Jan 5, 2026

On OpenHarmony a system pop-up will appear, if a USB connection is established. This pop-up will overlay servo, and intercept touch events, such as a simulated slide event, causing the CI test to fail.
To avoid this, we now check if the focused window is servo, and attempt to close the pop-up speculatively by doing the back action, which dismisses regular dismissible popups. See also the relevant OpenHarmony documentation

Testing: Manually tested the fix on a device locally and mach try
Fixes: #41608

@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Jan 5, 2026
@yezhizhen yezhizhen self-requested a review January 6, 2026 01:39
Copy link
Copy Markdown
Member

@yezhizhen yezhizhen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems reasonable. I tried these combinations and it works.

@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Jan 6, 2026
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Jan 6, 2026
@yezhizhen yezhizhen enabled auto-merge January 6, 2026 02:46
@yezhizhen yezhizhen added this pull request to the merge queue Jan 6, 2026
@servo-highfive servo-highfive added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Jan 6, 2026
Merged via the queue into servo:main with commit 8cd11c9 Jan 6, 2026
32 checks passed
@servo-highfive servo-highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Jan 6, 2026
@jschwe jschwe deleted the ohos-ci-popup branch January 6, 2026 05:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-awaiting-review There is new code that needs to be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ohos CI: Close "use USB to" dialog before running tests

3 participants