From 8d96c2ac609a733435a7b0c89bf1219c8a80cc29 Mon Sep 17 00:00:00 2001
From: Alexander Surkov <surkov.alexander@gmail.com>
Date: Sat, 10 Nov 2012 02:37:27 +0900
Subject: [PATCH] Bug 757774 - Fire state change event when link is traversed,
 r=tbsaunde

---
 accessible/src/generic/DocAccessible.cpp      |  8 ++-
 .../tests/mochitest/states/test_link.html     | 62 +++++++++++++------
 2 files changed, 51 insertions(+), 19 deletions(-)

diff --git a/accessible/src/generic/DocAccessible.cpp b/accessible/src/generic/DocAccessible.cpp
index d78ca014c49..e89acfa16da 100644
--- a/accessible/src/generic/DocAccessible.cpp
+++ b/accessible/src/generic/DocAccessible.cpp
@@ -1264,7 +1264,13 @@ DocAccessible::ContentStateChanged(nsIDocument* aDocument,
     nsRefPtr<AccEvent> event =
       new AccStateChangeEvent(aContent, states::INVALID, true);
     FireDelayedAccessibleEvent(event);
-   }
+  }
+
+  if (aStateMask.HasState(NS_EVENT_STATE_VISITED)) {
+    nsRefPtr<AccEvent> event =
+      new AccStateChangeEvent(aContent, states::TRAVERSED, true);
+    FireDelayedAccessibleEvent(event);
+  }
 }
 
 void
diff --git a/accessible/tests/mochitest/states/test_link.html b/accessible/tests/mochitest/states/test_link.html
index 4ceecf704e2..a438952ef75 100644
--- a/accessible/tests/mochitest/states/test_link.html
+++ b/accessible/tests/mochitest/states/test_link.html
@@ -23,6 +23,40 @@
   <script type="application/javascript">
     //gA11yEventDumpToConsole = true; // debug stuff
 
+    function closeDocChecker()
+    {
+      this.__proto__ = new asyncInvokerChecker(EVENT_DOCUMENT_LOAD_COMPLETE);
+
+      this.check = function closeDocChecker_check(aEvent)
+      {
+        aEvent.accessible.rootDocument.window.close();
+      }
+
+      this.match = function closeDocChecker_match(aEvent)
+      {
+        return true;
+      }
+    }
+
+    function clickLink(aID)
+    {
+      this.eventSeq = [
+        new stateChangeChecker(STATE_TRAVERSED, false, true, "link_traversed"),
+        new closeDocChecker()
+      ];
+
+      this.invoke = function clickLink_invoke()
+      {
+        synthesizeMouse(getNode("link_traversed"), 1, 1, { shiftKey: true });
+      }
+
+      this.getID = function clickLink_getID()
+      {
+        return "link + '" + aID + "' clicked.";
+      }
+    }
+
+    var gQueue = null;
     function doTest()
     {
       // a@href and its text node
@@ -47,28 +81,15 @@
       // a (no @href, no click event listener)
       testStates("link_notlink", 0, 0, STATE_LINKED);
 
-      // a: traversal state
+      // a: no traversed state
       testStates("link_traversed", 0, 0, STATE_TRAVERSED);
 
-      //enableLogging("docload"); // debug stuff
-      registerA11yEventListener(EVENT_DOCUMENT_LOAD_COMPLETE,
-                                traversedLinkTester);
-
-      synthesizeMouse(getNode("link_traversed"), 1, 1, { shiftKey: true });
+      // a: traversed state
+      gQueue = new eventQueue();
+      gQueue.push(new clickLink("link_traversed"));
+      gQueue.invoke(); // will call SimpleTest.finsih();
     }
 
-    var traversedLinkTester = {
-      handleEvent: function traversedLinkTester_handleEvent(aEvent) {
-        unregisterA11yEventListener(EVENT_DOCUMENT_LOAD_COMPLETE,
-                                    traversedLinkTester);
-        aEvent.accessible.rootDocument.window.close();
-
-        testStates("link_traversed", STATE_TRAVERSED);
-        //disableLogging();
-        SimpleTest.finish();
-      }
-    };
-
     SimpleTest.waitForExplicitFinish();
     addA11yLoadEvent(doTest);
   </script>
@@ -87,6 +108,11 @@
      title="Calculate link states separately">
     Mozilla Bug 754830
   </a>
+  <a target="_blank"
+     href="https://bugzilla.mozilla.org/show_bug.cgi?id=757774"
+     title="Fire state change event when link is traversed">
+    Mozilla Bug 757774
+  </a>
   <p id="display"></p>
   <div id="content" style="display: none"></div>
   <pre id="test">