Skip to content

Upgrade from Chromium 88 to Chromium 89 (1.21.x). #8009

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

Merged
merged 7 commits into from
Feb 23, 2021
Merged

Conversation

mkarolin
Copy link
Collaborator

@mkarolin mkarolin commented Feb 19, 2021

Fixes brave/brave-browser#13238
Fixes brave/brave-browser#14065
Fixes brave/brave-browser#14112
Fixes brave/brave-browser#14165
Fixes brave/brave-browser#14167
Fixes brave/brave-browser#14227
Fixes brave/brave-browser#14235
Fixes brave/brave-browser#14284

Related PR: brave/brave-browser#14255

Uplift from #7460
Uplift from #7900
Uplift from #7936
Uplift from #7975
Uplift from #7988
Uplift from #8000
Uplift from #8027

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 and others added 6 commits February 19, 2021 10:34
Upgrade from Chromium 88 to Chromium 89.
[Settings] fixes clicking 'Additional settings'.
Upgrade patches from Chromium 89.0.4389.40 to Chromium 89.0.4389.48
Upgrade patches from Chromium 89.0.4389.48 to Chromium 89.0.4389.58.
Disables Safe Browsing Enhanced Protection message in interstitials.
@mkarolin mkarolin added the CI/run-network-audit Run network-audit label Feb 19, 2021
@mkarolin mkarolin requested a review from a team February 19, 2021 15:45
@mkarolin mkarolin self-assigned this Feb 19, 2021
</if>
<message name="IDS_SETTINGS_PEOPLE_SIGN_IN_PROMPT_SECONDARY_WITH_ACCOUNT" desc="The secondary text displayed to prompt users to enable sync for an account that is alredy present in Brave.">
Sync and personalize Brave across your devices
</message>
<if expr="chromeos">
<message name="IDS_SETTINGS_ACCOUNT_MANAGER_DESCRIPTION_V2" desc="Description of the Account Manager Settings page. Shown just below the title of the page.">
Copy link
Contributor

Choose a reason for hiding this comment

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

Seems Chromium leftover, Chromebook => Bravebook

Copy link
Contributor

@AlexeyBarabash AlexeyBarabash left a comment

Choose a reason for hiding this comment

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

sync changes look good

@kjozwiak kjozwiak added this to the 1.21.x - Beta milestone Feb 23, 2021
Copy link
Member

@kjozwiak kjozwiak left a comment

Choose a reason for hiding this comment

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

Uplift into 1.21.x approved after deliberating with @brave/uplift-approvers. QA went through the manual passes as per brave/brave-browser#13968 & brave/qa-resources#207.

@kjozwiak kjozwiak merged commit a47ea64 into 1.21.x Feb 23, 2021
@kjozwiak kjozwiak deleted the cr89-1.21.x branch February 23, 2021 08:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI/run-network-audit Run network-audit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants