RFC: Rewrite of cookie support in plugin
dbhole at redhat.com
Tue Jul 7 13:39:28 PDT 2009
* Deepak Bhole <dbhole at redhat.com> [2009-07-07 16:25]:
> * Omair Majid <omajid at redhat.com> [2009-07-07 15:48]:
> > Hi,
> > Deepak Bhole wrote:
> >> Oops. Original message is missing 2 new files from the patch. Sorry
> >> about that. New patch attached.
> >> Deepak
> >> * Deepak Bhole <dbhole at redhat.com> [2009-07-03 17:48]:
> >>> This patch rewrites cookie support in the plugin. The old way was
> >>> static, set at initialization time. Additionally, it did not support
> >>> cookie provision for connection requests made by the applet internally.
> >>> This patch fixes all of that. Cookie information is now supplied to Java
> >>> in real-time from Mozilla side, as it should be.
> >>> Thanks to Omair for the initial error trace.
> >>> This patch fixes rhbz 506730:
> >>> https://bugzilla.redhat.com/show_bug.cgi?id=506730
> >>> http://www.sbm.no/ will now load correctly with this patch.
> >>> ChangeLog:
> >>> * IcedTeaPlugin.cc: Add suppport for cookie info requests from applets.
> >>> * plugin/icedtea/sun/applet/PluginAppletViewer.java: Rework cookie support
> >>> to make it dynamic.
> >>> * plugin/icedtea/sun/applet/PluginMain.java: Wire in custom cookie store
> >>> that dynamically requests cookie information from C++ side.
> >>> * rt/net/sourceforge/jnlp/JNLPFile.java: Remove old cookie handling code.
> >>> * rt/net/sourceforge/jnlp/Launcher.java: Same.
> >>> * rt/net/sourceforge/jnlp/NetxPanel.java: Same.
> >>> * rt/net/sourceforge/jnlp/PluginBridge.java: Same.
> >>> * rt/net/sourceforge/jnlp/cache/CacheUtil.java: Same.
> >>> * rt/net/sourceforge/jnlp/cache/Resource.java: Same.
> >>> * rt/net/sourceforge/jnlp/cache/ResourceTracker.java: Same.
> >>> * rt/net/sourceforge/jnlp/runtime/JNLPClassLoader.java: Same.
> >>> Comments?
> >>> Deepak
> > I am assuming that aside from the classes that manage cookies, nothing
> > should be storing or passing them around. However, there are still many
> > references to cookies. For example,net.sourceforge.jnlp.ExtensionDesc
> > contains:
> > public ExtensionDesc(String name, Version version, URL location, String
> > cookieStr)
> > Please remove all unused references to cookies.
> Good catch! New patch attached.
Just my luck. That patch breaks javaws (not sure how it compiled in the
first place when I tested it :/). Anyways, THIS one should work. The
previous one was missing a change to the JNLPFile constructor call in
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 33780 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20090707/93ce679c/ITP-cookie-support-rewrite3.patch
More information about the distro-pkg-dev