Changes to add duration into history database.

chrome/browser/history/history.h

 // Copyright (c) 2012 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.
 
 #ifndef CHROME_BROWSER_HISTORY_HISTORY_H_
 #define CHROME_BROWSER_HISTORY_HISTORY_H_
 #pragma once
 
 #include <set>
 #include <vector>
@@ skipping 202 matching lines @@
   // only be used when bookmarking a page, otherwise the row leaks in the
   // history db (it never gets cleaned).
   void AddPageNoVisitForBookmark(const GURL& url);
 
   // Sets the title for the given page. The page should be in history. If it
   // is not, this operation is ignored. This call will not update the full
   // text index. The last title set when the page is indexed will be the
   // title in the full text index.
   void SetPageTitle(const GURL& url, const string16& title);
 
+  // Updates the history database with a page's 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 UpdateWithPageEndTime(const void* host,
+                             int32 page_id,
+                             const GURL& url,
+                             const base::Time end_ts);

[brettw, 2012/03/28 17:51:45]

You don't need the const here (since you pass by value which is fine for the
Time class).

[Wei Li, 2012/03/28 18:53:13]

Done.

+
   // Indexing ------------------------------------------------------------------
 
   // Notifies history of the body text of the given recently-visited URL.
   // If the URL was not visited "recently enough," the history system may
   // discard it.
   void SetPageContents(const GURL& url, const string16& contents);
 
   // Querying ------------------------------------------------------------------
 
   // Callback class that a client can implement to iterate over URLs. The
@@ skipping 649 matching lines @@
   // True if needs top site migration.
   bool needs_top_sites_migration_;
 
   // The index used for quick history lookups.
   scoped_ptr<history::InMemoryURLIndex> in_memory_url_index_;
 
   DISALLOW_COPY_AND_ASSIGN(HistoryService);
 };
 
 #endif  // CHROME_BROWSER_HISTORY_HISTORY_H_