Title: [235046] releases/WebKitGTK/webkit-2.22/Source/WebCore
Revision
235046
Author
carlo...@webkit.org
Date
2018-08-20 04:11:54 -0700 (Mon, 20 Aug 2018)

Log Message

Merge r234967 - Remove unused allowScriptsToCloseWindows setting
https://bugs.webkit.org/show_bug.cgi?id=188602

Reviewed by Simon Fraser.

No change in behaviour. It was always false.

* page/DOMWindow.cpp:
(WebCore::DOMWindow::close):
* page/Settings.yaml:

Modified Paths

Diff

Modified: releases/WebKitGTK/webkit-2.22/Source/WebCore/ChangeLog (235045 => 235046)


--- releases/WebKitGTK/webkit-2.22/Source/WebCore/ChangeLog	2018-08-20 11:11:49 UTC (rev 235045)
+++ releases/WebKitGTK/webkit-2.22/Source/WebCore/ChangeLog	2018-08-20 11:11:54 UTC (rev 235046)
@@ -1,3 +1,16 @@
+2018-08-16  Alex Christensen  <achristen...@webkit.org>
+
+        Remove unused allowScriptsToCloseWindows setting
+        https://bugs.webkit.org/show_bug.cgi?id=188602
+
+        Reviewed by Simon Fraser.
+
+        No change in behaviour. It was always false.
+
+        * page/DOMWindow.cpp:
+        (WebCore::DOMWindow::close):
+        * page/Settings.yaml:
+
 2018-08-16  Ryosuke Niwa  <rn...@webkit.org>
 
         Custom element constructor doesn't use HTMLElement in new.target's realm

Modified: releases/WebKitGTK/webkit-2.22/Source/WebCore/page/DOMWindow.cpp (235045 => 235046)


--- releases/WebKitGTK/webkit-2.22/Source/WebCore/page/DOMWindow.cpp	2018-08-20 11:11:49 UTC (rev 235045)
+++ releases/WebKitGTK/webkit-2.22/Source/WebCore/page/DOMWindow.cpp	2018-08-20 11:11:54 UTC (rev 235046)
@@ -1086,9 +1086,7 @@
     if (!m_frame->isMainFrame())
         return;
 
-    bool allowScriptsToCloseWindows = m_frame->settings().allowScriptsToCloseWindows();
-
-    if (!(page->openedByDOM() || page->backForward().count() <= 1 || allowScriptsToCloseWindows)) {
+    if (!(page->openedByDOM() || page->backForward().count() <= 1)) {
         console()->addMessage(MessageSource::JS, MessageLevel::Warning, "Can't close the window since it was not opened by _javascript_"_s);
         return;
     }

Modified: releases/WebKitGTK/webkit-2.22/Source/WebCore/page/Settings.yaml (235045 => 235046)


--- releases/WebKitGTK/webkit-2.22/Source/WebCore/page/Settings.yaml	2018-08-20 11:11:49 UTC (rev 235045)
+++ releases/WebKitGTK/webkit-2.22/Source/WebCore/page/Settings.yaml	2018-08-20 11:11:54 UTC (rev 235046)
@@ -192,8 +192,6 @@
   initial: true
 usesEncodingDetector:
   initial: false
-allowScriptsToCloseWindows:
-  initial: false
 canvasUsesAcceleratedDrawing:
   initial: false
 acceleratedDrawingEnabled:
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to