Propagate inert state to OOPIFs when a modal dialog is active

third_party/WebKit/Source/core/dom/Node.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All
  * rights reserved.
  * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies)
  * Copyright (C) 2009 Torch Mobile Inc. All rights reserved.
  * (http://www.torchmobile.com/)
  *
@@ skipping 833 matching lines @@
   DCHECK(IsFocused());
   return true;
 }
 
 bool Node::IsInert() const {
   const HTMLDialogElement* dialog = GetDocument().ActiveModalDialog();
   if (dialog && this != GetDocument() &&
       (!CanParticipateInFlatTree() ||
        !FlatTreeTraversal::ContainsIncludingPseudoElement(*dialog, *this)))
     return true;
-  return GetDocument().LocalOwner() && GetDocument().LocalOwner()->IsInert();
+  return GetDocument().GetFrame() && GetDocument().GetFrame()->IsInert();

[dmazzoni, 2017/05/18 15:47:05]

I don't think this changes the logic, but I'm just checking - when would
GetFrame() return nullptr, and if so what should IsInert return?

[kenrb, 2017/05/19 17:46:28]

It should be fine to remove that condition, on the basis that this code
shouldn't be reachable for a Document with a null frame_ pointer. If any tests
prove me wrong I can change it back.

 }
 
 unsigned Node::NodeIndex() const {
   const Node* temp_node = previousSibling();
   unsigned count = 0;
   for (count = 0; temp_node; count++)
     temp_node = temp_node->previousSibling();
   return count;
 }
 
@@ skipping 1757 matching lines @@
   if (node) {
     std::stringstream stream;
     node->PrintNodePathTo(stream);
     LOG(INFO) << stream.str();
   } else {
     LOG(INFO) << "Cannot showNodePath for <null>";
   }
 }
 
 #endif