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

Unified Diff: components/sync_sessions/revisit/sessions_page_revisit_observer.cc

Issue 1387253004: [Sync] Creating sync_sessions component, moving revisit logic. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Removing test_support target. Created 5 years, 2 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
Index: components/sync_sessions/revisit/sessions_page_revisit_observer.cc
diff --git a/components/sync_driver/revisit/sessions_page_revisit_observer.cc b/components/sync_sessions/revisit/sessions_page_revisit_observer.cc
similarity index 84%
rename from components/sync_driver/revisit/sessions_page_revisit_observer.cc
rename to components/sync_sessions/revisit/sessions_page_revisit_observer.cc
index 1cd56dd90d0b7e55535038675600d66c88378b2f..1947b3e5d7348a37e5c4ddd7c895ede61cf7394e 100644
--- a/components/sync_driver/revisit/sessions_page_revisit_observer.cc
+++ b/components/sync_sessions/revisit/sessions_page_revisit_observer.cc
@@ -2,17 +2,20 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "components/sync_driver/revisit/sessions_page_revisit_observer.h"
+#include "components/sync_sessions/revisit/sessions_page_revisit_observer.h"
+
+#include <utility>
#include "base/metrics/histogram_macros.h"
#include "base/thread_task_runner_handle.h"
+#include "base/time/time.h"
#include "components/sessions/core/session_types.h"
#include "components/sync_driver/glue/synced_session.h"
-#include "components/sync_driver/revisit/current_tab_matcher.h"
-#include "components/sync_driver/revisit/offset_tab_matcher.h"
-#include "components/sync_driver/revisit/page_equality.h"
+#include "components/sync_sessions/revisit/current_tab_matcher.h"
+#include "components/sync_sessions/revisit/offset_tab_matcher.h"
+#include "components/sync_sessions/revisit/page_equality.h"
-namespace sync_driver {
+namespace sync_sessions {
SessionsPageRevisitObserver::SessionsPageRevisitObserver(
scoped_ptr<ForeignSessionsProvider> provider)
@@ -20,8 +23,9 @@ SessionsPageRevisitObserver::SessionsPageRevisitObserver(
SessionsPageRevisitObserver::~SessionsPageRevisitObserver() {}
-void SessionsPageRevisitObserver::OnPageVisit(const GURL& url,
- const TransitionType transition) {
+void SessionsPageRevisitObserver::OnPageVisit(
+ const GURL& url,
+ const PageVisitObserver::TransitionType transition) {
// We need to be invoked and eventually execute on the thread which owns the
// session objects the provider will give us. However, this work is not
// especially time sensitive, and so we post a task to perform this to get out
@@ -36,7 +40,7 @@ void SessionsPageRevisitObserver::OnPageVisit(const GURL& url,
void SessionsPageRevisitObserver::CheckForRevisit(
const GURL& url,
- const TransitionType transition) {
+ const PageVisitObserver::TransitionType transition) {
base::TimeTicks start(base::TimeTicks::Now());
// We want to match tabs/navigation entries in two slightly different ways. We
@@ -75,4 +79,4 @@ void SessionsPageRevisitObserver::CheckForRevisit(
UMA_HISTOGRAM_TIMES("Sync.PageRevisitSessionDuration", duration);
}
-} // namespace sync_driver
+} // namespace sync_sessions

Powered by Google App Engine
This is Rietveld 408576698