Is this correct? It looks like you're saying "for every blocked site, delete the corresponding entry in userSettings". But can't sites have userSettings other than "blocked"?

Even if you can't (quick grep doesn't show any other settings, which confuses me because of all the things you can do to sites in the UX, like hide, etc.) this code is still confusing. Why not just reset `userSettings = {}`?