From 072ef0a23df2d57f02cffe3b865e4355e7bc14e6 Mon Sep 17 00:00:00 2001
From: David Greenspan <dgreenspan@alum.mit.edu>
Date: Mon, 7 Jul 2014 11:56:11 -0700
Subject: [PATCH] Test focus and blur event cleanup

---
 packages/spacebars-tests/template_tests.html | 12 +++++
 packages/spacebars-tests/template_tests.js   | 54 ++++++++++++++++++++
 2 files changed, 66 insertions(+)

diff --git a/packages/spacebars-tests/template_tests.html b/packages/spacebars-tests/template_tests.html
index 775c9ad878..5aa0f609a1 100644
--- a/packages/spacebars-tests/template_tests.html
+++ b/packages/spacebars-tests/template_tests.html
@@ -868,3 +868,15 @@ Hi there!
     </div>
   </div>
 </template>
+
+<template name="spacebars_test_focus_blur_outer">
+  {{#if cond}}
+    a {{> spacebars_test_focus_blur_inner}}
+  {{else}}
+    b {{> spacebars_test_focus_blur_inner}}
+  {{/if}}
+</template>
+
+<template name="spacebars_test_focus_blur_inner">
+  <input type="text">
+</template>
diff --git a/packages/spacebars-tests/template_tests.js b/packages/spacebars-tests/template_tests.js
index 5bf25c4e06..133b620499 100644
--- a/packages/spacebars-tests/template_tests.js
+++ b/packages/spacebars-tests/template_tests.js
@@ -2409,3 +2409,57 @@ Tinytest.add(
     $(div.querySelector('.toremove')).remove();
     assertCallsAndListeners(0, 0, 0, 0);
   });
+
+Tinytest.add(
+  "spacebars-tests - template_tests - focus/blur get cleaned up",
+  function (test) {
+    var tmpl = Template.spacebars_test_focus_blur_outer;
+    var cond = ReactiveVar(true);
+    tmpl.cond = function () {
+      return cond.get();
+    };
+    var buf = [];
+    Template.spacebars_test_focus_blur_inner.events({
+      'focus input': function () {
+        buf.push('FOCUS');
+      },
+      'blur input': function () {
+        buf.push('BLUR');
+      }
+    });
+
+    var div = renderToDiv(tmpl);
+    document.body.appendChild(div);
+
+    // check basic focus and blur to make sure
+    // everything is sane
+    test.equal(div.querySelectorAll('input').length, 1);
+    var input;
+    focusElement(input = div.querySelector('input'));
+    // We don't get focus events when the Chrome Dev Tools are focused,
+    // unfortunately, as of Chrome 35.  I think this is a regression in
+    // Chrome 34.
+    if (buf.length === 0 && document.activeElement === input)
+      test.fail("NOTE: You might need to defocus the Chrome Dev Tools!");
+    test.equal(buf.join(), 'FOCUS');
+    blurElement(div.querySelector('input'));
+    test.equal(buf.join(), 'FOCUS,BLUR');
+
+    // now switch the IF and check again.  The failure mode
+    // we observed was that DOMBackend would not correctly
+    // unbind the old event listener at the jQuery level,
+    // so the old event listener would fire and cause an
+    // exception inside Blaze ("Must be attached" in
+    // DOMRange#containsElement), which would show up in
+    // the console and cause our handler not to fire.
+    cond.set(false);
+    buf.length = 0;
+    Deps.flush();
+    test.equal(div.querySelectorAll('input').length, 1);
+    focusElement(div.querySelector('input'));
+    test.equal(buf.join(), 'FOCUS');
+    blurElement(div.querySelector('input'));
+    test.equal(buf.join(), 'FOCUS,BLUR');
+
+    document.body.removeChild(div);
+  });