sync: centralize sync startup decisions in TryStart.

chrome/browser/signin/token_service.cc

Index: chrome/browser/signin/token_service.cc
diff --git a/chrome/browser/signin/token_service.cc b/chrome/browser/signin/token_service.cc
index 5eef15c5314d9cc6a76f4bc3260595c938b76d74..ef94d1532818a3ae8c4544950dea1a6b429357ac 100644
--- a/chrome/browser/signin/token_service.cc
+++ b/chrome/browser/signin/token_service.cc
@@ -191,6 +191,10 @@ bool TokenService::TokensLoadedFromDB() const {
   return tokens_loaded_;
 }
 
+void TokenService::set_tokens_loaded(bool loaded) {
+  tokens_loaded_ = loaded;
+}

[kochi, 2012/11/30 07:42:02]

Why not inline (define in header)?
(maybe this is just a preference, but I'm curious)

[tim (not reviewing), 2012/11/30 18:51:13]

No solid reason. I guess just overzealous application of
http://dev.chromium.org/developers/coding-style/cpp-dos-and-donts#TOC-Stop-in....
 I changed / inlined it.

+
 // static
 int TokenService::GetServiceIndex(const std::string& service) {
   for (size_t i = 0; i < arraysize(kServices); ++i) {