Skip to content
This repository was archived by the owner on Feb 26, 2024. It is now read-only.

fix #536, add notification api patch#599

Merged
mhevery merged 1 commit intoangular:masterfrom
JiaLiPassion:webapi
Jan 17, 2017
Merged

fix #536, add notification api patch#599
mhevery merged 1 commit intoangular:masterfrom
JiaLiPassion:webapi

Conversation

@JiaLiPassion
Copy link
Copy Markdown
Collaborator

Patch notification API,
because EventTarget has been patched already, so

  notification.addEventListener

is already patched.

This PR just patched onclick, onerror property.

  Zone.current.fork({name: 'test'}).run(function() {
      n.onclick = function() {
          expect(Zone.current.name).toEqual('test');
      }
 });

@mhevery mhevery merged commit 83dfa97 into angular:master Jan 17, 2017
@JiaLiPassion JiaLiPassion deleted the webapi branch January 17, 2017 22:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants