From 535b704dc6ea64b78c36dfc593fe16772191d49c Mon Sep 17 00:00:00 2001 From: wolfbeast Date: Wed, 2 May 2018 21:58:04 +0200 Subject: Remove content process sandbox code. --- accessible/ipc/DocAccessibleParent.cpp | 25 -------------------- .../windows/msaa/HTMLWin32ObjectAccessible.cpp | 27 ---------------------- .../windows/msaa/HTMLWin32ObjectAccessible.h | 7 ------ 3 files changed, 59 deletions(-) (limited to 'accessible') diff --git a/accessible/ipc/DocAccessibleParent.cpp b/accessible/ipc/DocAccessibleParent.cpp index 0e9dfc0806..4d368b1bc4 100644 --- a/accessible/ipc/DocAccessibleParent.cpp +++ b/accessible/ipc/DocAccessibleParent.cpp @@ -492,32 +492,7 @@ bool DocAccessibleParent::RecvGetWindowedPluginIAccessible( const WindowsHandle& aHwnd, IAccessibleHolder* aPluginCOMProxy) { -#if defined(MOZ_CONTENT_SANDBOX) - // We don't actually want the accessible object for aHwnd, but rather the - // one that belongs to its child (see HTMLWin32ObjectAccessible). - HWND childWnd = ::GetWindow(reinterpret_cast(aHwnd), GW_CHILD); - if (!childWnd) { - // We're seeing this in the wild - the plugin is windowed but we no longer - // have a window. - return true; - } - - IAccessible* rawAccPlugin = nullptr; - HRESULT hr = ::AccessibleObjectFromWindow(childWnd, OBJID_WINDOW, - IID_IAccessible, - (void**)&rawAccPlugin); - if (FAILED(hr)) { - // This might happen if the plugin doesn't handle WM_GETOBJECT properly. - // We should not consider that a failure. - return true; - } - - aPluginCOMProxy->Set(IAccessibleHolder::COMPtrType(rawAccPlugin)); - - return true; -#else return false; -#endif } #endif // defined(XP_WIN) diff --git a/accessible/windows/msaa/HTMLWin32ObjectAccessible.cpp b/accessible/windows/msaa/HTMLWin32ObjectAccessible.cpp index 3644db68d7..75a6f647bf 100644 --- a/accessible/windows/msaa/HTMLWin32ObjectAccessible.cpp +++ b/accessible/windows/msaa/HTMLWin32ObjectAccessible.cpp @@ -62,25 +62,6 @@ HTMLWin32ObjectAccessible::HTMLWin32ObjectAccessible(void* aHwnd, { mHwnd = aHwnd; if (mHwnd) { -#if defined(MOZ_CONTENT_SANDBOX) - if (XRE_IsContentProcess()) { - DocAccessibleChild* ipcDoc = aDoc->IPCDoc(); - MOZ_ASSERT(ipcDoc); - if (!ipcDoc) { - return; - } - - IAccessibleHolder proxyHolder; - if (!ipcDoc->SendGetWindowedPluginIAccessible( - reinterpret_cast(mHwnd), &proxyHolder)) { - return; - } - - mCOMProxy.reset(proxyHolder.Release()); - return; - } -#endif - // The plugin is not windowless. In this situation we use // use its inner child owned by the plugin so that we don't get // in an infinite loop, where the WM_GETOBJECT's get forwarded @@ -92,14 +73,6 @@ HTMLWin32ObjectAccessible::HTMLWin32ObjectAccessible(void* aHwnd, void HTMLWin32ObjectAccessible::GetNativeInterface(void** aNativeAccessible) { -#if defined(MOZ_CONTENT_SANDBOX) - if (XRE_IsContentProcess()) { - RefPtr addRefed = mCOMProxy.get(); - addRefed.forget(aNativeAccessible); - return; - } -#endif - if (mHwnd) { ::AccessibleObjectFromWindow(static_cast(mHwnd), OBJID_WINDOW, IID_IAccessible, diff --git a/accessible/windows/msaa/HTMLWin32ObjectAccessible.h b/accessible/windows/msaa/HTMLWin32ObjectAccessible.h index 786d52191b..b473061a2a 100644 --- a/accessible/windows/msaa/HTMLWin32ObjectAccessible.h +++ b/accessible/windows/msaa/HTMLWin32ObjectAccessible.h @@ -8,10 +8,6 @@ #include "BaseAccessibles.h" -#if defined(MOZ_CONTENT_SANDBOX) -#include "mozilla/mscom/Ptr.h" -#endif - struct IAccessible; namespace mozilla { @@ -59,9 +55,6 @@ public: protected: void* mHwnd; -#if defined(MOZ_CONTENT_SANDBOX) - mscom::ProxyUniquePtr mCOMProxy; -#endif }; } // namespace a11y -- cgit v1.2.3