[SCM] WebKit Debian packaging branch, debian/experimental, updated. upstream/1.3.3-9427-gc2be6fc
jer.noble at apple.com
jer.noble at apple.com
Wed Dec 22 16:05:26 UTC 2010
The following commit has been merged in the debian/experimental branch:
commit 83186701494bf0a0ae7e234675dcabfa5a603e6e
Author: jer.noble at apple.com <jer.noble at apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date: Thu Nov 18 18:09:00 2010 +0000
2010-11-16 Jer Noble <jer.noble at apple.com>
Reviewed by Eric Carlson.
REGRESSION (Safari 5.0.1): HTML5 videos on YouTube never start playing on Windows
https://bugs.webkit.org/show_bug.cgi?id=44439
<rdar://problem/8342407>
setUpCookiesForQuickTime() previously depended on having setFrameView() called first
in order to get a Frame, and when it wasn't set, the cookie-copy failed. Now,
setUpCookiesForQuickTime uses the MediaPlayerClient function mediaPlayerOwningDocument()
to retrieve the document and subsequently, the Frame.
* platform/graphics/win/MediaPlayerPrivateQuickTimeVisualContext.cpp:
(WebCore::MediaPlayerPrivateQuickTimeVisualContext::setUpCookiesForQuickTime):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@72304 268f45cc-cd09-0410-ab3c-d52691b4dbfc
diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index c6c32c1..cd98c02 100644
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,19 @@
+2010-11-16 Jer Noble <jer.noble at apple.com>
+
+ Reviewed by Eric Carlson.
+
+ REGRESSION (Safari 5.0.1): HTML5 videos on YouTube never start playing on Windows
+ https://bugs.webkit.org/show_bug.cgi?id=44439
+ <rdar://problem/8342407>
+
+ setUpCookiesForQuickTime() previously depended on having setFrameView() called first
+ in order to get a Frame, and when it wasn't set, the cookie-copy failed. Now,
+ setUpCookiesForQuickTime uses the MediaPlayerClient function mediaPlayerOwningDocument()
+ to retrieve the document and subsequently, the Frame.
+
+ * platform/graphics/win/MediaPlayerPrivateQuickTimeVisualContext.cpp:
+ (WebCore::MediaPlayerPrivateQuickTimeVisualContext::setUpCookiesForQuickTime):
+
2010-11-18 Joseph Pecoraro <joepeck at webkit.org>
Reviewed by Darin Adler.
diff --git a/WebCore/platform/graphics/win/MediaPlayerPrivateQuickTimeVisualContext.cpp b/WebCore/platform/graphics/win/MediaPlayerPrivateQuickTimeVisualContext.cpp
index 34a8817..8eaf2a9 100644
--- a/WebCore/platform/graphics/win/MediaPlayerPrivateQuickTimeVisualContext.cpp
+++ b/WebCore/platform/graphics/win/MediaPlayerPrivateQuickTimeVisualContext.cpp
@@ -254,7 +254,8 @@ void MediaPlayerPrivateQuickTimeVisualContext::setUpCookiesForQuickTime(const St
// WebCore loaded the page with the movie URL with CFNetwork but QuickTime will
// use WinINet to download the movie, so we need to copy any cookies needed to
// download the movie into WinInet before asking QuickTime to open it.
- Frame* frame = m_player->frameView() ? m_player->frameView()->frame() : 0;
+ Document* document = m_player->mediaPlayerClient()->mediaPlayerOwningDocument();
+ Frame* frame = document ? document->frame() : 0;
if (!frame || !frame->page() || !frame->page()->cookieEnabled())
return;
--
WebKit Debian packaging
More information about the Pkg-webkit-commits
mailing list