Closed Bug 915139 Opened 11 years ago Closed 11 years ago

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

Categories

(Mozilla QA Graveyard :: Mozmill Tests, defect, P1)

defect

Tracking

(firefox26 fixed)

RESOLVED FIXED
Tracking Status
firefox26 --- fixed

People

(Reporter: cosmin-malutan, Assigned: AndreeaMatei)

References

()

Details

(Whiteboard: [mozmill-test-failure])

Attachments

(2 files)

It reproduces locally so it should be P1 .
Severity: normal → critical
Priority: -- → P1
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
Attached patch skip patchSplinter Review
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 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+
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)
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.
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.
Attached patch patch v1Splinter Review
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)
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 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+
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [mozmill-test-failure][mozmill-test-skipped] → [mozmill-test-failure]
Product: Mozilla QA → Mozilla QA Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: