Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Meta] multi windows support #13997

Open
2 of 5 tasks
paulrouget opened this issue Oct 31, 2016 · 5 comments
Open
2 of 5 tasks

[Meta] multi windows support #13997

paulrouget opened this issue Oct 31, 2016 · 5 comments
Labels
B-meta This issue tracks the status of multiple, related pieces of work

Comments

@paulrouget
Copy link
Contributor

paulrouget commented Oct 31, 2016

@paulrouget
Copy link
Contributor Author

There's most likely a lot of work to be done. But I think we should start by suppressing the assumption that there is only one root frame, and only one compositor, and that a top level pipeline is drawn full window.

@jdm jdm added the B-meta This issue tracks the status of multiple, related pieces of work label Oct 31, 2016
@paulrouget
Copy link
Contributor Author

paulrouget commented Nov 2, 2016

Here is how I imagine this would work (as a first step):

  • supports command line: servo http://site1.com http://site2.com (opening new windows from window.open will come later)
  • in servo/main:
    • create 2 glutin windows, 2 compositors, 1 constellation with 2 browsers
    • spawn a thread for each window event handler
    • wait until both windows close to exit
  • in constellation,
    • remove root_frame_id
    • add a map root_frame <-> compositor
    • sort out compositor messages (find right frame for compositor, for example, key events)

I'm sure there are plenty of other things to take into consideration.

I have no idea how this will impact the compositor code.

@paulrouget
Copy link
Contributor Author

More about compositor / webrender implications here: #13338

@paulrouget
Copy link
Contributor Author

We will also need to fix glutin/winit to support multiwindow. See rust-windowing/winit#56

@pshaughn
Copy link
Member

pshaughn commented Nov 29, 2019

WPT xhr/open-url-multi-window-6.htm cares (probably among many other WPT tests I haven't dug into yet)

@jdm jdm added this to To do in web-platform-test failures via automation Nov 29, 2019
@delan delan mentioned this issue Jan 18, 2024
39 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
B-meta This issue tracks the status of multiple, related pieces of work
Projects
Development

No branches or pull requests

3 participants