1193742 - [Control Center] Grey and yellow exclamation marks are missing from Control Center for mixed content pages

Status: VERIFIED FIXED

(Firefox :: General, defect, P1)

Description

[Catalin Varga [QA][:VarCat]]

Attachment: exclamation_mark.jpg

Environment:

FF 42
Build Id: 20150811030206
OS: Win 7 x64, Ubuntu 12.04 x86

1. Go to https://people.mozilla.org/~tvyas/mixedcontent.html
2. Open Control Center
3. Open security subview

Issue:
The gray exclamation mark icon is missing from both Control Center and the security subview.

Note:
This issue is reproducible only on Ubuntu and Linux it works fine on Mac OS X.
I've attached a printscreen with the issue.

Comment 1

[Tim Taubert [:ttaubert] (inactive)]

Gah, my bad. Thanks again Catalin!

Comment 2

[Tim Taubert [:ttaubert] (inactive)]

Attachment: 0001-Bug-1193742-Add-missing-yield-signs-for-MCB-to-Linux.patch

Guess I simply forgot about that.

Comment 3

[Catalin Varga [QA][:VarCat]]

In the description of the bug I said that the bug is reproducible only on Linux and Ubuntu I meant to say Windows and Ubuntu, sorry about that.

Comment 4

[Tim Taubert [:ttaubert] (inactive)]

I actually read it that way, my brain adjusted :)

Comment 5

[:Paolo Amadini]

Comment on attachment 8646927 [details] [diff] [review]
0001-Bug-1193742-Add-missing-yield-signs-for-MCB-to-Linux.patch

Sorry for missing those in the previous review!

Comment 6

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/05b681d6a209

Comment 7

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/05b681d6a209

Comment 8

[Paul Silaghi, QA [:pauly]]

Verified fixed FF 43.0a1 (2015-08-24) Win 7, Ubuntu 14.04, OS X 10.10.

Comment 9

[Tim Taubert [:ttaubert] (inactive)]

Comment on attachment 8646927 [details] [diff] [review]
0001-Bug-1193742-Add-missing-yield-signs-for-MCB-to-Linux.patch

Approval Request Comment
[Feature/regressing bug #]:
[User impact if declined]: Missing warning icons (yield signs) with mixed content.
[Describe test coverage new/current, TreeHerder]:
[Risks and why]: Virtually no risk, simply adds two images to Linux and Windows themes.
[String/UUID change made/needed]: None.

Comment 10

[Sylvestre Ledru [:Sylvestre]]

Comment on attachment 8646927 [details] [diff] [review]
0001-Bug-1193742-Add-missing-yield-signs-for-MCB-to-Linux.patch

Fix an issue in a new feature, taking it.

Comment 11

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/ee62cbdfe5b3

Comment 12

[Paul Silaghi, QA [:pauly]]

Verified fixed FF 42.0a2 (2015-09-02) Win 7

Comment 13

[Paul Silaghi, QA [:pauly]]

Are 
https://wiki.mozilla.org/File:Active-blocked_dv.png
https://wiki.mozilla.org/File:Active-blocked_dv_extended.png
still expected?
Because I'm seeing something else in 43.0a1 (2015-09-14) Win 7 on https://www.maxcdn.com/one/tutorial/solving-mixed-content/

Comment 14

[Tim Taubert [:ttaubert] (inactive)]

(In reply to Paul Silaghi, QA [:pauly] from comment #13)
> Because I'm seeing something else in 43.0a1 (2015-09-14) Win 7 on
> https://www.maxcdn.com/one/tutorial/solving-mixed-content/

They have probably fixed their page in the meantime. Here's a more reliable test page:

https://people.mozilla.org/~tvyas/mixedcontent.html

It's EV but the control center doesn't treat DV differently, except showing a few more info.