Reduce usage of HTMLElementFactory::createHTMLElement

The function contains autonomous custom element processing.  It's
unexpected in many cases.

If it's called with known HTML tags, custom element processing is
unnecessary and it should be replaced with
Document::CreateRawElement().

If it's called with arbitrary tag names, custom element processing
is necessary and it should be replaced with Document::createElement().

This CL is a preparation to remove HTMLElementFactory::createElement().

Bug: 806641
Change-Id: I00dae6c2cdac18eccd1580a72d94862b5cb2d657
Reviewed-on: https://chromium-review.googlesource.com/895203Reviewed-by: Yoichi Osato <yoichio@chromium.org>
Commit-Queue: Kent Tamura <tkent@chromium.org>
Cr-Commit-Position: refs/heads/master@{#533235}

third_party/WebKit/Source/core/editing/commands/EditingCommandsUtilities.cpp

@@ -33,7 +33,6 @@
 #include "core/editing/SelectionTemplate.h"
 #include "core/editing/VisiblePosition.h"
 #include "core/editing/VisibleSelection.h"
-#include "core/html_element_factory.h"
 #include "core/layout/LayoutObject.h"
 
 namespace blink {
@@ -332,8 +331,9 @@ bool LineBreakExistsAtVisiblePosition(const VisiblePosition& visible_position) {
 }
 
 HTMLElement* CreateHTMLElement(Document& document, const QualifiedName& name) {
-  return HTMLElementFactory::createHTMLElement(name.LocalName(), document,
-                                               kCreatedByCloneNode);
+  DCHECK_EQ(name.NamespaceURI(), HTMLNames::xhtmlNamespaceURI)
+      << "Unexpected namespace: " << name;
+  return ToHTMLElement(document.createElement(name, kCreatedByCloneNode));
 }
 
 HTMLElement* EnclosingList(const Node* node) {

third_party/WebKit/Source/core/html/track/vtt/VTTElement.cpp

@@ -26,7 +26,7 @@
 #include "core/html/track/vtt/VTTElement.h"
 
 #include "core/css/StyleChangeReason.h"
-#include "core/html_element_factory.h"
+#include "core/dom/Document.h"
 
 namespace blink {
 
@@ -82,37 +82,37 @@ Element* VTTElement::CloneElementWithoutAttributesAndChildren() {
 }
 
 HTMLElement* VTTElement::CreateEquivalentHTMLElement(Document& document) {
-  HTMLElement* html_element = nullptr;
+  Element* html_element = nullptr;
   switch (web_vtt_node_type_) {
     case kVTTNodeTypeClass:
     case kVTTNodeTypeLanguage:
     case kVTTNodeTypeVoice:
-      html_element = HTMLElementFactory::createHTMLElement(
-          HTMLNames::spanTag.LocalName(), document);
+      html_element =
+          document.CreateRawElement(HTMLNames::spanTag, kCreatedByParser);
       html_element->setAttribute(HTMLNames::titleAttr,
                                  getAttribute(VoiceAttributeName()));
       html_element->setAttribute(HTMLNames::langAttr,
                                  getAttribute(LangAttributeName()));
       break;
     case kVTTNodeTypeItalic:
-      html_element = HTMLElementFactory::createHTMLElement(
-          HTMLNames::iTag.LocalName(), document);
+      html_element =
+          document.CreateRawElement(HTMLNames::iTag, kCreatedByParser);
       break;
     case kVTTNodeTypeBold:
-      html_element = HTMLElementFactory::createHTMLElement(
-          HTMLNames::bTag.LocalName(), document);
+      html_element =
+          document.CreateRawElement(HTMLNames::bTag, kCreatedByParser);
       break;
     case kVTTNodeTypeUnderline:
-      html_element = HTMLElementFactory::createHTMLElement(
-          HTMLNames::uTag.LocalName(), document);
+      html_element =
+          document.CreateRawElement(HTMLNames::uTag, kCreatedByParser);
       break;
     case kVTTNodeTypeRuby:
-      html_element = HTMLElementFactory::createHTMLElement(
-          HTMLNames::rubyTag.LocalName(), document);
+      html_element =
+          document.CreateRawElement(HTMLNames::rubyTag, kCreatedByParser);
       break;
     case kVTTNodeTypeRubyText:
-      html_element = HTMLElementFactory::createHTMLElement(
-          HTMLNames::rtTag.LocalName(), document);
+      html_element =
+          document.CreateRawElement(HTMLNames::rtTag, kCreatedByParser);
       break;
     default:
       NOTREACHED();
@@ -120,7 +120,7 @@ HTMLElement* VTTElement::CreateEquivalentHTMLElement(Document& document) {
 
   html_element->setAttribute(HTMLNames::classAttr,
                              getAttribute(HTMLNames::classAttr));
-  return html_element;
+  return ToHTMLElement(html_element);
 }
 
 void VTTElement::SetIsPastNode(bool is_past_node) {