Skip to content
  • shinyak@chromium.org's avatar
    [Refactoring] Use isActiveInsertionPoint() instead of isInsertionPoint() · dc90debf
    shinyak@chromium.org authored
    https://bugs.webkit.org/show_bug.cgi?id=100459
    
    Reviewed by Hajime Morita.
    
    Checking InsertionPoint and its activeness with two if-statement is error-prone. We would like to
    use a utility function which checks both at once.
    
    We rewrite some lines with such function.
    
    No new tests, simple refactoring.
    
    * dom/ComposedShadowTreeWalker.cpp:
    (WebCore::ComposedShadowTreeWalker::traverseNode):
    * html/shadow/ContentDistributor.cpp:
    (WebCore::ContentDistributor::populate):
    (WebCore::ContentDistributor::distribute):
    (WebCore::ContentDistributor::distributeNodeChildrenTo):
    * html/shadow/InsertionPoint.h:
    (WebCore::isInsertionPoint): Since our convention is the argument of this kind of function should not be null,
    we would like to make it similar to the other functions.
    (WebCore::toInsertionPoint):
    (WebCore::isLowerEncapsulationBoundary):
    
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@132791 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    dc90debf