Safebrowsing: change gethash caching to match api 2.3 rules, fix some corner cases.

chrome/browser/safe_browsing/protocol_manager.h

Index: chrome/browser/safe_browsing/protocol_manager.h
diff --git a/chrome/browser/safe_browsing/protocol_manager.h b/chrome/browser/safe_browsing/protocol_manager.h
index d53fbeaa665fbbd530d564a705f5cc91c63bf252..5588b697540f551f5616a956d84f03d25c36cf5b 100644
--- a/chrome/browser/safe_browsing/protocol_manager.h
+++ b/chrome/browser/safe_browsing/protocol_manager.h
@@ -59,10 +59,10 @@ class SafeBrowsingProtocolManager : public net::URLFetcherDelegate,
   // Parameters:
   //   - The vector of full hash results. If empty, indicates that there
   //     were no matches, and that the resource is safe.
-  //   - Whether the result can be cached. This may not be the case when
-  //     the result did not come from the SB server, for example.
+  //   - The cache lifetime of the result. A lifetime of 0 indicates the results
+  //     should not be cached.
   typedef base::Callback<void(const std::vector<SBFullHashResult>&,
-                              bool)> FullHashCallback;
+                              const base::TimeDelta&)> FullHashCallback;
 
   virtual ~SafeBrowsingProtocolManager();