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

Unified Diff: components/sync_sessions/task_tracker.h

Issue 2901533003: Revert of Track task ids for navigations cross multiple tabs. (Closed)
Patch Set: Created 3 years, 7 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/task_tracker.h
diff --git a/components/sync_sessions/task_tracker.h b/components/sync_sessions/task_tracker.h
index e7987b6058799109322bef0f8d7a39a797003c52..44ee36368a6f879822bf5f8a15dc8cb2878d9d02 100644
--- a/components/sync_sessions/task_tracker.h
+++ b/components/sync_sessions/task_tracker.h
@@ -29,10 +29,10 @@
// TODO(shenchao): If the tab is restored, then the input navigation is not
// necessarily the first navigation in this case. Need to fix it by initalizing
// the object with restored data.
+// TODO(shenchao): Support to track tasks cross tabs.
class TabTasks {
public:
TabTasks();
- explicit TabTasks(const TabTasks* source_tab);
virtual ~TabTasks();
// Gets top-down task id list of ancestors and itself for
@@ -64,14 +64,6 @@
int64_t task_id;
};
- // Get position within task_ids_ for the navigation at |navigation_index| of
- // the tab.
- int GetTaskIdPositionFromNavigationIndex(int navigation_index) const;
-
- // Get index of corresponding navigation of the tab at |task_id_position|
- // within task_ids_.
- int GetNavigationIndexFromTaskIdPosition(int task_id_position) const;
-
// Task ids (with root task) for the navigations of the tab. The vector is
// corresponding to the sequence of navigations of the tab.
std::vector<TaskIdAndRoot> task_ids_;
@@ -80,8 +72,6 @@
// Number of oldest ancestors which have been excluded from being tracked in
// task_ids_;
int excluded_navigation_num_ = 0;
- // Number of tasks from source tab.
- int source_tab_task_num_ = 0;
DISALLOW_COPY_AND_ASSIGN(TabTasks);
};
@@ -97,11 +87,6 @@
// given |tab_id|.
TabTasks* GetTabTasks(SessionID::id_type tab_id);
- // Returns a TabTasks pointer, which is owned by this object, for the tab of
- // given |tab_id|, which is created from a source tab |source_tab_id|.
- TabTasks* GetTabTasks(SessionID::id_type tab_id,
- SessionID::id_type source_tab_id);
-
// Cleans tracked task ids of navigations in the tab of |tab_id|.
void CleanTabTasks(SessionID::id_type tab_id);
« no previous file with comments | « components/sync_sessions/sessions_sync_manager_unittest.cc ('k') | components/sync_sessions/task_tracker.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698