[Bug 749] New: sun.applet.PluginStreamHandler#handleMessage(String) really slow

[bugzilla-daemon at icedtea.classpath.org]

http://icedtea.classpath.org/bugzilla/show_bug.cgi?id=749

           Summary: sun.applet.PluginStreamHandler#handleMessage(String)
                    really slow
           Product: IcedTea-Web
           Version: unspecified
          Platform: 64-bit
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: P5
         Component: Plugin
        AssignedTo: dbhole at redhat.com
        ReportedBy: rickyepoderi at yahoo.es
                CC: unassigned at icedtea.classpath.org


Testing an old portlet I worked with some time ago inside your plugin I've
discovered that the communication from the browser to the JVM is extremely
slow. I'm using debian packaged version 1.1 (testing). This applet renders
scatter plots which can have a lot of data. More or less the applet works well
when the number of points is not very big but when the points to draw are many
(around ten thousand) it works painfully slow. The data is requested via
browser and then received by the applet. 

Checking with jconsole I saw the time was consumed inside the following method
sun.applet.PluginStreamHandler#handleMessage(String) and exactly in this loop:

   161         // rest
   162         for (int i = nextIndex; i < msgComponents.length; i++) {
   163             rest += msgComponents[i];
   164             rest += " ";
   165         }

The problem here is that msgComponents var has more than 100.000 elements and
the concatenating part lasts for more than 5 minutes. I did some changes in
your code to try to increase performance. My idea is simple, I don't split the
message and just pick up the pairs using indexOf and substring methods, the
rest is again computed using substring. I've implemented a new readPair method
in order to do that. Now icedtea plugin renders the plot more or less in the
same time that the non-free plugin. I'm going to attach the patch right now.

It's only an idea but please change the way you deal with the message in this
method, it's really slow when it's very long (in my case more than 100.000
words). You can check the applet here:

http://www.puertos.es/en/oceanografia_y_meteorologia/redes_de_medida/index.html

Select Sea Level (this magnitude is the one with a lot of samples), select any
gauge in the map and then mark Plot. The plot opens in a new div below. I'm not
a current developer of the applet so please consider I have no access to the
code.

Thank you very much!


-- 
Configure bugmail: http://icedtea.classpath.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.