Port HistoryService::GetVisibleVisitCountToHost to CancelableTaskTracker

chrome/browser/history/history_types.h

Index: chrome/browser/history/history_types.h
diff --git a/chrome/browser/history/history_types.h b/chrome/browser/history/history_types.h
index e0ba22447699c280d7723d8c0b247cfcf5775214..96d57e19067fc7847a181f1b8c1746c7a315449d 100644
--- a/chrome/browser/history/history_types.h
+++ b/chrome/browser/history/history_types.h
@@ -510,6 +510,19 @@ struct QueryRedirectsResult {
   RedirectList redirects;
 };
 
+// VisibleVisitCountToHostResult ----------------------------------------------
+
+// VisibleVisitCountToHostResult encapsulate the result of a call to

[blundell, 2014/06/24 14:08:58]

nit: encapsulates.

[sdefresne, 2014/06/25 12:20:36]

Done.

+// HistoryBackend::GetVisibleVisitCountToHost.
+struct VisibleVisitCountToHostResult {
+  // Indicates whether the call to HistoryBackend::GetVisibleVisitCountToHost

[blundell, 2014/06/24 14:08:58]

nit: should you declare the constructor/destructor is is done above?

[sdefresne, 2014/06/25 12:20:36]

I declared it for the other structure because they contains complex objects
(std::vector) and android compilation complains that the constructor/destructor
should not be inlined for complex objects. Here the structure only contains
simple types, so I don't think the constructor/destructor are required. I'll add
them if compilation of android complains.

+  // was successfull or not. If false, then both |count| and |first_visit| are
+  // undefined.
+  bool success;
+  int count;
+  base::Time first_visit;
+};
+
 // KeywordSearchTermVisit -----------------------------------------------------
 
 // KeywordSearchTermVisit is returned from GetMostRecentKeywordSearchTerms. It