Fix based on review comments

web-platform-tests · Feb 24, 2021 · b318f8c · b318f8c
1 parent 90988a7
commit b318f8c
Show file tree

Hide file tree

Showing 5 changed files with 16 additions and 16 deletions.
diff --git a/infrastructure/testdriver/set_timezone.html b/infrastructure/testdriver/set_timezone.html
@@ -12,7 +12,7 @@
 }
 async_test(t => {
   let timeZone = "Asia/Taipei";
-  let timeZone2 = "Asia/Hong_Kong";
+  let timeZone2 = "Pacific/Fakaofo";
   test_driver
     .set_time_zone(timeZone)
     .then(() => {

diff --git a/service-workers/timezonechange/resources/service-worker-timezonechange.js b/service-workers/timezonechange/resources/service-worker-timezonechange.js
@@ -1,14 +1,14 @@
 self.addEventListener('message', function(e) {
-    const message = e.data;
-    if ('port' in message) {
-      const port = message.port;
-      const oldTimeZone =
-          (new Intl.DateTimeFormat()).resolvedOptions().timeZone;
-      self.addEventListener('timezonechange', function(evt) {
-        const newTimeZone =
-          (new Intl.DateTimeFormat()).resolvedOptions().timeZone;
-        port.postMessage('SUCCESS:' + newTimeZone);
-      });
-      port.postMessage('READY:' + oldTimeZone);
-    }
+  const message = e.data;
+  if ('port' in message) {
+    const port = message.port;
+    const oldTimeZone =
+        (new Intl.DateTimeFormat()).resolvedOptions().timeZone;
+    self.addEventListener('timezonechange', function(evt) {
+      const newTimeZone =
+        (new Intl.DateTimeFormat()).resolvedOptions().timeZone;
+      port.postMessage('SUCCESS:' + newTimeZone);
+    });
+    port.postMessage('READY:' + oldTimeZone);
+  }
 });
diff --git a/timezonechange/addeventlistener-timezonechange-fired.html b/timezonechange/addeventlistener-timezonechange-fired.html
@@ -18,7 +18,7 @@
     window.addEventListener('timezonechange', r);
     window.test_driver.set_time_zone(newTimeZone);
   }).then(e => {
-      assert_equals(defaultTimeZone(), newTimeZone);
+    assert_equals(defaultTimeZone(), newTimeZone);
   });
 }, "Test that the timezonechange event fires on window.addEventListener('timezonechange')");
 </script>
diff --git a/timezonechange/resources/shared-worker-timezonechange.js b/timezonechange/resources/shared-worker-timezonechange.js
@@ -6,4 +6,4 @@ onconnect = connectEvent => {
     port.postMessage("SUCCESS:" + timezone);
   };
   port.postMessage("READY:" + oldtimezone);  // (the html will change the timezone)
-}
+};
diff --git a/timezonechange/resources/worker-timezonechange.js b/timezonechange/resources/worker-timezonechange.js
@@ -2,5 +2,5 @@ let oldtimezone = (new Intl.DateTimeFormat()).resolvedOptions().timeZone;
 ontimezonechange = evt => {
   let timezone = (new Intl.DateTimeFormat()).resolvedOptions().timeZone;
   postMessage("SUCCESS:" + timezone);
-}
+};
 postMessage("READY:" + oldtimezone);