Closed Bug 1010348 Opened 7 years ago Closed 7 years ago

"learn more" link in SiteIdentifyPopup links to desktop kb article.

Categories

(Firefox for Android Graveyard :: General, defect)

ARM
Android
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 32

People

(Reporter: ckitching, Assigned: ckitching)

References

Details

Attachments

(1 file)

Here:
https://mxr.mozilla.org/mozilla-central/source/mobile/android/base/toolbar/SiteIdentityPopup.java#36

We find,

    // FIXME: Update this URL for mobile. See bug 885923.
    private static final String MIXED_CONTENT_SUPPORT_URL =
        "https://support.mozilla.org/kb/how-does-content-isnt-secure-affect-my-safety";


Bug 885923 is now fixed, and there's a shiny new article at:
https://support.mozilla.org/kb/how-does-insecure-content-affect-safety-android

Presumably the URL should be updated accordingly?
Since Margaret filed the original bug I guess you're an appropriate reviewer, although I suspect this one is going to be fairly trivial.
Attachment #8422547 - Flags: review?(margaret.leibovic)
See Also: → 885923
Comment on attachment 8422547 [details] [diff] [review]
Update the link to the article produced by the other bug.

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

Good catch! That must have fallen through the cracks :(
Attachment #8422547 - Flags: review?(margaret.leibovic) → review+
https://hg.mozilla.org/integration/fx-team/rev/d1269bdb70e2
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
(In reply to Chris Kitching [:ckitching] from comment #3)

Please don't resolve bugs that haven't landed on m-c yet.
Target Milestone: --- → Firefox 32
(In reply to Ryan VanderMeulen [:RyanVM UTC-4] from comment #5)
> (In reply to Chris Kitching [:ckitching] from comment #3)
> 
> Please don't resolve bugs that haven't landed on m-c yet.

Apologies. Haven't landed many of my own bugs, wasn't sure when to set the flag. Won't be repeating that error.
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.