Bug 1460734 - Remove macro DEBUG_DOCSHELL_FOCUS. r?bz
The code guarded by it doesn't compile.
MozReview-Commit-ID: IgIp4pv2OwX
--- a/docshell/base/nsDocShell.cpp
+++ b/docshell/base/nsDocShell.cpp
@@ -3495,95 +3495,23 @@ nsDocShell::GetTreeOwner(nsIDocShellTree
{
NS_ENSURE_ARG_POINTER(aTreeOwner);
*aTreeOwner = mTreeOwner;
NS_IF_ADDREF(*aTreeOwner);
return NS_OK;
}
-#ifdef DEBUG_DOCSHELL_FOCUS
-static void
-PrintDocTree(nsIDocShellTreeItem* aParentNode, int aLevel)
-{
- for (int32_t i = 0; i < aLevel; i++) {
- printf(" ");
- }
-
- int32_t childWebshellCount;
- aParentNode->GetChildCount(&childWebshellCount);
- nsCOMPtr<nsIDocShell> parentAsDocShell(do_QueryInterface(aParentNode));
- int32_t type = aParentNode->ItemType();
- nsCOMPtr<nsIPresShell> presShell = parentAsDocShell->GetPresShell();
- RefPtr<nsPresContext> presContext;
- parentAsDocShell->GetPresContext(getter_AddRefs(presContext));
- nsIDocument* doc = presShell->GetDocument();
-
- nsCOMPtr<nsPIDOMWindowOuter> domwin(doc->GetWindow());
-
- nsCOMPtr<nsIWidget> widget;
- nsViewManager* vm = presShell->GetViewManager();
- if (vm) {
- vm->GetWidget(getter_AddRefs(widget));
- }
- dom::Element* rootElement = doc->GetRootElement();
-
- printf("DS %p Ty %s Doc %p DW %p EM %p CN %p\n",
- (void*)parentAsDocShell.get(),
- type == nsIDocShellTreeItem::typeChrome ? "Chr" : "Con",
- (void*)doc, (void*)domwin.get(),
- (void*)presContext->EventStateManager(), (void*)rootElement);
-
- if (childWebshellCount > 0) {
- for (int32_t i = 0; i < childWebshellCount; i++) {
- nsCOMPtr<nsIDocShellTreeItem> child;
- aParentNode->GetChildAt(i, getter_AddRefs(child));
- PrintDocTree(child, aLevel + 1);
- }
- }
-}
-
-static void
-PrintDocTree(nsIDocShellTreeItem* aParentNode)
-{
- NS_ASSERTION(aParentNode, "Pointer is null!");
-
- nsCOMPtr<nsIDocShellTreeItem> parentItem;
- aParentNode->GetParent(getter_AddRefs(parentItem));
- while (parentItem) {
- nsCOMPtr<nsIDocShellTreeItem> tmp;
- parentItem->GetParent(getter_AddRefs(tmp));
- if (!tmp) {
- break;
- }
- parentItem = tmp;
- }
-
- if (!parentItem) {
- parentItem = aParentNode;
- }
-
- PrintDocTree(parentItem, 0);
-}
-#endif
-
NS_IMETHODIMP
nsDocShell::SetTreeOwner(nsIDocShellTreeOwner* aTreeOwner)
{
if (mIsBeingDestroyed && aTreeOwner) {
return NS_ERROR_FAILURE;
}
-#ifdef DEBUG_DOCSHELL_FOCUS
- nsCOMPtr<nsIDocShellTreeItem> item(do_QueryInterface(aTreeOwner));
- if (item) {
- PrintDocTree(item);
- }
-#endif
-
// Don't automatically set the progress based on the tree owner for frames
if (!IsFrame()) {
nsCOMPtr<nsIWebProgress> webProgress =
do_QueryInterface(GetAsSupports(this));
if (webProgress) {
nsCOMPtr<nsIWebProgressListener> oldListener =
do_QueryInterface(mTreeOwner);
--- a/dom/events/EventStateManager.cpp
+++ b/dom/events/EventStateManager.cpp
@@ -98,18 +98,16 @@
#ifdef XP_MACOSX
#import <ApplicationServices/ApplicationServices.h>
#endif
namespace mozilla {
using namespace dom;
-//#define DEBUG_DOCSHELL_FOCUS
-
static const LayoutDeviceIntPoint kInvalidRefPoint = LayoutDeviceIntPoint(-1,-1);
static uint32_t gMouseOrKeyboardEventCounter = 0;
static nsITimer* gUserInteractionTimer = nullptr;
static nsITimerCallback* gUserInteractionTimerCallback = nullptr;
static const double kCursorLoadingTimeout = 1000; // ms
static AutoWeakFrame gLastCursorSourceFrame;
@@ -117,76 +115,16 @@ static TimeStamp gLastCursorUpdateTime;
static inline int32_t
RoundDown(double aDouble)
{
return (aDouble > 0) ? static_cast<int32_t>(floor(aDouble)) :
static_cast<int32_t>(ceil(aDouble));
}
-#ifdef DEBUG_DOCSHELL_FOCUS
-static void
-PrintDocTree(nsIDocShellTreeItem* aParentItem, int aLevel)
-{
- for (int32_t i=0;i<aLevel;i++) printf(" ");
-
- int32_t childWebshellCount;
- aParentItem->GetChildCount(&childWebshellCount);
- nsCOMPtr<nsIDocShell> parentAsDocShell(do_QueryInterface(aParentItem));
- int32_t type = aParentItem->ItemType();
- nsCOMPtr<nsIPresShell> presShell = parentAsDocShell->GetPresShell();
- RefPtr<nsPresContext> presContext;
- parentAsDocShell->GetPresContext(getter_AddRefs(presContext));
- nsCOMPtr<nsIContentViewer> cv;
- parentAsDocShell->GetContentViewer(getter_AddRefs(cv));
- nsCOMPtr<nsIDOMDocument> domDoc;
- if (cv)
- cv->GetDOMDocument(getter_AddRefs(domDoc));
- nsCOMPtr<nsIDocument> doc = do_QueryInterface(domDoc);
- nsCOMPtr<nsIDOMWindow> domwin = doc ? doc->GetWindow() : nullptr;
- nsIURI* uri = doc ? doc->GetDocumentURI() : nullptr;
-
- printf("DS %p Type %s Cnt %d Doc %p DW %p EM %p%c",
- static_cast<void*>(parentAsDocShell.get()),
- type==nsIDocShellTreeItem::typeChrome?"Chrome":"Content",
- childWebshellCount, static_cast<void*>(doc.get()),
- static_cast<void*>(domwin.get()),
- static_cast<void*>(presContext ? presContext->EventStateManager() : nullptr),
- uri ? ' ' : '\n');
- if (uri) {
- nsAutoCString spec;
- uri->GetSpec(spec);
- printf("\"%s\"\n", spec.get());
- }
-
- if (childWebshellCount > 0) {
- for (int32_t i = 0; i < childWebshellCount; i++) {
- nsCOMPtr<nsIDocShellTreeItem> child;
- aParentItem->GetChildAt(i, getter_AddRefs(child));
- PrintDocTree(child, aLevel + 1);
- }
- }
-}
-
-static void
-PrintDocTreeAll(nsIDocShellTreeItem* aItem)
-{
- nsCOMPtr<nsIDocShellTreeItem> item = aItem;
- for(;;) {
- nsCOMPtr<nsIDocShellTreeItem> parent;
- item->GetParent(getter_AddRefs(parent));
- if (!parent)
- break;
- item = parent;
- }
-
- PrintDocTree(item, 0);
-}
-#endif
-
/******************************************************************/
/* mozilla::UITimerCallback */
/******************************************************************/
class UITimerCallback final :
public nsITimerCallback,
public nsINamed
{