Remove multiple instances of div#fb-root

Author: pierlon

includes/embeds/class-amp-facebook-embed.php

@@ -94,7 +94,12 @@ public function sanitize_raw_embeds( Document $dom ) {
 					$parent_node->parentNode->removeChild( $parent_node );
 				}
 			}
-			$fb_root->parentNode->removeChild( $fb_root );
+
+			// Remove other instances of <div id="fb-root">.
+			$fb_root_query = $dom->xpath->query( '//div[ @id = "fb-root" ]' );
+			foreach ( $fb_root_query as $fb_root ) {
+				$fb_root->parentNode->removeChild( $fb_root );
+			}
 		}
 	}
 

tests/php/test-amp-facebook-embed.php

@@ -219,6 +219,11 @@ public function get_raw_embed_dataset() {
 				'',
 			],
 
+			'remove_multiple_fb_root'       => [
+				str_repeat( '<div id="fb-root"></div>', 5 ) . str_repeat( '<script async defer crossorigin="anonymous" src="https://connect.facebook.net/en_US/sdk.js#xfbml=1&version=v3.2"></script>', 5 ), // phpcs:ignore WordPress.WP.EnqueuedResources.NonEnqueuedScript
+				'',
+			],
+
 			'remove_empty_p_tag'            => [
 				'<p><script async defer crossorigin="anonymous" src="https://connect.facebook.net/en_US/sdk.js#xfbml=1&version=v3.2"></script></p><div id="fb-root"></div>', // phpcs:ignore WordPress.WP.EnqueuedResources.NonEnqueuedScript
 				'',