Changeset 116457 in webkit


Ignore:
Timestamp:
May 8, 2012 3:04:23 PM (12 years ago)
Author:
rafaelw@chromium.org
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):

Location:
trunk/Source/WebCore
Files:
3 edited

Legend:

Unmodified
Added
Removed

  • trunk/Source/WebCore/ChangeLog

    r116456 r116457  
     12012-05-08  Rafael Weinstein  <rafaelw@chromium.org>
     2
     3        HTMLElementStack::hasOnlyHTMLElementsInScope is no longer called
     4        https://bugs.webkit.org/show_bug.cgi?id=85908
     5
     6        Reviewed by Eric Seidel.
     7
     8        This patch just removes the dead code.
     9
     10        No tests needed. Cleanup only.
     11
     12        * html/parser/HTMLElementStack.cpp:
     13        * html/parser/HTMLElementStack.h:
     14        (HTMLElementStack):
     15
    1162012-05-08  W. James MacLean  <wjmaclean@chromium.org>
    217

  • trunk/Source/WebCore/html/parser/HTMLElementStack.cpp

    r115763 r116457  
    463463}
    464464
    465 bool HTMLElementStack::hasOnlyHTMLElementsInScope() const
    466 {
    467     for (ElementRecord* record = m_top.get(); record; record = record->next()) {
    468         ContainerNode* node = record->node();
    469         if (!isInHTMLNamespace(node))
    470             return false;
    471         if (isScopeMarker(node))
    472             return true;
    473     }
    474     ASSERT_NOT_REACHED(); // <html> is always on the stack and is a scope marker.
    475     return true;
    476 }
    477 
    478465bool HTMLElementStack::hasNumberedHeaderElementInScope() const
    479466{

  • trunk/Source/WebCore/html/parser/HTMLElementStack.h

    r107700 r116457  
    141141    bool inSelectScope(const QualifiedName&) const;
    142142
    143     bool hasOnlyHTMLElementsInScope() const;
    144143    bool hasNumberedHeaderElementInScope() const;
    145144
Note: See TracChangeset for help on using the changeset viewer.