Collapse all the resize events before fullscreen to the last one

NEW
Unassigned

Status

()

Core
DOM: Events
3 years ago
3 years ago

People

(Reporter: xidorn, Unassigned)

Tracking

(Blocks: 1 bug, {leave-open})

Trunk
leave-open
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox41 affected)

Details

(Whiteboard: [parity-blink])

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
Currently, we dispatch a bunch of resize event before entering fullscreen. We should collapse all of them and only dispatch the last one.
(Reporter)

Updated

3 years ago
Summary: Reduce the resize event before fullscreen to one the last one → Collapse all the resize events before fullscreen to the last one
(Reporter)

Comment 1

3 years ago
The resize events are from the change of chrome controls. We probably should prevent those changes to trigger resize event on content.

Comment 2

3 years ago
(In reply to Xidorn Quan [:xidorn] (UTC+12) from comment #1)
> The resize events are from the change of chrome controls. We probably should
> prevent those changes to trigger resize event on content.

That sounds right.
(Reporter)

Updated

3 years ago
Depends on: 1168705
(Reporter)

Updated

3 years ago
Keywords: leave-open
(Reporter)

Comment 3

3 years ago
Created attachment 8612584 [details] [diff] [review]
patch 1 - avoid reflow for hiding window marigns on Windows
Attachment #8612584 - Flags: review?(jmathies)

Comment 4

3 years ago
Comment on attachment 8612584 [details] [diff] [review]
patch 1 - avoid reflow for hiding window marigns on Windows

Review of attachment 8612584 [details] [diff] [review]:
-----------------------------------------------------------------

I didn't see much of a difference visually, seems fine though if it cuts down on event traffic during the transition.
Attachment #8612584 - Flags: review?(jmathies) → review+
(Reporter)

Comment 5

3 years ago
(In reply to Jim Mathies [:jimm] from comment #4)
> Comment on attachment 8612584 [details] [diff] [review]
> patch 1 - avoid reflow for hiding window marigns on Windows
> 
> Review of attachment 8612584 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> I didn't see much of a difference visually, seems fine though if it cuts
> down on event traffic during the transition.

It removes one resize event on Windows according to my test :)
You need to log in before you can comment on or make changes to this bug.