Stop using static arrays for *Tags / *Attrs

Source/core/html/HTMLObjectElement.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2000 Stefan Schimanski (1Stein@gmx.de)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2011 Apple Inc. All rights reserved.
  * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies)
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
@@ skipping 372 matching lines @@
         }
     }
 
     m_useFallbackContent = true;
 
     // FIXME: Style gets recalculated which is suboptimal.
     reattachFallbackContent();
 }
 
 // FIXME: This should be removed, all callers are almost certainly wrong.
 static bool isRecognizedTagName(const QualifiedName& tagName)

[Inactive, 2013/10/30 18:35:36]

I am getting rid of this method in:
https://codereview.chromium.org/50453004/

Once this is gone, HTMLNames::getHTMLTags() will only be called in one place
(html/parser/HTMLIdentifier.cpp).

[abarth-chromium, 2013/10/30 19:30:11]

Great.

 {
     DEFINE_STATIC_LOCAL(HashSet<StringImpl*>, tagList, ());
     if (tagList.isEmpty()) {
-        const QualifiedName* const* tags = HTMLNames::getHTMLTags();
-        for (size_t i = 0; i < HTMLNames::HTMLTagsCount; i++) {
+        Vector<const QualifiedName*> tags;
+        HTMLNames::getHTMLTags(tags);
+        size_t HTMLTagsCount = tags.size();
+        for (size_t i = 0; i < HTMLTagsCount; ++i) {
             if (*tags[i] == bgsoundTag
                 || *tags[i] == commandTag
                 || *tags[i] == detailsTag
                 || *tags[i] == figcaptionTag
                 || *tags[i] == figureTag
                 || *tags[i] == summaryTag
                 || *tags[i] == trackTag) {
                 // Even though we have atoms for these tags, we don't want to
                 // treat them as "recognized tags" for the purpose of parsing
                 // because that changes how we parse documents.
@@ skipping 96 matching lines @@
 {
     return FormAssociatedElement::form();
 }
 
 bool HTMLObjectElement::isInteractiveContent() const
 {
     return fastHasAttribute(usemapAttr);
 }
 
 }