diff options
| author | mattm@chromium.org <mattm@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-07-14 21:16:09 +0000 |
|---|---|---|
| committer | mattm@chromium.org <mattm@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-07-14 21:16:09 +0000 |
| commit | c36caf72ef5dfd8ed53c2541a402fa283982402b (patch) | |
| tree | 099d624951a72bb980f91822d44eac16224669bc /chrome | |
| parent | a0fc50d70d4f98debbc99c312cec44c3eff5bed5 (diff) | |
| download | chromium_src-c36caf72ef5dfd8ed53c2541a402fa283982402b.zip chromium_src-c36caf72ef5dfd8ed53c2541a402fa283982402b.tar.gz chromium_src-c36caf72ef5dfd8ed53c2541a402fa283982402b.tar.bz2 | |
Coverity: missing break in SyncSetupFlow::OnDialogClosed switch.
This may have lead to some bad stats in Sync.UserPerceivedAuthorizationTime histogram.
CID=11400
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/2976003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52403 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome')
| -rw-r--r-- | chrome/browser/sync/sync_setup_flow.cc | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/chrome/browser/sync/sync_setup_flow.cc b/chrome/browser/sync/sync_setup_flow.cc index 84cbf11..769d4b0 100644 --- a/chrome/browser/sync/sync_setup_flow.cc +++ b/chrome/browser/sync/sync_setup_flow.cc @@ -299,6 +299,7 @@ void SyncSetupFlow::OnDialogClosed(const std::string& json_retval) { case SyncSetupWizard::CHOOSE_DATA_TYPES: ProfileSyncService::SyncEvent( ProfileSyncService::CANCEL_FROM_CHOOSE_DATA_TYPES); + break; case SyncSetupWizard::DONE_FIRST_TIME: case SyncSetupWizard::DONE: // TODO(sync): rename this histogram; it's tracking authorization AND |