Closed Bug 1634402 Opened 4 years ago Closed 2 years ago

Date information are shown in about:logins page for RTL builds only after the page is refreshed

Categories

(Firefox :: about:logins, defect, P3)

defect

Tracking

()

RESOLVED WORKSFORME
Tracking Status
firefox-esr68 --- unaffected
firefox-esr78 --- fix-optional
firefox75 --- unaffected
firefox76 --- wontfix
firefox77 --- wontfix
firefox78 --- wontfix
firefox79 --- fix-optional

People

(Reporter: asoncutean, Unassigned)

Details

(Keywords: regression)

Attachments

(1 file)

Attached image screenshot.png

Affected versions

  • 77.0a1
  • 76.0

Affected platforms

  • Windows 10
  • macOS 10.15
  • Ubuntu 18.04

Steps to reproduce

  1. Open a RTL Firefox build (eg. ar)
  2. Save credentials for any website (eg. https://www.facebook.com/)
  3. Open the about:logins page
  4. Observe the تاريخ الإنشاء (Created) / آخر تعديل (Last modified) / آخر استخدام (Last used) fields

Expected result

  • The date of the corresponding action is displayed for every section

Actual result

  • “???” is shown instead of dates

Regression range

Additional notes

  • The dates are shown after the about:loggins page is refreshed once
  • Reproducible on he locale (RTL build as well)
  • Not reproducible on en-US, de, zh-CN
Has Regression Range: --- → yes
Has STR: --- → yes
Component: about:logins → Localization
Product: Firefox → Core
Component: Localization → Internationalization

m_kato, could you prioritize this one? Also do you think this should be tracked for a dot release?

Flags: needinfo?(m_kato)

Possibly regressed by bug 1622042?

Severity: normal → S2
Flags: needinfo?(m_kato)
Priority: -- → P2

Zibi, known issue? Could you handle this?

Flags: needinfo?(gandalf)
Severity: S2 → S1

I looked into it, and found two-layer issue:

  1. https://searchfox.org/mozilla-central/source/browser/components/aboutlogins/content/components/login-item.js#198-206

This code fallbacks are wrong. They use "" as a fallback, but since this is used as epoch it should be a number.

  1. With (1) fixed, the values of this._timeCreated|this._timeUsed|this._timeChanged is falsy in ar but not in pl or en-US

I'm not sure what causes the difference, it may be a race condition maybe? But it seems like some timing bug since I don't believe this code should be called with falsy values.

It also explains why after reload it looks good (since the values are not falsy anymore).

I'm tentatively reassigning it to about:logins component because while the bug is visible depending on locale, it seems like the solution requires understanding why those variables are falsy.

I also couldn't reproduce it in pseudolocalization, so my first hypothesis is that since ar is incomplete, we load two locales, which takes a bit more time, and somehow in that scenario those variables are falsy.

If we end up finding out that some Intl code is causing them to be falsy, it may be moved back to Core::Internationalization.

NI on Jared, since he wrong those lines.

Component: Internationalization → about:logins
Flags: needinfo?(gandalf) → needinfo?(jaws)
Product: Core → Firefox

What's the status for this bug wrt our upcoming releases? Thanks

S1 or S2 bugs need an assignee - could you find someone for this bug?

Flags: needinfo?(MattN+bmo)

This is on Jared. See comment 4.

Severity: S1 → S2
Flags: needinfo?(MattN+bmo)
Summary: Date information are shown in about:logins page for RTL builds only after the page is refreshed → Date information are shown in about:logins page for RTL builds only after the page is refreshed

I question putting this at S1 or S2. The metadata is not critical functionality and a workaround does exist.

Severity: S2 → S3
Flags: needinfo?(jaws)

Moving to P3 as this is not happening in the next release cycle.

Priority: P2 → P3

I can no longer reproduce this issue with the latest RTL - ar Nightly (103.0a1->2022-06-21) on Windows 10 and MacOS 11.0.

Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: