Skip to content

Disable Chromium translate dialog via managed policy and E2E test#712

Open
guysoft wants to merge 1 commit into
develfrom
bugfix/translation-disable
Open

Disable Chromium translate dialog via managed policy and E2E test#712
guysoft wants to merge 1 commit into
develfrom
bugfix/translation-disable

Conversation

@guysoft

@guysoft guysoft commented Jul 2, 2026

Copy link
Copy Markdown
Owner

Add /etc/chromium/policies/managed/disable_translate.json with TranslateEnabled=false and remove the obsolete --disable-features=TranslateUI flag from start_chromium_browser.

Add test_translation_disabled.sh: uploads a local German page, launches windowed Chromium, captures translation_disabled.png as visual evidence, then restores the kiosk session. Remove the broken branch-gated reproduction logic from prepare-image.sh (no .git in E2E container).

@guysoft

guysoft commented Jul 3, 2026

Copy link
Copy Markdown
Owner Author

Has a nice new test.
Before:
translation_disabled

After:

translation_disabled

Add /etc/chromium/policies/managed/disable_translate.json with
TranslateEnabled=false and remove the obsolete --disable-features=TranslateUI
flag from start_chromium_browser.

Add test_translation_disabled.sh: uploads a local German page, launches
windowed Chromium, captures translation_disabled.png as visual evidence,
then restores the kiosk session. Remove the broken branch-gated reproduction
logic from prepare-image.sh (no .git in E2E container).
@guysoft guysoft force-pushed the bugfix/translation-disable branch from 2c65c65 to 0531b66 Compare July 6, 2026 21:35
@guysoft

guysoft commented Jul 6, 2026

Copy link
Copy Markdown
Owner Author

Regression proof after rebasing on current devel and hardening the translation screenshot wait:

This confirms the E2E test exercises the regression path: removing the managed TranslateEnabled=false policy makes the dialog appear, while the PR branch suppresses it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant