From 19ca206a62d08ed290bcda7f5974b1cdb6a765c1 Mon Sep 17 00:00:00 2001
From: Mustafa Emre Acer <meacer@chromium.org>
Date: Mon, 02 Feb 2026 14:19:58 -0800
Subject: [PATCH] Disable ligatures in permission dialogs on Android

This change disables rendering of ligatures in permission dialogs
for security and readability. A similar change was previously made
for the omnibox in crrev.com/c/7199504.

Bug: 418214610
Change-Id: I4f0e61d87ea50304ef0909f939908393a81d9e78
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7536052
Reviewed-by: Sinan Sahin <sinansahin@google.com>
Commit-Queue: Mustafa Emre Acer <meacer@chromium.org>
Reviewed-by: Elias Klim <elklm@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1578393}
---

diff --git a/components/permissions/android/java/src/org/chromium/components/permissions/PermissionDialogCoordinator.java b/components/permissions/android/java/src/org/chromium/components/permissions/PermissionDialogCoordinator.java
index bd4876d..08329d0b 100644
--- a/components/permissions/android/java/src/org/chromium/components/permissions/PermissionDialogCoordinator.java
+++ b/components/permissions/android/java/src/org/chromium/components/permissions/PermissionDialogCoordinator.java
@@ -23,6 +23,7 @@
 import org.chromium.components.browser_ui.util.DimensionCompat;
 import org.chromium.components.content_settings.ContentSetting;
 import org.chromium.ui.LayoutInflaterUtils;
+import org.chromium.ui.UiUtils;
 import org.chromium.ui.modaldialog.ModalDialogManager;
 import org.chromium.ui.modaldialog.ModalDialogManager.ModalDialogManagerObserver;
 import org.chromium.ui.modaldialog.ModalDialogProperties;
@@ -76,12 +77,16 @@
                 recordOutOfScreenNegativeButton(lastButton);
                 return;
             }
+            final View rootView = dialogView;
+            UiUtils.disableLigaturesForSecurity(rootView);
+
             lastButton
                     .getViewTreeObserver()
                     .addOnGlobalLayoutListener(
                             new ViewTreeObserver.OnGlobalLayoutListener() {
                                 @Override
                                 public void onGlobalLayout() {
+                                    UiUtils.disableLigaturesForSecurity(rootView);
                                     if (!lastButton.isLaidOut()) {
                                         return;
                                     }
diff --git a/ui/android/java/src/org/chromium/ui/UiUtils.java b/ui/android/java/src/org/chromium/ui/UiUtils.java
index c27408c..17795258 100644
--- a/ui/android/java/src/org/chromium/ui/UiUtils.java
+++ b/ui/android/java/src/org/chromium/ui/UiUtils.java
@@ -77,10 +77,30 @@
     // this long after the prompt is displayed.
     public static long PROMPT_INPUT_PROTECTION_SHORT_DELAY_MS = 600;
 
+    // Font feature setting to disable ligature rendering.
+    private static final String NO_LIGATURES = "\"liga\" 0, \"clig\" 0";
+
     /** Guards this class from being instantiated. */
     private UiUtils() {}
 
     /**
+     * Recursively walks the view tree and disables ligatures on all TextViews
+     *
+     * @param view The root view{@link View}
+     */
+    public static void disableLigaturesForSecurity(View view) {
+        if (view instanceof TextView) {
+            ((TextView) view).setFontFeatureSettings(NO_LIGATURES);
+        }
+        if (view instanceof ViewGroup) {
+            ViewGroup group = (ViewGroup) view;
+            for (int i = 0; i < group.getChildCount(); i++) {
+                disableLigaturesForSecurity(group.getChildAt(i));
+            }
+        }
+    }
+
+    /**
      * Gets the set of locales supported by the current enabled Input Methods.
      *
      * @param context A {@link Context} instance.