Skip to content

/ servo

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

Jump to bottom

Running browser.html with -M causes cross origin loads to fail #12807

Open
cbrewster opened this issue Aug 10, 2016 · 5 comments
Open

Running browser.html with -M causes cross origin loads to fail #12807

cbrewster opened this issue Aug 10, 2016 · 5 comments
Labels
I-panic

Comments

@cbrewster
Copy link
Member

@cbrewster cbrewster commented Aug 10, 2016

Steps:

  1. Run browser.html with -M (currently default)
  2. Open a tab.
  3. Close that tab.
  4. Open a cross origin tab.
  5. The page will not load

Assertion failure:

ERROR:servo: assertion failed: mach_sys::mach_port_mod_refs(mach_task_self(), self.port,
                             MACH_PORT_RIGHT_SEND, 1) == KERN_SUCCESS
@jdm jdm added the I-panic label Aug 10, 2016
@cbrewster
Copy link
Member Author

@cbrewster cbrewster commented Aug 10, 2016

@Gozala has determined that this is an issue with iframe visibility.
@Gozala
Copy link
Contributor

@Gozala Gozala commented Aug 10, 2016

Ok here is the as reduced test case as I was able to produce:

/Applications/Servo.app/Contents/MacOS/servo -w -b -S -M --pref dom.mozbrowser.enabled --pref dom.forcetouch.enabled --pref  shell.builtin-key-shortcuts.enabled=false https://Gozala.jsbin.com/dehoji

Without -M issue does not occur, without use of visibility: hidden in here issue does not occur either https://jsbin.com/dehoji/edit?html,css,js,output
@Gozala Gozala mentioned this issue Aug 10, 2016
Open
@Gozala
Copy link
Contributor

@Gozala Gozala commented Aug 10, 2016 •
edited

Also worth pointing out that servo nightly is one commit behind the browser.html head, once it's up to date this will prompt a crash reporter. @paulrouget could you do sync them up plz.
@paulrouget
Copy link
Contributor

@paulrouget paulrouget commented Aug 11, 2016

@Gozala #12812
This was referenced Aug 15, 2016
Merged
Open
@paulrouget
Copy link
Contributor

@paulrouget paulrouget commented Aug 15, 2016

Let's disable multiprocess (#12867) and keep this issue open
bors-servo added a commit that referenced this issue Aug 15, 2016
@bors-servo
Auto merge of #12867 - paulrouget:noMS, r=emilio
Loading status checks…
a617ea3 
Revert "Enable multiprocess and sandbox"

Multiprocess and sandboxing still have some issues:
* #12791
* #12807

Let's disable these features for now and reenable them later: #12868.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/12867)
<!-- Reviewable:end -->
gecko-dev-updater pushed a commit to marco-c/gecko-dev-comments-removed that referenced this issue Oct 1, 2019
@marco-c
servo: Merge #12867 - Revert "Enable multiprocess and sandbox" (from …
234e5ce 
…paulrouget:noMS); r=emilio

Multiprocess and sandboxing still have some issues:
* servo/servo#12791
* servo/servo#12807

Let's disable these features for now and reenable them later: servo/servo#12868.

Source-Repo: https://github.com/servo/servo
Source-Revision: a617ea353a059486a117e1c04a60e3d13fc5f9cc

UltraBlame original commit: 166e02c3b7a1db2ef2989dc0c2e12fe05909364a
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified-and-comments-removed that referenced this issue Oct 1, 2019
@marco-c
servo: Merge #12867 - Revert "Enable multiprocess and sandbox" (from …
e6d4423 
…paulrouget:noMS); r=emilio

Multiprocess and sandboxing still have some issues:
* servo/servo#12791
* servo/servo#12807

Let's disable these features for now and reenable them later: servo/servo#12868.

Source-Repo: https://github.com/servo/servo
Source-Revision: a617ea353a059486a117e1c04a60e3d13fc5f9cc

UltraBlame original commit: 166e02c3b7a1db2ef2989dc0c2e12fe05909364a
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified that referenced this issue Oct 1, 2019
@marco-c
servo: Merge #12867 - Revert "Enable multiprocess and sandbox" (from …
651e984 
…paulrouget:noMS); r=emilio

Multiprocess and sandboxing still have some issues:
* servo/servo#12791
* servo/servo#12807

Let's disable these features for now and reenable them later: servo/servo#12868.

Source-Repo: https://github.com/servo/servo
Source-Revision: a617ea353a059486a117e1c04a60e3d13fc5f9cc

UltraBlame original commit: 166e02c3b7a1db2ef2989dc0c2e12fe05909364a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
I-panic
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
@Gozala @jdm @paulrouget @cbrewster
You can’t perform that action at this time.