Remove unused variable from media.module and from UrlResolver.php files

I've tried this patch. Looking good to me. Unused variables are removed.

+++ b/core/modules/media/src/OEmbed/UrlResolver.php
@@ -123,7 +123,7 @@ protected function findUrl(\DOMXPath $xpath, $format) {
     // Check the URL against every scheme of every endpoint of every provider
     // until we find a match.
-    foreach ($this->providers->getAll() as $provider_name => $provider_info) {
+    foreach ($this->providers->getAll() as $provider_info) {
       foreach ($provider_info->getEndpoints() as $endpoint) {

People often specify the key regardless of usage in foreach statements for readability. Please remove that hunk from the patch and let's open a coding standards issue if we want to standardise either way.

@Hardik_Patel_12 and @catch
Updated the patch as per the standards.

@Hardik_Patel_12 and @catch
Updated the patch as per the standards.

Thanks @Rangaswini for the patch, I have reviewed the above patch and we can definitely remove the unused variable ,also confirmed it in the community with @longwave as well :)

Committed 3459966 and pushed to 9.0.x. Thanks!