commit 942d3d6b2c683e4b2afb6e8f554bce9df4eda223
author scottb@google.com <scottb@google.com@8db76d5a-ed1c-0410-87a9-c151d255dfc7> Wed Mar 10 23:31:25 2010 +0000
committer scottb@google.com <scottb@google.com@8db76d5a-ed1c-0410-87a9-c151d255dfc7> Wed Mar 10 23:31:25 2010 +0000
tree e4df2f5f8e993efc6e06e0350fbb1c240bee35cc
parent e5120fa01d351aee75e433b106146c202c1cb43a

Fixes JsDuplicateFunctionRemover to be more robust.

Any JsNameRef JsName that is *not* the immediate LHS of a JsInvocation is blacklisted, and therefore, cannot be replaced nor replace another ref. For example, defineSeed(ctor1, ctor2) would stop ctor1 or ctor2 from being considered, and so would ctor1.prototype = ... .

It looks like JsNameOf nodes can refer to functions as well, so these are blacklisted too.

http://gwt-code-reviews.appspot.com/169801
Patch by: cromwellian
Review by: me


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