Skip to content

Commit

Permalink
2300: *workers info fixup
Browse files Browse the repository at this point in the history
"Push and web notifications require service workers, which in turn require workers." - this is clearly not (or no longer) true. See #256 where workers are disabled, but service workers enabled, and service workers create IDB entries on Youtube
  • Loading branch information
Thorin-Oakenpants authored Oct 31, 2017
1 parent ce66da1 commit df01fb8
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions user.js
Original file line number Diff line number Diff line change
Expand Up @@ -1086,9 +1086,8 @@ user_pref("dom.disable_beforeunload", true);
/*** 2300: WEB WORKERS [SETUP]
A worker is a JS "background task" running in a global context, i.e. it is different from
the current window. Workers can spawn new workers (must be the same origin & scheme),
including service and shared workers. Shared workers can be utilized by multiple scripts
and communicate between browsing contexts (windows/tabs/iframes) and can even control your
cache. Push and web notifications require service workers, which in turn require workers.
including service and shared workers. Shared workers can be utilized by multiple scripts and
communicate between browsing contexts (windows/tabs/iframes) and can even control your cache.
[WARNING] Disabling workers *will* break sites (e.g. Google Street View, Twitter).
It is recommended that you use a separate profile for these sorts of sites.
Expand Down

1 comment on commit df01fb8

@Thorin-Oakenpants
Copy link
Contributor Author

@Thorin-Oakenpants Thorin-Oakenpants commented on df01fb8 Oct 31, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

correction: I meant #234 for the serviceworkers example with YT

Please sign in to comment.