Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4108)

Unified Diff: chrome/browser/history/history_tab_helper.cc

Issue 111633003: Removes some dead database code (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: leave idle comment Created 7 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/history/history_tab_helper.cc
diff --git a/chrome/browser/history/history_tab_helper.cc b/chrome/browser/history/history_tab_helper.cc
index 22567445123d5b32c5e3d4fcedc4abd3a38660a9..03ccc8157f2f73e4ad75c88d34b47967bfb6f21f 100644
--- a/chrome/browser/history/history_tab_helper.cc
+++ b/chrome/browser/history/history_tab_helper.cc
@@ -154,9 +154,6 @@ HistoryService* HistoryTabHelper::GetHistoryService() {
}
void HistoryTabHelper::WebContentsDestroyed(WebContents* tab) {
- // TODO(sky): nuke this since no one is using visit_duration (and this is all
- // wrong).
-
// We update the history for this URL.
// The content returned from web_contents() has been destroyed by now.
// We need to use tab value directly.

Powered by Google App Engine
This is Rietveld 408576698