linux: write a unit test for plugin mime parsing

webkit/glue/plugins/plugin_lib_linux.cc

 // Copyright (c) 2009 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "webkit/glue/plugins/plugin_lib.h"
 
 #include <dlfcn.h>
 #include <elf.h>
 
 #include "base/string_util.h"
@@ skipping 64 matching lines @@
   // See comments in plugin_lib_mac regarding this symbol.
   typedef const char* (*NP_GetMimeDescriptionType)();
   NP_GetMimeDescriptionType NP_GetMIMEDescription =
       reinterpret_cast<NP_GetMimeDescriptionType>(
           dlsym(dl, "NP_GetMIMEDescription"));
   const char* mime_description = NULL;
   if (NP_GetMIMEDescription)
     mime_description = NP_GetMIMEDescription();
 
   if (mime_description) {
-    // We parse the description here into WebPluginMimeType structures.
-    // Description for Flash 10 looks like (all as one string):
-    //   "application/x-shockwave-flash:swf:Shockwave Flash;"
-    //   "application/futuresplash:spl:FutureSplash Player"
-    std::vector<std::string> descriptions;
-    SplitString(mime_description, ';', &descriptions);
-    for (size_t i = 0; i < descriptions.size(); ++i) {
-      if (descriptions[i].empty())
-        continue;  // Don't warn if they have trailing semis.
-
-      std::vector<std::string> fields;
-      SplitString(descriptions[i], ':', &fields);
-      if (fields.size() != 3) {
-        LOG(WARNING) << "Couldn't parse plugin info: " << descriptions[i];
-        continue;
-      }
-
-      WebPluginMimeType mime_type;
-      mime_type.mime_type = fields[0];
-      SplitString(fields[1], ',', &mime_type.file_extensions);
-      mime_type.description = UTF8ToWide(fields[2]);
-      info->mime_types.push_back(mime_type);
+    if (!ParseMIMEDescription(mime_description, &info->mime_types)) {
+      base::UnloadNativeLibrary(dl);
+      return false;
     }
   }
 
   // The plugin name and description live behind NP_GetValue calls.
   typedef NPError (*NP_GetValueType)(void* unused,
                                      nsPluginVariable variable,
                                      void* value_out);
   NP_GetValueType NP_GetValue =
       reinterpret_cast<NP_GetValueType>(dlsym(dl, "NP_GetValue"));
   if (NP_GetValue) {
     const char* name = NULL;
     NP_GetValue(NULL, nsPluginVariable_NameString, &name);
     if (name)
       info->name = UTF8ToWide(name);
 
     const char* description = NULL;
     NP_GetValue(NULL, nsPluginVariable_DescriptionString, &description);
     if (description)
       info->desc = UTF8ToWide(description);
   }
 
   base::UnloadNativeLibrary(dl);
 
   return true;
 }
 
+// static
+bool PluginLib::ParseMIMEDescription(
+    const char* description,
+    std::vector<WebPluginMimeType>* mime_types) {
+  // TODO(evanm): rewrite this to better match Firefox; see
+  // ParsePluginMimeDescription near
+  // http://mxr.mozilla.org/firefox/source/modules/plugin/base/src/nsPluginsDirUtils.h#53
+
+  // We parse the description here into WebPluginMimeType structures.
+  // Description for Flash 10 looks like (all as one string):
+  //   "application/x-shockwave-flash:swf:Shockwave Flash;"
+  //   "application/futuresplash:spl:FutureSplash Player"
+  std::vector<std::string> descriptions;
+  SplitString(description, ';', &descriptions);
+  for (size_t i = 0; i < descriptions.size(); ++i) {
+    if (descriptions[i].empty())
+      continue;  // Don't warn if they have trailing semis.
+
+    std::vector<std::string> fields;
+    SplitString(descriptions[i], ':', &fields);
+    if (fields.size() != 3) {
+      LOG(WARNING) << "Couldn't parse plugin info: " << description;
+      // This plugin's got something weird going on; abort.
+      return false;
+    }
+
+    WebPluginMimeType mime_type;
+    mime_type.mime_type = fields[0];
+    SplitString(fields[1], ',', &mime_type.file_extensions);
+    mime_type.description = UTF8ToWide(fields[2]);
+    mime_types->push_back(mime_type);
+  }
+
+  return true;
+}
+
 }  // namespace NPAPI