[X11] Allow empty plugin name and description during scan plugin
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 27 Nov 2012 11:41:46 +0000 (11:41 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 27 Nov 2012 11:41:46 +0000 (11:41 +0000)
https://bugs.webkit.org/show_bug.cgi?id=103234

Patch by KwangYong Choi <ky0.choi@samsung.com> on 2012-11-27
Reviewed by Simon Hausmann.

Empty entries should be allowed when split the output string of scan plugin
because the plugin name and the description can be empty string.

* UIProcess/Plugins/qt/PluginProcessProxyQt.cpp:
(WebKit::PluginProcessProxy::scanPlugin):
* UIProcess/Plugins/unix/PluginProcessProxyUnix.cpp:
(WebKit::PluginProcessProxy::scanPlugin):

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

Source/WebKit2/ChangeLog
Source/WebKit2/UIProcess/Plugins/qt/PluginProcessProxyQt.cpp
Source/WebKit2/UIProcess/Plugins/unix/PluginProcessProxyUnix.cpp

index c0df159..8f8124f 100644 (file)
@@ -1,3 +1,18 @@
+2012-11-27  KwangYong Choi  <ky0.choi@samsung.com>
+
+        [X11] Allow empty plugin name and description during scan plugin
+        https://bugs.webkit.org/show_bug.cgi?id=103234
+
+        Reviewed by Simon Hausmann.
+
+        Empty entries should be allowed when split the output string of scan plugin
+        because the plugin name and the description can be empty string.
+
+        * UIProcess/Plugins/qt/PluginProcessProxyQt.cpp:
+        (WebKit::PluginProcessProxy::scanPlugin):
+        * UIProcess/Plugins/unix/PluginProcessProxyUnix.cpp:
+        (WebKit::PluginProcessProxy::scanPlugin):
+
 2012-11-27  Sheriff Bot  <webkit.review.bot@gmail.com>
 
         Unreviewed, rolling out r135819.
index 679c633..8b4e359 100644 (file)
@@ -207,7 +207,7 @@ bool PluginProcessProxy::scanPlugin(const String& pluginPath, RawPluginMetaData&
 
         String output(reinterpret_cast<const UChar*>(outputBytes.constData()), outputBytes.size() / sizeof(UChar));
         Vector<String> lines;
-        output.split(UChar('\n'), lines);
+        output.split(UChar('\n'), true, lines);
         ASSERT(lines.size() == 3);
 
         result.name.swap(lines[0]);
index d24c391..730dafe 100644 (file)
@@ -96,7 +96,7 @@ bool PluginProcessProxy::scanPlugin(const String& pluginPath, RawPluginMetaData&
     free(stdOut);
 
     Vector<String> lines;
-    stdOutString.split(UChar('\n'), lines);
+    stdOutString.split(UChar('\n'), true, lines);
 
     if (lines.size() < 3)
         return false;