From 774915f3967655ab800cc3c1ac73f0746618d3de Mon Sep 17 00:00:00 2001
From: Stephen Blott
Date: Tue, 30 Dec 2014 06:07:20 +0000
Subject: Minor changes to vomnibar-in-iframe.

From top to bottom on the diff:

- The echo handler on the background page is no longer required.
- Simplify/refactor vomnibarUI message handler.
- Initialise vomnibar query to "" (rather than null) and simplify.
- No need to focus parent window when vomnibar closes; that's handled by
  the iframe framework.  Also no need to blur.
---
 content_scripts/vomnibar.coffee | 10 ++--------
 1 file changed, 2 insertions(+), 8 deletions(-)

(limited to 'content_scripts')

diff --git a/content_scripts/vomnibar.coffee b/content_scripts/vomnibar.coffee
index 6b82d31c..0d5197a5 100644
--- a/content_scripts/vomnibar.coffee
+++ b/content_scripts/vomnibar.coffee
@@ -37,12 +37,8 @@ Vomnibar =
 
   init: ->
     unless @vomnibarUI?
-      @vomnibarUI = new UIComponent "pages/vomnibar.html", "vomnibarFrame", @handleMessage.bind this
-
-  handleMessage: (event) ->
-    if event.data == "hide"
-      @hide()
-
+      @vomnibarUI = new UIComponent "pages/vomnibar.html", "vomnibarFrame", =>
+        @vomnibarUI.hide()
 
   # This function opens the vomnibar. It accepts options, a map with the values:
   #   completer   - The completer to fetch results from.
@@ -51,7 +47,5 @@ Vomnibar =
   #   newTab      - Optional, boolean. Whether to open the result in a new tab.
   open: (options) -> @vomnibarUI.activate options
 
-  hide: -> @vomnibarUI?.hide()
-
 root = exports ? window
 root.Vomnibar = Vomnibar
-- 
cgit v1.2.3