auto import from //depot/cupcake/@136594

1 files changed, 5 insertions, 26 deletions

diff --git a/WebCore/platform/posix/FileSystemPOSIX.cpp b/WebCore/platform/posix/FileSystemPOSIX.cpp
index 82ae087..c1bef60 100644
--- a/WebCore/platform/posix/FileSystemPOSIX.cpp
+++ b/WebCore/platform/posix/FileSystemPOSIX.cpp
@@ -163,31 +163,10 @@ String directoryName(const String& path)
     return dirname(fsRep.mutableData());
 }
 
-Vector<String> listDirectory(const String& path, const String& filter)
-{
-#ifdef ANDROID_PLUGINS
-    CString fsRepPath = fileSystemRepresentation(path);
-    CString fsRepFilter = fileSystemRepresentation(filter);
-#endif
-    Vector<String> entries;
-#ifdef ANDROID_PLUGINS
-    DIR *dir = opendir(fsRepPath.data());
-    if (dir == NULL)
-        return entries;
-    for (;;) {
-        struct dirent *entry = readdir(dir);
-        if (entry == NULL)
-            break;
-        if (!fnmatch(fsRepFilter.data(), entry->d_name, FNM_NOESCAPE)) {
-            String fullPath = path + "/" + entry->d_name;
-            entries.append(fullPath);
-        }
-    }
-    closedir(dir);
-#else
-    notImplemented();
-#endif
-    return entries;
-}
+// OK to not implement listDirectory at the moment, because it's only used for plug-ins, and
+// all platforms that use the shared plug-in implementation have implementations. We'd need
+// to implement it if we wanted to use PluginDatabase.cpp on the Mac. Better to not implement
+// at all and get a link error in case this arises, rather than having a stub here, because
+// with a stub you learn about the problem at runtime instead of link time.
 
 } // namespace WebCore