Ensure active document for hasEditableLevel

third_party/WebKit/Source/core/editing/EditingUtilities.cpp

 /*
  * Copyright (C) 2004, 2005, 2006, 2007 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 254 matching lines @@
                      const PositionWithAffinity& b) {
   return comparePositions(a.position(), b.position());
 }
 
 int comparePositions(const VisiblePosition& a, const VisiblePosition& b) {
   return comparePositions(a.deepEquivalent(), b.deepEquivalent());
 }
 
 enum EditableLevel { Editable, RichlyEditable };
 static bool hasEditableLevel(const Node& node, EditableLevel editableLevel) {
-  // TODO(yoichio): We should have this check.
-  // DCHECK(!needsLayoutTreeUpdate(node));
+  DCHECK(node.document().isActive());
+  // TODO(editing-dev): We should have this check:
+  // DCHECK_GE(node.document().lifecycle().state(),
+  //           DocumentLifecycle::StyleClean);
   if (node.isPseudoElement())
     return false;
 
   // Ideally we'd call DCHECK(!needsStyleRecalc()) here, but
   // ContainerNode::setFocus() calls setNeedsStyleRecalc(), so the assertion
   // would fire in the middle of Document::setFocusedNode().
 
   for (const Node& ancestor : NodeTraversal::inclusiveAncestorsOf(node)) {
     if ((ancestor.isHTMLElement() || ancestor.isDocumentNode()) &&
         ancestor.layoutObject()) {
       switch (ancestor.layoutObject()->style()->userModify()) {
         case READ_ONLY:
           return false;
         case READ_WRITE:
           return true;
         case READ_WRITE_PLAINTEXT_ONLY:
           return editableLevel != RichlyEditable;
       }
       NOTREACHED();
       return false;
     }
   }
 
   return false;
 }
 
 bool hasEditableStyle(const Node& node) {
+  // TODO(editing-dev): We shouldn't check editable style in inactive documents.
+  // We should hoist this check in the call stack, replace it by a DCHECK of
+  // active document and ultimately cleanup the code paths with inactive
+  // documents.  See crbug.com/667681
+  if (!node.document().isActive())
+    return false;
+
   return hasEditableLevel(node, Editable);
 }
 
 bool hasRichlyEditableStyle(const Node& node) {
+  // TODO(editing-dev): We shouldn't check editable style in inactive documents.
+  // We should hoist this check in the call stack, replace it by a DCHECK of
+  // active document and ultimately cleanup the code paths with inactive
+  // documents.  See crbug.com/667681
+  if (!node.document().isActive())
+    return false;
+
   return hasEditableLevel(node, RichlyEditable);
 }
 
 bool isRootEditableElement(const Node& node) {
   return hasEditableStyle(node) && node.isElementNode() &&
          (!node.parentNode() || !hasEditableStyle(*node.parentNode()) ||
           !node.parentNode()->isElementNode() ||
           &node == node.document().body());
 }
 
@@ skipping 1808 matching lines @@
         return InputType::DeleteWordBackward;
       if (granularity == LineBoundary)
         return InputType::DeleteLineBackward;
       return InputType::DeleteContentBackward;
     default:
       return InputType::None;
   }
 }
 
 }  // namespace blink