/hg/release/icedtea6-1.8: 3 new changesets
andrew at icedtea.classpath.org
andrew at icedtea.classpath.org
Tue Jun 15 09:11:07 PDT 2010
changeset a033b01ed09b in /hg/release/icedtea6-1.8
details: http://icedtea.classpath.org/hg/release/icedtea6-1.8?cmd=changeset;node=a033b01ed09b
author: Omair Majid <omajid at redhat.com>
date: Mon Jun 14 14:55:08 2010 -0400
PR icedtea/480: (NPPlugin with NoScript extension).
2010-06-14 Omair Majid <omajid at redhat.com>
* plugin/icedteanp/IcedTeaNPPlugin.cc: Use getproperty NPAPI
call instead of evaluate, to get page URL.
changeset b7e6bac9cd18 in /hg/release/icedtea6-1.8
details: http://icedtea.classpath.org/hg/release/icedtea6-1.8?cmd=changeset;node=b7e6bac9cd18
author: Omair Majid <omajid at redhat.com>
date: Tue Jun 15 17:09:31 2010 +0100
PR icedtea/488: Question mark changing into underscore in URL.
2010-06-14 Omair Majid <omajid at redhat.com>
PR icedtea/488
* plugin/icedteanp/IcedTeaPluginUtils.h: Bug #488. Fix bug due to
incorrect assumption that 'A' > 'a'.
changeset 8de4ce5137b8 in /hg/release/icedtea6-1.8
details: http://icedtea.classpath.org/hg/release/icedtea6-1.8?cmd=changeset;node=8de4ce5137b8
author: Deepak Bhole <dbhole at redhat.com>
date: Mon Jun 14 15:03:55 2010 -0400
Fix bug causing 100% CPU usage (rhbz# 592553).
2010-06-14 Deepak Bhole <dbhole at redhat.com>
*
plugin/icedteanp/java/sun/applet/PluginMessageConsumer.java: Fix bug
causing 100% CPU usage (rhbz# 592553).
diffstat:
5 files changed, 37 insertions(+), 22 deletions(-)
ChangeLog | 17 +++++
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 (131 lines):
diff -r a59c9d0cda71 -r 8de4ce5137b8 ChangeLog
--- a/ChangeLog Fri May 07 16:57:27 2010 -0400
+++ b/ChangeLog Mon Jun 14 15:03:55 2010 -0400
@@ -1,3 +1,20 @@ 2010-05-07 Deepak Bhole <dbhole at redhat.
+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>
+
+ PR icedtea/488
+ * plugin/icedteanp/IcedTeaPluginUtils.h: Bug #488. Fix bug due to
+ incorrect assumption that 'A' > 'a'.
+
+2010-06-14 Omair Majid <omajid at redhat.com>
+
+ PR icedtea/480
+ * plugin/icedteanp/IcedTeaNPPlugin.cc: Use getproperty NPAPI call instead
+ of evaluate, to get page URL.
+
2010-05-07 Deepak Bhole <dbhole at redhat.com>
PR icedtea/436:
diff -r a59c9d0cda71 -r 8de4ce5137b8 plugin/icedteanp/IcedTeaNPPlugin.cc
--- a/plugin/icedteanp/IcedTeaNPPlugin.cc Fri May 07 16:57:27 2010 -0400
+++ 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 a59c9d0cda71 -r 8de4ce5137b8 plugin/icedteanp/IcedTeaPluginUtils.h
--- a/plugin/icedteanp/IcedTeaPluginUtils.h Fri May 07 16:57:27 2010 -0400
+++ 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 a59c9d0cda71 -r 8de4ce5137b8 plugin/icedteanp/java/sun/applet/PluginMessageConsumer.java
--- a/plugin/icedteanp/java/sun/applet/PluginMessageConsumer.java Fri May 07 16:57:27 2010 -0400
+++ 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 a59c9d0cda71 -r 8de4ce5137b8 plugin/icedteanp/java/sun/applet/PluginMessageHandlerWorker.java
--- a/plugin/icedteanp/java/sun/applet/PluginMessageHandlerWorker.java Fri May 07 16:57:27 2010 -0400
+++ 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