Remove blacklist on .innerHTML and .outerHTML

third_party/WebKit/Source/core/html/HTMLElement.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  * Copyright (C) 2004-2008, 2013, 2014 Apple Inc. All rights reserved.
  * Copyright (C) 2009 Torch Mobile Inc. All rights reserved.
  * (http://www.torchmobile.com/)
  * Copyright (C) 2011 Motorola Mobility. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ skipping 118 matching lines @@
   // FIXME: We should have a way to detect XHTML elements and replace the
   // hasPrefix() check with it.
   if (document().isHTMLDocument()) {
     if (!tagQName().hasPrefix())
       return tagQName().localNameUpper();
     return Element::nodeName().upper();
   }
   return Element::nodeName();
 }
 
-bool HTMLElement::ieForbidsInsertHTML() const {
-  // FIXME: Supposedly IE disallows settting innerHTML, outerHTML
-  // and createContextualFragment on these tags.  We have no tests to
-  // verify this however, so this list could be totally wrong.
-  // This list was moved from the previous endTagRequirement() implementation.
-  // This is also called from editing and assumed to be the list of tags
-  // for which no end tag should be serialized. It's unclear if the list for
-  // IE compat and the list for serialization sanity are the same.
+bool HTMLElement::shouldSerializeEndTag() const {
+  // See https://www.w3.org/TR/DOM-Parsing/
   if (hasTagName(areaTag) || hasTagName(baseTag) || hasTagName(basefontTag) ||
-      hasTagName(brTag) || hasTagName(colTag) || hasTagName(embedTag) ||
-      hasTagName(frameTag) || hasTagName(hrTag) || hasTagName(imageTag) ||
+      hasTagName(bgsoundTag) || hasTagName(brTag) || hasTagName(colTag) ||
+      hasTagName(embedTag) || hasTagName(frameTag) || hasTagName(hrTag) ||
       hasTagName(imgTag) || hasTagName(inputTag) || hasTagName(keygenTag) ||
-      hasTagName(linkTag) || (RuntimeEnabledFeatures::contextMenuEnabled() &&
-                              hasTagName(menuitemTag)) ||
-      hasTagName(metaTag) || hasTagName(paramTag) || hasTagName(sourceTag) ||
-      hasTagName(trackTag) || hasTagName(wbrTag))
-    return true;
-  return false;
+      hasTagName(linkTag) || hasTagName(menuitemTag) || hasTagName(metaTag) ||
+      hasTagName(paramTag) || hasTagName(sourceTag) || hasTagName(trackTag) ||
+      hasTagName(wbrTag))
+    return false;
+  return true;
 }
 
 static inline CSSValueID unicodeBidiAttributeForDirAuto(HTMLElement* element) {
   if (element->hasTagName(preTag) || element->hasTagName(textareaTag))
     return CSSValueWebkitPlaintext;
   // FIXME: For bdo element, dir="auto" should result in "bidi-override isolate"
   // but we don't support having multiple values in unicode-bidi yet.
   // See https://bugs.webkit.org/show_bug.cgi?id=73164.
   return CSSValueWebkitIsolate;
 }
@@ skipping 333 matching lines @@
       if (c == '\r' && i + 1 < length && text[i + 1] == '\n')
         i++;
     }
 
     start = i + 1;  // Character after line break.
   }
 
   return fragment;
 }
 
-static inline bool shouldProhibitSetInnerOuterText(const HTMLElement& element) {
-  return element.hasTagName(colTag) || element.hasTagName(colgroupTag) ||
-         element.hasTagName(framesetTag) || element.hasTagName(headTag) ||
-         element.hasTagName(htmlTag) || element.hasTagName(tableTag) ||
-         element.hasTagName(tbodyTag) || element.hasTagName(tfootTag) ||
-         element.hasTagName(theadTag) || element.hasTagName(trTag);
-}
-
 void HTMLElement::setInnerText(const String& text,
                                ExceptionState& exceptionState) {
-  if (ieForbidsInsertHTML()) {
-    exceptionState.throwDOMException(
-        NoModificationAllowedError,
-        "The '" + localName() + "' element does not support text insertion.");
-    return;
-  }
-  if (shouldProhibitSetInnerOuterText(*this)) {
-    exceptionState.throwDOMException(
-        NoModificationAllowedError,
-        "The '" + localName() + "' element does not support text insertion.");
-    return;
-  }
-
   // FIXME: This doesn't take whitespace collapsing into account at all.
 
   if (!text.contains('\n') && !text.contains('\r')) {
     if (text.isEmpty()) {
       removeChildren();
       return;
     }
     replaceChildrenWithText(this, text, exceptionState);
     return;
   }
@@ skipping 17 matching lines @@
   }
 
   // Add text nodes and <br> elements.
   DocumentFragment* fragment = textToFragment(text, exceptionState);
   if (!exceptionState.hadException())
     replaceChildrenWithFragment(this, fragment, exceptionState);
 }
 
 void HTMLElement::setOuterText(const String& text,
                                ExceptionState& exceptionState) {
-  if (ieForbidsInsertHTML()) {
-    exceptionState.throwDOMException(
-        NoModificationAllowedError,
-        "The '" + localName() + "' element does not support text insertion.");
-    return;
-  }
-  if (shouldProhibitSetInnerOuterText(*this)) {
-    exceptionState.throwDOMException(
-        NoModificationAllowedError,
-        "The '" + localName() + "' element does not support text insertion.");
-    return;
-  }
-
   ContainerNode* parent = parentNode();
   if (!parent) {
     exceptionState.throwDOMException(NoModificationAllowedError,
                                      "The element has no parent.");
     return;
   }
 
   Node* prev = previousSibling();
   Node* next = nextSibling();
   Node* newChild = nullptr;
@@ skipping 627 matching lines @@
 
 #ifndef NDEBUG
 
 // For use in the debugger
 void dumpInnerHTML(blink::HTMLElement*);
 
 void dumpInnerHTML(blink::HTMLElement* element) {
   printf("%s\n", element->innerHTML().ascii().data());
 }
 #endif