1229107 - Undefined entity safeb.blocked.forbiddenPage.title

Status: RESOLVED INVALID

(Firefox for Android Graveyard :: General, defect)

Description

[Sebastian Kaspari (:sebastian; :pocmo)]

STR:
* Go to http://www.itisatrap.org/firefox/its-an-attack.html

Expected:
* "Reported Attack" page should show up

Actual:
* Error about undefined entity (&safeb.blocked.forbiddenPage.title;) is shown

Comment 1

[Sebastian Kaspari (:sebastian; :pocmo)]

It seems to be in listed in phishing.dtd though:
https://dxr.mozilla.org/mozilla-central/source/mobile/android/locales/en-US/chrome/phishing.dtd

Comment 2

[James Willcox (:snorp) ([e-mail]) (he/him)]

Margaret, can you find an assignee?

Comment 3

[Gian-Carlo Pascutto [:gcp]]

Works for me. Sebastian, is this a localized build?

Comment 4

[François Marier [:francois]]

Should be fixed in bug 1227800 but could happen again since we don't have any Safe Browsing UI tests on Fennec :(

Comment 5

[Gian-Carlo Pascutto [:gcp]]

(In reply to François Marier [:francois] from comment #4)
> Should be fixed in bug 1227800 

That landed the 26th, but this was reported the 30th.

> but could happen again since we don't have any Safe Browsing UI tests on Fennec :(

Why don the normal mochitests not cover the Android error pages?

Comment 6

[Sebastian Kaspari (:sebastian; :pocmo)]

Yikes, Nightly was lagging behind on this device. With the latest version the issue is gone.

Comment 7

[:Margaret Leibovic]

(In reply to Gian-Carlo Pascutto [:gcp] from comment #5)
> (In reply to François Marier [:francois] from comment #4)
> > Should be fixed in bug 1227800 
> 
> That landed the 26th, but this was reported the 30th.
> 
> > but could happen again since we don't have any Safe Browsing UI tests on Fennec :(
> 
> Why don the normal mochitests not cover the Android error pages?

These tests likely don't run on Android. If we have a forked version of the page, we probably need the tests to be slightly different, so it might not be as easy as just enabling the tests.

But we should look into getting some of these tests to run on Android. Here are docs about mochitest on Android:

https://wiki.mozilla.org/Mobile/Fennec/Android/Testing#mochitest_.28plain_and_chrome.29

Comment 8

[Chris Peterson [:cpeterson]]

Removing regressionwindow-wanted keyword because this bug has been resolved.

Comment 9

[Chris Peterson [:cpeterson]]

Removing regressionwindow-wanted keyword because this bug has been resolved.

Comment 10

[Chris Peterson [:cpeterson]]

Removing regressionwindow-wanted keyword because this bug has been resolved.