1567826 - Don't mark localhost as insecure

Status: RESOLVED FIXED

(Firefox :: Site Identity, enhancement, P1)

Description

[Johann Hofmann [:johannh]]

localhost is SecureContext already, so we shouldn't give it the broken lock.

Comment 1

[Johann Hofmann [:johannh]]

Attachment: Bug 1567826 - Don't mark any secureContext pages as insecure. r=nhnt11!,keeler!,ehsan!

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ded87cc3f3ee
Don't mark any secureContext pages as insecure. r=nhnt11,keeler,Ehsan

Comment 3

[Andrei Ciure[:aciure]]

Backed out changeset ded87cc3f3ee (bug 1567826) for causing browser_check_identity_state.js to perma fail

push that caused the backout: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception%2Cretry%2Cusercancel%2Crunning%2Cpending%2Crunnable&revision=ded87cc3f3eed779fd0b525921f76dc5f6818216

failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&fromchange=3133ef9a77a3f178d3a8b77dea0abb0b2e3d349f&searchStr=os%2Cx%2C10.14%2Cdebug%2Cmochitests%2Ctest-macosx1014-64%2Fdebug-mochitest-browser-chrome-e10s-1%2Cm%28bc1%29

backout: https://hg.mozilla.org/integration/autoland/rev/92c752e9d0581404db0350db525c5950f5e69542

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/0fceabdb8b9d
Don't mark any secureContext pages as insecure. r=nhnt11,keeler,Ehsan

Comment 5

[Daniel Varga [:dvarga]]

https://hg.mozilla.org/mozilla-central/rev/0fceabdb8b9d