Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(110)

Unified Diff: Source/WebCore/html/HTMLCollection.cpp

Issue 14028014: Remove MicroData implementation (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Also delete all the tests Created 7 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/WebCore/html/HTMLAreaElement.cpp ('k') | Source/WebCore/html/HTMLElement.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/WebCore/html/HTMLCollection.cpp
diff --git a/Source/WebCore/html/HTMLCollection.cpp b/Source/WebCore/html/HTMLCollection.cpp
index af50b56360ce7f9e28cad6feddec1fbcc341f3c1..be140e5635ebc0170ed2d954c0ad0efa928deb16 100644
--- a/Source/WebCore/html/HTMLCollection.cpp
+++ b/Source/WebCore/html/HTMLCollection.cpp
@@ -33,11 +33,6 @@
#include "NodeRareData.h"
#include "NodeTraversal.h"
-#if ENABLE(MICRODATA)
-#include "HTMLPropertiesCollection.h"
-#include "PropertyNodeList.h"
-#endif
-
#include <utility>
namespace WebCore {
@@ -62,9 +57,6 @@ static bool shouldOnlyIncludeDirectChildren(CollectionType type)
case SelectedOptions:
case DataListOptions:
case WindowNamedItems:
-#if ENABLE(MICRODATA)
- case ItemProperties:
-#endif
case FormControls:
return false;
case NodeChildren:
@@ -79,8 +71,6 @@ static bool shouldOnlyIncludeDirectChildren(CollectionType type)
case HTMLTagNodeListType:
case RadioNodeListType:
case LabelsNodeListType:
- case MicroDataItemListType:
- case PropertyNodeListType:
break;
}
ASSERT_NOT_REACHED();
@@ -100,9 +90,6 @@ static NodeListRootType rootTypeFromCollectionType(CollectionType type)
case DocAll:
case WindowNamedItems:
case DocumentNamedItems:
-#if ENABLE(MICRODATA)
- case ItemProperties:
-#endif
case FormControls:
return NodeListIsRootedAtDocument;
case NodeChildren:
@@ -122,8 +109,6 @@ static NodeListRootType rootTypeFromCollectionType(CollectionType type)
case HTMLTagNodeListType:
case RadioNodeListType:
case LabelsNodeListType:
- case MicroDataItemListType:
- case PropertyNodeListType:
break;
}
ASSERT_NOT_REACHED();
@@ -159,10 +144,6 @@ static NodeListInvalidationType invalidationTypeExcludingIdAndNameAttributes(Col
return InvalidateOnIdNameAttrChange;
case DocumentNamedItems:
return InvalidateOnIdNameAttrChange;
-#if ENABLE(MICRODATA)
- case ItemProperties:
- return InvalidateOnItemAttrChange;
-#endif
case FormControls:
return InvalidateForFormControls;
case ChildNodeListType:
@@ -172,8 +153,6 @@ static NodeListInvalidationType invalidationTypeExcludingIdAndNameAttributes(Col
case HTMLTagNodeListType:
case RadioNodeListType:
case LabelsNodeListType:
- case MicroDataItemListType:
- case PropertyNodeListType:
break;
}
ASSERT_NOT_REACHED();
@@ -245,10 +224,6 @@ template <> inline bool isMatchingElement(const HTMLCollection* htmlCollection,
case DocAll:
case NodeChildren:
return true;
-#if ENABLE(MICRODATA)
- case ItemProperties:
- return element->fastHasAttribute(itempropAttr);
-#endif
case FormControls:
case DocumentNamedItems:
case TableRows:
@@ -260,8 +235,6 @@ template <> inline bool isMatchingElement(const HTMLCollection* htmlCollection,
case HTMLTagNodeListType:
case RadioNodeListType:
case LabelsNodeListType:
- case MicroDataItemListType:
- case PropertyNodeListType:
ASSERT_NOT_REACHED();
}
return false;
@@ -445,13 +418,6 @@ Node* LiveNodeListBase::item(unsigned offset) const
if (isLengthCacheValid() && cachedLength() <= offset)
return 0;
-#if ENABLE(MICRODATA)
- if (type() == ItemProperties)
- static_cast<const HTMLPropertiesCollection*>(this)->updateRefElements();
- else if (type() == PropertyNodeListType)
- static_cast<const PropertyNodeList*>(this)->updateRefElements();
-#endif
-
ContainerNode* root = rootContainerNode();
if (!root) {
// FIMXE: In someTextNode.childNodes case the root is Text. We shouldn't even make a LiveNodeList for that.
« no previous file with comments | « Source/WebCore/html/HTMLAreaElement.cpp ('k') | Source/WebCore/html/HTMLElement.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698