From 7c9064f5b88e5cb1fa20e2119d4020b394584ef3 Mon Sep 17 00:00:00 2001 From: Philip Tellis Date: Mon, 16 Jan 2012 20:08:30 -0800 Subject: [PATCH] Fix issue 18: beacon not sent if using https When the page_ready event fires, all event subscribers are called in order. By default, the order is: - BOOMR.plugins.RT.done - BOOMR.plugins.BW.run each is supposed to call BOOMR.sendBeacon() when done. The bug in the https flow is that the BW plugin never calls BOOMR.sendBeacon() in that block. --- boomerang.js | 1 + 1 file changed, 1 insertion(+) diff --git a/boomerang.js b/boomerang.js index f56e1e12a..9429c8cff 100644 --- a/boomerang.js +++ b/boomerang.js @@ -1237,6 +1237,7 @@ BOOMR.plugins.BW = { BOOMR.info("HTTPS detected, skipping bandwidth test", "bw"); impl.complete = true; + BOOMR.sendBeacon(); return this; }