Closed Bug 1658356 Opened 5 years ago Closed 5 years ago

[compatibility tooltip] Wrong key usage while accessing compatibility message id

Categories

(DevTools :: Inspector: Rules, defect, P3)

Firefox 81
defect

Tracking

(firefox81 fixed)

RESOLVED FIXED
81 Branch
Tracking Status
firefox81 --- fixed

People

(Reporter: lelouch.cpp, Assigned: lelouch.cpp)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:81.0) Gecko/20100101 Firefox/81.0

Steps to reproduce:

Line resulting in error: https://searchfox.org/mozilla-central/source/devtools/client/inspector/rules/models/text-property.js#347

  1. Toggle devtools.inspector.ruleview.inline-compatibility-warning.enabled pref to true
  2. Go to https://developer.mozilla.org/en-US/
  3. Inspect the Learn web development button
  4. Hover over the tooltip icon

Actual results:

The tooltip doesn't show logging an error in the browser console

Expected results:

The tooltip should appear with experimental warning but supported on all platform

The key of the object is incorrectly used as css-compatibility-experimental-supported-message
instead of experimental-supported. This leads to the l10n translation
for undefined msgId leading to failure of appearance of tooltip.
This patch fixes the problem and adds a test to prevent regression.

Assignee: nobody → lelouch.cpp
Severity: -- → S3
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Priority: -- → P3
Pushed by mtigley@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/ebe12173ab02 Fix the incorrect object key for the Tooltip message ID r=daisuke,mtigley
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → 81 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: