Move the parts of RunLoopMac.mm that aren't Mac-specific into RunLoopCF.cpp
authoraestes@apple.com <aestes@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 15 May 2012 00:05:12 +0000 (00:05 +0000)
committeraestes@apple.com <aestes@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 15 May 2012 00:05:12 +0000 (00:05 +0000)
https://bugs.webkit.org/show_bug.cgi?id=86411

Reviewed by Anders Carlsson.

Most of RunLoopMac.mm was written in terms of CF API. Move these bits
to a new file called RunLoopCF.cpp, leaving only the bits dealing with
NSApplication.

* WebCore.xcodeproj/project.pbxproj: Xcode decided to give
VisitedLinks.* new UUIDs. I'll allow it.
* platform/cf/RunLoopCF.cpp: Copied from Source/WebCore/platform/mac/RunLoopMac.mm.
* platform/mac/RunLoopMac.mm:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@117018 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/WebCore.xcodeproj/project.pbxproj
Source/WebCore/platform/cf/RunLoopCF.cpp [new file with mode: 0644]
Source/WebCore/platform/mac/RunLoopMac.mm

index 7139478..d804646 100644 (file)
@@ -1,3 +1,19 @@
+2012-05-14  Andy Estes  <aestes@apple.com>
+
+        Move the parts of RunLoopMac.mm that aren't Mac-specific into RunLoopCF.cpp
+        https://bugs.webkit.org/show_bug.cgi?id=86411
+
+        Reviewed by Anders Carlsson.
+
+        Most of RunLoopMac.mm was written in terms of CF API. Move these bits
+        to a new file called RunLoopCF.cpp, leaving only the bits dealing with
+        NSApplication.
+
+        * WebCore.xcodeproj/project.pbxproj: Xcode decided to give
+        VisitedLinks.* new UUIDs. I'll allow it.
+        * platform/cf/RunLoopCF.cpp: Copied from Source/WebCore/platform/mac/RunLoopMac.mm.
+        * platform/mac/RunLoopMac.mm:
+
 2012-05-14  Erik Arvidsson  <arv@chromium.org>
 
         [V8] Add "stack" property to DOMException
index 44df1f5..5270ff5 100644 (file)
                293EAE1F1356B2FE0067ACF9 /* RuntimeApplicationChecks.h in Headers */ = {isa = PBXBuildFile; fileRef = 293EAE1E1356B2FE0067ACF9 /* RuntimeApplicationChecks.h */; settings = {ATTRIBUTES = (Private, ); }; };
                293EAE211356B32E0067ACF9 /* RuntimeApplicationChecks.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 293EAE201356B32E0067ACF9 /* RuntimeApplicationChecks.cpp */; };
                29489FC712C00F0300D83F0F /* AccessibilityScrollView.h in Headers */ = {isa = PBXBuildFile; fileRef = 29489FC512C00F0300D83F0F /* AccessibilityScrollView.h */; };
+               29A309AC1561C99B00BBD8E5 /* RunLoopCF.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 29A309AB1561C99B00BBD8E5 /* RunLoopCF.cpp */; };
+               29A309B01561CAC400BBD8E5 /* VisitedLinks.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 29A309AE1561CAC400BBD8E5 /* VisitedLinks.cpp */; };
+               29A309B11561CAC400BBD8E5 /* VisitedLinks.h in Headers */ = {isa = PBXBuildFile; fileRef = 29A309AF1561CAC400BBD8E5 /* VisitedLinks.h */; };
                29A8122B0FBB9C1D00510293 /* AccessibilityTable.h in Headers */ = {isa = PBXBuildFile; fileRef = 29A8120D0FBB9C1D00510293 /* AccessibilityTable.h */; };
                29A8122C0FBB9C1D00510293 /* AccessibilityList.h in Headers */ = {isa = PBXBuildFile; fileRef = 29A8120E0FBB9C1D00510293 /* AccessibilityList.h */; };
                29A8122E0FBB9C1D00510293 /* AccessibilityARIAGridCell.h in Headers */ = {isa = PBXBuildFile; fileRef = 29A812100FBB9C1D00510293 /* AccessibilityARIAGridCell.h */; };
                2981CAAD131822EC00D12F2A /* AccessibilityTableHeaderContainer.cpp */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.cpp.cpp; path = AccessibilityTableHeaderContainer.cpp; sourceTree = "<group>"; };
                2981CAAE131822EC00D12F2A /* AccessibilityTableRow.cpp */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.cpp.cpp; path = AccessibilityTableRow.cpp; sourceTree = "<group>"; };
                2981CAAF131822EC00D12F2A /* AXObjectCache.cpp */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.cpp.cpp; path = AXObjectCache.cpp; sourceTree = "<group>"; };
+               29A309AB1561C99B00BBD8E5 /* RunLoopCF.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = RunLoopCF.cpp; sourceTree = "<group>"; };
+               29A309AE1561CAC400BBD8E5 /* VisitedLinks.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = VisitedLinks.cpp; sourceTree = "<group>"; };
+               29A309AF1561CAC400BBD8E5 /* VisitedLinks.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = VisitedLinks.h; sourceTree = "<group>"; };
                29A8120D0FBB9C1D00510293 /* AccessibilityTable.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = AccessibilityTable.h; sourceTree = "<group>"; };
                29A8120E0FBB9C1D00510293 /* AccessibilityList.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = AccessibilityList.h; sourceTree = "<group>"; };
                29A812100FBB9C1D00510293 /* AccessibilityARIAGridCell.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = AccessibilityARIAGridCell.h; sourceTree = "<group>"; };
                                938192040F87E1EC00D5352A /* BinaryPropertyList.h */,
                                5160306B0CC4362300C8AC25 /* FileSystemCF.cpp */,
                                1A98956A0AA78F80005EF5EF /* KURLCFNet.cpp */,
+                               29A309AB1561C99B00BBD8E5 /* RunLoopCF.cpp */,
                                1C63A2470F71646600C09D5A /* RunLoopTimerCF.cpp */,
                                1CEFC9B80D78DC8C007D2579 /* SchedulePair.cpp */,
                                1CEFC9B70D78DC8C007D2579 /* SchedulePair.h */,
                                2E3BBF051162DA1100B9409A /* UUID.cpp */,
                                2E3BBF061162DA1100B9409A /* UUID.h */,
                                515788C01207852C00A37C4A /* VisitedLinkStrategy.h */,
-                               D6FDAEF2149C06190037B1E5 /* VisitedLinks.cpp */,
-                               D630E2AB149BF344005B2F97 /* VisitedLinks.h */,
+                               29A309AE1561CAC400BBD8E5 /* VisitedLinks.cpp */,
+                               29A309AF1561CAC400BBD8E5 /* VisitedLinks.h */,
                                9380F47109A11AB4001FDB34 /* Widget.cpp */,
                                9380F47209A11AB4001FDB34 /* Widget.h */,
                                E1E1BEFF115FF6FB006F52CA /* WindowsKeyboardCodes.h */,
                                93309E20099E64920056E581 /* VisiblePosition.h in Headers */,
                                A883DF280F3D045D00F19BF6 /* VisibleSelection.h in Headers */,
                                515788C11207852C00A37C4A /* VisitedLinkStrategy.h in Headers */,
-                               D630E2AC149BF344005B2F97 /* VisitedLinks.h in Headers */,
                                E44613B60CD6344E00FADA75 /* VoidCallback.h in Headers */,
                                FD7F299213D4C0CB00AD9535 /* WaveShaperDSPKernel.h in Headers */,
                                FD7F299413D4C0CB00AD9535 /* WaveShaperNode.h in Headers */,
                                31FE6DFA15004C2A0004EBC4 /* NotificationPermissionCallback.h in Headers */,
                                B658FFA21522EF3A00DD5595 /* JSRadioNodeList.h in Headers */,
                                B658FFA61522EFAA00DD5595 /* RadioNodeList.h in Headers */,
+                               29A309B11561CAC400BBD8E5 /* VisitedLinks.h in Headers */,
                        );
                        runOnlyForDeploymentPostprocessing = 0;
                };
                                93309E1D099E64920056E581 /* visible_units.cpp in Sources */,
                                93309E1F099E64920056E581 /* VisiblePosition.cpp in Sources */,
                                A883DF270F3D045D00F19BF6 /* VisibleSelection.cpp in Sources */,
-                               D6FDAEF3149C06190037B1E5 /* VisitedLinks.cpp in Sources */,
                                FD7F299113D4C0CB00AD9535 /* WaveShaperDSPKernel.cpp in Sources */,
                                FD7F299313D4C0CB00AD9535 /* WaveShaperNode.cpp in Sources */,
                                FD7F299613D4C0CB00AD9535 /* WaveShaperProcessor.cpp in Sources */,
                                31FE6E6D1501A3BF0004EBC4 /* JSNotificationCustom.cpp in Sources */,
                                B658FFA11522EF3A00DD5595 /* JSRadioNodeList.cpp in Sources */,
                                B658FFA51522EFAA00DD5595 /* RadioNodeList.cpp in Sources */,
+                               29A309AC1561C99B00BBD8E5 /* RunLoopCF.cpp in Sources */,
+                               29A309B01561CAC400BBD8E5 /* VisitedLinks.cpp in Sources */,
                        );
                        runOnlyForDeploymentPostprocessing = 0;
                };
diff --git a/Source/WebCore/platform/cf/RunLoopCF.cpp b/Source/WebCore/platform/cf/RunLoopCF.cpp
new file mode 100644 (file)
index 0000000..5f4e38e
--- /dev/null
@@ -0,0 +1,179 @@
+/*
+ * Copyright (C) 2012 Apple Inc. All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ *    notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ *    notice, this list of conditions and the following disclaimer in the
+ *    documentation and/or other materials provided with the distribution.
+ *
+ * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
+ * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
+ * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
+ * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
+ * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
+ * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
+ * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
+ * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
+ * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
+ * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
+ * THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#include "config.h"
+#include "RunLoop.h"
+
+#include "AutodrainedPool.h"
+#include <CoreFoundation/CoreFoundation.h>
+#include <dispatch/dispatch.h>
+
+namespace WebCore {
+    
+static RunLoop* s_mainRunLoop;
+
+void RunLoop::initializeMainRunLoop()
+{
+    static dispatch_once_t onceToken;
+    dispatch_once(&onceToken, ^{
+        s_mainRunLoop = new RunLoop(CFRunLoopGetMain());
+    });
+}
+
+RunLoop* RunLoop::current()
+{
+    if (pthread_main_np())
+        return RunLoop::main();
+    
+    DEFINE_STATIC_LOCAL(WTF::ThreadSpecific<RunLoop>, runLoopData, ());
+    return &*runLoopData;
+}
+
+RunLoop* RunLoop::main()
+{
+    ASSERT(s_mainRunLoop);
+    return s_mainRunLoop;
+}
+
+void RunLoop::performWork(void* context)
+{
+    // Wrap main thread in an Autorelease pool. Sending messages can call 
+    // into objc code and accumulate memory.  
+    if (current() == main()) {
+        AutodrainedPool pool;
+        static_cast<RunLoop*>(context)->performWork();
+    } else
+        static_cast<RunLoop*>(context)->performWork();
+}
+
+RunLoop::RunLoop()
+    : m_runLoop(CFRunLoopGetCurrent())
+    , m_nestingLevel(0)
+{
+    CFRunLoopSourceContext context = { 0, this, 0, 0, 0, 0, 0, 0, 0, performWork };
+    m_runLoopSource = CFRunLoopSourceCreate(kCFAllocatorDefault, 0, &context);
+    CFRunLoopAddSource(m_runLoop, m_runLoopSource, kCFRunLoopCommonModes);
+}
+
+RunLoop::RunLoop(CFRunLoopRef runLoop)
+    : m_runLoop(runLoop)
+    , m_nestingLevel(0)
+{
+    CFRunLoopSourceContext context = { 0, this, 0, 0, 0, 0, 0, 0, 0, performWork };
+    m_runLoopSource = CFRunLoopSourceCreate(kCFAllocatorDefault, 0, &context);
+    CFRunLoopAddSource(m_runLoop, m_runLoopSource, kCFRunLoopCommonModes);
+}
+
+RunLoop::~RunLoop()
+{
+    // FIXME: Tear down the work item queue here.
+    CFRunLoopSourceInvalidate(m_runLoopSource);
+    CFRelease(m_runLoopSource);
+}
+
+void RunLoop::runForDuration(double duration)
+{
+    CFRunLoopRunInMode(kCFRunLoopDefaultMode, duration, true);
+}
+
+void RunLoop::wakeUp()
+{
+    CFRunLoopSourceSignal(m_runLoopSource);
+    CFRunLoopWakeUp(m_runLoop);
+}
+
+#if !PLATFORM(MAC) || PLATFORM(IOS)
+void RunLoop::run()
+{
+    current()->m_nestingLevel++;
+    
+    {
+        AutodrainedPool pool;
+        CFRunLoopRun();
+    }
+    
+    current()->m_nestingLevel--;
+}
+
+void RunLoop::stop()
+{
+    ASSERT(m_runLoop == CFRunLoopGetCurrent());
+    CFRunLoopStop(m_runLoop);
+}
+#endif
+
+// RunLoop::Timer
+
+void RunLoop::TimerBase::timerFired(CFRunLoopTimerRef, void* context)
+{
+    TimerBase* timer = static_cast<TimerBase*>(context);
+    
+    // Wrap main thread in an Autorelease pool. The timer can call 
+    // into objc code and accumulate memory outside of the main event loop.
+    if (current() == main()) {
+        AutodrainedPool pool;
+        timer->fired();
+    } else
+        timer->fired();
+}
+
+RunLoop::TimerBase::TimerBase(RunLoop* runLoop)
+    : m_runLoop(runLoop)
+    , m_timer(0)
+{
+}
+
+RunLoop::TimerBase::~TimerBase()
+{
+    stop();
+}
+
+void RunLoop::TimerBase::start(double nextFireInterval, bool repeat)
+{
+    if (m_timer)
+        stop();
+    
+    CFRunLoopTimerContext context = { 0, this, 0, 0, 0 };
+    CFTimeInterval repeatInterval = repeat ? nextFireInterval : 0;
+    m_timer = CFRunLoopTimerCreate(kCFAllocatorDefault, CFAbsoluteTimeGetCurrent() + nextFireInterval, repeatInterval, 0, 0, timerFired, &context);
+    CFRunLoopAddTimer(m_runLoop->m_runLoop, m_timer, kCFRunLoopCommonModes);
+}
+
+void RunLoop::TimerBase::stop()
+{
+    if (!m_timer)
+        return;
+    
+    CFRunLoopTimerInvalidate(m_timer);
+    CFRelease(m_timer);
+    m_timer = 0;
+}
+
+bool RunLoop::TimerBase::isActive() const
+{
+    return m_timer && CFRunLoopTimerIsValid(m_timer);
+}
+
+} // namespace WebCore
index 717dd8d..ab2c9a0 100644 (file)
 #import "config.h"
 #import "RunLoop.h"
 
-#import <dispatch/dispatch.h>
-
 namespace WebCore {
 
-static RunLoop* s_mainRunLoop;
-
-void RunLoop::initializeMainRunLoop()
-{
-    static dispatch_once_t onceToken;
-    dispatch_once(&onceToken, ^{
-        s_mainRunLoop = new RunLoop(CFRunLoopGetMain());
-    });
-}
-
-RunLoop* RunLoop::current()
-{
-    if (pthread_main_np())
-        return RunLoop::main();
-
-    DEFINE_STATIC_LOCAL(WTF::ThreadSpecific<RunLoop>, runLoopData, ());
-    return &*runLoopData;
-}
-
-RunLoop* RunLoop::main()
-{
-    ASSERT(s_mainRunLoop);
-    return s_mainRunLoop;
-}
-
-void RunLoop::performWork(void* context)
-{
-    // Wrap main thread in an Autorelease pool.  Sending messages can call 
-    // into objc code and accumulate memory.  
-    if (current() == main()) {
-        NSAutoreleasePool *pool = [[NSAutoreleasePool alloc] init];
-        static_cast<RunLoop*>(context)->performWork();
-        [pool drain];
-    } else
-        static_cast<RunLoop*>(context)->performWork();
-}
-
-RunLoop::RunLoop()
-    : m_runLoop(CFRunLoopGetCurrent())
-    , m_nestingLevel(0)
-{
-    CFRunLoopSourceContext context = { 0, this, 0, 0, 0, 0, 0, 0, 0, performWork };
-    m_runLoopSource = CFRunLoopSourceCreate(kCFAllocatorDefault, 0, &context);
-    CFRunLoopAddSource(m_runLoop, m_runLoopSource, kCFRunLoopCommonModes);
-}
-
-RunLoop::RunLoop(CFRunLoopRef runLoop)
-    : m_runLoop(runLoop)
-    , m_nestingLevel(0)
-{
-    CFRunLoopSourceContext context = { 0, this, 0, 0, 0, 0, 0, 0, 0, performWork };
-    m_runLoopSource = CFRunLoopSourceCreate(kCFAllocatorDefault, 0, &context);
-    CFRunLoopAddSource(m_runLoop, m_runLoopSource, kCFRunLoopCommonModes);
-}
-
-RunLoop::~RunLoop()
-{
-    // FIXME: Tear down the work item queue here.
-    CFRunLoopSourceInvalidate(m_runLoopSource);
-    CFRelease(m_runLoopSource);
-}
-
 void RunLoop::run()
 {
     current()->m_nestingLevel++;
@@ -106,11 +42,6 @@ void RunLoop::run()
     current()->m_nestingLevel--;
 }
 
-void RunLoop::runForDuration(double duration)
-{
-    CFRunLoopRunInMode(kCFRunLoopDefaultMode, duration, true);
-}
-
 void RunLoop::stop()
 {
     ASSERT(m_runLoop == CFRunLoopGetCurrent());
@@ -131,63 +62,4 @@ void RunLoop::stop()
         CFRunLoopStop(m_runLoop);
 }
 
-void RunLoop::wakeUp()
-{
-    CFRunLoopSourceSignal(m_runLoopSource);
-    CFRunLoopWakeUp(m_runLoop);
-}
-
-// RunLoop::Timer
-
-void RunLoop::TimerBase::timerFired(CFRunLoopTimerRef, void* context)
-{
-    TimerBase* timer = static_cast<TimerBase*>(context);
-    
-    // Wrap main thread in an Autorelease pool.  The timer can call 
-    // into objc code and accumulate memory outside of the main event loop.
-    if (current() == main()) {
-        NSAutoreleasePool *pool = [[NSAutoreleasePool alloc] init]; 
-        timer->fired();
-        [pool drain];
-    } else
-        timer->fired();
-}
-
-RunLoop::TimerBase::TimerBase(RunLoop* runLoop)
-    : m_runLoop(runLoop)
-    , m_timer(0)
-{
-}
-
-RunLoop::TimerBase::~TimerBase()
-{
-    stop();
-}
-
-void RunLoop::TimerBase::start(double nextFireInterval, bool repeat)
-{
-    if (m_timer)
-        stop();
-
-    CFRunLoopTimerContext context = { 0, this, 0, 0, 0 };
-    CFTimeInterval repeatInterval = repeat ? nextFireInterval : 0;
-    m_timer = CFRunLoopTimerCreate(kCFAllocatorDefault, CFAbsoluteTimeGetCurrent() + nextFireInterval, repeatInterval, 0, 0, timerFired, &context);
-    CFRunLoopAddTimer(m_runLoop->m_runLoop, m_timer, kCFRunLoopCommonModes);
-}
-
-void RunLoop::TimerBase::stop()
-{
-    if (!m_timer)
-        return;
-
-    CFRunLoopTimerInvalidate(m_timer);
-    CFRelease(m_timer);
-    m_timer = 0;
-}
-
-bool RunLoop::TimerBase::isActive() const
-{
-    return m_timer && CFRunLoopTimerIsValid(m_timer);
-}
-
 } // namespace WebCore