Update controllers to conform to controller-messenger pattern

[MajorLift]

Motivation

A number of controllers currently undergoing active development are extending from the deprecated BaseController class, and not conforming to the controller-messenger pattern. As part of Shared Libraries team OKRs, these controllers should be updated so that they adhere to current practices.

• Shared Libraries - Reduce maintenance burden for core wallet functionality - Q423 #1780
• Shared Libraries - 2023/Q4/O1/KR3 - Create and apply consistent patterns across controllers for multichain support #1782

Tasks

• Controllers that currently extend from BaseController should be updated to extend from BaseControllerV2.

  • core

    • AbstractMessageManager
    • AccountTrackerController
    • AddressBookController
    • AssetsContractController
    • ComposableController
    • NftController
    • PhishingController
    • PreferencesController
    • RateLimitController
    • TokenBalancesController
    • TokensController
    • TransactionController

  • external

    • SwapsController

• Controllers that currently extend from PollingControllerOnly or PollingControllerV1 should be updated to extend from PollingController.

  • core

    • NftDetectionController
    • TokenDetectionController
    • TokenRatesController

  • external

    • SmartTransactionsController

[MajorLift]

Closing as duplicate of #1509