corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 1689496 - Add showSponsoredTopSites to user_prefs r=thecount

Browse Source 
Differential Revision: https://phabricator.services.mozilla.com/D103403
This commit is contained in:
Nan Jiang
2021-01-29 18:47:01 +00:00
parent 565c703353
commit f9c899e733
2 changed files with 6 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
browser/components/newtab/docs/v2-system-addon/data_dictionary.md
View File

@@ -322,20 +322,22 @@ This encoding mapping was defined in `system-addon/lib/TelemetryFeed.jsm`
+-------------------+------------------------+
| `showCFRFeatures` | 128 (10000000) |
+-------------------+------------------------+
| `showSponsoredTopSites` | 256 (100000000) |
+-------------------+------------------------+
```
Each item above could be combined with other items through bitwise OR (`|`) operation.
Examples:
* Everything is on, `user_prefs = 1 | 2 | 4 | 8 | 16 | 32 | 64 | 128 = 255`
* Everything is on, `user_prefs = 1 | 2 | 4 | 8 | 16 | 32 | 64 | 128 | 256 = 511`
* Everything is off, `user_prefs = 0`
* Only show search and Top Stories, `user_prefs = 1 | 4 = 5`
* Everything except Highlights, `user_prefs = 1 | 2 | 4 | 16 | 32 | 64 | 128 = 247`
* Everything except Highlights, `user_prefs = 1 | 2 | 4 | 16 | 32 | 64 | 128 | 256 = 503`
Likewise, one can use bitwise AND (`&`) for decoding.
* Check if everything is shown, `user_prefs & (1 | 2 | 4 | 8 | 16 | 32 | 64 | 128)` or `user_prefs == 255`
* Check if everything is shown, `user_prefs & (1 | 2 | 4 | 8 | 16 | 32 | 64 | 128 | 256)` or `user_prefs == 511`
* Check if everything is off, `user_prefs == 0`
* Check if search is shown, `user_prefs & 1`
* Check if both Top Sites and Top Stories are shown, `(user_prefs & 2) && (user_prefs & 4)`, or `(user_prefs & (2 | 4)) == (2 | 4)`