Title: [145049] trunk/Source/WebCore
Revision
145049
Author
commit-qu...@webkit.org
Date
2013-03-07 00:51:42 -0800 (Thu, 07 Mar 2013)

Log Message

[v8] remove latin-1 transition flag
https://bugs.webkit.org/show_bug.cgi?id=111692

Patch by Dan Carney <dcar...@google.com> on 2013-03-07
Reviewed by Jochen Eisinger.

No new tests. No change in functionality.

* bindings/v8/V8ValueCache.cpp:
(WebCore::makeExternalString):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (145048 => 145049)


--- trunk/Source/WebCore/ChangeLog	2013-03-07 08:27:14 UTC (rev 145048)
+++ trunk/Source/WebCore/ChangeLog	2013-03-07 08:51:42 UTC (rev 145049)
@@ -1,3 +1,15 @@
+2013-03-07  Dan Carney  <dcar...@google.com>
+
+        [v8] remove latin-1 transition flag
+        https://bugs.webkit.org/show_bug.cgi?id=111692
+
+        Reviewed by Jochen Eisinger.
+
+        No new tests. No change in functionality.
+
+        * bindings/v8/V8ValueCache.cpp:
+        (WebCore::makeExternalString):
+
 2013-03-07  Vsevolod Vlasov  <vse...@chromium.org>
 
         Web Inspector: NavigatorView should show (program) for uiSourceCodes with empty name.

Modified: trunk/Source/WebCore/bindings/v8/V8ValueCache.cpp (145048 => 145049)


--- trunk/Source/WebCore/bindings/v8/V8ValueCache.cpp	2013-03-07 08:27:14 UTC (rev 145048)
+++ trunk/Source/WebCore/bindings/v8/V8ValueCache.cpp	2013-03-07 08:51:42 UTC (rev 145049)
@@ -42,11 +42,7 @@
 
 static v8::Local<v8::String> makeExternalString(const String& string)
 {
-#ifdef V8_ONE_BYTE_STRINGS_ENABLED
     if (string.is8Bit()) {
-#else
-    if (string.is8Bit() && string.containsOnlyASCII()) {
-#endif
         WebCoreStringResource8* stringResource = new WebCoreStringResource8(string);
         v8::Local<v8::String> newString = v8::String::NewExternal(stringResource);
         if (newString.IsEmpty())
@@ -58,7 +54,6 @@
     v8::Local<v8::String> newString = v8::String::NewExternal(stringResource);
     if (newString.IsEmpty())
         delete stringResource;
-
     return newString;
 }
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to