From 6591b6f4a6ce2435cc255be2e021723a119f7a40 Mon Sep 17 00:00:00 2001
From: "aaronleventhal%moonset.net" <aaronleventhal%moonset.net>
Date: Mon, 2 Aug 2004 17:56:43 +0000
Subject: [PATCH] Bug 244179. Tabindex not honoured if form element is focused
 via <label>. Buggy part of code pinpointed by Nian Liu. r=nian.liu@sun.com,
 sr=bryner

---
 content/events/src/nsEventStateManager.cpp | 12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)

diff --git a/content/events/src/nsEventStateManager.cpp b/content/events/src/nsEventStateManager.cpp
index a2c9038f88f..539a1e70fb2 100644
--- a/content/events/src/nsEventStateManager.cpp
+++ b/content/events/src/nsEventStateManager.cpp
@@ -4509,7 +4509,17 @@ nsEventStateManager::GetDocSelectionLocation(nsIContent **aStartContent,
   if (startContent) {
     rv = shell->GetPrimaryFrameFor(startContent, &startFrame);
     if (isCollapsed && NS_SUCCEEDED(rv)) {
-      // First check to see if our caret is at the very end of a node
+      // First check to see if we're in a <label>
+      // We don't want to return the selection in a label, because
+      // we we should be tabbing relative to what the label 
+      // points to (the current focus), not relative to the label itself.
+      nsIContent *parentContent = startContent;
+      while ((parentContent = parentContent->GetParent()) != nsnull) {
+        if (parentContent->Tag() == nsHTMLAtoms::label) {
+          return NS_OK; // Don't return selection location, we're on a label
+        }
+      }
+      // Next check to see if our caret is at the very end of a node
       // If so, the caret is actually sitting in front of the next
       // logical frame's primary node - so for this case we need to
       // change caretContent to that node.