Fix a typo in RealModelTypeToNotificationType.

Fix a typo in RealModelTypeToNotificationType.

BUG=139229
TEST=sync_integration_tests should still pass

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=148695

[akalin, 2012-07-26 23:14:16]

Lgtm
On Jul 26, 2012 4:00 PM, <dcheng@chromium.org> wrote:

> Reviewers: akalin,
>
> Description:
> Fix a typo in RealModelTypeToNotificationTyp**e.
>
> BUG=139229
> TEST=sync_integration_tests should still pass
>
> Please review this at
http://codereview.chromium.**org/10825047/<http://codereview.chromium.org/108...
>
> SVN Base:
svn://svn.chromium.org/chrome/**trunk/src<http://svn.chromium.org/chrome/trunk/src>
>
> Affected files:
>   M chrome/browser/sync/test/**integration/sync_test.cc
>   M sync/syncable/model_type.cc
>
>
> Index: chrome/browser/sync/test/**integration/sync_test.cc
> diff --git a/chrome/browser/sync/test/**integration/sync_test.cc
> b/chrome/browser/sync/test/**integration/sync_test.cc
> index da07fd4d9b779a636c9d9785f99b9f**929cd22de0..**
> b0dd0005bb8cb0f74951430fbaf5db**d5301baf79 100644
> --- a/chrome/browser/sync/test/**integration/sync_test.cc
> +++ b/chrome/browser/sync/test/**integration/sync_test.cc
> @@ -335,7 +335,7 @@ bool SyncTest::SetupSync() {
>    // This value must be updated whenever new permanent items are added
> (although
>    // this should handle new datatype-specific top level folders).
>    number_of_default_sync_items_ = GetClient(0)->GetNumEntries() -
> -                                  GetClient(0)->GetNumDatatypes(**) - 7;
> +                                  GetClient(0)->GetNumDatatypes(**) - 6;
>    DVLOG(1) << "Setting " << number_of_default_sync_items_ << " as default
> "
>             << " number of entries.";
>
> Index: sync/syncable/model_type.cc
> diff --git a/sync/syncable/model_type.cc b/sync/syncable/model_type.cc
> index 4c75f9158c4aaf1598dad6770253a4**b4371aa7da..**
> 83e803cdd58c451f6b8eae1efa8e35**e56e156203 100644
> --- a/sync/syncable/model_type.cc
> +++ b/sync/syncable/model_type.cc
> @@ -453,7 +453,7 @@ bool RealModelTypeToNotificationTyp**e(ModelType
> model_type,
>        *notification_type = kNigoriNotificationType;
>        return true;
>      case APP_SETTINGS:
> -      *notification_type = kAppNotificationType;
> +      *notification_type = kAppSettingNotificationType;
>        return true;
>      case APPS:
>        *notification_type = kAppNotificationType;
>
>
>

[commit-bot: I haz the power, 2012-07-27 00:48:04]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dcheng@chromium.org/10825047/1