emoji [nfc]: Move popularEmojiCandidates implementation down out of mixin

Author: chrisbobbe

lib/model/emoji.dart

@@ -115,9 +115,7 @@ mixin EmojiStore {
   ///
   /// See description in the web code:
   ///   https://github.com/zulip/zulip/blob/83a121c7e/web/shared/src/typeahead.ts#L3-L21
-  Iterable<EmojiCandidate> popularEmojiCandidates() {
-    return EmojiStoreImpl._popularCandidates;
-  }
+  Iterable<EmojiCandidate> popularEmojiCandidates();
 
   Iterable<EmojiCandidate> allEmojiCandidates();
 
@@ -216,6 +214,11 @@ class EmojiStoreImpl extends PerAccountStoreBase with EmojiStore {
 
   static final _popularCandidates = _generatePopularCandidates();
 
+  @override
+  Iterable<EmojiCandidate> popularEmojiCandidates() {
+    return _popularCandidates;
+  }
+
   static List<EmojiCandidate> _generatePopularCandidates() {
     EmojiCandidate candidate(String emojiCode, List<String> names) {
       final emojiName = names.removeAt(0);

lib/model/store.dart

@@ -611,6 +611,9 @@ class PerAccountStore extends PerAccountStoreBase with ChangeNotifier, EmojiStor
     notifyListeners();
   }
 
+  @override
+  Iterable<EmojiCandidate> popularEmojiCandidates() => _emoji.popularEmojiCandidates();
+
   @override
   Iterable<EmojiCandidate> allEmojiCandidates() => _emoji.allEmojiCandidates();