Removes site settings for delete sites

[NejcZdovc]

Resolves #14245

Submitter Checklist:

• Submitted a ticket for my issue if one did not already exist.
• Used Github auto-closing keywords in the commit message.
• Added/updated tests for this change (for new code or code which already has tests).
• Ran git rebase -i to squash commits (if needed).
• Tagged reviewers and labelled the pull request as needed.
• Request a security/privacy review as needed. (Ask a Brave employee to help if you cannot access this document.)

Test Plan:

defined in #14245

Reviewer Checklist:

• Request a security/privacy review as needed if one was not already requested.

Tests

• Adequate test coverage exists to prevent regressions
• Tests should be independent and work correctly when run individually or as a suite ref
• New files have MPL2 license header

[jasonrsadler]

If auto-include is left on and site is removed from ledger completely, revisiting it relists, but does not include it. Is this right?

Also, is this issue possible dupe?
#13283 (comment)

[NejcZdovc]

@jasonrsadler I can't reproduce this. What are the steps exactly?

[jasonrsadler]

Now I can't either. Disregard.

[ryanml]

Could not repro issue mentioned above, code looks good, test plan works well

[NejcZdovc]

master 21f24d0
0.23 116e53f
0.22 84183b0