2009-05-24 Holger Hans Peter Freyther <zecke@selfish.org>
authorzecke@webkit.org <zecke@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 25 May 2009 03:29:06 +0000 (03:29 +0000)
committerzecke@webkit.org <zecke@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 25 May 2009 03:29:06 +0000 (03:29 +0000)
        Reviewed by Maciej Stachowiak.

        https://bugs.webkit.org/show_bug.cgi?id=25988

        Move ::hash, ::equal and ::compareFileVersion from
        the duplicated copies in PluginPackageQt.cpp and
        PluginPackageGtk to PluginPackage.cpp. We need to #ifdef
        this for the PluginPackageWin version. Use the new style
        ENABLE() for this feature/policy and enable it in the Gtk+
        and Qt buildsystem.

        * GNUmakefile.am:
        * WebCore.pro:
        * plugins/PluginPackage.cpp:
        (WebCore::PluginPackage::hash):
        (WebCore::PluginPackage::equal):
        (WebCore::PluginPackage::compareFileVersion):
        * plugins/gtk/PluginPackageGtk.cpp:
        * plugins/qt/PluginPackageQt.cpp:

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

WebCore/ChangeLog
WebCore/GNUmakefile.am
WebCore/WebCore.pro
WebCore/plugins/PluginPackage.cpp
WebCore/plugins/gtk/PluginPackageGtk.cpp
WebCore/plugins/qt/PluginPackageQt.cpp

index 781ff97..9b9c0c1 100644 (file)
@@ -4,6 +4,28 @@
 
         https://bugs.webkit.org/show_bug.cgi?id=25988
 
+        Move ::hash, ::equal and ::compareFileVersion from
+        the duplicated copies in PluginPackageQt.cpp and
+        PluginPackageGtk to PluginPackage.cpp. We need to #ifdef
+        this for the PluginPackageWin version. Use the new style
+        ENABLE() for this feature/policy and enable it in the Gtk+
+        and Qt buildsystem.
+
+        * GNUmakefile.am:
+        * WebCore.pro:
+        * plugins/PluginPackage.cpp:
+        (WebCore::PluginPackage::hash):
+        (WebCore::PluginPackage::equal):
+        (WebCore::PluginPackage::compareFileVersion):
+        * plugins/gtk/PluginPackageGtk.cpp:
+        * plugins/qt/PluginPackageQt.cpp:
+
+2009-05-24  Holger Hans Peter Freyther  <zecke@selfish.org>
+
+        Reviewed by Maciej Stachowiak.
+
+        https://bugs.webkit.org/show_bug.cgi?id=25988
+
         So far the initialization of the browserfuncs struct
         is done in PluginPackageQt, PluginPackageGtk and
         PluginPackageWin there is no need for this duplication.
index 8fb164a..5ec8fd1 100644 (file)
@@ -50,6 +50,7 @@ webcore_cppflags += \
 
 webcoregtk_cppflags += \
        -DWTF_USE_SOUP=1 \
+        -DENABLE_PLUGIN_PACKAGE_SIMPLE_HASH=1 \
        -I$(srcdir)/WebCore/accessibility/gtk \
        -I$(srcdir)/WebCore/loader/gtk \
        -I$(srcdir)/WebCore/page/gtk \
index 035821f..49dcacf 100644 (file)
@@ -136,7 +136,7 @@ CONFIG(QTDIR_build) {
     }
 }
 
-DEFINES += WTF_USE_JAVASCRIPTCORE_BINDINGS=1 WTF_CHANGES=1
+DEFINES += WTF_USE_JAVASCRIPTCORE_BINDINGS=1 WTF_CHANGES=1 ENABLE_PLUGIN_PACKAGE_SIMPLE_HASH=1
 
 include($$PWD/../JavaScriptCore/JavaScriptCore.pri)
 
index f95c6e7..a13e63c 100644 (file)
@@ -293,4 +293,30 @@ void PluginPackage::initializeBrowserFuncs()
     m_browserFuncs.construct = _NPN_Construct;
 }
 
+#if ENABLE(PLUGIN_PACKAGE_SIMPLE_HASH)
+unsigned PluginPackage::hash() const
+{
+    unsigned hashCodes[2] = {
+        m_path.impl()->hash(),
+        m_lastModified
+    };
+
+    return StringImpl::computeHash(reinterpret_cast<UChar*>(hashCodes), 2 * sizeof(unsigned) / sizeof(UChar));
+}
+
+bool PluginPackage::equal(const PluginPackage& a, const PluginPackage& b)
+{
+    return a.m_description == b.m_description;
+}
+
+int PluginPackage::compareFileVersion(const PlatformModuleVersion& compareVersion) const
+{
+    // return -1, 0, or 1 if plug-in version is less than, equal to, or greater than
+    // the passed version
+    if (m_moduleVersion != compareVersion)
+        return m_moduleVersion > compareVersion ? 1 : -1;
+    return 0;
+}
+#endif
+
 }
index 7ada5b9..31c4ed3 100644 (file)
@@ -189,28 +189,4 @@ abort:
     return false;
 }
 
-unsigned PluginPackage::hash() const
-{ 
-    unsigned hashCodes[2] = {
-        m_path.impl()->hash(),
-        m_lastModified
-    };
-
-    return StringImpl::computeHash(reinterpret_cast<UChar*>(hashCodes), 2 * sizeof(unsigned) / sizeof(UChar));
-}
-
-bool PluginPackage::equal(const PluginPackage& a, const PluginPackage& b)
-{
-    return a.m_description == b.m_description;
-}
-
-int PluginPackage::compareFileVersion(const PlatformModuleVersion& compareVersion) const
-{
-    // return -1, 0, or 1 if plug-in version is less than, equal to, or greater than
-    // the passed version
-    if (m_moduleVersion != compareVersion)
-        return m_moduleVersion > compareVersion ? 1 : -1;
-    return 0;
-}
-
 }
index c23caf1..d9e3765 100644 (file)
@@ -127,28 +127,4 @@ abort:
     return false;
 }
 
-unsigned PluginPackage::hash() const
-{ 
-    unsigned hashCodes[2] = {
-        m_path.impl()->hash(),
-        m_lastModified
-    };
-
-    return StringImpl::computeHash(reinterpret_cast<UChar*>(hashCodes), 2 * sizeof(unsigned) / sizeof(UChar));
-}
-
-bool PluginPackage::equal(const PluginPackage& a, const PluginPackage& b)
-{
-    return a.m_description == b.m_description;
-}
-
-int PluginPackage::compareFileVersion(const PlatformModuleVersion& compareVersion) const
-{
-    // return -1, 0, or 1 if plug-in version is less than, equal to, or greater than
-    // the passed version
-    if (m_moduleVersion != compareVersion)
-        return m_moduleVersion > compareVersion ? 1 : -1;
-    return 0;
-}
-
 }