From 702d0d5527123ec84d25bb2d6a596b49818157ca Mon Sep 17 00:00:00 2001
From: Phil Ringnalda <philringnalda@gmail.com>
Date: Wed, 30 Nov 2016 21:34:42 -0800
Subject: [PATCH] Backed out changeset 079ee7b41c3a (bug 1156160) for selection
 reftest failures CLOSED TREE

---
 layout/base/AccessibleCaretEventHub.cpp | 34 +------------------------
 layout/base/AccessibleCaretEventHub.h   | 12 +++------
 layout/base/AccessibleCaretManager.cpp  |  7 -----
 layout/base/AccessibleCaretManager.h    |  4 ---
 4 files changed, 5 insertions(+), 52 deletions(-)

diff --git a/layout/base/AccessibleCaretEventHub.cpp b/layout/base/AccessibleCaretEventHub.cpp
index 89a1fe2640b1..b897e4d776d8 100644
--- a/layout/base/AccessibleCaretEventHub.cpp
+++ b/layout/base/AccessibleCaretEventHub.cpp
@@ -32,8 +32,7 @@ namespace mozilla {
 #define AC_LOGV(message, ...)                                                  \
   AC_LOGV_BASE("AccessibleCaretEventHub (%p): " message, this, ##__VA_ARGS__);
 
-NS_IMPL_ISUPPORTS(AccessibleCaretEventHub,
-                  nsIDocumentActivity, nsIReflowObserver, nsIScrollObserver,
+NS_IMPL_ISUPPORTS(AccessibleCaretEventHub, nsIReflowObserver, nsIScrollObserver,
                   nsISelectionListener, nsISupportsWeakReference);
 
 // -----------------------------------------------------------------------------
@@ -434,12 +433,6 @@ AccessibleCaretEventHub::Init()
 
   mDocShell = static_cast<nsDocShell*>(docShell);
 
-  nsIDocument* doc = mPresShell->GetDocument();
-  if (doc) {
-    doc->RegisterActivityObserver(
-      NS_ISUPPORTS_CAST(nsIDocumentActivity*, this));
-  }
-
   if (sUseLongTapInjector) {
     mLongTapInjectorTimer = do_CreateInstance("@mozilla.org/timer;1");
   }
@@ -464,12 +457,6 @@ AccessibleCaretEventHub::Terminate()
     docShell->RemoveWeakScrollObserver(this);
   }
 
-  nsIDocument* doc = mPresShell->GetDocument();
-  if (doc) {
-    doc->UnregisterActivityObserver(
-      NS_ISUPPORTS_CAST(nsIDocumentActivity*, this));
-  }
-
   if (mLongTapInjectorTimer) {
     mLongTapInjectorTimer->Cancel();
   }
@@ -714,25 +701,6 @@ AccessibleCaretEventHub::ReflowInterruptible(DOMHighResTimeStamp aStart,
   return Reflow(aStart, aEnd);
 }
 
-void
-AccessibleCaretEventHub::NotifyOwnerDocumentActivityChanged()
-{
-  if (!mInitialized) {
-    return;
-  }
-
-  MOZ_ASSERT(mRefCnt.get() > 1, "Expect caller holds us as well!");
-
-  nsIDocument* doc = mPresShell->GetDocument();
-  if (!doc) {
-    return;
-  }
-
-  if (!doc->Hidden()) {
-    mManager->OnDocumentVisible();
-  }
-}
-
 void
 AccessibleCaretEventHub::AsyncPanZoomStarted()
 {
diff --git a/layout/base/AccessibleCaretEventHub.h b/layout/base/AccessibleCaretEventHub.h
index 1d5974f1211c..6681f8669e07 100644
--- a/layout/base/AccessibleCaretEventHub.h
+++ b/layout/base/AccessibleCaretEventHub.h
@@ -11,7 +11,6 @@
 #include "mozilla/UniquePtr.h"
 #include "mozilla/WeakPtr.h"
 #include "nsCOMPtr.h"
-#include "nsIDocumentActivity.h"
 #include "nsIFrame.h"
 #include "nsIReflowObserver.h"
 #include "nsIScrollObserver.h"
@@ -61,12 +60,10 @@ class WidgetTouchEvent;
 // Please see the wiki page for more information.
 // https://wiki.mozilla.org/AccessibleCaret
 //
-class AccessibleCaretEventHub
-  : public nsIDocumentActivity
-  , public nsIReflowObserver
-  , public nsIScrollObserver
-  , public nsISelectionListener
-  , public nsSupportsWeakReference
+class AccessibleCaretEventHub : public nsIReflowObserver,
+                                public nsIScrollObserver,
+                                public nsISelectionListener,
+                                public nsSupportsWeakReference
 {
 public:
   explicit AccessibleCaretEventHub(nsIPresShell* aPresShell);
@@ -79,7 +76,6 @@ public:
   void NotifyBlur(bool aIsLeavingDocument);
 
   NS_DECL_ISUPPORTS
-  NS_DECL_NSIDOCUMENTACTIVITY
   NS_DECL_NSIREFLOWOBSERVER
   NS_DECL_NSISELECTIONLISTENER
 
diff --git a/layout/base/AccessibleCaretManager.cpp b/layout/base/AccessibleCaretManager.cpp
index c490d29649c7..589675a61dcd 100644
--- a/layout/base/AccessibleCaretManager.cpp
+++ b/layout/base/AccessibleCaretManager.cpp
@@ -751,13 +751,6 @@ AccessibleCaretManager::OnFrameReconstruction()
   mSecondCaret->EnsureApzAware();
 }
 
-void
-AccessibleCaretManager::OnDocumentVisible()
-{
-  AC_LOG("%s: UpdateCarets()", __FUNCTION__);
-  UpdateCarets();
-}
-
 void
 AccessibleCaretManager::SetLastInputSource(uint16_t aInputSource)
 {
diff --git a/layout/base/AccessibleCaretManager.h b/layout/base/AccessibleCaretManager.h
index d3df5341b9ae..1cfe85d01b20 100644
--- a/layout/base/AccessibleCaretManager.h
+++ b/layout/base/AccessibleCaretManager.h
@@ -103,10 +103,6 @@ public:
   // was reconstructed, resulting in the content elements getting cloned.
   virtual void OnFrameReconstruction();
 
-  // Called by AccessibleCaretEventHub to inform us that the document
-  // becomes visible.
-  virtual void OnDocumentVisible();
-
   // Update the manager with the last input source that was observed. This
   // is used in part to determine if the carets should be shown or hidden.
   void SetLastInputSource(uint16_t aInputSource);