Fix bug in service worker push tests: don't cache subscription options

chrome/test/data/push_messaging/service_worker.js

Index: chrome/test/data/push_messaging/service_worker.js
diff --git a/chrome/test/data/push_messaging/service_worker.js b/chrome/test/data/push_messaging/service_worker.js
index f32068849a8fb2b3995e84fd895d82133efc97a0..15332d4478714b08199296d3cfb0f2686f81d92a 100644
--- a/chrome/test/data/push_messaging/service_worker.js
+++ b/chrome/test/data/push_messaging/service_worker.js
@@ -6,10 +6,6 @@
 // which would affect the scope of the importScripts call here.
 self.importScripts('/push_messaging/push_constants.js');
 
-var pushSubscriptionOptions = {
-    userVisibleOnly: true
-};
-
 // The "onpush" event currently understands two values as message payload
 // data coming from the test. Any other input is passed through to the
 // document unchanged.
@@ -47,6 +43,9 @@ this.onpush = function(event) {
 };
 
 self.addEventListener('message', function handler (event) {
+  let pushSubscriptionOptions = {
+      userVisibleOnly: true
+  };
   if (event.data.command == 'workerSubscribe') {
     pushSubscriptionOptions.applicationServerKey = kApplicationServerKey.buffer;
   } else if (event.data.command == 'workerSubscribeNoKey') {