Changes to add duration into history database.

chrome/browser/history/history.h

Index: chrome/browser/history/history.h
diff --git a/chrome/browser/history/history.h b/chrome/browser/history/history.h
index 230eb4f419eb81ee26c3aff0a032fd58420561f3..7c4ae1f7983aaf019aeedea8a2211bdbc79ffd68 100644
--- a/chrome/browser/history/history.h
+++ b/chrome/browser/history/history.h
@@ -220,6 +220,17 @@ class HistoryService : public CancelableRequestProvider,
   // title in the full text index.
   void SetPageTitle(const GURL& url, const string16& title);
 
+  // Updates a page with its ending time stamp information. The page can be
+  // identified by the combination of the pointer to a RenderProcessHost, the
+  // page id and the url.
+  //
+  // The given pointer will not be dereferenced, it is only used for
+  // identification purposes, hence it is a void*.
+  void UpdatePageInfo(const void* host,

[brettw, 2012/03/26 20:25:06]

Updating a page's ending time stamp seems pretty different than updating
"PageInfo" which is what the function is called. Can you pick a more appropriate
name for this?

[Wei Li, 2012/03/27 18:11:53]

Changed to UpdateDBWithPageInfo which could be more specific . But I would think
this as a general service method name. What do you think?

[brettw, 2012/03/28 16:33:48]

My problem wasn't that "update" was ambiguous, but that "page info" is. This
updates the visit time. It should have that in the name, the fact that it
updates the DB is implied.

[Wei Li, 2012/03/28 17:37:38]

Hope the name is clearer this time.

+                      int32 page_id,
+                      const GURL& url,
+                      const base::Time end_ts);
+
   // Indexing ------------------------------------------------------------------
 
   // Notifies history of the body text of the given recently-visited URL.