Closed Bug 1551114 Opened 5 years ago Closed 5 years ago

Layout inspector crashes tab for flexbox in print preview

Categories

(DevTools :: Inspector: Layout, defect, P2)

64 Branch
defect

Tracking

(firefox-esr60 unaffected, firefox66 wontfix, firefox67 wontfix, firefox67.0.1 wontfix, firefox68 wontfix, firefox69 fix-optional, firefox70 fix-optional)

RESOLVED DUPLICATE of bug 1563163
Tracking Status
firefox-esr60 --- unaffected
firefox66 --- wontfix
firefox67 --- wontfix
firefox67.0.1 --- wontfix
firefox68 --- wontfix
firefox69 --- fix-optional
firefox70 --- fix-optional

People

(Reporter: Oriol, Assigned: bradwerth)

References

(Regression)

Details

(Keywords: crash, regression, testcase)

Crash Data

Attachments

(1 file)

Attached file testcase-crash.htm
  1. Load a document with a flex container
  2. Open print preview
  3. Right-click and inspect the flex container
  4. Choose the Layout view (this happens by default since bug 1499483)

Result: the tab crashes
https://crash-stats.mozilla.org/report/index/3292582e-a8b9-47de-a8fe-b00640190512

Regression-window: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=9d702b7e5d80bc365790fe49001882b8dca80c9c&tochange=920027e02ee128e0bbe86e824395f006045c1cbc

Flags: needinfo?(gl)
Assignee: nobody → gl
Status: NEW → ASSIGNED
Flags: needinfo?(gl)
Priority: -- → P1

I have not been able to reproduce this on mac. Looking at the crash-stats, it seems this might be reproducible on linux.

Needinfoing bradwerth to see if he sees anything in particular in the crash-stats report since it's pointing to Flex.cpp https://crash-stats.mozilla.org/report/index/3292582e-a8b9-47de-a8fe-b00640190512#tab-details

Assignee: gl → nobody
Status: ASSIGNED → UNCONFIRMED
Ever confirmed: false
Flags: needinfo?(bwerth)
Priority: P1 → --

I don't have a mac, I can reproduce on linux and windows.
But on windows the crash is not immediate, I have two wait like 2-3 seconds after inspecting the element.

Assignee: nobody → bwerth
Flags: needinfo?(bwerth)
Priority: -- → P2
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true

Too late for a fix in 68 but we could still take a patch for 69/70.

This is P2 and assigned, so I'm removing it from weekly regression triage by marking it fix-optional.
Happy to take a patch for 70 or in future.

Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → DUPLICATE
Has Regression Range: --- → yes
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: