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 df438f22f7d2 from chromium #27609

Merged
merged 3 commits into from Feb 12, 2021

Conversation

nornagon
Copy link
Member

@nornagon nornagon commented Feb 3, 2021

[navigation] Fix navigation request reset logic

Do not delete navigation request which has started upon receiving
notification about beforeunload dialog being cancelled, as a) this
navigation request is not waiting for beforeunload and b) it might have
been this navigation request which canceled this beforeunload dialog.

M86 merge conflicts and resolution:

  • content/browser/frame_host/*
    In ToT files the affected under frame_host directory are moved to
    renderer_host dir. Applied patch to frame_host, no further conflicts.

R=​alexmos@chromium.org
BUG=1161705

(cherry picked from commit 23c110b5b81dc401ded5d4dcecfab65d5d88fdfa)

(cherry picked from commit 87550e04d9fed4bbedff4546f4161e3c02415d7e)

Change-Id: I7d385d4326fac6f67d17a003679471806b5ad3b2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2624733
Commit-Queue: Alexander Timin altimin@chromium.org
Reviewed-by: Alex Moshchuk alexmos@chromium.org
Cr-Original-Original-Commit-Position: refs/heads/master@{#843343}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2652791
Commit-Queue: Alex Moshchuk alexmos@chromium.org
Auto-Submit: Alexander Timin altimin@chromium.org
Cr-Original-Commit-Position: refs/branch-heads/4324@{#2040}
Cr-Original-Branched-From: c73b5a651d37a6c4d0b8e3262cc4015a5579c6c8-refs/heads/master@{#827102}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2666397
Reviewed-by: Victor-Gabriel Savu vsavu@google.com
Commit-Queue: Artem Sumaneev asumaneev@google.com
Cr-Commit-Position: refs/branch-heads/4240@{#1537}
Cr-Branched-From: f297677702651916bbf65e59c0d4bbd4ce57d1ee-refs/heads/master@{#800218}

Notes: Security: backported fix for https://crbug.com/1161705.

@nornagon nornagon requested a review from a team as a code owner February 3, 2021 17:52
@nornagon nornagon added 11-x-y backport-check-skip Skip trop's backport validity checking semver/patch backwards-compatible bug fixes labels Feb 3, 2021
@electron-cation electron-cation bot added new-pr 🌱 PR opened in the last 24 hours and removed new-pr 🌱 PR opened in the last 24 hours labels Feb 3, 2021
@nornagon nornagon merged commit 4636dfe into 11-x-y Feb 12, 2021
@release-clerk
Copy link

release-clerk bot commented Feb 12, 2021

Release Notes Persisted

Security: backported fix for https://crbug.com/1161705.

@nornagon nornagon deleted the cherry-pick/11-x-y/chromium/df438f22f7d2 branch February 12, 2021 00:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
11-x-y backport-check-skip Skip trop's backport validity checking semver/patch backwards-compatible bug fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants