feat: Add Sync policy

[bsmth]

Description:

Adding Sync

Motivation:

Sync is incoming

For reviewers:

• Check case sensitivity, such as:

Software\Policies\Mozilla\Firefox\Sync\bookmarks
Software\Policies\Mozilla\Firefox\Sync\Bookmarks

See also:

• https://bugzilla.mozilla.org/show_bug.cgi?id=2015810
• https://github.com/mozilla/enterprise-firefox/pull/313/changes#diff-53e9ab73465fad9f328175c11477a4271e6956a01722ec137eee894397604a62R57

[bsmth]

Having valid JSON here feels like a good thing, although this is likely an unrealistic example. Same with others, like:

{
  "policies": {
    "Sync": {
      "Enabled": true,
      "addresses": true,
      "bookmarks": true,
      "history": true,
      "openTabs": true,
      "passwords": true,
      "paymentMethods": true,
      "addons": true,
      "settings": true,
      "Locked": true
    }
  }
}

v

{
  "policies": {
    "Sync": {
      "Enabled": true | false,
      "addresses": true | false,
      "bookmarks": true | false,
      "history": true | false,
      "openTabs": true | false,
      "passwords": true | false,
      "paymentMethods": true | false,
      "addons": true | false,
      "settings": true | false,
      "Locked": true | false
    }
  }
}

[1rneh]

Thanks for pointing out the inconsistently capitalized properties form the JSDoc - a correction has landed.

I left a notes on the Sync description. If we follow the Acorn convention to not capitalize the first letter of "sync", we should update each occurrence in this document accordingly.

Aside from those comments, the docs look good to me.

[cadeyrn]

Is the policy exclusive to Firefox Enterprise? I can't see in the linked Bugzilla ticket that code has been added to regular Firefox, it only references a pull request for Firefox Enterprise.

If so: 1) Why? 2) It should be noted in the documentation, I guess.

[bsmth]

Is the policy exclusive to Firefox Enterprise? I can't see in the linked Bugzilla ticket that code has been added to regular Firefox, it only references a pull request for Firefox Enterprise.

If so: 1) Why? 2) It should be noted in the documentation, I guess.

Hi @cadeyrn thanks for flagging it.

1. Currently it's in Firefox Enterprise only. It's needed for a specific use case in Enterprise, although there's a bug here tracking progress for landing it in Firefox -> https://bugzilla.mozilla.org/show_bug.cgi?id=2017722 we aim to have it supported everywhere eventually.
2. I'll update the compat information so it's at least tracked in prose for now.