In r618, I made a change to stop qualifying statics with the window.  This required me to do some comma-expression juggling in assignment statements, since a comma expression could now (illegally) be the LHS of an assignment. Well, I totally missed the fact that there is another context where an LHS is needed - modifying unary operations, "++" and "--".  This patch implements the juggling in these cases as well.  Putting the patch in neatly without duplicating a bunch of code required refactoring JsPrefixOperation and JsPostfixOperation to have a common superclass; most likely it should have always been that way.

While I was at it, I went ahead and reflected the refactorings into the Java AST, even though it's not critical for this patch.

Review by: mmendez
           alex.tkachman

git-svn-id: https://google-web-toolkit.googlecode.com/svn/trunk@728 8db76d5a-ed1c-0410-87a9-c151d255dfc7
9 files changed