[gwt-contrib] Re: Changed method dependencies report to show method code sizes. Call stacks are now a dropdown act... (issue1443809)

2011-05-26 Thread zundel

LGTM.

Updated report looks great!

http://gwt-code-reviews.appspot.com/1443809/

--
http://groups.google.com/group/Google-Web-Toolkit-Contributors


[gwt-contrib] Re: Changed method dependencies report to show method code sizes. Call stacks are now a dropdown act... (issue1443809)

2011-05-26 Thread dconnelly

http://gwt-code-reviews.appspot.com/1443809/

--
http://groups.google.com/group/Google-Web-Toolkit-Contributors


[gwt-contrib] Re: Changed method dependencies report to show method code sizes. Call stacks are now a dropdown act... (issue1443809)

2011-05-26 Thread zundel

And, as you pointed out, there is no need for the class name headers to
be clickable - could you remove that non-functioning clicking?


http://gwt-code-reviews.appspot.com/1443809/diff/1/dev/core/src/com/google/gwt/soyc/MakeTopLevelHtmlForPerm.java
File dev/core/src/com/google/gwt/soyc/MakeTopLevelHtmlForPerm.java
(right):

http://gwt-code-reviews.appspot.com/1443809/diff/1/dev/core/src/com/google/gwt/soyc/MakeTopLevelHtmlForPerm.java#newcode209
dev/core/src/com/google/gwt/soyc/MakeTopLevelHtmlForPerm.java:209:
outFile.print("b1(" + nameArray + ",");
Rename b1() and b2() to be one character names in order to save space in
the generated output.

http://gwt-code-reviews.appspot.com/1443809/diff/1/dev/core/src/com/google/gwt/soyc/MakeTopLevelHtmlForPerm.java#newcode279
dev/core/src/com/google/gwt/soyc/MakeTopLevelHtmlForPerm.java:279:
outFile.println("\"open\" : \"images/play-g16-down.png\",");
I assume these files will be checked in with the patch.

http://gwt-code-reviews.appspot.com/1443809/diff/1/dev/core/src/com/google/gwt/soyc/MakeTopLevelHtmlForPerm.java#newcode327
dev/core/src/com/google/gwt/soyc/MakeTopLevelHtmlForPerm.java:327:
outFile.println("document.write(\"http://gwt-code-reviews.appspot.com/1443809/diff/1/dev/core/src/com/google/gwt/soyc/resources/soyc.css
File dev/core/src/com/google/gwt/soyc/resources/soyc.css (right):

http://gwt-code-reviews.appspot.com/1443809/diff/1/dev/core/src/com/google/gwt/soyc/resources/soyc.css#newcode208
dev/core/src/com/google/gwt/soyc/resources/soyc.css:208:
.soyc-method-size-table {
? needed

http://gwt-code-reviews.appspot.com/1443809/

--
http://groups.google.com/group/Google-Web-Toolkit-Contributors