[MPArch] Remove kIgnoreOpenURLFromInactiveRFH
This CL is a follow-up CL of crrev.com/4016008 and cleans up the
code added to minimize the risk of ignoring OpenURL from
non-active RenderFrameHosts.
Bug: 1350580
Change-Id: Iab46d4ef7fcd3986a4ce8aabf436d28f3aa4d01e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4193930
Reviewed-by: Arthur Sonzogni <arthursonzogni@chromium.org>
Commit-Queue: Miyoung Shin <myid.shin@igalia.com>
Cr-Commit-Position: refs/heads/main@{#1096683}
diff --git a/content/browser/renderer_host/render_frame_host_impl.cc b/content/browser/renderer_host/render_frame_host_impl.cc
index 21f335f30..337aab0 100644
--- a/content/browser/renderer_host/render_frame_host_impl.cc
+++ b/content/browser/renderer_host/render_frame_host_impl.cc
@@ -314,11 +314,6 @@
#endif
);
-// Feature to ignore OpenURL from inactive RFH.
-BASE_FEATURE(kIgnoreOpenURLFromInactiveRFH,
- "IgnoreOpenURLFromInactiveRFH",
- base::FEATURE_ENABLED_BY_DEFAULT);
-
} // namespace features
namespace content {
@@ -7327,12 +7322,7 @@
// Also, see a similar check in RenderFrameHostImpl::BeginNavigation at
// https://source.chromium.org/chromium/chromium/src/+/main:content/browser/renderer_host/render_frame_host_impl.cc;l=7761-7769;drc=6dc39d60fea45c003424272efdb4c366119a9d7f
if (!owner) {
- if (base::FeatureList::IsEnabled(features::kIgnoreOpenURLFromInactiveRFH)) {
- return;
- } else {
- // TODO(mshin): Remove after 2023/Jan/21.
- owner = frame_tree_node_;
- }
+ return;
}
owner->GetCurrentNavigator().RequestOpenURL(
this, validated_url, base::OptionalToPtr(params->initiator_frame_token),
