Merge pull request #3233 from hypothesis/work-on-file-urls

Fix Hypothesis on local HTML documents

h/static/scripts/discovery.coffee

@@ -86,6 +86,15 @@ module.exports = class Discovery
   _onMessage: (event) =>
     {source, origin, data} = event
 
+    # If `origin` is 'null' the source frame is a file URL or loaded over some
+    # other scheme for which the `origin` is undefined. In this case, the only
+    # way to ensure the message arrives is to use the wildcard origin. See:
+    #
+    #   https://developer.mozilla.org/en-US/docs/Web/API/Window/postMessage
+    #
+    if origin is 'null'
+      origin = '*'
+
     # Check if the message is at all related to our discovery mechanism
     match = data.match? /^__cross_frame_dhcp_(discovery|offer|request|ack)(?::(\d+))?$/
     return unless match

h/static/scripts/test/discovery-test.coffee

@@ -74,6 +74,18 @@ describe 'Discovery', ->
       matcher = sinon.match(/__cross_frame_dhcp_ack:\d+/)
       assert.calledWith(fakeTopWindow.postMessage, matcher, 'top')
 
+    it 'sends an "ack" to the wildcard origin if a request comes from a frame with null origin', ->
+      fakeFrameWindow.addEventListener.yields({
+        data: '__cross_frame_dhcp_request'
+        source: fakeTopWindow
+        origin: 'null'
+      })
+      server.startDiscovery(->)
+
+      assert.called(fakeTopWindow.postMessage)
+      matcher = sinon.match(/__cross_frame_dhcp_ack:\d+/)
+      assert.calledWith(fakeTopWindow.postMessage, matcher, '*')
+
     it 'calls the discovery callback on receiving "request"', ->
       fakeFrameWindow.addEventListener.yields({
         data: '__cross_frame_dhcp_request'