Title: [92437] trunk/Source
Revision
92437
Author
le...@chromium.org
Date
2011-08-04 17:51:19 -0700 (Thu, 04 Aug 2011)

Log Message

CStringBuffer should have thread safety checks turned on.
https://bugs.webkit.org/show_bug.cgi?id=58093

Reviewed by Dmitry Titov.

Source/_javascript_Core:

* wtf/text/CString.h:
(WTF::CStringBuffer::CStringBuffer): Removed the ifdef that
turned this off for Chromium.

Source/WebKit/chromium:

* DEPS: Roll far enough to pick up the latest WebKit::initialize
fix for Linux.

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (92436 => 92437)


--- trunk/Source/_javascript_Core/ChangeLog	2011-08-05 00:39:28 UTC (rev 92436)
+++ trunk/Source/_javascript_Core/ChangeLog	2011-08-05 00:51:19 UTC (rev 92437)
@@ -1,3 +1,14 @@
+2011-08-04  David Levin  <le...@chromium.org>
+
+        CStringBuffer should have thread safety checks turned on.
+        https://bugs.webkit.org/show_bug.cgi?id=58093
+
+        Reviewed by Dmitry Titov.
+
+        * wtf/text/CString.h:
+        (WTF::CStringBuffer::CStringBuffer): Removed the ifdef that
+        turned this off for Chromium.
+
 2011-08-04  Mark Rowe  <mr...@apple.com>
 
         Future-proof Xcode configuration settings.

Modified: trunk/Source/_javascript_Core/wtf/text/CString.h (92436 => 92437)


--- trunk/Source/_javascript_Core/wtf/text/CString.h	2011-08-05 00:39:28 UTC (rev 92436)
+++ trunk/Source/_javascript_Core/wtf/text/CString.h	2011-08-05 00:51:19 UTC (rev 92437)
@@ -41,14 +41,7 @@
     friend class CString;
 
     static PassRefPtr<CStringBuffer> create(size_t length) { return adoptRef(new CStringBuffer(length)); }
-    CStringBuffer(size_t length) : m_vector(length)
-    {
-#if PLATFORM(CHROMIUM)
-        // FIXME: Need to fix some start-up code in Chromium Linux and some test code for
-        // Chromium OS X in order to remove this.
-        deprecatedTurnOffVerifier();
-#endif
-    }
+    CStringBuffer(size_t length) : m_vector(length) { }
     char* mutableData() { return m_vector.data(); }
 
     Vector<char> m_vector;

Modified: trunk/Source/WebKit/chromium/ChangeLog (92436 => 92437)


--- trunk/Source/WebKit/chromium/ChangeLog	2011-08-05 00:39:28 UTC (rev 92436)
+++ trunk/Source/WebKit/chromium/ChangeLog	2011-08-05 00:51:19 UTC (rev 92437)
@@ -1,3 +1,13 @@
+2011-08-04  David Levin  <le...@chromium.org>
+
+        CStringBuffer should have thread safety checks turned on.
+        https://bugs.webkit.org/show_bug.cgi?id=58093
+
+        Reviewed by Dmitry Titov.
+
+        * DEPS: Roll far enough to pick up the latest WebKit::initialize
+        fix for Linux.
+
 2011-08-04  Sheriff Bot  <webkit.review....@gmail.com>
 
         Unreviewed, rolling out r92415.

Modified: trunk/Source/WebKit/chromium/DEPS (92436 => 92437)


--- trunk/Source/WebKit/chromium/DEPS	2011-08-05 00:39:28 UTC (rev 92436)
+++ trunk/Source/WebKit/chromium/DEPS	2011-08-05 00:51:19 UTC (rev 92437)
@@ -32,7 +32,7 @@
 
 vars = {
   'chromium_svn': 'http://src.chromium.org/svn/trunk/src',
-  'chromium_rev': '94850'
+  'chromium_rev': '95533'
 }
 
 deps = {
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to