Diff
Modified: trunk/LayoutTests/ChangeLog (224540 => 224541)
--- trunk/LayoutTests/ChangeLog 2017-11-07 20:01:08 UTC (rev 224540)
+++ trunk/LayoutTests/ChangeLog 2017-11-07 20:02:48 UTC (rev 224541)
@@ -1,3 +1,17 @@
+2017-11-07 Ryan Haddad <ryanhad...@apple.com>
+
+ Unreviewed, rolling out r224498.
+
+ The LayoutTest for this change is flaky and affecting EWS
+ results.
+
+ Reverted changeset:
+
+ "Add tests to ensure that <source> tags are only preloaded
+ when the `type`"
+ https://bugs.webkit.org/show_bug.cgi?id=179231
+ https://trac.webkit.org/changeset/224498
+
2017-11-07 Chris Dumez <cdu...@apple.com>
LayoutTest http/tests/security/cross-frame-access-put.html is a flaky failure
Deleted: trunk/LayoutTests/http/tests/loading/preload-picture-type-expected.txt (224540 => 224541)
--- trunk/LayoutTests/http/tests/loading/preload-picture-type-expected.txt 2017-11-07 20:01:08 UTC (rev 224540)
+++ trunk/LayoutTests/http/tests/loading/preload-picture-type-expected.txt 2017-11-07 20:02:48 UTC (rev 224541)
@@ -1,52 +0,0 @@
-main frame - didStartProvisionalLoadForFrame
-main frame - didCommitLoadForFrame
-main frame - willPerformClientRedirectToURL: http://127.0.0.1:8000/loading/preload-picture-type.html
-main frame - didFinishDocumentLoadForFrame
-main frame - didFinishLoadForFrame
-main frame - didStartProvisionalLoadForFrame
-main frame - didCancelClientRedirectForFrame
-main frame - didCommitLoadForFrame
-main frame - didFinishDocumentLoadForFrame
-main frame - didHandleOnloadEventsForFrame
-main frame - didFinishLoadForFrame
-PASS window.devicePixelRatio is 1
-PASS internals.isPreloaded('resources/base-image1.png?0'); is true
-PASS internals.isPreloaded('resources/preload-test.jpg?0'); is false
-PASS internals.isPreloaded('resources/base-image1.png?1'); is true
-PASS internals.isPreloaded('resources/preload-test.jpg?1'); is false
-PASS internals.isPreloaded('resources/base-image1.png?2'); is true
-PASS internals.isPreloaded('resources/preload-test.jpg?2'); is false
-PASS internals.isPreloaded('resources/base-image1.png?3'); is false
-PASS internals.isPreloaded('resources/preload-test.jpg?3'); is true
-PASS internals.isPreloaded('resources/base-image1.png?4'); is false
-PASS internals.isPreloaded('resources/base-image2.png?4'); is false
-PASS internals.isPreloaded('resources/base-image3.png?4'); is true
-PASS internals.isPreloaded('resources/preload-test.jpg?4'); is false
-PASS internals.isPreloaded('resources/base-image1.png?5'); is false
-PASS internals.isPreloaded('resources/base-image2.png?5'); is false
-PASS internals.isPreloaded('resources/base-image3.png?5'); is false
-PASS internals.isPreloaded('resources/preload-test.jpg?5'); is true
-PASS internals.isPreloaded('resources/base-image1.png?6'); is true
-PASS internals.isPreloaded('resources/preload-test.jpg?6'); is false
-PASS internals.isPreloaded('resources/base-image1.png?7'); is false
-PASS internals.isPreloaded('resources/preload-test.jpg?7'); is true
-PASS internals.isPreloaded('resources/base-image1.png?8'); is true
-PASS internals.isPreloaded('resources/preload-test.jpg?8'); is false
-PASS internals.isPreloaded('resources/base-image1.png?9'); is false
-PASS internals.isPreloaded('resources/preload-test.jpg?9'); is true
-PASS internals.isPreloaded('resources/base-image1.png?10'); is true
-PASS internals.isPreloaded('resources/base-image2.png?10'); is false
-PASS internals.isPreloaded('resources/preload-test.jpg?10'); is false
-PASS internals.isPreloaded('resources/base-image1.png?11'); is false
-PASS internals.isPreloaded('resources/base-image2.png?11'); is true
-PASS internals.isPreloaded('resources/preload-test.jpg?11'); is false
-PASS internals.isPreloaded('resources/base-image1.png?12'); is true
-PASS internals.isPreloaded('resources/base-image2.png?12'); is false
-PASS internals.isPreloaded('resources/preload-test.jpg?12'); is false
-PASS internals.isPreloaded('resources/base-image1.png?13'); is false
-PASS internals.isPreloaded('resources/base-image2.png?13'); is false
-PASS internals.isPreloaded('resources/preload-test.jpg?13'); is true
-PASS successfullyParsed is true
-
-TEST COMPLETE
-
Deleted: trunk/LayoutTests/http/tests/loading/preload-picture-type.html (224540 => 224541)
--- trunk/LayoutTests/http/tests/loading/preload-picture-type.html 2017-11-07 20:01:08 UTC (rev 224540)
+++ trunk/LayoutTests/http/tests/loading/preload-picture-type.html 2017-11-07 20:02:48 UTC (rev 224541)
@@ -1,137 +0,0 @@
-<html>
-<head>
- <script>
- testRunner.waitUntilDone();
- testRunner.setBackingScaleFactor(1);
- if (!sessionStorage.pageReloaded) {
- sessionStorage.pageReloaded = true
- // have to force a restart because of a bug in dynamic changes with srcset
- // we have to force the restart even if DPR=1 for consistency
- document.location.reload(true);
- }
- </script>
- <script src=""
- <script src=""
-</head>
-<body>
-<script>
- if (window.testRunner)
- testRunner.dumpAsText();
-
- shouldBe("window.devicePixelRatio", "1");
- shouldBeTrue("internals.isPreloaded('resources/base-image1.png?0');");
- shouldBeFalse("internals.isPreloaded('resources/preload-test.jpg?0');");
- shouldBeTrue("internals.isPreloaded('resources/base-image1.png?1');");
- shouldBeFalse("internals.isPreloaded('resources/preload-test.jpg?1');");
- shouldBeTrue("internals.isPreloaded('resources/base-image1.png?2');");
- shouldBeFalse("internals.isPreloaded('resources/preload-test.jpg?2');");
- shouldBeFalse("internals.isPreloaded('resources/base-image1.png?3');");
- shouldBeTrue("internals.isPreloaded('resources/preload-test.jpg?3');");
-
- shouldBeFalse("internals.isPreloaded('resources/base-image1.png?4');");
- shouldBeFalse("internals.isPreloaded('resources/base-image2.png?4');");
- shouldBeTrue("internals.isPreloaded('resources/base-image3.png?4');");
- shouldBeFalse("internals.isPreloaded('resources/preload-test.jpg?4');");
-
- shouldBeFalse("internals.isPreloaded('resources/base-image1.png?5');");
- shouldBeFalse("internals.isPreloaded('resources/base-image2.png?5');");
- shouldBeFalse("internals.isPreloaded('resources/base-image3.png?5');");
- shouldBeTrue("internals.isPreloaded('resources/preload-test.jpg?5');");
-
- shouldBeTrue("internals.isPreloaded('resources/base-image1.png?6');");
- shouldBeFalse("internals.isPreloaded('resources/preload-test.jpg?6');");
- shouldBeFalse("internals.isPreloaded('resources/base-image1.png?7');");
- shouldBeTrue("internals.isPreloaded('resources/preload-test.jpg?7');");
-
- shouldBeTrue("internals.isPreloaded('resources/base-image1.png?8');");
- shouldBeFalse("internals.isPreloaded('resources/preload-test.jpg?8');");
-
- shouldBeFalse("internals.isPreloaded('resources/base-image1.png?9');");
- shouldBeTrue("internals.isPreloaded('resources/preload-test.jpg?9');");
-
- shouldBeTrue("internals.isPreloaded('resources/base-image1.png?10');");
- shouldBeFalse("internals.isPreloaded('resources/base-image2.png?10');");
- shouldBeFalse("internals.isPreloaded('resources/preload-test.jpg?10');");
-
- shouldBeFalse("internals.isPreloaded('resources/base-image1.png?11');");
- shouldBeTrue("internals.isPreloaded('resources/base-image2.png?11');");
- shouldBeFalse("internals.isPreloaded('resources/preload-test.jpg?11');");
-
- shouldBeTrue("internals.isPreloaded('resources/base-image1.png?12');");
- shouldBeFalse("internals.isPreloaded('resources/base-image2.png?12');");
- shouldBeFalse("internals.isPreloaded('resources/preload-test.jpg?12');");
-
- shouldBeFalse("internals.isPreloaded('resources/base-image1.png?13');");
- shouldBeFalse("internals.isPreloaded('resources/base-image2.png?13');");
- shouldBeTrue("internals.isPreloaded('resources/preload-test.jpg?13');");
- testRunner.notifyDone();
-</script>
-<!-- Control group -->
-<picture>
- <source srcset="resources/base-image1.png?0">
- <img src=""
-</picture>
-<!-- All permutations of type, media, srcset and sizes -->
-
-<picture>
- <source type="image/png" srcset="resources/base-image1.png?1">
- <img src=""
-</picture>
-<picture>
- <source srcset="resources/base-image1.png?2" type="image/png">
- <img src=""
-</picture>
-<picture>
- <source type="image/bad" srcset="resources/base-image1.png?3">
- <img src=""
-</picture>
-<picture>
- <source type="image/png" sizes="400px" srcset="resources/base-image1.png?4 200w, resources/base-image3.png?4 400w, resources/base-image2.png?4 800w">
- <img src=""
-</picture>
-<picture>
- <source type="image/bad" sizes="400px" srcset="resources/base-image1.png?5 200w, resources/base-image3.png?5 400w, resources/base-image2.png?5 800w" >
- <img src=""
-</picture>
-<picture>
- <source type="image/png" media="(min-width: 1px)" srcset="resources/base-image1.png?6">
- <img src=""
-</picture>
-<picture>
- <source type="image/bad" media="(min-width: 1px)" srcset="resources/base-image1.png?7">
- <img src=""
-</picture>
-
-<!-- Duplicate attributes -->
-<picture>
- <source type="image/png" type="image/bad" srcset="resources/base-image1.png?8">
- <img src=""
-</picture>
-<picture>
- <source type="image/bad" type="image/png" srcset="resources/base-image1.png?9">
- <img src=""
-</picture>
-<!-- Multiple sources -->
-<picture>
- <source type="image/png" media="(min-width: 1px)" srcset="resources/base-image1.png?10">
- <source type="image/png" media="(min-width: 1px)" srcset="resources/base-image2.png?10">
- <img src=""
-</picture>
-<picture>
- <source type="image/bad" media="(min-width: 1px)" srcset="resources/base-image1.png?11">
- <source type="image/png" media="(min-width: 1px)" srcset="resources/base-image2.png?11">
- <img src=""
-</picture>
-<picture>
- <source type="image/png" media="(min-width: 1px)" srcset="resources/base-image1.png?12">
- <source type="image/bad" media="(min-width: 1px)" srcset="resources/base-image2.png?12">
- <img src=""
-</picture>
-<picture>
- <source type="image/bad" media="(min-width: 1px)" srcset="resources/base-image1.png?13">
- <source type="image/invalid" media="(min-width: 1px)" srcset="resources/base-image2.png?13">
- <img src=""
-</picture>
-
-</body>
-</html>
Modified: trunk/Source/WebCore/ChangeLog (224540 => 224541)
--- trunk/Source/WebCore/ChangeLog 2017-11-07 20:01:08 UTC (rev 224540)
+++ trunk/Source/WebCore/ChangeLog 2017-11-07 20:02:48 UTC (rev 224541)
@@ -1,3 +1,17 @@
+2017-11-07 Ryan Haddad <ryanhad...@apple.com>
+
+ Unreviewed, rolling out r224498.
+
+ The LayoutTest for this change is flaky and affecting EWS
+ results.
+
+ Reverted changeset:
+
+ "Add tests to ensure that <source> tags are only preloaded
+ when the `type`"
+ https://bugs.webkit.org/show_bug.cgi?id=179231
+ https://trac.webkit.org/changeset/224498
+
2017-11-07 Joseph Pecoraro <pecor...@apple.com>
Web Inspector: Add some fast returns in cases where we only call through to a NetworkAgent
Modified: trunk/Source/WebCore/html/parser/HTMLPreloadScanner.cpp (224540 => 224541)
--- trunk/Source/WebCore/html/parser/HTMLPreloadScanner.cpp 2017-11-07 20:01:08 UTC (rev 224540)
+++ trunk/Source/WebCore/html/parser/HTMLPreloadScanner.cpp 2017-11-07 20:02:48 UTC (rev 224541)
@@ -36,7 +36,6 @@
#include "LinkLoader.h"
#include "LinkRelAttribute.h"
#include "Logging.h"
-#include "MIMETypeRegistry.h"
#include "MediaList.h"
#include "MediaQueryEvaluator.h"
#include "RenderView.h"
@@ -122,7 +121,7 @@
processAttribute(attributeName, attributeValue, document, pictureState);
}
- if (m_tagId == TagId::Source && !pictureState.isEmpty() && !pictureState.last() && m_mediaMatched && m_typeMatched && !m_srcSetAttribute.isEmpty()) {
+ if (m_tagId == TagId::Source && !pictureState.isEmpty() && !pictureState.last() && m_mediaMatched && !m_srcSetAttribute.isEmpty()) {
auto sourceSize = SizesAttributeParser(m_sizesAttribute, document).length();
ImageCandidate imageCandidate = bestFitSourceForImageAttributes(m_deviceScaleFactor, m_urlToLoad, m_srcSetAttribute, sourceSize);
@@ -220,11 +219,6 @@
LOG(MediaQueries, "HTMLPreloadScanner %p processAttribute evaluating media queries", this);
m_mediaMatched = MediaQueryEvaluator { document.printing() ? "print" : "screen", document, documentElement ? documentElement->computedStyle() : nullptr }.evaluate(mediaSet.get());
}
- if (match(attributeName, typeAttr) && m_typeAttribute.isNull()) {
- // when multiple type attributes present: first value wins, ignore subsequent (to match ImageElement parser and Blink behaviours)
- m_typeAttribute = attributeValue;
- m_typeMatched &= MIMETypeRegistry::isSupportedImageOrSVGMIMEType(m_typeAttribute);
- }
break;
case TagId::Script:
if (match(attributeName, typeAttr)) {
@@ -347,7 +341,6 @@
String m_srcSetAttribute;
String m_sizesAttribute;
bool m_mediaMatched { true };
- bool m_typeMatched { true };
String m_charset;
String m_crossOriginMode;
bool m_linkIsStyleSheet;