Bug 1280370 - Remove unneeded DomainIsWildcard check draft
authorRob Wu <rob@robwu.nl>
Wed, 08 Aug 2018 18:56:53 +0200
changeset 827706 aef8a69d4d11efe927216837d02679e532ca6509
parent 827705 fdf039bc1cc469e958e74d85062c02582198822c
push id118573
push userbmo:rob@robwu.nl
push dateWed, 08 Aug 2018 21:39:11 +0000
bugs1280370
milestone63.0a1
Bug 1280370 - Remove unneeded DomainIsWildcard check The DomainIsWildcard check is already covered by MatchesDomain.
toolkit/components/extensions/MatchPattern.cpp
--- a/toolkit/components/extensions/MatchPattern.cpp
+++ b/toolkit/components/extensions/MatchPattern.cpp
@@ -427,17 +427,17 @@ MatchPattern::Matches(const URLInfo& aUR
   if (aExplicit && mMatchSubdomain) {
     return false;
   }
 
   if (!mSchemes->Contains(aURL.Scheme())) {
     return false;
   }
 
-  if (!DomainIsWildcard() && !MatchesDomain(aURL.Host())) {
+  if (!MatchesDomain(aURL.Host())) {
     return false;
   }
 
   if (mPath && !mPath->IsWildcard() && !mPath->Matches(aURL.Path())) {
     return false;
   }
 
   return true;