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)
Tracking
()
NEW
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 totrue
Steps to reproduce
- Launch Firefox.
- Open a new tab and access the following link
- Open the print preview.
- 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.
Reporter | ||
Updated•4 years ago
|
Whiteboard: [print2020_v81]
Updated•4 years ago
|
Severity: -- → S3
Updated•4 years ago
|
Priority: -- → P2
Comment 1•4 years ago
|
||
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
Updated•4 years ago
|
Has STR: --- → yes
Comment 2•4 years ago
|
||
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]
Comment 3•4 years ago
|
||
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)
Comment 4•4 years ago
|
||
[Moving front-end printing v82 bugs to v83.]
Whiteboard: [print2020_v82] → [print2020_v83]
Updated•4 years ago
|
Whiteboard: [print2020_v83] → [print2020_v84]
Updated•4 years ago
|
Whiteboard: [print2020_v84] → [print2020_v85]
Updated•4 years ago
|
Whiteboard: [print2020_v86] → [print2020_v88]
Updated•4 years ago
|
Whiteboard: [print2020_v88] → [print2020]
Updated•2 years ago
|
Priority: P2 → P3
You need to log in
before you can comment on or make changes to this bug.
Description
•