[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation changes
[ ] Other... Please describe:
PR Checklist
Please check your PR fulfills the following requirements:
Bugfixes:
[ ] Regression testing has been carried out using the example project to ensure the intended bug is fixed and no regression bugs have been inadvertently introduced.
Didnt run regression but might not be necessary for the change type.
New features/enhancements:
[ ] Exhaustive testing has been carried out for the new functionality
[ ] Regression testing has been carried out to ensure no existing functionality is adversely affected
[ ] Documentation has been added / updated
[ ] The example project has been update to validate/demonstrate the new functionality.
What is the purpose of this PR?
The current release of this plugin fails with the default version of Chrome that comes with SDK 24 & 25.
PR Type
What kind of change does this PR introduce?
PR Checklist
Please check your PR fulfills the following requirements:
Bugfixes:
Didnt run regression but might not be necessary for the change type.
New features/enhancements:
What is the purpose of this PR?
The current release of this plugin fails with the default version of Chrome that comes with SDK 24 & 25.
This issue is cause by the trailing comma in the function parameters. Support for trailing commas in function parameters was adde in Chrome 58.
SDK 24: WebView 53 (fails) SDK 25: WebView 55 (fails) SDK 26: WebView 58 (works)
Since the latest Cordova Android's minimum SDK support 24, and the type of change is minor, I thought maybe its OK to request.
Does this PR introduce a breaking change?
What testing has been done on the changes in the PR?
Manually deleting the trailing comma and rebuilt the app.
What testing has been done on existing functionality?
n/a
Other information