Move cookie defaults to GetCookieSetting

[pilgrim-brave]

Fixes brave/brave-browser#1449

Fixes brave/brave-browser#1592

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).
• Verified that these changes build without errors on
  • Windows
  • macOS
  • Linux
• Verified that these changes pass automated tests (npm test brave_unit_tests && npm test brave_browser_tests) on
  • Windows
  • macOS
  • Linux
• Ran git rebase master (if needed).
• 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.
• Add appropriate QA labels (QA/Yes or QA/No) to include the closed issue in milestone

Test Plan:

Reviewer Checklist:

• New files have MPL-2.0 license header.
• Request a security/privacy review as needed.
• Adequate test coverage exists to prevent regressions
• Verify test plan is specified in PR before merging to source

[yrliou]

Change looks good to me, I also tested it on mac, works fine.

[yrliou]

Is code in brave_cookie_settings.cc already have test coverage?

[yrliou]

Override the approve just to make sure it already has test coverage before merging.

[yrliou]

Covered in brave_content_settings_observer_browsertest.cc, thanks for confirming, Mark!

[bbondy]

master: a52c537
0.57.x: 27b75a6
0.56.x: 2e22eff
0.55.x: dc6c2bd