Changeset 162355 in webkit

Timestamp:

Jan 20, 2014 9:25:38 AM (10 years ago)

Author:

andersca@apple.com

Message:

Remove an unused member variable
​https://bugs.webkit.org/show_bug.cgi?id=127295

Reviewed by Antti Koivisto.

• wtf/MainThread.cpp:

(WTF::FunctionWithContext::FunctionWithContext):
(WTF::FunctionWithContext::operator == ):
(WTF::dispatchFunctionsFromMainThread):

Location:

trunk/Source/WTF

Files:

• ChangeLog (modified) (1 diff)
• wtf/MainThread.cpp (modified) (2 diffs)

trunk/Source/WTF/ChangeLog

@@ +1 @@
+2014-01-20  Anders Carlsson  <andersca@apple.com>
+
+        Remove an unused member variable
+        https://bugs.webkit.org/show_bug.cgi?id=127295
+
+        Reviewed by Antti Koivisto.
+
+        * wtf/MainThread.cpp:
+        (WTF::FunctionWithContext::FunctionWithContext):
+        (WTF::FunctionWithContext::operator == ):
+        (WTF::dispatchFunctionsFromMainThread):
+
 2014-01-20  Anders Carlsson  <andersca@apple.com>
 

trunk/Source/WTF/wtf/MainThread.cpp

@@ -42 +42 @@
     MainThreadFunction* function;
     void* context;
-    ThreadCondition* syncFlag;
-
-    FunctionWithContext(MainThreadFunction* function = 0, void* context = 0, ThreadCondition* syncFlag = 0)
+
+    FunctionWithContext(MainThreadFunction* function = nullptr, void* context = nullptr)
         : function(function)
         , context(context)
-        , syncFlag(syncFlag)
-    { 
+    {
     }
     bool operator == (const FunctionWithContext& o)
     {
-        return function == o.function
-            && context == o.context
-            && syncFlag == o.syncFlag;
+        return function == o.function && context == o.context;
     }
 };
@@ -166 +162 @@
 
         invocation.function(invocation.context);
-        if (invocation.syncFlag) {
-            MutexLocker locker(mainThreadFunctionQueueMutex());
-            invocation.syncFlag->signal();
-        }
 
         // If we are running accumulated functions for too long so UI may become unresponsive, we need to