Closed Bug 1717890 Opened 4 years ago Closed 3 years ago

Slack hangs indefinitely when user interacts with some links

Categories

(Web Compatibility :: Site Reports, defect, P3)

Tracking

(firefox92 affected, firefox99 affected)

RESOLVED WORKSFORME
Tracking Status
firefox92 --- affected
firefox99 --- affected

People

(Reporter: aryx, Assigned: denschub)

References

(Depends on 1 open bug)

Details

(Keywords: webcompat:needs-diagnosis)

Firefox 91.0a1 20210623095324 on Windows 8.1, jcristau also noticed similar behavior (should be on Linux)

If I go into MoCo's #berlin slack and click with the right mouse button on the email address posted today, the browser hangs until a slow script warning is shown. It still seems to be necessary to reload the slack tab after one stopped the script.

Profiler recording: https://share.firefox.dev/3gPVPFR

Emilio, the same kind of issue (as bug 1640441) seems to be happening again here, judging by the profile - the jank period in the profile is entirely taken up by setTimeout combined with a mutation handler and a call into measureContents in the slack JS code which I expect flushes layout... I don't suppose you can take a look?

Flags: needinfo?(emilio)

This is... not something easy to address on our side. This is just bug 1640441 again.

We do layout in app units, which are 1/60th of a CSS pixel, and can't be represented exactly by a float. Slack is relying on the values we return to be roundtripping / comparing in a way that they just can't right now.

I filed bug 1719314 to consider changing our internal representation of app-units since this bug keeps coming back and I found no open bug to track such thing, but that's a sad-ish change because 1/60 has some other nice properties.

I contacted slack on the #ext-slack-mozilla-performance channel anyhow, fixing this on their side should be trivial...

Flags: needinfo?(emilio)

Needs Triage.

Flags: needinfo?(raul.bucata)
Flags: needinfo?(oana.arbuzov)

We appreciate your report. I was able to reproduce the issue. I have noticed this also to be reproducible on the "Get Started" page, when selecting a workspace or when accepting an invite for a workspace. It seems that the page returns an error regarding the mail used for signing in.

Tested with:

Browser / Version: Firefox Nightly 92.0a1 (2021-07-20) (64-bit)
Operating System: Windows 10 PRO x64

Notes:

  1. Reproducible regardless of the status of ETP
  2. Reproducible on the latest build of Firefox Nightly
  3. Works as expected using Chrome
Flags: needinfo?(raul.bucata)
Flags: needinfo?(oana.arbuzov)
Priority: -- → P3
Assignee: nobody → dschubert
Status: NEW → ASSIGNED

I was not able to reproduce the issue on my side. Right clicking or hovering any links on Slack no longer hangs.
Also https://bugzilla.mozilla.org/show_bug.cgi?id=1640441 is fixed.

Tested with:
Browser / Version: Firefox Nightly 103.0a1 (2022-06-23)
Operating System: Windows 10 Pro

Sebastian can you still reproduce it?

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