1268335 - Getting screen size in content process costs an additional sync IPC

Status: RESOLVED DUPLICATE of bug 1194751

(Core :: DOM: Core & HTML, defect)

Description

[Xidorn Quan [:xidorn] UTC+11]

For avoid potential additional resize reflow, we change the window dimensions to the screen dimensions in advance for fullscreen change in the content process.

Currently, we get the size of the screen in the content process, which costs an additional IPC call, which could be avoided if we pass the screen together with the fullscreen change message itself.

It isn't too expensive, but it's still worth several tens of ms we may want to avoid.

Comment 1

[Xidorn Quan [:xidorn] UTC+11]

It's worth 10% of the time.

Comment 2

[Olli Pettay [:smaug][[e-mail]]]

tens of ms? huh, parent process must be busy doing something else if it is that slow to reply.

Comment 3

[Xidorn Quan [:xidorn] UTC+11]

The parent process could be reflowing the UI.

Comment 4

[Anthony Jones (:ajones, :kentuckyfriedtakahe, :k17e)]

(In reply to Olli Pettay [:smaug] from comment #2)
> tens of ms? huh, parent process must be busy doing something else if it is
> that slow to reply.

We often have several CPU cores decoding video while transitioning to full screen.

Comment 5

[Andrew Overholt [:overholt]]

I'm guessing this is something we/Xidorn aim to fix in the next few months => "fixlater".