[WIP] Firefox support #225
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This bug drove me insane for virtually all the time I'm in
porting this extension to Firefox. Sometimes when I switch
to a tab and then click unsuspend, it may load not to an
intended URL, but to about:blank instead. Few mounths into
the work this bug caught my attention, and after dozens of
unsuspends I've come with this:
Immediately after tab switch, TGS starts a checker to
check and match suspended tabs with the extention state.
But it forks not immediately, and in a short time window
it's possible to unsuspend unchecked tab. It leads to the
following log: 'Resuspending unresponsive suspended tab'
and the call of `tabsReload(tab.id);`. On Chromium it's
perfectly fine, but in FF it just breaks everything
(posibly due to a bad code inside the browser engine) and
leads to loading blank page insted of a desired.
Patch adds a flag so checker won't reload the tab and
let it unsuspend properly. This flag later will be cleared
by existing code in `handleUnsuspendedTabStateChanged`.
Remove autodiscardable since it's not currently supported by Firefox.
Firefox by default comes with conflicting shortcut in Firefox Screenshots system extension.
This flag allows replacing URL in tab history therefore making tab history clean and usable. NOTE: loadReplace is unique to Firefox, in Chrome achieved via history manipulations.
Firefox does not support callbacks for chrome.tabs.discard, it uses promisses instead.
Internal views for tabs in different containers are currently unavailable in Firefox. Discard container tabs instead of suspending achieves the best possible effect without redesigning entire extension.
Add `wrappedJSObject` to fix "Operation is insecure" exception. Credits: https://bugzilla.mozilla.org/show_bug.cgi?id=1613779#c1
Use promise instead of Chrome's callback
Use the correct argument name to prevent key event errors
dvalter
commented
Dec 3, 2023
| }, | ||
| "applications": { | ||
| "gecko": { | ||
| "id": "themarvelloussuspender@dvalter" |
dvalter
commented
Dec 3, 2023
| "content_security_policy": "script-src 'self'; object-src 'self'; child-src 'self'; connect-src 'self'; img-src 'self' data: chrome:; style-src 'self'; default-src 'self'", | ||
| "incognito": "split", | ||
| "incognito": "not_allowed", |
There was a problem hiding this comment.
I could not make it work in private mode in FF, the easiest workaround would be to split manifests and package different ones for Chrome and FF
dvalter
commented
Dec 3, 2023
| @@ -303,6 +303,10 @@ var gsTabSuspendManager = (function() { | |||
| let discardInPlaceOfSuspend = gsStorage.getOption( | |||
| gsStorage.DISCARD_IN_PLACE_OF_SUSPEND, | |||
| ); | |||
| if (tab.cookieStoreId != "firefox-default") { | |||
There was a problem hiding this comment.
Could not make it work with FF tab containers, however, discarding works fine here
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is a draft of a Firefox support PR.
It is based on my TGS fork with all the necessary commits rebased on the MS master branch.
It would probably break some (or most) of functionality in Chrome, so some changes are likely needed
Fixes #18