docs: fix grammar and spelling mistakes (#895)

DEVELOPER.md

@@ -62,7 +62,7 @@ chmod u+x .git/hooks/pre-commit
 Webdriver Test
 --------------
 
-Now `zone.js` also support run webdriver e2e test.
+`zone.js` also supports running webdriver e2e tests.
 
 1. run locally
 

MODULE.md

@@ -1,9 +1,8 @@
 # Modules
 
-zone.js patches the async APIs which described above, but those patch will have some overhead,
-from zone.js v0.8.9, you can choose which web API module you want to patch, for example, 
-the below samples show how to disable some modules, you just need to define some global variables 
-before load zone.js.
+Starting from zone.js v0.8.9, you can choose which web API modules you want to patch as to reduce overhead introduced by the patching of these modules. For example, 
+the below samples show how to disable some modules. You just need to define a few global variables 
+before loading zone.js.
 
 ```
   <script>
@@ -17,7 +16,7 @@ before load zone.js.
   <script src="../dist/zone.js"></script>
 ```
 
-Below is the full list of current support modules.
+Below is the full list of currently supported modules.
 
 - Common 
 
@@ -58,7 +57,7 @@ Below is the full list of current support modules.
 
 - on_property
 
-you can also disable specified on_property by setting `__Zone_ignore_on_properties`, for example,
+You can also disable specific on_properties by setting `__Zone_ignore_on_properties` as follows: for example,
 if you want to disable `window.onmessage` and `HTMLElement.prototype.onclick` from zone.js patching,
 you can do like this.
 
@@ -79,7 +78,7 @@ you can do like this.
 
 - Angular(2+)
 
-Angular use zone.js to manage async operations and decide when to perform change detection, so in Angular, 
+Angular uses zone.js to manage async operations and decide when to perform change detection. Thus, in Angular, 
 the following APIs should be patched, otherwise Angular may not work as expected.
 
 1. ZoneAwarePromise

NON-STANDARD-APIS.md

@@ -1,10 +1,9 @@
 # Zone.js's support for non standard apis
 
-Zone.js patched most standard APIs so they can be in zone. such as DOM events listener, XMLHttpRequest in Browser
- and EventEmitter, fs API in nodejs. 
+Zone.js patched most standard APIs such as DOM event listeners, XMLHttpRequest in Browser, EventEmitter and fs API in Node.js so they can be in zone.
 
-But there are still a lot of non standard APIs are not patched by default, such as MediaQuery, Notification, 
- WebAudio and so on. We are adding the support to those APIs, and the progress will be updated here.
+But there are still a lot of non standard APIs that are not patched by default, such as MediaQuery, Notification, 
+ WebAudio and so on. We are adding support to those APIs, and our progress is updated here.
 
 ## Currently supported non standard Web APIs 
 
@@ -95,9 +94,9 @@ to load the patch, you should load in the following order.
 ## Usage
 
 By default, those APIs' support will not be loaded in zone.js or zone-node.js,
-so if you want to load those API's support, you should load those files by yourself
+so if you want to load those API's support, you should load those files by yourself.
 
-for example, if you want to add MediaQuery patch, you should do like this. 
+For example, if you want to add MediaQuery patch, you should do like this: 
 
 ```
   <script src="path/zone.js"></script> 
@@ -107,7 +106,7 @@ for example, if you want to add MediaQuery patch, you should do like this.
 * rxjs
 
 `zone.js` also provide a `rxjs` patch to make sure rxjs Observable/Subscription/Operator run in correct zone.
-for detail please refer to [pull request 843](https://github.com/angular/zone.js/pull/843), the following sample code describe the idea.
+For details please refer to [pull request 843](https://github.com/angular/zone.js/pull/843). The following sample code describes the idea.
 
 ```
 const constructorZone = Zone.current.fork({name: 'constructor'});
@@ -143,19 +142,19 @@ operatorZone.run(() => {
 });
 ```
 
-currently basically all `rxjs` API include
+Currently basically everything the `rxjs` API includes
 
 - Observable
 - Subscription
 - Subscriber
 - Operators 
 - Scheduler 
 
-are patched, so they will run in the correct zone.
+is patched, so each asynchronous call will run in the correct zone.
 
 ## Usage.
 
-for example, in angular application, you can load this patch in your `app.module.ts`.
+For example, in an Angular application, you can load this patch in your `app.module.ts`.
 
 ```
 import 'zone.js/dist/zone-patch-rxjs';

README.md

@@ -7,7 +7,7 @@ Implements _Zones_ for JavaScript, inspired by [Dart](https://www.dartlang.org/a
 
 > If you're using zone.js via unpkg please provide a query param `?main=browser`  
 `https://unpkg.com/zone.js?main=browser`  
-> If you're using the following library, make sure you import them first 
+> If you're using any of the following libraries, make sure you import them first 
 
 > * 'newrelic' as it patches global.Promise before zone.js does
 > * 'async-listener' as it patches global.setTimeout, global.setInterval before zone.js does
@@ -37,19 +37,19 @@ See this video from ng-conf 2014 for a detailed explanation:
 ## Standard API support
 
 zone.js patched most standard web APIs(such as DOM events, XMLHttpRequest...) and nodejs APIs 
-(EventEmitter,fs ...), for more details, please see [STANDARD-APIS.md](STANDARD-APIS.md) for 
-more details.
+(EventEmitter,fs ...), for more details, please see [STANDARD-APIS.md](STANDARD-APIS.md).
 
 ## Non standard API support
 
-And now we are adding support to some non standard APIs, such as MediaQuery,
+We are adding support to some non standard APIs, such as MediaQuery and
 Notification. Please see [NON-STANDARD-APIS.md](NON-STANDARD-APIS.md) for more details.
 
 ## Modules
 
-zone.js patches the async APIs which described above, but those patch will have some overhead,
-from zone.js v0.8.9, you can choose which web API module you want to patch, for detail, please 
-see [MODULE.md](MODULE.md) for more details.
+zone.js patches the async APIs described above, but those patches will have some overhead.
+Starting from zone.js v0.8.9, you can choose which web API module you want to patch. 
+For more details, please 
+see [MODULE.md](MODULE.md).
 
 ## Promise A+ test passed
 [![Promises/A+ 1.1 compliant](https://promisesaplus.com/assets/logo-small.png)](https://promisesaplus.com/)

STANDARD-APIS.md

@@ -1,24 +1,22 @@
 # Zone.js's support for standard apis
 
-Zone.js patched most standard APIs so they can be in zone. such as DOM events listener, XMLHttpRequest in Browser
- and EventEmitter, fs API in nodejs. 
+Zone.js patched most standard APIs such as DOM event listeners, XMLHttpRequest in Browser, EventEmitter and fs API in Node.js so they can be in zone.
 
-In this document, all patched API will be listed here. 
+In this document, all patched API are listed. 
 
-for non standard API, please reference to [NON-STANDARD-APIS.md](NON-STANDARD-APIS.md)
+For non-standard APIs, please see [NON-STANDARD-APIS.md](NON-STANDARD-APIS.md)
 
-## Patch Mechanism 
+## Patch Mechanisms
 
-There are several patch mechanism
+There are several patch mechanisms
 
-- wrap, wrap will make callback in zone, and application can receive onInvoke, onIntercept callback 
-- Task, just like javascript vm, application can receive onScheduleTask, onInvokeTask, onCancelTask, onHasTask callback 
+- wrap: makes callbacks run in zones, and makes applications able to receive onInvoke and onIntercept callbacks
+- Task: just like in the JavaScript VM, applications can receive onScheduleTask, onInvokeTask, onCancelTask and onHasTask callbacks
   1. MacroTask
-  2. MicroTask     
+  2. MicroTask
   3. EventTask
 
-Now there are some APIs should be treated as Tasks, but still be patched in wrap way, they will be 
-patched as Task soon.
+Some APIs which should be treated as Tasks, but are currently still patched in the wrap way. These will be patched as Tasks soon.
 
 ## Browser 
 
@@ -54,9 +52,9 @@ Web APIs
 
 EventTarget
 
-- For browser support eventTarget, zone.js just patch EventTarget, so everything 
-inherit from EventTarget will also be patched.
-- For browser does not support eventTarget, zone.js will patch the following APIs in the IDL
+- For browsers supporting EventTarget, Zone.js just patches EventTarget, so everything that inherits
+from EventTarget will also be patched.
+- For browsers that do not support EventTarget, Zone.js will patch the following APIs in the IDL
  that inherit from EventTarget
 
  |||||
@@ -69,8 +67,7 @@ inherit from EventTarget will also be patched.
  |IDBRequest|IDBOpenDBRequest|IDBDatabase|IDBTransaction|
  |IDBCursor|DBIndex|WebSocket|
 
-on properties, such as onclick, onreadystatechange, the following on properties will 
-be patched as EventTask, the following is the on properties zone.js patched  
+The following 'on' properties, such as onclick, onreadystatechange, are patched in Zone.js as EventTasks
 
  |||||
  |---|---|---|---|
@@ -104,28 +101,28 @@ be patched as EventTask, the following is the on properties zone.js patched
 | crypto | MacroTask |  |
 | fs | MacroTask | all async methods are patched |
 
-EventEmitter, addEventListener, prependEventListener, on, once will be patched as EventTask, and removeEventListener,
-removeAllListeners will remove those eventTasks
+EventEmitter, addEventListener, prependEventListener and 'on' will be patched once as EventTasks, and removeEventListener and
+removeAllListeners will remove those EventTasks
 
 ## Electron 
 
-zone.js did not patch electron API, but in electron, both browser APIs and node APIs are patched, so 
-when you want to include zone.js in Electron, please use dist/zone-mix.js
+Zone.js does not patch the Electron API, although in Electron both browser APIs and node APIs are patched, so 
+if you want to include Zone.js in Electron, please use dist/zone-mix.js
 
 ## ZoneAwareError
 
-ZoneAwareError replace global Error, and it will add zone information to stack trace.
-ZoneAwareError will also handle 'this' issue.
-such as create an error without new, then this will be undefined in strict mode, and global in
-non-strict mode. it will cause some issues and very difficult to detect.
+ZoneAwareError replaces global Error, and adds zone information to stack trace.
+ZoneAwareError also handles 'this' issue.
+This type of issue would happen when creating an error without `new`: `this` would be `undefined` in strict mode, and `global` in
+non-strict mode. It could cause some very difficult to detect issues.
 
 ```javascript
   const error = Error(); 
 ```
 
-ZoneAwareError will make sure that `this` is ZoneAwareError even without new.
+ZoneAwareError makes sure that `this` is ZoneAwareError even without new.
 
 ## ZoneAwarePromise
 
-ZoneAwarePromise wrap the global Promise and make it in zone and run as microTask, 
-it also passes promise A+ tests.
+ZoneAwarePromise wraps the global Promise and makes it run in zones as a MicroTask.
+It also passes promise A+ tests.

lib/closure/zone_externs.js

@@ -178,7 +178,7 @@ var ZoneSpec = function() {};
 ZoneSpec.prototype.name;
 
 /**
- * @type {Object<string, Object>|undefined} A set of properties to be associated with Zone. Use [Zone.get] to retrive them.
+ * @type {Object<string, Object>|undefined} A set of properties to be associated with Zone. Use [Zone.get] to retrieve them.
  */
 ZoneSpec.prototype.properties;
 
@@ -266,21 +266,21 @@ var ZoneDelegate = function() {};
  */
 ZoneDelegate.prototype.zone;
 /**
- * @param {!Zone} targetZone the [zone] which originally received the request.
+ * @param {!Zone} targetZone the [Zone] which originally received the request.
  * @param {!ZoneSpec} zoneSpec the argument passed into the `fork` method.
  * @returns {!Zone} the new forked zone
  */
 ZoneDelegate.prototype.fork = function(targetZone, zoneSpec) {};
 /**
- * @param {!Zone} targetZone the [zone] which originally received the request.
+ * @param {!Zone} targetZone the [Zone] which originally received the request.
  * @param {!Function} callback the callback function passed into `wrap` function
- * @param {string=} source the argument passed into the `warp` method.
+ * @param {string=} source the argument passed into the `wrap` method.
  * @returns {!Function} 
  */
 ZoneDelegate.prototype.intercept = function(targetZone, callback, source) {};
 
 /**
- * @param {Zone} targetZone the [zone] which originally received the request.
+ * @param {Zone} targetZone the [Zone] which originally received the request.
  * @param {!Function} callback the callback which will be invoked.
  * @param {?Object=} applyThis the argument passed into the `run` method.
  * @param {?Array=} applyArgs the argument passed into the `run` method.
@@ -289,14 +289,14 @@ ZoneDelegate.prototype.intercept = function(targetZone, callback, source) {};
  */
 ZoneDelegate.prototype.invoke = function(targetZone, callback, applyThis, applyArgs, source) {};
 /**
- * @param {!Zone} targetZone the [zone] which originally received the request.
- * @param {!Object} error the argument passed into the `handleerror` method.
+ * @param {!Zone} targetZone the [Zone] which originally received the request.
+ * @param {!Object} error the argument passed into the `handleError` method.
  * @returns {boolean}
  */
 ZoneDelegate.prototype.handleError = function(targetZone, error) {};
 /**
- * @param {!Zone} targetZone the [zone] which originally received the request.
- * @param {!Task} task the argument passed into the `scheduletask` method.
+ * @param {!Zone} targetZone the [Zone] which originally received the request.
+ * @param {!Task} task the argument passed into the `scheduleTask` method.
  * @returns {!Task} task
  */
 ZoneDelegate.prototype.scheduleTask = function(targetZone, task) {};