1830282 - Replace Focus UrlUtils.java methods with URLStringUtils from AC

Status: VERIFIED FIXED

(Focus :: General, task, P2)

Description

[Mihai Adrian Carare [:mcarare]]

We should move to using the AC Kotlin implementation instead of the Java one.

This includes completely removing the file and replace methods with the AC ones, as well as migrating the methods that do not have an equivalent in AC to the AC URLStringUtils ( including tests).

Comment 1

[BMO Github Automation]

Attachment: [mozilla-mobile/firefox-android] Bug 1830282 - Replace Focus UrlUtils.java methods with URLStringUtils from AC. (#1807)

Comment 2

[BMO Github Automation]

Authored by https://github.com/mcarare
https://github.com/mozilla-mobile/firefox-android/commit/bf5869050d6c3621986cd10ff5b494a99d93c36e
[main] Bug 1830282 - Replace Focus UrlUtils.java methods with URLStringUtils from AC.

Comment 3

[Mihai Adrian Carare [:mcarare]]

For QA: Test that the following flows work as expected:

• Adding a new search engine.
• Opening a new page by typing the URL.
• Adding a website to device home screen shortcuts ( check proper default page title generated).
• Adding a website to app home screen shortcuts (AKA topsites - check proper representative image generated)

Thank you!

Comment 4

[Mira Lobontiu (Android QA)]

Verified as implemented on the latest Focus Nightly 115.0a1 build from 5/10, with the following devices:

• Google Pixel 6 (Android 13),
• HTC 10 (Android 8),
• Lenovo tab M10 (Android 10).