Revert "Cache content_scripts/vimium.css in chrome.storage.local."

This reverts commit 0a49cc45732175c65651b5f054c677d6c42a28c0.

1 files changed, 26 insertions, 3 deletions

diff --git a/content_scripts/ui_component.coffee b/content_scripts/ui_component.coffee
index 7422aada..1a7a1634 100644
--- a/content_scripts/ui_component.coffee
+++ b/content_scripts/ui_component.coffee
@@ -5,6 +5,7 @@ class UIComponent
   iframeFrameId: null
   options: null
   shadowDOM: null
+  styleSheetGetter: null
 
   toggleIframeElementClasses: (removeClass, addClass) ->
     @iframeElement.classList.remove removeClass
@@ -17,9 +18,10 @@ class UIComponent
       # Default to everything hidden while the stylesheet loads.
       styleSheet.innerHTML = "iframe {display: none;}"
 
-      # Fetch "content_scripts/vimium.css" from chrome.storage.local; the background page caches it there.
-      chrome.storage.local.get "vimiumCSSInChromeStorage", (items) ->
-        styleSheet.innerHTML = items.vimiumCSSInChromeStorage
+      # Use an XMLHttpRequest, possibly via the background page, to fetch the stylesheet. This allows us to
+      # catch and recover from failures that we could not have caught when using CSS @include (eg. #1817).
+      UIComponent::styleSheetGetter ?= new AsyncDataFetcher @fetchFileContents "content_scripts/vimium.css"
+      @styleSheetGetter.use (styles) -> styleSheet.innerHTML = styles
 
       @iframeElement = DomUtils.createElement "iframe"
       extend @iframeElement,
@@ -97,5 +99,26 @@ class UIComponent
         @options = null
         @postMessage "hidden" # Inform the UI component that it is hidden.
 
+  # Fetch a Vimium file/resource (such as "content_scripts/vimium.css").
+  # We try making an XMLHttpRequest request.  That can fail (see #1817), in which case we fetch the
+  # file/resource via the background page.
+  fetchFileContents: (file) -> (callback) ->
+    request = new XMLHttpRequest()
+
+    request.onload = ->
+      if request.status == 200
+        callback request.responseText
+      else
+        request.onerror()
+
+    request.onerror = ->
+      chrome.runtime.sendMessage
+        handler: "fetchFileContents"
+        fileName: file
+      , callback
+
+    request.open "GET", (chrome.runtime.getURL file), true
+    request.send()
+
 root = exports ? window
 root.UIComponent = UIComponent