Never cast Attr to ContainerNode based on nodeType ATTRIBUTE_NODE

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/)
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ skipping 1384 matching lines @@
 void Node::setTextContent(const String& text)
 {
     switch (nodeType()) {
         case TEXT_NODE:
         case CDATA_SECTION_NODE:
         case COMMENT_NODE:
         case PROCESSING_INSTRUCTION_NODE:
             setNodeValue(text);
             return;
         case ELEMENT_NODE:
-        case ATTRIBUTE_NODE:
         case DOCUMENT_FRAGMENT_NODE: {
             // FIXME: Merge this logic into replaceChildrenWithText.
             RefPtrWillBeRawPtr<ContainerNode> container = toContainerNode(this);
 
             // Note: This is an intentional optimization.
             // See crbug.com/352836 also.
             // No need to do anything if the text is identical.
             if (container->hasOneTextChild() && toText(container->firstChild())->data() == text)
                 return;
 
             ChildListMutationScope mutation(*this);
             // Note: This API will not insert empty text nodes:
             // http://dom.spec.whatwg.org/#dom-node-textcontent
             if (text.isEmpty()) {
                 container->removeChildren(DispatchSubtreeModifiedEvent);
             } else {
                 container->removeChildren(OmitSubtreeModifiedEvent);
                 container->appendChild(document().createTextNode(text), ASSERT_NO_EXCEPTION);
             }
             return;
         }
+        case ATTRIBUTE_NODE:
         case DOCUMENT_NODE:
         case DOCUMENT_TYPE_NODE:
             // Do nothing.
             return;
     }
     ASSERT_NOT_REACHED();
 }
 
 bool Node::offsetInCharacters() const
 {
@@ skipping 1000 matching lines @@
 {
     // This switch statement must be consistent with that of Range::processContentsBetweenOffsets.
     switch (nodeType()) {
     case Node::TEXT_NODE:
     case Node::CDATA_SECTION_NODE:
     case Node::COMMENT_NODE:
         return toCharacterData(this)->length();
     case Node::PROCESSING_INSTRUCTION_NODE:
         return toProcessingInstruction(this)->data().length();
     case Node::ELEMENT_NODE:
-    case Node::ATTRIBUTE_NODE:
     case Node::DOCUMENT_NODE:
     case Node::DOCUMENT_FRAGMENT_NODE:
         return toContainerNode(this)->countChildren();
+    case Node::ATTRIBUTE_NODE:
     case Node::DOCUMENT_TYPE_NODE:
         return 0;
     }
     ASSERT_NOT_REACHED();
     return 0;
 }
 
 v8::Local<v8::Object> Node::wrap(v8::Isolate* isolate, v8::Local<v8::Object> creationContext)
 {
     // It's possible that no one except for the new wrapper owns this object at
@@ skipping 40 matching lines @@
 
 void showNodePath(const blink::Node* node)
 {
     if (node)
         node->showNodePathForThis();
     else
         fprintf(stderr, "Cannot showNodePath for (nil)\n");
 }
 
 #endif