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

chore: cherry-pick 3b5f65c0aeca from chromium #25859

Merged
merged 2 commits into from
Oct 14, 2020

Conversation

ppontes
Copy link
Member

@ppontes ppontes commented Oct 9, 2020

[m85] Reland "Add more checks for chrome.debugger extensions"

TBR=rdevlin.cronin@chromium.org

This reverts commit 5a809a08fd5ca32cb8d594664416db2f2dc8ebdc.

Reason for revert: I don't think the test failure is related. Please note it stopped before the revert landed (build no 91007 vs. 91010). This must have been a flake, or a independent failure that has been fixed by one of the front-end rolls.

Original change's description:

Revert "Add more checks for chrome.debugger extensions"

This reverts commit 4838b76ae48797760fd8a362b4dc15325ccddcf5.

Reason for revert: 1119297

Original change's description:

Add more checks for chrome.debugger extensions

Bug: 1113558, 1113565
Change-Id: I99f2e030f9a38f1ffd6b6adc760ba15e5d231f96
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2342277
Commit-Queue: Andrey Kosyakov caseq@chromium.org
Reviewed-by: Sigurd Schneider sigurds@chromium.org
Reviewed-by: Yang Guo yangguo@chromium.org
Reviewed-by: Devlin rdevlin.cronin@chromium.org
Reviewed-by: Dmitry Gozman dgozman@chromium.org
Cr-Commit-Position: refs/heads/master@{#799514}

TBR=dgozman@chromium.org,rdevlin.cronin@chromium.org,caseq@chromium.org,yangguo@chromium.org,sigurds@chromium.org

Change-Id: I01ad12ca99ac75197f9073e2c6c9d0eaa0d95147
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 1113558
Bug: 1113565
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2362920
Reviewed-by: Christian Dullweber dullweber@chromium.org
Commit-Queue: Christian Dullweber dullweber@chromium.org
Cr-Commit-Position: refs/heads/master@{#799558}

TBR=dgozman@chromium.org,rdevlin.cronin@chromium.org,caseq@chromium.org,yangguo@chromium.org,sigurds@chromium.org,dullweber@chromium.org

(cherry picked from commit a064db74c8734fbf47de2f3a3503832514857173)

(cherry picked from commit 9940472e708a4003aee9edf9da42d68fde591e08)

Bug: 1113558
Bug: 1113565
Change-Id: Ic98fc037028a210204b7935b0b8e50e4e36e2397
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2368446
Reviewed-by: Andrey Kosyakov caseq@chromium.org
Commit-Queue: Andrey Kosyakov caseq@chromium.org
Cr-Original-Original-Commit-Position: refs/heads/master@{#800682}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2398884
Cr-Original-Commit-Position: refs/branch-heads/4240@{#506}
Cr-Original-Branched-From: f297677702651916bbf65e59c0d4bbd4ce57d1ee-refs/heads/master@{#800218}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2419133
Cr-Commit-Position: refs/branch-heads/4183@{#1863}
Cr-Branched-From: 740e9e8a40505392ba5c8e022a8024b3d018ca65-refs/heads/master@{#782793}

Notes: Security: backported fix for 1113558.

@ppontes ppontes requested a review from a team as a code owner October 9, 2020 18:48
@ppontes ppontes added 9-x-y backport-check-skip Skip trop's backport validity checking labels Oct 9, 2020
@electron-cation electron-cation bot added the new-pr 🌱 PR opened in the last 24 hours label Oct 9, 2020
@ppontes ppontes force-pushed the cherry-pick/9-x-y/chromium/3b5f65c0aeca branch from 72465f7 to 249533d Compare October 9, 2020 20:38
@electron-cation electron-cation bot removed the new-pr 🌱 PR opened in the last 24 hours label Oct 10, 2020
@ppontes ppontes force-pushed the cherry-pick/9-x-y/chromium/3b5f65c0aeca branch 2 times, most recently from 2dad661 to 95dc5d6 Compare October 12, 2020 12:40
@@ -137,6 +137,5 @@ backport_1122684.patch
backport_1111737.patch
cherry-pick-0e61c69ebd47.patch
cherry-pick-814a27f8522b.patch
cherry-pick-adc731d678c4.patch
Copy link
Contributor

Choose a reason for hiding this comment

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

Why are these patches removed?

Copy link
Member Author

Choose a reason for hiding this comment

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

Great catch, @zcbenz , this was probably the result of an interactive rebase of the patch commit log in the Chromium source gone wrong. Fixed.

@ppontes ppontes force-pushed the cherry-pick/9-x-y/chromium/3b5f65c0aeca branch from 702ac0e to 95f60d4 Compare October 13, 2020 20:24
@zcbenz zcbenz merged commit 437701d into 9-x-y Oct 14, 2020
@release-clerk
Copy link

release-clerk bot commented Oct 14, 2020

Release Notes Persisted

Security: backported fix for 1113558.

@zcbenz zcbenz deleted the cherry-pick/9-x-y/chromium/3b5f65c0aeca branch October 14, 2020 01:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
9-x-y backport-check-skip Skip trop's backport validity checking
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants