Title: [109249] trunk/Source/WebKit2
Revision
109249
Author
[email protected]
Date
2012-02-29 13:13:16 -0800 (Wed, 29 Feb 2012)

Log Message

CFURLDownloadScheduleWithCurrentMessageQueue only exists on Windows
https://bugs.webkit.org/show_bug.cgi?id=79936

Reviewed by Brady Eidson.

CFURLDownloadScheduleWithCurrentMessageQueue only exists on Windows
platforms. Non-Windows platforms that use CFNetwork-based downloads
should omit this call.

* WebProcess/Downloads/cfnet/DownloadCFNet.cpp:
(WebKit::Download::start):

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (109248 => 109249)


--- trunk/Source/WebKit2/ChangeLog	2012-02-29 21:10:11 UTC (rev 109248)
+++ trunk/Source/WebKit2/ChangeLog	2012-02-29 21:13:16 UTC (rev 109249)
@@ -1,3 +1,17 @@
+2012-02-29  Andy Estes  <[email protected]>
+
+        CFURLDownloadScheduleWithCurrentMessageQueue only exists on Windows
+        https://bugs.webkit.org/show_bug.cgi?id=79936
+
+        Reviewed by Brady Eidson.
+
+        CFURLDownloadScheduleWithCurrentMessageQueue only exists on Windows
+        platforms. Non-Windows platforms that use CFNetwork-based downloads
+        should omit this call.
+
+        * WebProcess/Downloads/cfnet/DownloadCFNet.cpp:
+        (WebKit::Download::start):
+
 2012-02-29  Rafael Brandao  <[email protected]>
 
         [Qt][WK2] We should not add NetscapeBrowserFuncs.cpp as header

Modified: trunk/Source/WebKit2/WebProcess/Downloads/cfnet/DownloadCFNet.cpp (109248 => 109249)


--- trunk/Source/WebKit2/WebProcess/Downloads/cfnet/DownloadCFNet.cpp	2012-02-29 21:10:11 UTC (rev 109248)
+++ trunk/Source/WebKit2/WebProcess/Downloads/cfnet/DownloadCFNet.cpp	2012-02-29 21:13:16 UTC (rev 109249)
@@ -100,7 +100,9 @@
     // FIXME: Allow this to be changed by the client.
     CFURLDownloadSetDeletesUponFailure(m_download.get(), false);
 
+#if OS(WINDOWS)
     CFURLDownloadScheduleWithCurrentMessageQueue(m_download.get());
+#endif
     CFURLDownloadScheduleDownloadWithRunLoop(m_download.get(), loaderRunLoop(), kCFRunLoopDefaultMode);
 
     CFURLDownloadStart(m_download.get());
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to