/hg/icedtea6: 3 new changesets
dbhole at icedtea.classpath.org
dbhole at icedtea.classpath.org
Mon Jun 14 12:04:04 PDT 2010
changeset df96f563510a in /hg/icedtea6
details: http://icedtea.classpath.org/hg/icedtea6?cmd=changeset;node=df96f563510a
author: Deepak Bhole <dbhole at redhat.com>
date: Mon Jun 14 14:55:08 2010 -0400
Patch for Bug# 480 (NPPlugin with NoScript extension), by Omair
Majid.
changeset 3dc9772300a0 in /hg/icedtea6
details: http://icedtea.classpath.org/hg/icedtea6?cmd=changeset;node=3dc9772300a0
author: Deepak Bhole <dbhole at redhat.com>
date: Mon Jun 14 14:58:37 2010 -0400
Fix for Bug# 488 (Question mark changing into underscore in URL), by
Omair M.
changeset 9a9ed35b3f80 in /hg/icedtea6
details: http://icedtea.classpath.org/hg/icedtea6?cmd=changeset;node=9a9ed35b3f80
author: Deepak Bhole <dbhole at redhat.com>
date: Mon Jun 14 15:03:55 2010 -0400
Fix bug causing 100% CPU usage (rhbz# 592553).
diffstat:
5 files changed, 35 insertions(+), 22 deletions(-)
ChangeLog | 15 +++++
plugin/icedteanp/IcedTeaNPPlugin.cc | 30 ++++------
plugin/icedteanp/IcedTeaPluginUtils.h | 4 -
plugin/icedteanp/java/sun/applet/PluginMessageConsumer.java | 4 -
plugin/icedteanp/java/sun/applet/PluginMessageHandlerWorker.java | 4 -
diffs (129 lines):
diff -r be0a17ddbdb8 -r 9a9ed35b3f80 ChangeLog
--- a/ChangeLog Fri Jun 11 14:45:52 2010 +0100
+++ b/ChangeLog Mon Jun 14 15:03:55 2010 -0400
@@ -1,3 +1,18 @@ 2010-06-11 Andrew John Hughes <ahughes@
+2010-06-14 Deepak Bhole <dbhole at redhat.com>
+
+ * plugin/icedteanp/java/sun/applet/PluginMessageConsumer.java: Fix bug
+ causing 100% CPU usage (rhbz# 592553).
+
+2010-06-14 Omair Majid <omajid at redhat.com>
+
+ * plugin/icedteanp/IcedTeaPluginUtils.h: Bug #488. Fix bug due to
+ incorrect assumption that 'A' > 'a'.
+
+2010-06-14 Omair Majid <omajid at redhat.com>
+
+ * plugin/icedteanp/IcedTeaNPPlugin.cc: Use getproperty NPAPI call instead
+ of evaluate, to get page URL.
+
2010-06-11 Andrew John Hughes <ahughes at redhat.com>
* Makefile.am:
diff -r be0a17ddbdb8 -r 9a9ed35b3f80 plugin/icedteanp/IcedTeaNPPlugin.cc
--- a/plugin/icedteanp/IcedTeaNPPlugin.cc Fri Jun 11 14:45:52 2010 +0100
+++ b/plugin/icedteanp/IcedTeaNPPlugin.cc Mon Jun 14 15:03:55 2010 -0400
@@ -1021,30 +1021,26 @@ plugin_get_documentbase (NPP instance)
// Additionally, since it is insecure, we cannot use it for making
// security decisions.
NPObject* window;
- NPString script = NPString();
- std::string script_str = std::string();
- NPVariant* location = new NPVariant();
- std::string location_str = std::string();
+ browser_functions.getvalue(instance, NPNVWindowNPObject, &window);
- browser_functions.getvalue(instance, NPNVWindowNPObject, &window);
- script_str += "window.location.href";
-#if MOZILLA_VERSION_COLLAPSED < 1090200
- script.utf8characters = script_str.c_str();
- script.utf8length = script_str.size();
-#else
- script.UTF8Characters = script_str.c_str();
- script.UTF8Length = script_str.size();
-#endif
- browser_functions.evaluate(instance, window, &script, location);
+ NPVariant location;
+ NPIdentifier location_id = browser_functions.getstringidentifier("location");
+ browser_functions.getproperty(instance, window, location_id, &location);
+
+ NPVariant href;
+ NPIdentifier href_id = browser_functions.getstringidentifier("href");
+ browser_functions.getproperty(instance, NPVARIANT_TO_OBJECT(location),
+ href_id, &href);
// Strip everything after the last "/"
#if MOZILLA_VERSION_COLLAPSED < 1090200
- gchar** parts = g_strsplit (NPVARIANT_TO_STRING(*location).utf8characters, "/", -1);
+ gchar** parts = g_strsplit (NPVARIANT_TO_STRING(href).utf8characters, "/", -1);
#else
- gchar** parts = g_strsplit (NPVARIANT_TO_STRING(*location).UTF8Characters, "/", -1);
+ gchar** parts = g_strsplit (NPVARIANT_TO_STRING(href).UTF8Characters, "/", -1);
#endif
guint parts_sz = g_strv_length (parts);
+ std::string location_str;
for (int i=0; i < parts_sz - 1; i++)
{
location_str += parts[i];
@@ -1054,6 +1050,8 @@ plugin_get_documentbase (NPP instance)
documentbase_copy = g_strdup (location_str.c_str());
// Release references.
+ browser_functions.releasevariantvalue(&href);
+ browser_functions.releasevariantvalue(&location);
cleanup_done:
PLUGIN_DEBUG_0ARG ("plugin_get_documentbase return\n");
PLUGIN_DEBUG_1ARG("plugin_get_documentbase returning: %s\n", documentbase_copy);
diff -r be0a17ddbdb8 -r 9a9ed35b3f80 plugin/icedteanp/IcedTeaPluginUtils.h
--- a/plugin/icedteanp/IcedTeaPluginUtils.h Fri Jun 11 14:45:52 2010 +0100
+++ b/plugin/icedteanp/IcedTeaPluginUtils.h Mon Jun 14 15:03:55 2010 -0400
@@ -137,8 +137,8 @@ exception statement from your version. *
}
#define HEX_TO_INT(c) \
- ((*c >= 'A') ? *c - 'A' + 10 : \
- (*c >= 'a') ? *c - 'a' + 10 : \
+ ((*c >= 'a') ? *c - 'a' + 10 : \
+ (*c >= 'A') ? *c - 'A' + 10 : \
*c - '0')
#define IS_VALID_HEX(c) \
diff -r be0a17ddbdb8 -r 9a9ed35b3f80 plugin/icedteanp/java/sun/applet/PluginMessageConsumer.java
--- a/plugin/icedteanp/java/sun/applet/PluginMessageConsumer.java Fri Jun 11 14:45:52 2010 +0100
+++ b/plugin/icedteanp/java/sun/applet/PluginMessageConsumer.java Mon Jun 14 15:03:55 2010 -0400
@@ -164,11 +164,11 @@ class PluginMessageConsumer {
registerPriorityWait("instance " + instanceNum + " handle");
registerPriorityWait("instance " + instanceNum + " width");
- } else if (msgParts[2].equals("handle") || msgParts[2].equals("width")) {
+ } else if (msgParts[2].equals("handle")) {
Integer instanceNum = new Integer(msgParts[1]);
// If this instance is not in init, return false immediately.
- // Handle/Width messages should NEVER go before tag messages
+ // Handle messages should NEVER go before tag messages
if (!isInInit(instanceNum))
return false;
}
diff -r be0a17ddbdb8 -r 9a9ed35b3f80 plugin/icedteanp/java/sun/applet/PluginMessageHandlerWorker.java
--- a/plugin/icedteanp/java/sun/applet/PluginMessageHandlerWorker.java Fri Jun 11 14:45:52 2010 +0100
+++ b/plugin/icedteanp/java/sun/applet/PluginMessageHandlerWorker.java Mon Jun 14 15:03:55 2010 -0400
@@ -72,7 +72,7 @@ class PluginMessageHandlerWorker extends
if (message != null) {
- PluginDebug.debug("Consumer thread " + id + " consuming " + message);
+ PluginDebug.debug("Consumer (priority=" + isPriorityWorker + ") thread " + id + " consuming " + message);
// ideally, whoever returns things object should mark it
// busy first, but just in case..
@@ -90,7 +90,7 @@ class PluginMessageHandlerWorker extends
this.message = null;
- PluginDebug.debug("Consumption completed by consumer thread " + id);
+ PluginDebug.debug("Consumption (priority=" + isPriorityWorker + ") completed by consumer thread " + id);
// mark ourselves free again
free();
More information about the distro-pkg-dev
mailing list