Avoid unnecessary event broadcasting for size mode change

NEW
Unassigned

Status

()

defect
3 years ago
3 years ago

People

(Reporter: wcpan, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

In bug 1104916, size mode change will be broadcasting to all remote children, no matter whether the size mode is really changed or not. (to prevent regression in bug 1259641)

It would be better to check this in nsWindow::OnWindowStateEvent.
However, in some cases (e.g.: toggle fullscreen by pressing F11), nsWindow::mSizeMode will be changed in other places first (e.g.: nsWindow::MakeFullScreen).
Maybe we need to unify the state change into same function.
(In reply to Wei-Cheng Pan [:wcpan] [:wcp] [:legnaleurc] from comment #0)
> In bug 1104916, size mode change will be broadcasting to all remote
> children, no matter whether the size mode is really changed or not. (to
> prevent regression in bug 1259641)

[edit]
In bug 1104916, size mode change will be broadcasting to all remote
children, no matter whether the size mode is really changed or not.
(which caused regression in bug 1259641)
You need to log in before you can comment on or make changes to this bug.