Test failure "The page has mixed content" in /testSecurity/testMixedContentPage.js

RESOLVED FIXED

Status

Mozilla QA
Mozmill Tests
P1
normal
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: cosmin, Assigned: AndreeaMatei)

Tracking

unspecified

Firefox Tracking Flags

(firefox26 fixed)

Details

(Whiteboard: [mozmill-test-failure], URL)

Attachments

(2 attachments)

(Reporter)

Updated

5 years ago
status-firefox26: --- → affected
(Reporter)

Comment 1

5 years ago
It reproduces locally so it should be P1 .
Severity: normal → critical
Priority: -- → P1
(Assignee)

Comment 2

5 years ago
Here it should be identity.mixed_display_loaded instead of identity.mixed_display_loaded_active_blocked:

http://mxr.mozilla.org/l10n-central/source/it/browser/chrome/browser/browser.properties#147

I tested with that and it fixes the issue. I wonder where was this changed.
Assignee: nobody → andreea.matei
Status: NEW → ASSIGNED
(Assignee)

Comment 3

5 years ago
Created attachment 802988 [details] [diff] [review]
skip patch

Adding skip patch until I can further see where this was changed and fully test on several locales. I see that some locales have different properties, for example [ar] doesn't have identity.mixed_display_loaded_active_blocked or identity.mixed_display_loaded:
http://mxr.mozilla.org/l10n-central/source/ar/browser/chrome/browser/browser.properties#214
Attachment #802988 - Flags: review?(andrei.eftimie)

Comment 4

5 years ago
Comment on attachment 802988 [details] [diff] [review]
skip patch

Review of attachment 802988 [details] [diff] [review]:
-----------------------------------------------------------------

Landed: http://hg.mozilla.org/qa/mozmill-tests/rev/ffe13db173e0 (default)
Attachment #802988 - Flags: review?(andrei.eftimie)
Attachment #802988 - Flags: review+
Attachment #802988 - Flags: checkin+
(Assignee)

Updated

5 years ago
status-firefox26: affected → disabled
Whiteboard: [mozmill-test-failure] → [mozmill-test-failure][mozmill-test-skipped]
Not sure why this has been set as critical. Moving back to normal.

(In reply to Andreea Matei [:AndreeaMatei] from comment #3)
> test on several locales. I see that some locales have different properties,
> for example [ar] doesn't have identity.mixed_display_loaded_active_blocked
> or identity.mixed_display_loaded:

Right. Not all locales are always up to date. If no property or DTD entity is available it will fall back to en-US during the repack. Flod might be able to explain that better or help us here.
Severity: critical → normal
Flags: needinfo?(francesco.lodolo)
You can see the status of locales on their dashboard. For example "ar"
https://l10n.mozilla.org/shipping/dashboard?locale=ar

And then click on "C" for the product you're interested in. For example this is the current status of Firefox on trunk (fx_central) for ar
https://l10n.mozilla.org/dashboard/compare?run=340853

At build time strings missing from the locale are automatically added from en-US, so localized builds are always string complete and fully functional (missing string = yellow screen of death or not working functions).
Flags: needinfo?(francesco.lodolo)
(Assignee)

Comment 7

5 years ago
The only related changeset from this range (yesterday build vs today's) I can find is this one:
http://hg.mozilla.org/mozilla-central/rev/e9a2e269aa9e

related to bug 912817.

Changing the property to identity.mixed_display_loaded is fixing today's failures (most popular locales), but not sure about other locales at the moment, as mentioned [ar] which doesn't have that property.
(Assignee)

Comment 8

5 years ago
Thanks for the links Francesco, good to know!
So in [ar] we fail at the favicon check, if I skip that part and replace the correct property, we're green. Investigating further the favicon issue and check some more locales to fix as many as possible.
(Assignee)

Comment 9

5 years ago
Created attachment 804435 [details] [diff] [review]
patch v1

Updated the property, checked on all platforms with several locales and it works fine.

Reports:
- http://mozmill-crowd.blargon7.com/#/remote/report/dc3d4d2f66af31b9e75e291fa048d893
- http://mozmill-crowd.blargon7.com/#/remote/report/dc3d4d2f66af31b9e75e291fa0486249
- http://mozmill-crowd.blargon7.com/#/remote/report/dc3d4d2f66af31b9e75e291fa048546a

I did noticed zh-tw on Ubuntu is failing at the favicon check, where it has the globe image, not the triangle. I'll file another bug to further investigate that (but I can reproduce manually). Might be related to bug 914410.
Attachment #804435 - Flags: review?(andrei.eftimie)
(Assignee)

Comment 10

5 years ago
I didn't had the last zh-TW apparently, but I do see the issue from bug 914410 if I click the favicon :)
Test works fine there as well:
http://mozmill-crowd.blargon7.com/#/remote/report/dc3d4d2f66af31b9e75e291fa04c2edc

Comment 11

5 years ago
Comment on attachment 804435 [details] [diff] [review]
patch v1

Review of attachment 804435 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good.

Landed: http://hg.mozilla.org/qa/mozmill-tests/rev/f4e461f75f13 (default)
Attachment #804435 - Flags: review?(andrei.eftimie)
Attachment #804435 - Flags: review+
Attachment #804435 - Flags: checkin+

Updated

5 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
status-firefox26: disabled → fixed
Resolution: --- → FIXED
Whiteboard: [mozmill-test-failure][mozmill-test-skipped] → [mozmill-test-failure]
You need to log in before you can comment on or make changes to this bug.