From b40145e60a8a1d5e4a4dc0bfd5ea7dfd6a110e7f Mon Sep 17 00:00:00 2001
From: Jason Robbins <jrobbins@gmail.com>
Date: Thu, 25 Feb 2021 16:14:18 -0500
Subject: [PATCH] Remove a few more references to unused service worker.

---
 app.yaml                        | 23 -----------------------
 developer-documentation.md      | 13 ++++++-------
 static/firebase-messaging-sw.js |  9 ---------
 3 files changed, 6 insertions(+), 39 deletions(-)
 delete mode 100644 static/firebase-messaging-sw.js

diff --git a/app.yaml b/app.yaml
index a66ca0bf..7183978e 100644
--- a/app.yaml
+++ b/app.yaml
@@ -19,20 +19,6 @@ handlers:
   static_files: static/robots.txt
   upload: static/robots\.txt
 
-- url: /service-worker\.js
-  static_files: static/dist/service-worker.js
-  upload: static/dist/service-worker\.js
-  application_readable: true
-  expiration: "0m"
-  secure: always
-
-- url: /firebase-messaging-sw\.js
-  static_files: static/firebase-messaging-sw.js
-  upload: static/firebase-messaging-sw\.js
-  # application_readable: true
-  expiration: "0m"
-  secure: always
-
 - url: /static
   static_dir: static
   #expiration: 30s
@@ -57,10 +43,6 @@ handlers:
   script: notifier.app
   login: admin # Prevents raw access to this handler.
 
-- url: /admin/gae/.*
-  script: google.appengine.ext.admin.application
-  login: admin
-
 - url: /admin/blink.*
   script: blink_handler.app
   login: required # non-admin
@@ -90,11 +72,6 @@ handlers:
   script: users.app
   secure: always
 
-- url: /admin/.*
-  script: google.appengine.ext.admin.application
-  login: admin
-  secure: always
-
 # Main server handlers ---------------------------------------------------------
 # features.xml needs to support HTTP as well as HTTPS, as some RSS backends don't support SNI certs.
 # Same issue as what's described at https://github.com/Polymer/blog/issues/7
diff --git a/developer-documentation.md b/developer-documentation.md
index 1866945c..76dc958b 100644
--- a/developer-documentation.md
+++ b/developer-documentation.md
@@ -20,7 +20,6 @@ All the pages are rendered in a combination of Django template (`/templates`) an
 1. All lit-element components are in `/static/elements`.
 1. All JavaScript files are in `/static/js-src/` and processed by gulp, then output to '/static/js/' and get included in templates.
 1. All CSS files are in `/static/sass/` and processed by gulp, then output to `/static/css/` and get included in templates.
-1. A service worker is created by gulp too. Output in `/static/dist/service-worker.js`.
 
 ## Some nice-to-fix
 
@@ -80,19 +79,19 @@ All the pages are rendered in a combination of Django template (`/templates`) an
 *
 * visiting /cron/histograms calls HistogramsHandler which retrieves FeatureObserver and
 * FeatureObserver histograms from chromium.googlesource.com.
-* 
+*
 * ACTION REQUIRED: we will need to replace histogramID with the appropriate ID.
 * This can be obtained from uma.googleplex.com/data/histograms/ids-chrome-histograms.txt
 * Searching this file for CSS Properties, once our histogram has data should give us a hex
 * value which should be converted to an integer.
 *
-* 
+*
 
 
 ** featurelevel.js
 *
 * Creates charts for the feature level page.
-* 
+*
 * drawVisualization()
 * This function takes in the name of the property for which the graph is being drawn.
 * (This should probably be changed to the propertyID/bucketID in the future.)
@@ -107,7 +106,7 @@ All the pages are rendered in a combination of Django template (`/templates`) an
 * Takes the desired name to have displayed on the chart. Checks if there is a correspinding property (prefixed or unprefixed)
 * adds the given name plus the corresponding name to an array and returns the array.
 *
-** 
+**
 
 
 
@@ -122,7 +121,7 @@ OPTIMIZATIONS
 - recalc style issue perf in chrome 28 (https://src.chromium.org/viewvc/blink?revision=150018&view=revision)
   -> turn on shadow dom polyfill for stable (e.g. Platform = {flags: {shadow: 'polyfill'}};)
 - insertBefore issue being fixed: https://code.google.com/p/chromium/issues/detail?id=255734.
-  -> Remove inline <style> in elements. 
+  -> Remove inline <style> in elements.
 - Calculate a property in a *Changed handler rather than getters. Latter sets
   up timers if o.Observe() is not present. Former is only calculated once when
   the prop changes.
@@ -152,7 +151,7 @@ all elements in SD, then filtered on :hover. Change was to be more specific and
 .views {
     @include display-flex;
     @include flex-wrap(wrap);
-    
+
     & > span { // was & > * {
       @include display-flex;
       @include align-items(center);
diff --git a/static/firebase-messaging-sw.js b/static/firebase-messaging-sw.js
deleted file mode 100644
index 56448a82..00000000
--- a/static/firebase-messaging-sw.js
+++ /dev/null
@@ -1,9 +0,0 @@
-/* global importScripts, firebase */
-
-importScripts('https://www.gstatic.com/firebasejs/4.1.3/firebase-app.js');
-importScripts('https://www.gstatic.com/firebasejs/4.1.3/firebase-messaging.js');
-
-firebase.initializeApp({messagingSenderId: '999517574127'});
-
-// Handle background messages.
-firebase.messaging();