Open Bug 1658285 Opened 4 years ago Updated 2 years ago

Print preview gets closed if the tab is moved to a new window

Categories

(Toolkit :: Printing, defect, P3)

Firefox 81
defect

Tracking

()

Tracking Status
firefox81 --- affected

People

(Reporter: emilghitta, Unassigned)

References

(Blocks 1 open bug)

Details

(Whiteboard: [print2020] )

Affected versions

  • 81.0a1 (BuildId:20200804215246)

Affected platforms

  • Windows 10 64bit
  • macOS 10.14
  • Ubuntu 18.04 64bit

Preconditions

  • Ensure that the print.tab_modal.enabled pref is set to true

Steps to reproduce

  1. Launch Firefox.
  2. Open a new tab and access the following link
  3. Open the print preview.
  4. Move the current tab (the tab from step 2) inside a new window or drag the tab in order to instantiate a new window containing the tab from step 2.

Expected result

  • Print preview remains open for the tab mentioned in step 2.

Actual result

  • Dragging the tab to a different window is closing the print preview.

Regression Range

  • I don’t think that this is a regression

Notes

  • The following error is being outputted inside the browser console each time the tab is moved in a new window while print preview is open: TypeError: can't access property "close", this._frame.contentWindow is nullSubDialog.jsm:296:7
  • [Suggested Severity] I think that S3 fits for this one.
Whiteboard: [print2020_v81]
Severity: -- → S3
Blocks: 1658287
Priority: -- → P2

This seems fine to me. I don't think it's a real issue that it closes. The error however should be looked into.

Since that error is in SubDialog it might go away on its own when we switch to TabDialogBox.

Severity: S3 → S4
Has STR: --- → yes

I'm fine with this happening for the print dialog, the error is fixed since the move to TabDialogBox.

@pbz is this something the other TabDialogBox use cases might need to handle?

Flags: needinfo?(pbz)
Whiteboard: [print2020_v81] → [print2020_v82]

Detaching the tab calls TabDialogBox#abortAllDialogs here: https://searchfox.org/mozilla-central/rev/969fc7fa6c3c7fc489f53b7b7f8c902028b5169f/browser/base/content/browser.js#8957
I think this is fine for now.

Flags: needinfo?(pbz)

[Moving front-end printing v82 bugs to v83.]

Whiteboard: [print2020_v82] → [print2020_v83]
Whiteboard: [print2020_v83] → [print2020_v84]
Whiteboard: [print2020_v84] → [print2020_v85]

(Moving bugs to 86, part 2.)

Whiteboard: [print2020_v85] → [print2020_v86]
Whiteboard: [print2020_v86] → [print2020_v88]
Whiteboard: [print2020_v88] → [print2020]
Priority: P2 → P3
You need to log in before you can comment on or make changes to this bug.