[Mac] Safari fails to fire page "load" event with video[preload=none]
https://bugs.webkit.org/show_bug.cgi?id=143147
Reviewed by Simon Fraser.
Source/WebCore:
Test: media/video-page-load-preload-none.html
When told to load() when preload == None, set the networkState to Idle.
This causes HTMLMediaElement to set the shouldDelayLoadEvent flag to
false, and allows the page's 'load' event to fire.
Drive-by fix: add setNetworkState() and setReadyState() methods to
MediaPlayerPrivateAVFoundation, reducing a bunch of code duplication.
* platform/graphics/avfoundation/MediaPlayerPrivateAVFoundation.cpp:
(WebCore::MediaPlayerPrivateAVFoundation::load):
(WebCore::MediaPlayerPrivateAVFoundation::setNetworkState):
(WebCore::MediaPlayerPrivateAVFoundation::setReadyState):
(WebCore::MediaPlayerPrivateAVFoundation::updateStates):
* platform/graphics/avfoundation/MediaPlayerPrivateAVFoundation.h:
LayoutTests:
* media/video-page-load-preload-none-expected.txt: Added.
* media/video-page-load-preload-none.html: Added.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@182081 268f45cc-cd09-0410-ab3c-d52691b4dbfc
diff --git a/LayoutTests/media/video-page-load-preload-none.html b/LayoutTests/media/video-page-load-preload-none.html
new file mode 100644
index 0000000..efd4695
--- /dev/null
+++ b/LayoutTests/media/video-page-load-preload-none.html
@@ -0,0 +1,29 @@
+<!doctype html>
+<html>
+ <head>
+ <title>video-page-load-preload-none</title>
+ <script src=video-test.js></script>
+ <script src=media-file.js></script>
+ <script>
+ var video;
+
+ </script>
+ </head>
+ <body>
+ <video id="video" preload="none" poster="content/counting.jpg"><source id="source"></video>
+
+ <p>Test the document's load event is not delayed when a movie's preload attribute is "none".</p>
+
+ <script>
+ consoleWrite("<br><b>inline script</b>");
+
+ video = document.getElementById('video');
+ source = document.getElementById('source');
+ source.src = findMediaFile("video", "content/test");
+
+ waitForEventAndFail('stalled');
+ waitForEventOn(window, 'load', endTest);
+ </script>
+
+ </body>
+</html>
