Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Disables TrustTokens feature. #8039

Merged
merged 4 commits into from
May 18, 2021
Merged

Disables TrustTokens feature. #8039

merged 4 commits into from
May 18, 2021

Conversation

mkarolin
Copy link
Collaborator

@mkarolin mkarolin commented Feb 22, 2021

Fixes brave/brave-browser#13886
Fixes brave/brave-browser#15847
Fixes brave/brave-browser#15848
Fixes brave/brave-browser#15902

Submitter Checklist:

  • There is a ticket for my issue
  • Used Github auto-closing keywords in the PR description above
  • Wrote a good PR/commit description
  • Added appropriate labels (QA/Yes or QA/No; release-notes/include or release-notes/exclude; OS/...) to the associated issue
  • Checked the PR locally: npm run test -- brave_browser_tests, npm run test -- brave_unit_tests, npm run lint, npm run gn_check, npm run tslint
  • Ran git rebase master (if needed)
  • Requested a security/privacy review as needed

Reviewer Checklist:

  • New files have MPL-2.0 license header
  • Adequate test coverage exists to prevent regressions
  • Major classes, functions and non-trivial code blocks are well-commented
  • Changes in component dependencies are properly reflected in gn
  • Code follows the style guide
  • Test plan is specified in PR before merging

After-merge Checklist:

Test Plan:

@mkarolin mkarolin requested a review from pes10k February 22, 2021 22:51
@mkarolin mkarolin requested a review from a team as a code owner February 22, 2021 22:51
@mkarolin mkarolin self-assigned this Feb 22, 2021
@mkarolin mkarolin force-pushed the maxk-disable-trust-tokens branch 2 times, most recently from 5f2af2b to 3f462ef Compare May 17, 2021 23:47
@mkarolin mkarolin requested a review from bridiver as a code owner May 17, 2021 23:47
@mkarolin
Copy link
Collaborator Author

mkarolin commented May 18, 2021

CI failures:

  • Linux and MacOS: BraveAdsBrowserTest/BraveAdsUpgradeBrowserTest.UpgradePath browser test is known and unrelated to the PR.

Also, disables the origin trial.

Fixes brave/brave-browser#13886
Features and also the origin trial.

Fixes brave/brave-browser#15848
We need to follow up on this to proxy component download and also fiture
out API keys maybe as it currently doesn't work.

Chromium change:

https://source.chromium.org/chromium/chromium/src/+/9b46507cf5a5ee1460dc9c20307206d7cb004a2f

commit 9b46507cf5a5ee1460dc9c20307206d7cb004a2f
Author: evliu <evliu@google.com>
Date:   Fri Mar 5 00:49:23 2021 +0000

    Enable the Live Caption feature by default

    This CL enables the LiveCaption, UseSodaForLiveCaption, and IPH_LiveCaption features by default.

    Bug: 1182008
@mkarolin mkarolin merged commit 80e8ab3 into master May 18, 2021
@mkarolin mkarolin deleted the maxk-disable-trust-tokens branch May 18, 2021 12:48
@mkarolin
Copy link
Collaborator Author

Rebased due to a conflict and merged.

@mkarolin mkarolin added this to the 1.26.x - Nightly milestone May 18, 2021
mkarolin added a commit that referenced this pull request May 18, 2021
sangwoo108 added a commit that referenced this pull request Feb 9, 2023
[We disabled the "Trust token" feature](#8039),
so we shouldn't reveal this item.
sangwoo108 added a commit that referenced this pull request Mar 28, 2023
[We disabled the "Trust token" feature](#8039),
so we shouldn't reveal this item.
sangwoo108 added a commit that referenced this pull request Mar 30, 2023
* Hide "Trust Token" item from devtools

[We disabled the "Trust token" feature](#8039),
so we shouldn't reveal this item.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants