commit ba5e430dea9bd9b5689c8617da7c0f7c1f4e1860
parent b91a9601875b24a2d3b57f8b839e192b1da8d2ea
Author: Thomas Wisniewski <twisniewski@mozilla.com>
Date: Wed, 8 Oct 2025 13:38:21 +0000
Bug 1982878 - remove our UA override for fanplus.co.jp; r=denschub,webcompat-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D267803
Diffstat:
2 files changed, 4 insertions(+), 40 deletions(-)
diff --git a/browser/extensions/webcompat/data/interventions.json b/browser/extensions/webcompat/data/interventions.json
@@ -5154,21 +5154,6 @@
}
]
},
- "1982878": {
- "label": "fanplus.co.jp",
- "bugs": {
- "1982878": {
- "issue": "broken-redirect",
- "matches": ["*://fanplus.co.jp/*"]
- }
- },
- "interventions": [
- {
- "platforms": ["android"],
- "ua_string": ["Chrome_with_FxQuantum"]
- }
- ]
- },
"1934908": {
"label": "www.cbsnews.com",
"bugs": {
diff --git a/testing/webcompat/interventions/tests/test_1982878_fanplus_co_jp.py b/testing/webcompat/interventions/tests/test_1982878_fanplus_co_jp.py
@@ -1,35 +1,14 @@
import pytest
-from webdriver.bidi.error import UnknownErrorException
-# This site fails with a redirect loop with a Firefox UA
+# This site once failed with a redirect loop with a Firefox UA
URL = "https://fanplus.co.jp/feature/artistarena"
-REDIR_FAILURE_CSS = "[data-l10n-id=redirectLoop-title]"
-REDIR_FAILURE_TEXT = "ERROR_REDIRECT_LOOP"
-SUCCESS_CSS = "#globaltop_header"
+SUCCESS_CSS = "#aacontents"
@pytest.mark.only_platforms("android")
@pytest.mark.asyncio
-@pytest.mark.with_interventions
-async def test_enabled(client):
+@pytest.mark.without_interventions
+async def test_regression(client):
await client.navigate(URL, wait="none")
assert client.await_css(SUCCESS_CSS)
-
-
-@pytest.mark.only_platforms("android")
-@pytest.mark.asyncio
-@pytest.mark.without_interventions
-async def test_disabled(client):
- # We will either get a static error page with the text, or
- # an UnknownErrorException from WebDriver.
- try:
- await client.navigate(URL, wait="none", no_skip=True)
- desktop, mobile = client.await_first_element_of(
- [client.css(REDIR_FAILURE_CSS), client.text(REDIR_FAILURE_TEXT)],
- is_displayed=True,
- timeout=30,
- )
- assert desktop or mobile
- except UnknownErrorException:
- assert True