Closed Bug 1709544 Opened 3 years ago Closed 3 years ago

variable tooltip is not displayed when hovering variable while paused in script using private fields

Categories

(DevTools :: Debugger, defect, P1)

defect

Tracking

(firefox90 verified)

VERIFIED FIXED
90 Branch
Tracking Status
firefox90 --- verified

People

(Reporter: nchevobbe, Assigned: nchevobbe)

References

Details

Attachments

(1 file)

Steps to reproduce

  1. Go to https://firefox-devtools-private-fields.glitch.me/
  2. Open the debugger
  3. On the page, click the Create instance and pause button, it should pauses in script.js
  4. In getSecreatAndSalt, hover the creationDate variable on line 17

Expected results

The variable tooltip shows the value of creationDate

Actual results

The variable tooltip does not show up


An error is displayed in the terminal:

new SyntaxError("This experimental syntax requires enabling one of the following parser plugin(s): 'classPrivateProperties, classPrivateMethods' (4:9)", "resource://devtools/client/debugger/dist/parser-worker.js", 17120)

We have to update the babel configuration to add support for private fields.

Note that inline variable preview does not work either, and we should check if fixing this bug resolves the issue, and if not, file another specific bug

Assignee: nobody → nchevobbe
Status: NEW → ASSIGNED
Blocks: 1709567

This patch adds 2 babel plugin so we can parse such scripts.

Pushed by nchevobbe@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b40e026a5d61
[devtools] Fix debugger preview tooltip on scripts with private fields. r=bomsy.
Severity: -- → S3
Priority: -- → P1
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 90 Branch
Flags: qe-verify+

I've reproduced this bug on macOS 10.5, using the steps from comment 0, on an affected Nightly build (2021-05-05).

The bug is verified as fixed on Beta 90.0b4, across platforms: Win 10 x64, macOS 10.15 and Ubuntu 18.04 x64.

Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: