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

Implement incumbent, entry, current settings objects / globals / realms #10963

Closed
Ms2ger opened this issue May 2, 2016 · 7 comments
Closed

Implement incumbent, entry, current settings objects / globals / realms #10963

Ms2ger opened this issue May 2, 2016 · 7 comments

Comments

@Ms2ger
Copy link
Contributor

@Ms2ger Ms2ger commented May 2, 2016

Incumbent in particular is needed for security checks (iframe.contentDocument, for example).

@jdm
Copy link
Member

@jdm jdm commented May 2, 2016

Note to self: find the WIP branch that was trying to implement these concepts.

@jdm
Copy link
Member

@jdm jdm commented May 12, 2016

This is not the WIP branch, but included a very minimal AutoEntryScript that kept a stack of pipeline IDs that were executing script: https://github.com/jdm/servo/tree/postmessage

@mrmiywj
Copy link
Contributor

@mrmiywj mrmiywj commented Aug 3, 2016

Mind if I try this and the related issues?
Any reference resource?

@jdm
Copy link
Member

@jdm jdm commented Aug 3, 2016

@mrmiywj This is not something I would feel comfortable having someone work on without discussing the necessary work and implementation plan first.

@jdm
Copy link
Member

@jdm jdm commented Aug 3, 2016

Also e3873c2 is a more recent implementation of the sort of thing we need here.

@mrmiywj
Copy link
Contributor

@mrmiywj mrmiywj commented Aug 4, 2016

Thx! I will take a look on it.

On Thu, Aug 4, 2016 at 2:53 AM Josh Matthews notifications@github.com
wrote:

Also e3873c2
e3873c2
is a more recent implementation of the sort of thing we need here.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#10963 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AHHT7Bfv1uXpwLGFFzO7QElNshn4fOw7ks5qcOOygaJpZM4IVP6B
.

@Ms2ger Ms2ger changed the title Implement incumbent, entry, current settings objects Implement incumbent, entry, current settings objects / globals / realms Nov 24, 2016
Ms2ger added a commit that referenced this issue Jan 4, 2017
Partial fix for #10963.
Ms2ger added a commit that referenced this issue Jan 5, 2017
Partial fix for #10963.
Ms2ger added a commit that referenced this issue Jan 5, 2017
Partial fix for #10963.
Ms2ger added a commit that referenced this issue Jan 5, 2017
Partial fix for #10963.
Ms2ger added a commit that referenced this issue Jan 5, 2017
Partial fix for #10963.
Ms2ger added a commit that referenced this issue Jan 5, 2017
Partial fix for #10963.
Ms2ger added a commit that referenced this issue Jan 5, 2017
Partial fix for #10963.
Ms2ger added a commit that referenced this issue Jan 5, 2017
Partial fix for #10963.
bors-servo added a commit that referenced this issue Jan 6, 2017
Implement the entry global.

Partial fix for #10963.

<!-- 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/14840)
<!-- Reviewable:end -->
Ms2ger added a commit that referenced this issue Jan 6, 2017
Partial fix for #10963.
bors-servo added a commit that referenced this issue Jan 6, 2017
Implement the entry global.

Partial fix for #10963.

<!-- 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/14840)
<!-- Reviewable:end -->
Ms2ger added a commit that referenced this issue Jan 9, 2017
Fixes #10963.
Ms2ger added a commit that referenced this issue Jan 10, 2017
Fixes #10963.
Ms2ger added a commit that referenced this issue Jan 10, 2017
Fixes #10963.
Ms2ger added a commit that referenced this issue Jan 13, 2017
Fixes #10963.
bors-servo added a commit that referenced this issue Jan 13, 2017
Implement the incumbent global.

Fixes #10963.

<!-- 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/14936)
<!-- Reviewable:end -->
Ms2ger added a commit that referenced this issue Jan 17, 2017
Fixes #10963.
bors-servo added a commit that referenced this issue Jan 17, 2017
Implement the incumbent global.

Fixes #10963.

<!-- 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/14936)
<!-- Reviewable:end -->
Ms2ger added a commit that referenced this issue Jan 17, 2017
Fixes #10963.
bors-servo added a commit that referenced this issue Jan 17, 2017
Implement the incumbent global.

Fixes #10963.

<!-- 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/14936)
<!-- Reviewable:end -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
You can’t perform that action at this time.