LocalAddressBook.renameAccount() doesn't prepare accounts the same way as create() does

[rfc2822]

During a rename address book operation (renameAccount()), LocalAddressBook doesn't call updateSyncFrameworkSettings() when creating address books. This may cause one or two extra unnecessary syncs from the sync framework. It fixes itself anyway because sooner or later LocalAddressBook.update() will be called, which in turn calls updateSyncFrameworkSettings.

LocalAddressBook.create() does call updateSyncFrameworkSettings, but create() is not used by renameAccount(). Shared code should be in a separate method.

addressBook.settings (ContactsContract.Settings.SHOULD_SYNC and UNGROUPED_VISIBLE) are also not set when address books are created during renaming or when address books are updated.

addressBook.readOnly is also not set when renamed, but at least when updated.

---

It should be exactly documented in which state an address book account should be and how and when it gets into that state.

• LocalAddressBook: call updateSyncFrameworkSettings when address books are renamed #1112
• LocalAddressBook: set/update ContactsContract.Settings.* when address books are renamed and updated #1113
• LocalAddressBook: set/update readOnly when address books are renamed #1114
  • Currently, when address books are updated, readOnly is only updated when forceReadOnly != null. However readOnly should always be calculated and the nowReadOnly != readOnly block executed, if applicable.
• [Architecture] Do sync framework integration in a separate class #1115
• TODO: add more specific steps, if necessary