[Chromium] Enable AtomicString && CString && WTFString && StringImpl unit tests
authorjchaffraix@webkit.org <jchaffraix@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 24 Aug 2012 03:03:10 +0000 (03:03 +0000)
committerjchaffraix@webkit.org <jchaffraix@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 24 Aug 2012 03:03:10 +0000 (03:03 +0000)
https://bugs.webkit.org/show_bug.cgi?id=94871

Reviewed by Benjamin Poulain.

Source/WTF:

* wtf/PassRefPtr.h:
We need to include Assertions.h due to the COMPILE_ASSERT added in r89283.
This matches other files in WTF though I am not sure why Chromium is the only
platform complaining about that.

Tools:

* TestWebKitAPI/TestWebKitAPI.gypi:
Added the missing files to Chromium's build system.

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

Source/WTF/ChangeLog
Source/WTF/wtf/PassRefPtr.h
Tools/ChangeLog
Tools/TestWebKitAPI/TestWebKitAPI.gypi

index 6e0439e80cfb721afe230387d268c64480db74e1..6fa29193f600756d7d60851c3f93f61df9a07b14 100644 (file)
@@ -1,3 +1,15 @@
+2012-08-23  Julien Chaffraix  <jchaffraix@webkit.org>
+
+        [Chromium] Enable AtomicString && CString && WTFString && StringImpl unit tests
+        https://bugs.webkit.org/show_bug.cgi?id=94871
+
+        Reviewed by Benjamin Poulain.
+
+        * wtf/PassRefPtr.h:
+        We need to include Assertions.h due to the COMPILE_ASSERT added in r89283.
+        This matches other files in WTF though I am not sure why Chromium is the only
+        platform complaining about that.
+
 2012-08-23  Mark Rowe  <mrowe@apple.com>
 
         Build fix for old, old versions of Clang.
index b858060836c2a43a27a5d75b90a2772be10d491c..5f50671135615ad8d28e4e3cfe70582c2ca2960d 100644 (file)
@@ -22,6 +22,7 @@
 #define WTF_PassRefPtr_h
 
 #include <wtf/AlwaysInline.h>
+#include <wtf/Assertions.h>
 #include <wtf/NullPtr.h>
 
 namespace WTF {
index f0a7e6d0d942725b5580def4fbd57c402ffb1bc7..f416b1d2fe2d5006fa5d388a58a44332e94619f9 100644 (file)
@@ -1,3 +1,13 @@
+2012-08-23  Julien Chaffraix  <jchaffraix@webkit.org>
+
+        [Chromium] Enable AtomicString && CString && WTFString && StringImpl unit tests
+        https://bugs.webkit.org/show_bug.cgi?id=94871
+
+        Reviewed by Benjamin Poulain.
+
+        * TestWebKitAPI/TestWebKitAPI.gypi:
+        Added the missing files to Chromium's build system.
+
 2012-08-23  Raphael Kubo da Costa  <rakuco@webkit.org>
 
         [watchlist] Unreviewed, subscribe to WebKit2-EFL bugs.
index e85476303cd662a6e333ceaf457248f27b38099c..197723344e9d17a724288c4e8915cd3f7005c73c 100644 (file)
 {
     'variables': {
         'TestWebKitAPI_files': [
+            'Tests/WTF/AtomicString.cpp',
             'Tests/WTF/CheckedArithmeticOperations.cpp',
+            'Tests/WTF/CString.cpp',
             'Tests/WTF/Functional.cpp',
             'Tests/WTF/HashMap.cpp',
             'Tests/WTF/MediaTime.cpp',
             'Tests/WTF/RedBlackTree.cpp',
             'Tests/WTF/SaturatedArithmeticOperations.cpp',
             'Tests/WTF/StringBuilder.cpp',
+            'Tests/WTF/StringImpl.cpp',
             'Tests/WTF/StringOperators.cpp',
             'Tests/WTF/TemporaryChange.cpp',
             'Tests/WTF/Vector.cpp',
             'Tests/WTF/VectorBasic.cpp',
             'Tests/WTF/VectorReverse.cpp',
+            'Tests/WTF/WTFString.cpp',
         ],
     },
 }