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)
Mozilla QA Graveyard
Mozmill Tests
Tracking
(firefox26 fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox26 | --- | fixed |
People
(Reporter: cosmin-malutan, Assigned: AndreeaMatei)
References
()
Details
(Whiteboard: [mozmill-test-failure])
Attachments
(2 files)
1.74 KB,
patch
|
andrei
:
review+
andrei
:
checkin+
|
Details | Diff | Splinter Review |
1.89 KB,
patch
|
andrei
:
review+
andrei
:
checkin+
|
Details | Diff | Splinter Review |
This happend on Linux Ubuntu 13.04 with Nightly en-US http://mozmill-daily.blargon7.com/#/functional/report/26d5854914a68aa13b333f2815977abe
Reporter | ||
Updated•11 years ago
|
status-firefox26:
--- → affected
Reporter | ||
Comment 1•11 years ago
|
||
It reproduces locally so it should be P1 .
Severity: normal → critical
Priority: -- → P1
Assignee | ||
Comment 2•11 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•11 years ago
|
||
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•11 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•11 years ago
|
Whiteboard: [mozmill-test-failure] → [mozmill-test-failure][mozmill-test-skipped]
Comment 5•11 years ago
|
||
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)
Comment 6•11 years ago
|
||
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•11 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•11 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•11 years ago
|
||
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•11 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•11 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•11 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [mozmill-test-failure][mozmill-test-skipped] → [mozmill-test-failure]
Updated•5 years ago
|
Product: Mozilla QA → Mozilla QA Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•