Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Disable domain reliability service #246

Merged
merged 1 commit into from
Jul 12, 2018
Merged

Disable domain reliability service #246

merged 1 commit into from
Jul 12, 2018

Conversation

bbondy
Copy link
Member

@bbondy bbondy commented Jul 12, 2018

Partially addresses brave/brave-browser#514

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.

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

@bbondy bbondy self-assigned this Jul 12, 2018
@bbondy bbondy requested a review from simonhong July 12, 2018 03:22
@bbondy bbondy removed the request for review from simonhong July 12, 2018 03:29
@bbondy bbondy merged commit 926df02 into master Jul 12, 2018
@@ -17,6 +17,9 @@ using BraveProfilePrefsBrowserTest = InProcessBrowserTest;
IN_PROC_BROWSER_TEST_F(BraveProfilePrefsBrowserTest, DownloadPromptDefault) {
EXPECT_TRUE(
browser()->profile()->GetPrefs()->GetBoolean(prefs::kPromptForDownload));
}

IN_PROC_BROWSER_TEST_F(BraveProfilePrefsBrowserTest, WidevineOptInDisabledByDefault) {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

widevine?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just happened to notice it was wrongly grouped in DownloadPromptDefault so I added a line to make it its own test.

@bbondy bbondy deleted the domain-reliability branch August 20, 2018 01:39
srirambv pushed a commit to srirambv/brave-core that referenced this pull request Aug 20, 2018
cezaraugusto added a commit that referenced this pull request Jul 17, 2019
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants