Apply suggestions from code review

Co-authored-by: Robert Knight <robertknight@gmail.com>

src/shared/port-provider.js

@@ -15,7 +15,7 @@ import { isMessageEqual } from './port-util';
  * frames.
  *
  * There are 4 types of frames:
- * - `host`: frame where the hypothesis client is initially loaded.
+ * - `host`: frame where the Hypothesis client is initially loaded.
  * - `guest`: frames with annotatable content. In some instances a `guest`
  *    frame can be the same as the `host` frame, in other cases, it is an iframe
  *    where either (1) the hypothesis client has been injected or (2) the
@@ -33,7 +33,7 @@ import { isMessageEqual } from './port-util';
  * |-> `notebook` iframe
  * |-> [`guest` iframes]
  *
- * Currently, we support the communication between the following pairs of frames:
+ * Currently, we support communication between the following pairs of frames:
  * - `guest-host`
  * - `guest-sidebar`
  * - `host-sidebar`
@@ -65,7 +65,7 @@ export class PortProvider {
   /**
    * @param {string} hypothesisAppsOrigin - the origin of the hypothesis apps
    *   is use to send the notebook and sidebar ports to only the frames that
-   *   matches the origin.
+   *   match the origin.
    */
   constructor(hypothesisAppsOrigin) {
     this._hypothesisAppsOrigin = hypothesisAppsOrigin;

src/shared/test/port-finder-test.js

@@ -28,7 +28,7 @@ describe('PortFinder', () => {
   });
 
   describe('#destroy', () => {
-    it('ignores `offer` messages of ports', async () => {
+    it('ignores subsequent `offer` messages of ports', async () => {
       let error;
       const channel = 'host-sidebar';
       const port = 'sidebar';

src/shared/test/port-provider-test.js

@@ -208,7 +208,7 @@ describe('PortProvider', () => {
       );
     });
 
-    it('responds to the first valid port request, ignore additional requests', async () => {
+    it('responds to the first valid port request but ignores additional requests', async () => {
       const data = {
         channel: 'guest-host',
         port: 'guest',