Eliminate unused AMP_Rule_Spec::$additional_allowed_tags

ampproject · Nov 26, 2019 · a2ff9d1 · a2ff9d1
1 parent 545611e
commit a2ff9d1
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 22 deletions.
diff --git a/includes/sanitizers/class-amp-rule-spec.php b/includes/sanitizers/class-amp-rule-spec.php
@@ -133,18 +133,4 @@ abstract class AMP_Rule_Spec {
 		'typemustmatch',
 		'visible',
 	];
-
-	/**
-	 * Additional allowed tags.
-	 *
-	 * @var array
-	 */
-	public static $additional_allowed_tags = [
-
-		// An experimental tag with no protoascii.
-		'amp-share-tracking' => [
-			'attr_spec_list' => [],
-			'tag_spec'       => [],
-		],
-	];
 }
diff --git a/includes/sanitizers/class-amp-tag-and-attribute-sanitizer.php b/includes/sanitizers/class-amp-tag-and-attribute-sanitizer.php
@@ -214,17 +214,11 @@ public function __construct( $dom, $args = [] ) {
 			];
 		}
 
-		// Prepare whitelists.
-		foreach ( AMP_Rule_Spec::$additional_allowed_tags as $tag_name => $tag_rule_spec ) {
-			$this->allowed_tags[ $tag_name ][] = $tag_rule_spec;
-		}
-
 		// @todo Do the same for body when !use_document_element?
 		if ( ! empty( $this->args['use_document_element'] ) ) {
 			foreach ( $this->allowed_tags['html'] as &$rule_spec ) {
 				unset( $rule_spec[ AMP_Rule_Spec::TAG_SPEC ][ AMP_Rule_Spec::MANDATORY_PARENT ] );
 			}
-
 			unset( $rule_spec );
 		}
 
@@ -234,10 +228,8 @@ public function __construct( $dom, $args = [] ) {
 					$tag_spec[ AMP_Rule_Spec::ATTR_SPEC_LIST ] = $this->process_alternate_names( $tag_spec[ AMP_Rule_Spec::ATTR_SPEC_LIST ] );
 				}
 			}
-
 			unset( $tag_spec );
 		}
-
 		unset( $tag_specs );
 
 		$this->globally_allowed_attributes = $this->process_alternate_names( $this->globally_allowed_attributes );