Browse Source

Remove window name on cross origin navigation

Carmelo Messina 3 years ago
parent
commit
f77ac33155
1 changed files with 31 additions and 0 deletions
  1. 31 0
      build/patches/Remove-window-name-on-cross-origin-navigation.patch

+ 31 - 0
build/patches/Remove-window-name-on-cross-origin-navigation.patch

@@ -0,0 +1,31 @@
+From: uazo <uazo@users.noreply.github.com>
+Date: Sat, 9 Jul 2022 06:59:18 +0000
+Subject: Remove window name on cross origin navigation
+
+see also https://trac.webkit.org/changeset/209076/webkit
+---
+ third_party/blink/renderer/core/loader/document_loader.cc | 3 ++-
+ 1 file changed, 2 insertions(+), 1 deletion(-)
+
+diff --git a/third_party/blink/renderer/core/loader/document_loader.cc b/third_party/blink/renderer/core/loader/document_loader.cc
+--- a/third_party/blink/renderer/core/loader/document_loader.cc
++++ b/third_party/blink/renderer/core/loader/document_loader.cc
+@@ -2412,7 +2412,7 @@ void DocumentLoader::CommitNavigation() {
+     // that the name would be nulled and if the name is accessed after we will
+     // fire a UseCounter. If we decide to move forward with this change, we'd
+     // actually clean the name here.
+-    // frame_->tree().setName(g_null_atom);
++    frame_->Tree().SetName(g_null_atom);
+     frame_->Tree().ExperimentalSetNulledName();
+   }
+ 
+@@ -2423,6 +2423,7 @@ void DocumentLoader::CommitNavigation() {
+     // TODO(shuuran): CrossSiteCrossBrowsingContextGroupSetNulledName will just
+     // record the fact that the name would be nulled and if the name is accessed
+     // after we will fire a UseCounter.
++    frame_->Tree().SetName(g_null_atom);
+     frame_->Tree().CrossSiteCrossBrowsingContextGroupSetNulledName();
+   }
+ 
+--
+2.25.1