Bug 189292 – isAsyncGeneratorMethodParseMode() should check for SourceParseMode::AsyncGeneratorWrapperMethodMode.

• Status: RESOLVED
• Resolution: FIXED
• Component: JavaScriptCore

[Mark Lam]

Currently, it's checking for SourceParseMode::AsyncGeneratorWrapperFunctionMode instead.

<rdar://problem/38907433>

[Mark Lam]

Created attachment 348881 [details]
proposed patch.

[Saam Barati]

Comment on attachment 348881 [details]
proposed patch.

View in context: https://bugs.webkit.org/attachment.cgi?id=348881&action=review

r=me

Did this make any test262 tests pass?

> Source/JavaScriptCore/ChangeLog:9
> +        Previously, it's checking for SourceParseMode::AsyncGeneratorWrapperFunctionMode instead.

You need to rewrite this sentence

> Source/JavaScriptCore/parser/ParserModes.h:192
> +        SourceParseMode::AsyncGeneratorWrapperMethodMode).contains(parseMode);

Why on two lines? The indentation here looks weird (though perhaps that’s a bugzilla bug)

[Mark Lam]

Comment on attachment 348881 [details]
proposed patch.

View in context: https://bugs.webkit.org/attachment.cgi?id=348881&action=review

Thanks for the review.

>> Source/JavaScriptCore/parser/ParserModes.h:192
>> +        SourceParseMode::AsyncGeneratorWrapperMethodMode).contains(parseMode);
> 
> Why on two lines? The indentation here looks weird (though perhaps that’s a bugzilla bug)

I'll put it back on 1 line.

[Mark Lam]

(In reply to Saam Barati from comment #2)
> Did this make any test262 tests pass?

I'm running test262 now.

[Mark Lam]

(In reply to Mark Lam from comment #4)
> (In reply to Saam Barati from comment #2)
> > Did this make any test262 tests pass?
> 
> I'm running test262 now.

The test262 results:

NEW PASS test/intl402/PluralRules/prototype/resolvedOptions/pluralCategories.js (strict mode)
NEW PASS test/intl402/PluralRules/prototype/resolvedOptions/pluralCategories.js (default)
NEW PASS test/intl402/NumberFormat/prototype/formatToParts/length.js (strict mode)
NEW PASS test/intl402/NumberFormat/prototype/formatToParts/formatToParts.js (strict mode)
NEW PASS test/intl402/NumberFormat/prototype/formatToParts/default-parameter.js (strict mode)
NEW PASS test/intl402/NumberFormat/prototype/formatToParts/name.js (strict mode)
NEW PASS test/intl402/NumberFormat/prototype/formatToParts/return-abrupt-tonumber.js (strict mode)
NEW PASS test/intl402/NumberFormat/prototype/formatToParts/main.js (strict mode)
NEW PASS test/intl402/NumberFormat/prototype/formatToParts/length.js (default)
NEW PASS test/intl402/NumberFormat/prototype/formatToParts/name.js (default)
NEW PASS test/intl402/NumberFormat/prototype/formatToParts/formatToParts.js (default)
NEW PASS test/intl402/NumberFormat/prototype/formatToParts/default-parameter.js (default)
NEW PASS test/intl402/NumberFormat/prototype/formatToParts/return-abrupt-tonumber.js (default)
NEW PASS test/intl402/NumberFormat/prototype/formatToParts/main.js (default)

49176 tests run
5031 test files skipped
2375 tests failed in total
0 tests newly fail
14 tests newly pass

These 14 new passes are all in the intl402 spec.  I don't think they have anything to do with my patch.

[Mark Lam]

Created attachment 348896 [details]
patch for landing.

[WebKit Commit Bot]

Comment on attachment 348896 [details]
patch for landing.

Clearing flags on attachment: 348896

Committed r235662: <https://trac.webkit.org/changeset/235662>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.