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

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

Issue 10963018: Rework arguments of HistoryService::AddPage() (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix Windows compile Created 8 years, 3 months 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
« no previous file with comments | « chrome/browser/history/history_types.h ('k') | chrome/browser/history/history_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/history/history_types.cc
diff --git a/chrome/browser/history/history_types.cc b/chrome/browser/history/history_types.cc
index dd7c0385996b48589e5d78c033713452a6de51d3..01ed9c3ea7376a540845a81e0853a3497b5196b9 100644
--- a/chrome/browser/history/history_types.cc
+++ b/chrome/browser/history/history_types.cc
@@ -265,35 +265,36 @@ TopSitesDelta::~TopSitesDelta() {}
// HistoryAddPageArgs ---------------------------------------------------------
+HistoryAddPageArgs::HistoryAddPageArgs()
+ : id_scope(NULL),
+ page_id(0),
+ transition(content::PAGE_TRANSITION_LINK),
+ visit_source(SOURCE_BROWSED),
+ did_replace_entry(false) {}
+
HistoryAddPageArgs::HistoryAddPageArgs(
- const GURL& arg_url,
- base::Time arg_time,
- const void* arg_id_scope,
- int32 arg_page_id,
- const GURL& arg_referrer,
- const history::RedirectList& arg_redirects,
- content::PageTransition arg_transition,
- VisitSource arg_source,
- bool arg_did_replace_entry)
- : url(arg_url),
- time(arg_time),
- id_scope(arg_id_scope),
- page_id(arg_page_id),
- referrer(arg_referrer),
- redirects(arg_redirects),
- transition(arg_transition),
- visit_source(arg_source),
- did_replace_entry(arg_did_replace_entry) {
+ const GURL& url,
+ base::Time time,
+ const void* id_scope,
+ int32 page_id,
+ const GURL& referrer,
+ const history::RedirectList& redirects,
+ content::PageTransition transition,
+ VisitSource source,
+ bool did_replace_entry)
+ : url(url),
+ time(time),
+ id_scope(id_scope),
+ page_id(page_id),
+ referrer(referrer),
+ redirects(redirects),
+ transition(transition),
+ visit_source(source),
+ did_replace_entry(did_replace_entry) {
}
HistoryAddPageArgs::~HistoryAddPageArgs() {}
-HistoryAddPageArgs* HistoryAddPageArgs::Clone() const {
- return new HistoryAddPageArgs(
- url, time, id_scope, page_id, referrer, redirects, transition,
- visit_source, did_replace_entry);
-}
-
ThumbnailMigration::ThumbnailMigration() {}
ThumbnailMigration::~ThumbnailMigration() {}
« no previous file with comments | « chrome/browser/history/history_types.h ('k') | chrome/browser/history/history_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698