Title: [116457] trunk/Source/WebCore
Revision
116457
Author
[email protected]
Date
2012-05-08 15:04:23 -0700 (Tue, 08 May 2012)

Log Message

HTMLElementStack::hasOnlyHTMLElementsInScope is no longer called
https://bugs.webkit.org/show_bug.cgi?id=85908

Reviewed by Eric Seidel.

This patch just removes the dead code.

No tests needed. Cleanup only.

* html/parser/HTMLElementStack.cpp:
* html/parser/HTMLElementStack.h:
(HTMLElementStack):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (116456 => 116457)


--- trunk/Source/WebCore/ChangeLog	2012-05-08 21:57:21 UTC (rev 116456)
+++ trunk/Source/WebCore/ChangeLog	2012-05-08 22:04:23 UTC (rev 116457)
@@ -1,3 +1,18 @@
+2012-05-08  Rafael Weinstein  <[email protected]>
+
+        HTMLElementStack::hasOnlyHTMLElementsInScope is no longer called
+        https://bugs.webkit.org/show_bug.cgi?id=85908
+
+        Reviewed by Eric Seidel.
+
+        This patch just removes the dead code.
+
+        No tests needed. Cleanup only.
+
+        * html/parser/HTMLElementStack.cpp:
+        * html/parser/HTMLElementStack.h:
+        (HTMLElementStack):
+
 2012-05-08  W. James MacLean  <[email protected]>
 
         [chromium] Create LinkHighlightLayerChromium class to provide link-highlight preview animations for GraphicsLayerChromium.

Modified: trunk/Source/WebCore/html/parser/HTMLElementStack.cpp (116456 => 116457)


--- trunk/Source/WebCore/html/parser/HTMLElementStack.cpp	2012-05-08 21:57:21 UTC (rev 116456)
+++ trunk/Source/WebCore/html/parser/HTMLElementStack.cpp	2012-05-08 22:04:23 UTC (rev 116457)
@@ -462,19 +462,6 @@
     return false;
 }
 
-bool HTMLElementStack::hasOnlyHTMLElementsInScope() const
-{
-    for (ElementRecord* record = m_top.get(); record; record = record->next()) {
-        ContainerNode* node = record->node();
-        if (!isInHTMLNamespace(node))
-            return false;
-        if (isScopeMarker(node))
-            return true;
-    }
-    ASSERT_NOT_REACHED(); // <html> is always on the stack and is a scope marker.
-    return true;
-}
-
 bool HTMLElementStack::hasNumberedHeaderElementInScope() const
 {
     for (ElementRecord* record = m_top.get(); record; record = record->next()) {

Modified: trunk/Source/WebCore/html/parser/HTMLElementStack.h (116456 => 116457)


--- trunk/Source/WebCore/html/parser/HTMLElementStack.h	2012-05-08 21:57:21 UTC (rev 116456)
+++ trunk/Source/WebCore/html/parser/HTMLElementStack.h	2012-05-08 22:04:23 UTC (rev 116457)
@@ -140,7 +140,6 @@
     bool inSelectScope(const AtomicString& tagName) const;
     bool inSelectScope(const QualifiedName&) const;
 
-    bool hasOnlyHTMLElementsInScope() const;
     bool hasNumberedHeaderElementInScope() const;
 
     bool hasOnlyOneElement() const;
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to