Fix memory leak in URLMatcherConditionFactory introduced in r122503

[email protected]
BUG=112155
TEST=no

Review URL: https://chromiumcodereview.appspot.com/9415049

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@122509 0039d316-1c4b-4281-b951-d872f2087c98
diff --git a/chrome/browser/extensions/api/declarative/url_matcher.cc b/chrome/browser/extensions/api/declarative/url_matcher.cc
index 77194e9..d92a4a3 100644
--- a/chrome/browser/extensions/api/declarative/url_matcher.cc
+++ b/chrome/browser/extensions/api/declarative/url_matcher.cc
@@ -334,10 +334,12 @@
       const std::set<SubstringPattern::ID>& used_patterns) {
   PatternSingletons::iterator i = pattern_singletons_.begin();
   while (i != pattern_singletons_.end()) {
-    if (used_patterns.find((*i)->id()) != used_patterns.end())
+    if (used_patterns.find((*i)->id()) != used_patterns.end()) {
       ++i;
-    else
+    } else {
+      delete *i;
       pattern_singletons_.erase(i++);
+    }
   }
 }