Bug 727244: tabbrowser exception when dropping links on tabs, r=fryn, a=test bustage
authorGavin Sharp <gavin@gavinsharp.com>
Tue, 14 Feb 2012 15:08:32 -0800
changeset 87221 c00445fcd26d3d99b76328b0382a1248bc4c10d0
parent 87220 b740b95280281e90cdd5d8941b675581cff066b3
child 87222 51619e45fb5e4d2bfd4acb4fd16ee6e5d9d86c9d
push id880
push usergsharp@mozilla.com
push dateTue, 21 Feb 2012 18:29:28 +0000
treeherdermozilla-aurora@c00445fcd26d [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersfryn, test
bugs727244
milestone12.0a2
Bug 727244: tabbrowser exception when dropping links on tabs, r=fryn, a=test bustage
browser/base/content/tabbrowser.xml
--- a/browser/base/content/tabbrowser.xml
+++ b/browser/base/content/tabbrowser.xml
@@ -3562,18 +3562,20 @@
             } catch(ex) {
               // Just ignore invalid urls
             }
           }
         }
 
         // these offsets are only used in dragend, but we need to free them here
         // as well
-        delete draggedTab._dragOffsetX;
-        delete draggedTab._dragOffsetY;
+        if (draggedTab) {
+          delete draggedTab._dragOffsetX;
+          delete draggedTab._dragOffsetY;
+        }
       ]]></handler>
 
       <handler event="dragend"><![CDATA[
         // Note: while this case is correctly handled here, this event
         // isn't dispatched when the tab is moved within the tabstrip,
         // see bug 460801.
 
         // * mozUserCancelled = the user pressed ESC to cancel the drag