per spec and the code in here, it makes more sense that the default phase is 3 (bubbling), not 0 (a non-existent phase)
authorAndrew Sutherland <asutherland@asutherland.org>
Tue, 13 Jan 2009 03:47:24 -0800
changeset 16 608d65fc4ff224d3c60dc30e0f07bdeb8bdb8e98
parent 15 61a9fa6989452b168f9312407e110913014f4f25
child 17 5a832fd81f4173a9cdb61dd2b274ea43b407a981
push id1
push userroot
push dateWed, 08 Apr 2009 01:46:05 +0000
per spec and the code in here, it makes more sense that the default phase is 3 (bubbling), not 0 (a non-existent phase)
client/xbl-src.js
--- a/client/xbl-src.js
+++ b/client/xbl-src.js
@@ -472,17 +472,17 @@ cXBLLanguage.elements.xbl.binding.handle
 
 			if (fHandler) {
 				cBinding.$handlers[sName].push(fHandler);
 
 				// Get attributes
 				var sValue;
 				// Event
 				if (sValue = oNode.getAttribute("phase"))
-					fHandler["phase"]			= sValue == "capture" ? 1 : sValue == "target" ? 2 /* : sValue == "default-action" ? 0x78626C44*/ : 0;
+					fHandler["phase"]			= sValue == "capture" ? 1 : sValue == "target" ? 2 /* : sValue == "default-action" ? 0x78626C44*/ : 3;
 				if (sValue = oNode.getAttribute("trusted"))
 					fHandler["trusted"]			= sValue == "true";
 				if (sValue = oNode.getAttribute("propagate"))
 					fHandler["propagate"]		= sValue != "stop";
 				if (sValue = oNode.getAttribute("default-action"))
 					fHandler["default-action"]	= sValue != "cancel";
 				// MouseEvent
 				if (sValue = oNode.getAttribute("button"))