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.

Sign up for GitHub

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

[Session Bookmarks] [e10s] Doesn't work correctly with enabled Electrolysis (multi-process mode) #38

Open
Infocatcher opened this issue Aug 21, 2015 · 0 comments
Open

[Session Bookmarks] [e10s] Doesn't work correctly with enabled Electrolysis (multi-process mode) #38

Infocatcher opened this issue Aug 21, 2015 · 0 comments
Labels
compatibility

Comments

@Infocatcher
Copy link
Owner

Throws here:

            let isPrivate = "PrivateBrowsingUtils" in window
                && PrivateBrowsingUtils.isWindowPrivate(tab.linkedBrowser.contentWindow) // <---
                || tab.hasAttribute(privateAttr);

Also not only here because remote frame not yet initialized.
Infocatcher added a commit that referenced this issue Aug 21, 2015
@Infocatcher
Wait for remote frame initialization under e10s
c2a06d7 
(#38)
Infocatcher added a commit that referenced this issue Aug 21, 2015
@Infocatcher
Don't parse data twice in case of not yet initialized remote frame
d29c376 
(#38)
Infocatcher added a commit that referenced this issue Aug 22, 2015
@Infocatcher
Add shortcut for tab.linkedBrowser
c02151c 
(related to #38)
Infocatcher added a commit that referenced this issue Aug 22, 2015
@Infocatcher
Use .contentWindowAsCPOW and .contentDocumentAsCPOW
89f9d4e 
(not needed now, for future versions)
(#38)
Infocatcher added a commit that referenced this issue Aug 24, 2015
@Infocatcher
Try prevent only e10s-related errors
44640e2 
(#38)
Infocatcher added a commit that referenced this issue Aug 31, 2015
@Infocatcher
Fix typo: accidentally copy-pasted code to toggle private state
f6187c9 
(#38)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
compatibility
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Infocatcher