Apply updates to static methods.

Apply updates to static methods.

R=johnniwinther@google.com

Committed: https://code.google.com/p/dart/source/detail?r=41201

[ahe, 2014-10-15 15:32:16]

Patchset #1 (id:1) has been deleted

[ahe, 2014-10-15 15:36:55]

ahe@google.com changed reviewers:
+ johnniwinther@google.com

[ahe, 2014-10-15 15:36:56]

Notice that only one of the new test cases are passing. I started out working on
instance methods, but realized that supporting static methods comes before
instance methods. But I kept the instance method test.

[Johnni Winther, 2014-10-16 07:48:46]

lgtm

https://codereview.chromium.org/659813002/diff/20001/dart/pkg/dart2js_increme...
File dart/pkg/dart2js_incremental/lib/diff.dart (right):

https://codereview.chromium.org/659813002/diff/20001/dart/pkg/dart2js_increme...
dart/pkg/dart2js_incremental/lib/diff.dart:34: Token token;
Add comment on the semantics of [token].

https://codereview.chromium.org/659813002/diff/20001/dart/tests/try/web/incre...
File dart/tests/try/web/incremental_compilation_update_test.dart (right):

https://codereview.chromium.org/659813002/diff/20001/dart/tests/try/web/incre...
dart/tests/try/web/incremental_compilation_update_test.dart:208:
print({'update': update});
Remove debug code.

[ahe, 2014-10-16 09:19:48]

Thank you, Johnni!

https://codereview.chromium.org/659813002/diff/20001/dart/pkg/dart2js_increme...
File dart/pkg/dart2js_incremental/lib/diff.dart (right):

https://codereview.chromium.org/659813002/diff/20001/dart/pkg/dart2js_increme...
dart/pkg/dart2js_incremental/lib/diff.dart:34: Token token;
On 2014/10/16 07:48:46, Johnni Winther wrote:
> Add comment on the semantics of [token].

Done.

https://codereview.chromium.org/659813002/diff/20001/dart/tests/try/web/incre...
File dart/tests/try/web/incremental_compilation_update_test.dart (right):

https://codereview.chromium.org/659813002/diff/20001/dart/tests/try/web/incre...
dart/tests/try/web/incremental_compilation_update_test.dart:208:
print({'update': update});
On 2014/10/16 07:48:46, Johnni Winther wrote:
> Remove debug code.

I prefer to keep it. This is a test, after all, and this information is really
important in tracking down problems.

[ahe, 2014-10-20 13:24:05]

Caught a few type errors during testing in checked mode.

[ahe, 2014-10-20 15:21:01]

Committed patchset #3 (id:60001) manually as 41201 (presubmit successful).