Try to fix the Windows build.
authorbeidson@apple.com <beidson@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 8 Jun 2016 17:46:06 +0000 (17:46 +0000)
committerbeidson@apple.com <beidson@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 8 Jun 2016 17:46:06 +0000 (17:46 +0000)
Unreviewed.

* TestWebKitAPI/Tests/WTF/CrossThreadTask.cpp:
(TestWebKitAPI::LifetimeLogger::log):
(TestWebKitAPI::LifetimeLogger::takeLogStr):
(TestWebKitAPI::testFunction):
(TestWebKitAPI::TEST):
(TestWebKitAPI::log): Deleted. (VS was seeing it as a dupe symbol)
(TestWebKitAPI::takeLogStr): Deleted. (VS was seeing it as a dupe symbol)

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

Tools/ChangeLog
Tools/TestWebKitAPI/Tests/WTF/CrossThreadTask.cpp

index 1e1a0fb..639803d 100644 (file)
@@ -1,5 +1,19 @@
 2016-06-08  Brady Eidson  <beidson@apple.com>
 
+        Try to fix the Windows build.
+
+        Unreviewed.
+
+        * TestWebKitAPI/Tests/WTF/CrossThreadTask.cpp:
+        (TestWebKitAPI::LifetimeLogger::log):
+        (TestWebKitAPI::LifetimeLogger::takeLogStr):
+        (TestWebKitAPI::testFunction):
+        (TestWebKitAPI::TEST):
+        (TestWebKitAPI::log): Deleted. (VS was seeing it as a dupe symbol)
+        (TestWebKitAPI::takeLogStr): Deleted. (VS was seeing it as a dupe symbol)
+
+2016-06-08  Brady Eidson  <beidson@apple.com>
+
         Make CrossThreadCopier more efficient (fewer copies!).
         https://bugs.webkit.org/show_bug.cgi?id=158456
 
index 8c646ac..89952e9 100644 (file)
 
 namespace TestWebKitAPI {
 
-inline std::ostringstream& log()
-{
-    static std::ostringstream log;
-    return log;
-}
-
-inline std::string takeLogStr()
-{
-    std::string string = log().str();
-    log().str("");
-    return string;
-}
-
 struct LifetimeLogger {
     LifetimeLogger()
     {
@@ -85,11 +72,25 @@ struct LifetimeLogger {
     const char& name { *"<default>" };
     int copyGeneration { 0 };
     int moveGeneration { 0 };
+
+    static std::ostringstream& log()
+    {
+        static std::ostringstream log;
+        return log;
+    }
+
+    static std::string takeLogStr()
+    {
+        std::string string = log().str();
+        log().str("");
+        return string;
+    }
+
 };
 
 void testFunction(const LifetimeLogger&, const LifetimeLogger&, const LifetimeLogger&)
 {
-    log() << "testFunction called" << " ";
+    LifetimeLogger::log() << "testFunction called" << " ";
 }
 
 TEST(WTF_CrossThreadTask, Basic)
@@ -102,7 +103,7 @@ TEST(WTF_CrossThreadTask, Basic)
         auto task = createCrossThreadTask(testFunction, logger1, logger2, logger3);
         task.performTask();
     }
-    ASSERT_STREQ("default_constructor(<default>-0-0) copy_constructor(<default>-1-0) name_constructor(logger-0-0) isolatedCopy() copy_constructor(<default>-1-0) isolatedCopy() copy_constructor(<default>-2-0) isolatedCopy() copy_constructor(logger-1-0) move_constructor(<default>-1-1) move_constructor(<default>-2-1) move_constructor(logger-1-1) destructor(logger-1-0) destructor(<default>-2-0) destructor(<default>-1-0) testFunction called destructor(logger-1-1) destructor(<default>-2-1) destructor(<default>-1-1) destructor(logger-0-0) destructor(<default>-1-0) destructor(<default>-0-0) ", takeLogStr().c_str());
+    ASSERT_STREQ("default_constructor(<default>-0-0) copy_constructor(<default>-1-0) name_constructor(logger-0-0) isolatedCopy() copy_constructor(<default>-1-0) isolatedCopy() copy_constructor(<default>-2-0) isolatedCopy() copy_constructor(logger-1-0) move_constructor(<default>-1-1) move_constructor(<default>-2-1) move_constructor(logger-1-1) destructor(logger-1-0) destructor(<default>-2-0) destructor(<default>-1-0) testFunction called destructor(logger-1-1) destructor(<default>-2-1) destructor(<default>-1-1) destructor(logger-0-0) destructor(<default>-1-0) destructor(<default>-0-0) ", LifetimeLogger::takeLogStr().c_str());
 }
     
 } // namespace TestWebKitAPI