* Removed num_consecutive_errors and consecutive_transient_error counters. These aren't very useful because they tell us nothing about the cause of the errors. They lead to confusing bugs, with titles like "Need a graceful way to recover from a non-zero 'Max consecutive error count'" (most users commenting on this bug were being throttled). Further adding to the confusion is a bug in AllStatus that caused this counter to be reset after every sync cycle, so the value displayed was the maximum number of consecutive errors *for the most recent sync cycle*. * Removed SyncInternal::RaiseAuthNeededEvent(). This method was undefined. * Removed notifiable commits counter. We no longer use P2P notifications, so the client no longer has any say about whether or not a commit is "notifiable". The terminology is out of date and the code to maintain the count seemed out of place in SyncManager. * Added some counters to report on the number of commit cycles. These counters should provide all the information that used to be made available through the notifiable commits counter, and more. BUG=98346 TEST= Review URL: http://codereview.chromium.org/9269025 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@120376 0039d316-1c4b-4281-b951-d872f2087c98