[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