[ https://issues.apache.org/jira/browse/PIVOT-754?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13047228#comment-13047228 ]
Piotr Kołaczkowski commented on PIVOT-754: ------------------------------------------ No problem, but that is really not any magic. We use the standard Proguard recipe for proguarding libraries - there is an example on the Proguard page. We noticed, that although proguarding a library doesn't shrink it significantly (you may get only 30% improvement), there is a huge impact of doing a pack200 on the resulting proguarded jar. It looks like proguard optimisations helped pack200 work better. Anyway, you can even try pack200 alone - 3 fold size reduction is almost sure. > Pivot displays ugly gray box sometimes before it loads the applet > ----------------------------------------------------------------- > > Key: PIVOT-754 > URL: https://issues.apache.org/jira/browse/PIVOT-754 > Project: Pivot > Issue Type: Bug > Components: wtk > Affects Versions: 2.0 > Environment: Observed both on Linux and Windows Vista, Firefox 3.x > and Firefox 4.0.1 > Java 6 update 25. > Reporter: Piotr Kołaczkowski > Priority: Minor > Fix For: 2.0.1 > > Attachments: pivot-graybox.patch > > > When the applet starts, a progress bar is displayed. When the applet finishes > downloading, the progress bar disappears, leaving a gray box until the pivot > toolkit loads the application. Even for the simpliest application displaying > an empty window, pivot requires about 0.3 s to start (on Core2Duo 2.2 GHz), > so the gray box effect can be easily seen. The gray box effect does not > happen always, but in about half of the cases. > Below I submit patch we made to the BrowserApplicationContext.java file. > Seems like setting the applet's background to white and delaying the > add(displayHost) call as late as possible does the trick: > [code] > 51a52 > > private boolean displayHostInstalled = false; > 151c152,154 > < add(displayHost); > --- > > > > // Don't add it now, add it later, to avoid ugly gray box: > > //add(displayHost); > 160c163,164 > < setBackground(null); > --- > > // Huh? WTF? > > // setBackground(null); > 191a196 > > installDisplayHostIfNeeded(); > 245a251,257 > > private void installDisplayHostIfNeeded() { > > if (!displayHostInstalled) { > > this.add(displayHost); > > displayHostInstalled = true; > > } > > } > > > 247a260 > > this.setBackground(Color.white); > 309a323 > > installDisplayHostIfNeeded(); > [/code] -- This message is automatically generated by JIRA. For more information on JIRA, see: http://www.atlassian.com/software/jira