Closed
Bug 1206230
Opened 9 years ago
Closed 9 years ago
Add back android:key for Manage Logins
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(firefox42 fixed, firefox43 fixed)
RESOLVED
FIXED
Firefox 43
People
(Reporter: liuche, Assigned: liuche)
Details
Attachments
(1 file)
40 bytes,
text/x-review-board-request
|
mcomella
:
review+
Sylvestre
:
approval-mozilla-beta+
|
Details |
The "Manage Logins" settings item doesn't have a key, so it shows up in telemetry as an empty string which is not useful. This is my bad, I forgot that we were using android:keys for more than just removing/adding Preference items, so this got removed at my request in bug 1114821.
Assignee | ||
Updated•9 years ago
|
status-firefox42:
--- → affected
Assignee | ||
Comment 1•9 years ago
|
||
Bug 1206230 - Add back android:key for Manage Logins. r=mcomella
Attachment #8663129 -
Flags: review?(michael.l.comella)
Assignee: nobody → liuche
Comment on attachment 8663129 [details] MozReview Request: Bug 1206230 - Add back android:key for Manage Logins. r=mcomella https://reviewboard.mozilla.org/r/19725/#review17753 (In reply to Chenxia Liu [:liuche] from comment #0) > This is my bad, I forgot that we were using android:keys for more than just > removing/adding Preference items, so this got removed at my request in bug > 1114821. I know nothing about this but that technique seems reasonable.
Attachment #8663129 -
Flags: review?(michael.l.comella) → review+
I'm guessing we should uplift this too?
Assignee | ||
Comment 5•9 years ago
|
||
Comment on attachment 8663129 [details] MozReview Request: Bug 1206230 - Add back android:key for Manage Logins. r=mcomella Approval Request Comment [Feature/regressing bug #]: Bug 1114821 [User impact if declined]: Telemetry for the Manage Logins setting will be blank [Describe test coverage new/current, TreeHerder]: local testing [Risks and why]: Very low, reverting one line that is in xml [String/UUID change made/needed]: none To be clear, this is a r? for 42, which we will probably miss in aurora :/
Attachment #8663129 -
Flags: approval-mozilla-aurora?
Comment 6•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/1b227fca027c
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 43
Comment 9•9 years ago
|
||
Sorry, disregard comment 7 & comment 8. That was for bug 1206203, and I typo'd the bug number in the commit message.
Comment 10•9 years ago
|
||
Comment on attachment 8663129 [details] MozReview Request: Bug 1206230 - Add back android:key for Manage Logins. r=mcomella Moving the flag to beta since this missed 42 aurora.
Attachment #8663129 -
Flags: approval-mozilla-aurora? → approval-mozilla-beta?
Comment 11•9 years ago
|
||
Comment on attachment 8663129 [details] MozReview Request: Bug 1206230 - Add back android:key for Manage Logins. r=mcomella Sure, telemetry is good!
Attachment #8663129 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Updated•3 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•