Bug 638465 - do not ignore inital documents, r=tbsaunde

accessible/src/base/DocManager.cpp

@@ -271,11 +271,6 @@ DocManager::HandleEvent(nsIDOMEvent* aEvent)
       logging::DocDestroy("received 'pagehide' event", document);
 #endif
 
-    // Ignore 'pagehide' on temporary documents since we ignore them entirely in
-    // accessibility.
-    if (document->IsInitialDocument())
-      return NS_OK;
-
     // Shutdown this one and sub document accessibles.
 
     // We're allowed to not remove listeners when accessible document is
@@ -370,10 +365,8 @@ DocManager::RemoveListeners(nsIDocument* aDocument)
 DocAccessible*
 DocManager::CreateDocOrRootAccessible(nsIDocument* aDocument)
 {
-  // Ignore temporary, hiding, resource documents and documents without
+  // Ignore hiding, resource documents and documents without docshell.
-  // docshell.
+  if (!aDocument->IsVisibleConsideringAncestors() ||
-  if (aDocument->IsInitialDocument() ||
-      !aDocument->IsVisibleConsideringAncestors() ||
       aDocument->IsResourceDoc() || !aDocument->IsActive())
     return nullptr;
 

accessible/tests/mochitest/relations/test_embeds.xul

@@ -52,6 +52,24 @@
       }
     }
 
+    function browserReorderChecker()
+    {
+      this.type = EVENT_REORDER;
+
+      this.match = function browserReorderChecker_match(aEvent)
+      {
+        // Reorder event might be duped because of temporary document creation.
+        if (aEvent.accessible == getAccessible(currentBrowser())) {
+          this.cnt++;
+          return this.cnt != 2;
+        }
+
+        return false;
+      }
+
+      this.cnt = 0;
+    }
+
     function loadOneTab(aURI)
     {
       this.invoke = function loadOneTab_invoke()
@@ -60,7 +78,7 @@
       }
 
       this.eventSeq = [
-        new invokerChecker(EVENT_REORDER, currentBrowser)
+        new browserReorderChecker()
       ];
 
       this.finalCheck = function loadURI_finalCheck()