Fix precache issue with cancel and resume

components/precache/core/precache_url_table.cc

 // Copyright 2013 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 "components/precache/core/precache_url_table.h"
 
 #include <string>
 
 #include "base/logging.h"
 #include "sql/connection.h"
@@ skipping 90 matching lines @@
   while (statement.Step()) {
     GURL url(statement.ColumnString(0));
     if (statement.ColumnInt(1))
       used_urls->push_back(url);
     else
       unused_urls->push_back(url);
   }
 }
 
 void PrecacheURLTable::ClearAllForReferrerHost(int64_t referrer_host_id) {
-  Statement statement(db_->GetCachedStatement(
+  // Delete the URLs that are not precached.
+  Statement delete_statement(

[sclittle, 2016/09/02 18:47:56]

Should a test be added for this new functionality?

[Raj, 2016/09/15 17:02:25]

Done.

+      db_->GetCachedStatement(SQL_FROM_HERE,
+                              "DELETE FROM precache_urls WHERE "
+                              "referrer_host_id=? AND is_precached=0"));
+  delete_statement.BindInt64(0, referrer_host_id);
+  delete_statement.Run();
+
+  // Clear was_used for precached URLs.
+  Statement update_statement(db_->GetCachedStatement(
       SQL_FROM_HERE,
       "UPDATE precache_urls SET was_used=0 WHERE referrer_host_id=?"));
-
-  statement.BindInt64(0, referrer_host_id);
-  statement.Run();
+  update_statement.BindInt64(0, referrer_host_id);
+  update_statement.Run();
 }
 
 void PrecacheURLTable::DeleteAllPrecachedBefore(const base::Time& delete_end) {
   Statement statement(db_->GetCachedStatement(
       SQL_FROM_HERE, "DELETE FROM precache_urls WHERE time < ?"));
 
   statement.BindInt64(0, delete_end.ToInternalValue());
   statement.Run();
 }
 
@@ skipping 35 matching lines @@
       return false;
     if (!db_->DoesColumnExist("precache_urls", "referrer_host_id") &&
         !db_->Execute(
             "ALTER TABLE precache_urls ADD COLUMN referrer_host_id INTEGER"))
       return false;
   }
   return true;
 }
 
 }  // namespace precache