Title: [91641] trunk/Source/WebCore
Revision
91641
Author
[email protected]
Date
2011-07-23 16:42:06 -0700 (Sat, 23 Jul 2011)

Log Message

Remove WTF namespace from isMainThread() calls
https://bugs.webkit.org/show_bug.cgi?id=65068

Reviewed by Sam Weinig.

Because of the using WTF::isMainThread in the header there is no
reason to include the namespace name when calling this function.

* bindings/js/JSMainThreadExecState.h:
* bindings/v8/DOMDataStore.cpp:
* bindings/v8/V8Binding.cpp:
* bindings/v8/V8DOMMap.cpp:
* bindings/v8/V8DOMWrapper.h:
* page/DOMWindow.cpp:
* page/History.cpp:

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (91640 => 91641)


--- trunk/Source/WebCore/ChangeLog	2011-07-23 22:44:44 UTC (rev 91640)
+++ trunk/Source/WebCore/ChangeLog	2011-07-23 23:42:06 UTC (rev 91641)
@@ -1,3 +1,21 @@
+2011-07-23  Patrick Gansterer  <[email protected]>
+
+        Remove WTF namespace from isMainThread() calls
+        https://bugs.webkit.org/show_bug.cgi?id=65068
+
+        Reviewed by Sam Weinig.
+
+        Because of the using WTF::isMainThread in the header there is no
+        reason to include the namespace name when calling this function.
+
+        * bindings/js/JSMainThreadExecState.h:
+        * bindings/v8/DOMDataStore.cpp:
+        * bindings/v8/V8Binding.cpp:
+        * bindings/v8/V8DOMMap.cpp:
+        * bindings/v8/V8DOMWrapper.h:
+        * page/DOMWindow.cpp:
+        * page/History.cpp:
+
 2011-07-23  Alok Priyadarshi  <[email protected]>
 
         Switching off acceleration for small canvas broke gpu tests

Modified: trunk/Source/WebCore/bindings/js/JSMainThreadExecState.h (91640 => 91641)


--- trunk/Source/WebCore/bindings/js/JSMainThreadExecState.h	2011-07-23 22:44:44 UTC (rev 91640)
+++ trunk/Source/WebCore/bindings/js/JSMainThreadExecState.h	2011-07-23 23:42:06 UTC (rev 91641)
@@ -38,7 +38,7 @@
 public:
     static JSC::ExecState* currentState()
     { 
-        ASSERT(WTF::isMainThread());
+        ASSERT(isMainThread());
         return s_mainThreadState;
     };
     
@@ -61,13 +61,13 @@
     explicit JSMainThreadExecState(JSC::ExecState* exec)
         : m_previousState(s_mainThreadState)
     {
-        ASSERT(WTF::isMainThread());
+        ASSERT(isMainThread());
         s_mainThreadState = exec;
     };
     
     ~JSMainThreadExecState()
     {
-        ASSERT(WTF::isMainThread());
+        ASSERT(isMainThread());
         s_mainThreadState = m_previousState;
     }
 

Modified: trunk/Source/WebCore/bindings/v8/DOMDataStore.cpp (91640 => 91641)


--- trunk/Source/WebCore/bindings/v8/DOMDataStore.cpp	2011-07-23 22:44:44 UTC (rev 91640)
+++ trunk/Source/WebCore/bindings/v8/DOMDataStore.cpp	2011-07-23 23:42:06 UTC (rev 91641)
@@ -141,7 +141,7 @@
 
 void DOMDataStore::weakNodeCallback(v8::Persistent<v8::Value> value, void* domObject)
 {
-    ASSERT(WTF::isMainThread());
+    ASSERT(isMainThread());
 
     Node* node = static_cast<Node*>(domObject);
     // Node wrappers must be JS objects.

Modified: trunk/Source/WebCore/bindings/v8/V8Binding.cpp (91640 => 91641)


--- trunk/Source/WebCore/bindings/v8/V8Binding.cpp	2011-07-23 22:44:44 UTC (rev 91640)
+++ trunk/Source/WebCore/bindings/v8/V8Binding.cpp	2011-07-23 23:42:06 UTC (rev 91641)
@@ -419,7 +419,7 @@
 {
     // Caching of small strings below is not thread safe: newly constructed AtomicString
     // are not safely published.
-    ASSERT(WTF::isMainThread());
+    ASSERT(isMainThread());
 
     // Most numbers used are <= 100. Even if they aren't used there's very little cost in using the space.
     const int kLowNumbers = 100;
@@ -440,7 +440,7 @@
 String int32ToWebCoreString(int value)
 {
     // If we are on the main thread (this should always true for non-workers), call the faster one.
-    if (WTF::isMainThread())
+    if (isMainThread())
         return int32ToWebCoreStringFast(value);
     return String::number(value);
 }

Modified: trunk/Source/WebCore/bindings/v8/V8DOMMap.cpp (91640 => 91641)


--- trunk/Source/WebCore/bindings/v8/V8DOMMap.cpp	2011-07-23 22:44:44 UTC (rev 91640)
+++ trunk/Source/WebCore/bindings/v8/V8DOMMap.cpp	2011-07-23 23:42:06 UTC (rev 91641)
@@ -86,7 +86,7 @@
     v8::HandleScope scope;
 
     // The DOM objects with the following types only exist on the main thread.
-    if (WTF::isMainThread()) {
+    if (isMainThread()) {
         // Remove all DOM nodes.
         DOMData::removeObjectsFromWrapperMap<Node>(&store, store.domNodeMap());
 

Modified: trunk/Source/WebCore/bindings/v8/V8DOMWrapper.h (91640 => 91641)


--- trunk/Source/WebCore/bindings/v8/V8DOMWrapper.h	2011-07-23 22:44:44 UTC (rev 91640)
+++ trunk/Source/WebCore/bindings/v8/V8DOMWrapper.h	2011-07-23 23:42:06 UTC (rev 91641)
@@ -127,7 +127,7 @@
 
         static v8::Handle<v8::Object> getWrapper(Node* node)
         {
-            ASSERT(WTF::isMainThread());
+            ASSERT(isMainThread());
             if (LIKELY(!IsolatedWorld::count())) {
                 v8::Persistent<v8::Object>* wrapper = node->wrapper();
                 if (wrapper)

Modified: trunk/Source/WebCore/page/DOMWindow.cpp (91640 => 91641)


--- trunk/Source/WebCore/page/DOMWindow.cpp	2011-07-23 22:44:44 UTC (rev 91640)
+++ trunk/Source/WebCore/page/DOMWindow.cpp	2011-07-23 23:42:06 UTC (rev 91641)
@@ -919,7 +919,7 @@
         return;
 
     if (context) {
-        ASSERT(WTF::isMainThread());
+        ASSERT(isMainThread());
         Frame* activeFrame = static_cast<Document*>(context)->frame();
         if (!activeFrame)
             return;

Modified: trunk/Source/WebCore/page/History.cpp (91640 => 91641)


--- trunk/Source/WebCore/page/History.cpp	2011-07-23 22:44:44 UTC (rev 91640)
+++ trunk/Source/WebCore/page/History.cpp	2011-07-23 23:42:06 UTC (rev 91641)
@@ -96,7 +96,7 @@
     if (!m_frame)
         return;
 
-    ASSERT(WTF::isMainThread());
+    ASSERT(isMainThread());
     Frame* activeFrame = static_cast<Document*>(context)->frame();
     if (!activeFrame)
         return;
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to