Implement correct logic for URLPatternSet set operators.

extensions/common/url_pattern.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "extensions/common/url_pattern.h"
 
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_piece.h"
 #include "base/strings/string_split.h"
 #include "base/strings/string_util.h"
@@ skipping 443 matching lines @@
     return true;
   return (MatchesAnyScheme(other.GetExplicitSchemes()) ||
           other.MatchesAnyScheme(GetExplicitSchemes()))
       && (MatchesHost(other.host()) || other.MatchesHost(host()))
       && (MatchesPortPattern(other.port()) || other.MatchesPortPattern(port()))
       && (MatchesPath(StripTrailingWildcard(other.path())) ||
           other.MatchesPath(StripTrailingWildcard(path())));
 }
 
 bool URLPattern::Contains(const URLPattern& other) const {
+  if (*this == other)
+    return true;
   if (match_all_urls())
     return true;
   return MatchesAllSchemes(other.GetExplicitSchemes())
       && MatchesHost(other.host())
       && MatchesPortPattern(other.port())
       && MatchesPath(StripTrailingWildcard(other.path()));
 }
 
 bool URLPattern::MatchesAnyScheme(
     const std::vector<std::string>& schemes) const {
@@ skipping 62 matching lines @@
   }
 
   return result;
 }
 
 // static
 const char* URLPattern::GetParseResultString(
     URLPattern::ParseResult parse_result) {
   return kParseResultMessages[parse_result];
 }