Title: [188051] trunk/Source/WebCore
Revision
188051
Author
eric.carl...@apple.com
Date
2015-08-06 11:45:01 -0700 (Thu, 06 Aug 2015)

Log Message

Do not enforce "content-disposition: attachment" sandbox restrictions on a MediaDocument
https://bugs.webkit.org/show_bug.cgi?id=147734
rdar://problem/22028179

Reviewed by Dean Jackson.

Test to follow, see https://bugs.webkit.org/show_bug.cgi?id=147735

* dom/Document.cpp:
(WebCore::Document::shouldEnforceContentDispositionAttachmentSandbox): Return
  early if the Document is a MediaDocument.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (188050 => 188051)


--- trunk/Source/WebCore/ChangeLog	2015-08-06 18:35:16 UTC (rev 188050)
+++ trunk/Source/WebCore/ChangeLog	2015-08-06 18:45:01 UTC (rev 188051)
@@ -1,3 +1,17 @@
+2015-08-06  Eric Carlson  <eric.carl...@apple.com>
+
+        Do not enforce "content-disposition: attachment" sandbox restrictions on a MediaDocument
+        https://bugs.webkit.org/show_bug.cgi?id=147734
+        rdar://problem/22028179
+
+        Reviewed by Dean Jackson.
+
+        Test to follow, see https://bugs.webkit.org/show_bug.cgi?id=147735
+
+        * dom/Document.cpp:
+        (WebCore::Document::shouldEnforceContentDispositionAttachmentSandbox): Return
+          early if the Document is a MediaDocument.
+
 2015-08-06  Matt Rajca  <mra...@apple.com>
 
         Media Session: remove media elements from the ID <-> element map on destruction

Modified: trunk/Source/WebCore/dom/Document.cpp (188050 => 188051)


--- trunk/Source/WebCore/dom/Document.cpp	2015-08-06 18:35:16 UTC (rev 188050)
+++ trunk/Source/WebCore/dom/Document.cpp	2015-08-06 18:45:01 UTC (rev 188051)
@@ -6717,6 +6717,9 @@
     if (m_isSynthesized)
         return false;
 
+    if (isMediaDocument())
+        return false;
+
     bool contentDispositionAttachmentSandboxEnabled = settings() && settings()->contentDispositionAttachmentSandboxEnabled();
     bool responseIsAttachment = false;
     if (DocumentLoader* documentLoader = m_frame ? m_frame->loader().activeDocumentLoader() : nullptr)
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to