tor-browser

The Tor Browser
git clone https://git.dasho.dev/tor-browser.git
Log | Files | Refs | README | LICENSE

commit aeff28708bcc06d6715ea1c56d6eb76578979eb5
parent ab4348da048e7952f5d0861c2cd75b7fca817c93
Author: Thomas Wisniewski <twisniewski@mozilla.com>
Date:   Fri, 19 Dec 2025 23:16:53 +0000

Bug 1458885 - remove our webcompat intervention ipstack.com; r=webcompat-reviewers,ksenia

Differential Revision: https://phabricator.services.mozilla.com/D276772

Diffstat:
Mbrowser/extensions/webcompat/data/interventions.json | 17-----------------
Dbrowser/extensions/webcompat/injections/css/bug1458885-ipstack.com-fix-line-break-on-button.css | 15---------------
Dtesting/webcompat/interventions/tests/test_1458885_ipstack_com.py | 29-----------------------------
3 files changed, 0 insertions(+), 61 deletions(-)

diff --git a/browser/extensions/webcompat/data/interventions.json b/browser/extensions/webcompat/data/interventions.json @@ -128,23 +128,6 @@ } ] }, - "1458885": { - "label": "ipstack.com", - "bugs": { - "1458885": { - "issue": "broken-layout", - "matches": ["*://ipstack.com/*"] - } - }, - "interventions": [ - { - "platforms": ["desktop"], - "content_scripts": { - "css": ["bug1458885-ipstack.com-fix-line-break-on-button.css"] - } - } - ] - }, "1457335": { "label": "histography.io", "bugs": { diff --git a/browser/extensions/webcompat/injections/css/bug1458885-ipstack.com-fix-line-break-on-button.css b/browser/extensions/webcompat/injections/css/bug1458885-ipstack.com-fix-line-break-on-button.css @@ -1,15 +0,0 @@ -/* This Source Code Form is subject to the terms of the Mozilla Public - * License, v. 2.0. If a copy of the MPL was not distributed with this - * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ - -/** - * ipstack.com - "Find IP Location" button wraps across lines. - * - * Bug #1458885 - https://bugzilla.mozilla.org/show_bug.cgi?id=1458885 - * - * The site is relying very precise font measurements which only happen in Chromium. - * We can fix the resulting issue by unsetting the font-weight, which looks better. - */ -.ipchecker .i_header label { - font-weight: unset; -} diff --git a/testing/webcompat/interventions/tests/test_1458885_ipstack_com.py b/testing/webcompat/interventions/tests/test_1458885_ipstack_com.py @@ -1,29 +0,0 @@ -import pytest - -URL = "https://ipstack.com/" - -BUTTON_CSS = "label[for=ipcheck_submit]" - - -async def is_button_text_on_one_line(client): - await client.navigate(URL) - return client.execute_script( - """ - return arguments[0].getBoxQuads().length == 1; - """, - client.await_css(BUTTON_CSS, is_displayed=True), - ) - - -@pytest.mark.skip_platforms("android") -@pytest.mark.asyncio -@pytest.mark.with_interventions -async def test_enabled(client): - assert await is_button_text_on_one_line(client) - - -@pytest.mark.skip_platforms("android") -@pytest.mark.asyncio -@pytest.mark.without_interventions -async def test_disabled(client): - assert not await is_button_text_on_one_line(client)