Remove 8-bit related assertion from NetscapePluginModuleX11.
authorrakuco@webkit.org <rakuco@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 18 Nov 2012 03:37:04 +0000 (03:37 +0000)
committerrakuco@webkit.org <rakuco@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 18 Nov 2012 03:37:04 +0000 (03:37 +0000)
https://bugs.webkit.org/show_bug.cgi?id=102588

Reviewed by Andreas Kling.

Follow-up to r135043: since String::fromUTF8() is not guaranteed
to give us a 16-bit string, the assertion in
truncateToSingleLine() will fail in that case.

This should make the EFL WK2 Debug bot stop failing all
plugin-related tests because "WebKit Test Plugin" is returned as
an 8-bit string.

* Shared/Plugins/Netscape/x11/NetscapePluginModuleX11.cpp:
(WebKit::truncateToSingleLine):

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

Source/WebKit2/ChangeLog
Source/WebKit2/Shared/Plugins/Netscape/x11/NetscapePluginModuleX11.cpp

index ddf64ac..b02ce28 100644 (file)
@@ -1,3 +1,21 @@
+2012-11-17  Raphael Kubo da Costa  <raphael.kubo.da.costa@intel.com>
+
+        Remove 8-bit related assertion from NetscapePluginModuleX11.
+        https://bugs.webkit.org/show_bug.cgi?id=102588
+
+        Reviewed by Andreas Kling.
+
+        Follow-up to r135043: since String::fromUTF8() is not guaranteed
+        to give us a 16-bit string, the assertion in
+        truncateToSingleLine() will fail in that case.
+
+        This should make the EFL WK2 Debug bot stop failing all
+        plugin-related tests because "WebKit Test Plugin" is returned as
+        an 8-bit string.
+
+        * Shared/Plugins/Netscape/x11/NetscapePluginModuleX11.cpp:
+        (WebKit::truncateToSingleLine):
+
 2012-11-17  Alexandru Chiculita  <achicu@adobe.com>
 
         [Texmap][CSS Shaders] Reuse the precompiled shader for custom filters in TextureMapperGL
index d6f4536..62b53f9 100644 (file)
@@ -160,8 +160,6 @@ void NetscapePluginModule::determineQuirks()
 
 static String truncateToSingleLine(const String& string)
 {
-    ASSERT_ARG(string, !string.is8Bit());
-
     unsigned oldLength = string.length();
     UChar* buffer;
     String stringBuffer(StringImpl::createUninitialized(oldLength + 1, buffer));