Skip to content

fix(zone.js): fix typo in zone_externs#40348

Closed
pwwpche wants to merge 1 commit intoangular:masterfrom
pwwpche:zonejs-extern-fix
Closed

fix(zone.js): fix typo in zone_externs#40348
pwwpche wants to merge 1 commit intoangular:masterfrom
pwwpche:zonejs-extern-fix

Conversation

@pwwpche
Copy link
Contributor

@pwwpche pwwpche commented Jan 8, 2021

This change fixes a typo in zone.js externs definition and avoids Closure
compiler renaming Task.cancelScheduleRequest() unexpectedly.

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • angular.io application / infrastructure changes
  • Other... Please describe:

What is the current behavior?

There are two scripts loaded in the browser:

  • lib.js: Containing zone.js, compiled with Closure, added zonejs_externs
  • app.js: Main app, containing a line "zone.cancelScheduleRequest()"

When app.js runs, getting error "zone.Xlb is not a function", because Closure renames cancelScheduleRequest().

Issue Number: N/A

What is the new behavior?

app.js loads without throwing this error.

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

This change fixes a typo in zone.js externs definition and avoids Closure
compiler renaming Task.cancelScheduleRequest() unexpectedly.
@google-cla google-cla bot added the cla: yes label Jan 8, 2021
@pullapprove pullapprove bot requested a review from mhevery January 8, 2021 02:24
@mhevery mhevery added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release labels Jan 8, 2021
@atscott atscott added the area: zones Issues related to zone.js label Jan 8, 2021
@ngbot ngbot bot added this to the Backlog milestone Jan 8, 2021
@atscott atscott closed this in 8116edb Jan 8, 2021
atscott pushed a commit that referenced this pull request Jan 8, 2021
This change fixes a typo in zone.js externs definition and avoids Closure
compiler renaming Task.cancelScheduleRequest() unexpectedly.

PR Close #40348
@pwwpche pwwpche deleted the zonejs-extern-fix branch January 8, 2021 21:57
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Feb 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker area: zones Issues related to zone.js cla: yes target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants