Allow const factory on methods in parser

Allow const factory on methods in parser

This got left out of an earlier patch.

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

[scheglov, 2012-03-14 00:36:29]

LGTM

[srdjan, 2012-03-14 00:41:21]

https://chromiumcodereview.appspot.com/9704006/diff/2001/compiler/java/com/go...
File compiler/java/com/google/dart/compiler/parser/DartParser.java (right):

https://chromiumcodereview.appspot.com/9704006/diff/2001/compiler/java/com/go...
compiler/java/com/google/dart/compiler/parser/DartParser.java:936: }
I do not understand the code completely. Does it guarantee that only native
factories can be const? Non-native factories are not allowed to be const.

[zundel, 2012-03-14 00:43:43]

https://chromiumcodereview.appspot.com/9704006/diff/2001/compiler/java/com/go...
File compiler/java/com/google/dart/compiler/parser/DartParser.java (right):

https://chromiumcodereview.appspot.com/9704006/diff/2001/compiler/java/com/go...
compiler/java/com/google/dart/compiler/parser/DartParser.java:936: }
On 2012/03/14 00:41:22, srdjan wrote:
> I do not understand the code completely. Does it guarantee that only native
> factories can be const? Non-native factories are not allowed to be const.

I added the 'native' check in the resolver in a previous patch to Resolver.java.
 This was enough to allow frog code to parse because it never used the 'factory'
modifier on native constructors.  I forgot to allow both 'const' and 'factory'
modifiers in the parse stage.