sync/internal_api/public/sessions/sync_session_snapshot.h - Issue 10917234: sync: make scheduling logic and job ownership more obvious.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: sync/internal_api/public/sessions/sync_session_snapshot.h

Issue 10917234: sync: make scheduling logic and job ownership more obvious. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src

Patch Set: test + comment + rebase 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:

Index: sync/internal_api/public/sessions/sync_session_snapshot.h

diff --git a/sync/internal_api/public/sessions/sync_session_snapshot.h b/sync/internal_api/public/sessions/sync_session_snapshot.h

index 74e972bf850bd5a622f00681635c009588006283..39aba45bfbf26e5982590cde8b6eca4aa69c374c 100644

--- a/sync/internal_api/public/sessions/sync_session_snapshot.h

+++ b/sync/internal_api/public/sessions/sync_session_snapshot.h

@@ -43,8 +43,7 @@ class SyncSessionSnapshot {

const SyncSourceInfo& source,

bool notifications_enabled,

size_t num_entries,

- base::Time sync_start_time,

- bool retry_scheduled);

+ base::Time sync_start_time);

~SyncSessionSnapshot();

// Caller takes ownership of the returned dictionary.

@@ -69,7 +68,6 @@ class SyncSessionSnapshot {

bool notifications_enabled() const;

size_t num_entries() const;

base::Time sync_start_time() const;

- bool retry_scheduled() const;

// Set iff this snapshot was not built using the default constructor.

bool is_initialized() const;

@@ -89,7 +87,6 @@ class SyncSessionSnapshot {

bool notifications_enabled_;

size_t num_entries_;

base::Time sync_start_time_;

- bool retry_scheduled_;

rlarocque 2012/09/24 21:56:47 Cool! I didn't realize this was no longer in use.

bool is_initialized_;

};