jenkins-bot has submitted this change and it was merged.
Change subject: Balancer: Introduce BalanceElement::isHtmlNamed()
......................................................................
Balancer: Introduce BalanceElement::isHtmlNamed()
An optimised special case of BalanceElement::isA, reducing benchmark
time by 4%
Change-Id: I1204de9454eb7b8f9f3a5ed137218c3293f9ab27
---
M includes/tidy/Balancer.php
1 file changed, 27 insertions(+), 17 deletions(-)
Approvals:
Cscott: Looks good to me, approved
jenkins-bot: Verified
diff --git a/includes/tidy/Balancer.php b/includes/tidy/Balancer.php
index a6bb575..00f6e4b 100644
--- a/includes/tidy/Balancer.php
+++ b/includes/tidy/Balancer.php
@@ -467,7 +467,7 @@
$blank = false;
}
}
- if ( $this->isA( 'mw:p-wrap' ) ) {
+ if ( $this->isHtmlNamed( 'mw:p-wrap' ) ) {
$this->localName = 'p';
} elseif ( $blank ) {
// Add 'mw-empty-elt' class so elements can be
hidden via CSS
@@ -539,6 +539,16 @@
# assume this is an HTML element name.
return $this->isHtml() && $this->localName === $set;
}
+ }
+
+ /**
+ * Determine if this element is an HTML element with the specified name
+ * @param string $tagName
+ * @return bool
+ */
+ public function isHtmlNamed( $tagName ) {
+ return $this->namespaceURI === BalanceSets::HTML_NAMESPACE
+ && $this->localName === $tagName;
}
/**
@@ -724,7 +734,7 @@
public function insertElement( $elt ) {
Assert::parameterType( 'MediaWiki\Tidy\BalanceElement', $elt,
'$elt' );
if (
- $this->currentNode->isA( 'mw:p-wrap' ) &&
+ $this->currentNode->isHtmlNamed( 'mw:p-wrap' ) &&
!$elt->isA( BalanceSets::$tidyInlineSet )
) {
// Tidy compatibility.
@@ -806,7 +816,7 @@
/**
* Generate implied end tags.
- * @param BalanceElement|array|string|null $butnot
+ * @param string $butnot
* @param bool $thorough True if we should generate end tags thoroughly.
* @see
https://html.spec.whatwg.org/multipage/syntax.html#generate-implied-end-tags
*/
@@ -815,7 +825,7 @@
BalanceSets::$thoroughImpliedEndTagsSet :
BalanceSets::$impliedEndTagsSet;
while ( $this->length() > 0 ) {
- if ( $butnot !== null && $this->currentNode->isA(
$butnot ) ) {
+ if ( $butnot !== null &&
$this->currentNode->isHtmlNamed( $butnot ) ) {
break;
}
if ( !$this->currentNode->isA( $endTagSet ) ) {
@@ -907,7 +917,7 @@
} else {
$this->currentNode = null;
}
- if ( !$elt->isA( 'mw:p-wrap' ) ) {
+ if ( !$elt->isHtmlNamed( 'mw:p-wrap' ) ) {
$elt->flatten( $this->tidyCompat );
}
}
@@ -1046,14 +1056,14 @@
}
} else {
// We're fostering an element; do we need to
merge p-wrappers?
- if ( $elt->isA( 'mw:p-wrap' ) ) {
+ if ( $elt->isHtmlNamed( 'mw:p-wrap' ) ) {
$idx = $before ?
array_search( $before,
$parent->children, true ) :
count( $parent->children );
$after = $idx > 0 ?
$parent->children[$idx - 1] : '';
if (
$after instanceof
BalanceElement &&
- $after->isA( 'mw:p-wrap' )
+ $after->isHtmlNamed(
'mw:p-wrap' )
) {
return $after; // Re-use
existing p-wrapper.
}
@@ -1085,7 +1095,7 @@
// elements, then pop the current node off the stack of open
// elements and abort these steps.
if (
- $this->currentNode->isA( $tag ) &&
+ $this->currentNode->isHtmlNamed( $tag ) &&
!$afe->isInList( $this->currentNode )
) {
$this->pop();
@@ -2085,10 +2095,10 @@
$stacklen = $this->stack->length();
for ( $j = $i + 1; $j < $stacklen-1;
$j++ ) {
$ancestor = $this->stack->node(
$stacklen-$j-1 );
- if ( $ancestor->isA( 'template'
) ) {
+ if ( $ancestor->isHtmlNamed(
'template' ) ) {
break;
}
- if ( $ancestor->isA( 'table' )
) {
+ if ( $ancestor->isHtmlNamed(
'table' ) ) {
$this->switchMode(
'inSelectInTableMode' );
return;
}
@@ -2334,7 +2344,7 @@
case 'li':
# OMITTED: frameset_ok
foreach ( $this->stack as $node ) {
- if ( $node->isA( 'li' ) ) {
+ if ( $node->isHtmlNamed( 'li' ) ) {
$this->inBodyMode( 'endtag',
'li' );
break;
}
@@ -2355,11 +2365,11 @@
case 'dt':
# OMITTED: frameset_ok
foreach ( $this->stack as $node ) {
- if ( $node->isA( 'dd' ) ) {
+ if ( $node->isHtmlNamed( 'dd' ) ) {
$this->inBodyMode( 'endtag',
'dd' );
break;
}
- if ( $node->isA( 'dt' ) ) {
+ if ( $node->isHtmlNamed( 'dt' ) ) {
$this->inBodyMode( 'endtag',
'dt' );
break;
}
@@ -2513,7 +2523,7 @@
case 'optgroup':
case 'option':
- if ( $this->stack->currentNode->isA( 'option' )
) {
+ if ( $this->stack->currentNode->isHtmlNamed(
'option' ) ) {
$this->inBodyMode( 'endtag', 'option' );
}
$this->afe->reconstruct( $this->stack );
@@ -2708,7 +2718,7 @@
// Any other end tag goes here
foreach ( $this->stack as $i => $node ) {
- if ( $node->isA( $value ) ) {
+ if ( $node->isHtmlNamed( $value ) ) {
$this->stack->generateImpliedEndTags(
$value );
$this->stack->popTo( $i ); # including
$i
break;
@@ -2923,7 +2933,7 @@
} elseif ( $token === 'endtag' ) {
switch ( $value ) {
case 'colgroup':
- if ( !$this->stack->currentNode->isA(
'colgroup' ) ) {
+ if ( !$this->stack->currentNode->isHtmlNamed(
'colgroup' ) ) {
return true; // Ignore the token.
}
$this->stack->pop();
@@ -2940,7 +2950,7 @@
}
// Anything else
- if ( !$this->stack->currentNode->isA( 'colgroup' ) ) {
+ if ( !$this->stack->currentNode->isHtmlNamed( 'colgroup' ) ) {
return true; // Ignore the token.
}
$this->inColumnGroupMode( 'endtag', 'colgroup' );
--
To view, visit https://gerrit.wikimedia.org/r/297949
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I1204de9454eb7b8f9f3a5ed137218c3293f9ab27
Gerrit-PatchSet: 5
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Tim Starling <[email protected]>
Gerrit-Reviewer: Cscott <[email protected]>
Gerrit-Reviewer: Tim Starling <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
