Site Identifier in Location Bar is broken

RESOLVED FIXED in Firefox 23

Status

()

Core
Security: PSM
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Alice0775 White, Assigned: briansmith)

Tracking

({regression})

23 Branch
mozilla23
x86_64
All
regression
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(firefox23 fixed)

Details

Attachments

(3 attachments)

(Reporter)

Description

5 years ago
Build Identifier:
http://hg.mozilla.org/mozilla-central/rev/1150403342b2
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:23.0) Gecko/20130422 Firefox/23.0 ID:20130422105838

Steps To Reproduce:
1. Open https://addons.mozilla.org/en-US/firefox/ (for example)

Actual Results:
Only Gray Pad Lock.

Expected Results:
Green Pad Lock and Organization Name

Regression window(m-i)
Good:
http://hg.mozilla.org/integration/mozilla-inbound/rev/c1483e6de03a
Mozilla/5.0 (X11; Linux i686; rv:23.0) Gecko/20130419 Firefox/23.0 ID:20130419153005
Bad:
http://hg.mozilla.org/integration/mozilla-inbound/rev/f9405d359a22
Mozilla/5.0 (X11; Linux i686; rv:23.0) Gecko/20130419 Firefox/23.0 ID:20130419153606
Pushlog:
http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=c1483e6de03a&tochange=f9405d359a22

Suspected : Bug 787155
Facepalm.
Assignee: nobody → bsmith
(Reporter)

Comment 2

5 years ago
Created attachment 740682 [details]
screen shot
Blocks: 864300
potential solution: (different approach to bug 787155)
 https://tbpl.mozilla.org/?tree=Try&rev=33cbe8740d4b
Looks like we will succed with fixes and a test case (from bug 813418)

 https://tbpl.mozilla.org/?tree=Try&rev=025cf62c3b30
Created attachment 741500 [details] [diff] [review]
fix v1
Attachment #741500 - Flags: review?
Attachment #741500 - Flags: review? → review?(bsmith)
Created attachment 741501 [details] [diff] [review]
ev-test-patch(from 813418)

keeping r+ from review in bug 813418.
Attachment #741501 - Flags: review+
Final push, all tests all os 

https://tbpl.mozilla.org/?tree=Try&rev=aae8ea461818
Comment on attachment 741500 [details] [diff] [review]
fix v1

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

Please verify that the b2g build actually picks up the option.

Thanks for taking this. Set the bug to assigned: cviecco.

::: configure.in
@@ +6446,5 @@
>  fi
>  AC_SUBST(MOZ_DISABLE_CRYPTOLEGACY)
>  
>  dnl ========================================================
> +dnl = Disable LIB_PKIX

s/LIB_PKIX/libpkix/
Attachment #741500 - Flags: review?(bsmith) → review+
https://tbpl.mozilla.org/?tree=Try&rev=4d49ed6b3663
https://tbpl.mozilla.org/?tree=Mozilla-Inbound&rev=45e73d8d89c9
https://hg.mozilla.org/mozilla-central/rev/0373ea7e7f54
https://hg.mozilla.org/mozilla-central/rev/45e73d8d89c9
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
status-firefox23: --- → fixed
tracking-firefox23: ? → ---
You need to log in before you can comment on or make changes to this bug.