2009-06-30 Xan Lopez <xlopez@igalia.com>
authorxan@webkit.org <xan@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 1 Jul 2009 06:51:07 +0000 (06:51 +0000)
committerxan@webkit.org <xan@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 1 Jul 2009 06:51:07 +0000 (06:51 +0000)
        Reviewed by Jan Alonzo.

        https://bugs.webkit.org/show_bug.cgi?id=26877
        [GTK] DRT fixes to pass new plugin test

        Silence compiler warnings.

        Newer GCC don't like using or returning a static string where a
        'char*' is expected. Ideally we'd change the function signatures
        to return or take 'const char*' I suppose, but since we can't do
        that just cast the strings to 'char*'.

        * DumpRenderTree/gtk/TestNetscapePlugin/TestNetscapePlugin.cpp:
        (webkit_test_plugin_get_value):

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

WebKitTools/ChangeLog
WebKitTools/DumpRenderTree/gtk/TestNetscapePlugin/TestNetscapePlugin.cpp

index 80d4fb5..948425b 100644 (file)
@@ -5,6 +5,23 @@
         https://bugs.webkit.org/show_bug.cgi?id=26877
         [GTK] DRT fixes to pass new plugin test
 
+        Silence compiler warnings.
+
+        Newer GCC don't like using or returning a static string where a
+        'char*' is expected. Ideally we'd change the function signatures
+        to return or take 'const char*' I suppose, but since we can't do
+        that just cast the strings to 'char*'.
+
+        * DumpRenderTree/gtk/TestNetscapePlugin/TestNetscapePlugin.cpp:
+        (webkit_test_plugin_get_value):
+
+2009-06-30  Xan Lopez  <xlopez@igalia.com>
+
+        Reviewed by Jan Alonzo.
+
+        https://bugs.webkit.org/show_bug.cgi?id=26877
+        [GTK] DRT fixes to pass new plugin test
+
         Use the common pluginLog function instead of a local copy.
 
         * DumpRenderTree/gtk/TestNetscapePlugin/TestNetscapePlugin.cpp:
index 760025a..a58862b 100644 (file)
@@ -227,10 +227,10 @@ webkit_test_plugin_get_value(NPP instance, NPPVariable variable, void *value)
 
     switch (variable) {
         case NPPVpluginNameString:
-            *((char **)value) = "WebKit Test PlugIn";
+            *((char **)value) = const_cast<char*>("WebKit Test PlugIn");
             break;
         case NPPVpluginDescriptionString:
-            *((char **)value) = "Simple Netscape plug-in that handles test content for WebKit";
+            *((char **)value) = const_cast<char*>("Simple Netscape plug-in that handles test content for WebKit");
             break;
         case NPPVpluginNeedsXEmbed:
             *((NPBool *)value) = TRUE;
@@ -266,7 +266,7 @@ webkit_test_plugin_set_value(NPP instance, NPNVariable variable, void *value)
 char *
 NP_GetMIMEDescription(void)
 {
-    return "application/x-webkit-test-netscape:testnetscape:test netscape content";
+    return const_cast<char*>("application/x-webkit-test-netscape:testnetscape:test netscape content");
 }
 
 NPError