Bug 824162 - Don't mess with focus until after the event target handles it. r=gavin, a=lsblakk
authorShane Caraveo <scaraveo@mozilla.com>
Sat, 26 Jan 2013 13:09:19 -0800
changeset 127364 13dc236d0fd868cca1809123f0c8df1318eae34b
parent 127363 481e98d4984e5fde7b5cbde6b61e352c4a3492c8
child 127365 b6269b3e8eefa01f09608b899689f293cffe3d24
push id2151
push userlsblakk@mozilla.com
push dateTue, 19 Feb 2013 18:06:57 +0000
treeherdermozilla-beta@4952e88741ec [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersgavin, lsblakk
bugs824162
milestone20.0a2
Bug 824162 - Don't mess with focus until after the event target handles it. r=gavin, a=lsblakk
browser/base/content/socialchat.xml
--- a/browser/base/content/socialchat.xml
+++ b/browser/base/content/socialchat.xml
@@ -76,17 +76,17 @@
       <method name="toggle">
         <body><![CDATA[
           this.minimized = !this.minimized;
         ]]></body>
       </method>
     </implementation>
 
     <handlers>
-      <handler event="focus" phase="capturing">
+      <handler event="focus">
         this.parentNode.selectedChat = this;
       </handler>
       <handler event="DOMTitleChanged"><![CDATA[
         this.setAttribute('label', this.iframe.contentDocument.title);
         this.parentNode.updateTitlebar(this);
       ]]></handler>
       <handler event="DOMLinkAdded"><![CDATA[
         // much of this logic is from DOMLinkHandler in browser.js