Don't update profile information when the sync state changes.

chrome/browser/ui/webui/options/browser_options_handler.cc

Index: chrome/browser/ui/webui/options/browser_options_handler.cc
diff --git a/chrome/browser/ui/webui/options/browser_options_handler.cc b/chrome/browser/ui/webui/options/browser_options_handler.cc
index 8831a3899be40ea6eb2effa83e5a736c070517c3..a63095236cb2b3fc261da330369789cacb0f4230 100644
--- a/chrome/browser/ui/webui/options/browser_options_handler.cc
+++ b/chrome/browser/ui/webui/options/browser_options_handler.cc
@@ -658,17 +658,16 @@ void BrowserOptionsHandler::RegisterMessages() {
 }
 
 void BrowserOptionsHandler::OnStateChanged() {
-  web_ui()->CallJavascriptFunction("BrowserOptions.updateSyncState",
-                                   *GetSyncStateDictionary());
-
-  SendProfilesInfo();

[Adrian Kuegel, 2013/08/06 10:05:35]

I haven't completely understood if removing this line might break something, but
from what I can tell the call should be covered by the
chrome::NOTIFICATION_PROFILE_CACHED_INFO_CHANGED.
Anyway, the documentation of the OnStateChanged method seems to suggest to query
the sync service about what kind of change occurred, so that might also be an
option.

[Bernhard Bauer, 2013/08/06 10:38:49]

Yes, I traced this back to when it was added originally, and the two updates
were just lumped together in the same method (OnStateChanged sounds kind of
general).
Well, ProfileSyncService sends the notification among other things after every
sync cycle, so nothing would even have changed there.

+  UpdateSyncState();
 }
 
-void BrowserOptionsHandler::OnSigninAllowedPrefChange() {
+void BrowserOptionsHandler::UpdateSyncState() {

[Adrian Kuegel, 2013/08/06 10:05:35]

Can you move this function (and OnSigninAllowedPrefChange) down corresponding to
their position in the header file?
I guess the functions in this file should at least be ordered by public
functions first, then private.

[Bernhard Bauer, 2013/08/06 10:38:49]

Done.

   web_ui()->CallJavascriptFunction("BrowserOptions.updateSyncState",
                                    *GetSyncStateDictionary());
+}
 
-  SendProfilesInfo();
+void BrowserOptionsHandler::OnSigninAllowedPrefChange() {
+  UpdateSyncState();
 }
 
 void BrowserOptionsHandler::PageLoadStarted() {