Log User Actions for Most Visited taps earlier.

User Actions are added to Breadscrumbs (log attached to steps to
reproduce). Logging the actions earler will include them into
Breadcrumbs if presentation code crashes (like what happens in
crbug.com/1050786).

Bug: 1046231
Change-Id: I6cde3c11320d41135bbeffc9d4943d5b9645a293
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2087874
Auto-Submit: Eugene But <eugenebut@chromium.org>
Commit-Queue: Gauthier Ambard <gambard@chromium.org>
Reviewed-by: Gauthier Ambard <gambard@chromium.org>
Cr-Commit-Position: refs/heads/master@{#747148}

ios/chrome/browser/ui/content_suggestions/ntp_home_mediator.mm

@@ -251,20 +251,20 @@ const char kNTPHelpURL[] =
             item);
     switch (mostVisitedItem.collectionShortcutType) {
       case NTPCollectionShortcutTypeBookmark:
-        [self.dispatcher showBookmarksManager];
         base::RecordAction(base::UserMetricsAction("MobileNTPShowBookmarks"));
+        [self.dispatcher showBookmarksManager];
         break;
       case NTPCollectionShortcutTypeReadingList:
-        [self.dispatcher showReadingList];
         base::RecordAction(base::UserMetricsAction("MobileNTPShowReadingList"));
+        [self.dispatcher showReadingList];
         break;
       case NTPCollectionShortcutTypeRecentTabs:
-        [self.dispatcher showRecentTabs];
         base::RecordAction(base::UserMetricsAction("MobileNTPShowRecentTabs"));
+        [self.dispatcher showRecentTabs];
         break;
       case NTPCollectionShortcutTypeHistory:
-        [self.dispatcher showHistory];
         base::RecordAction(base::UserMetricsAction("MobileNTPShowHistory"));
+        [self.dispatcher showHistory];
         break;
       case NTPCollectionShortcutTypeCount:
         NOTREACHED();