Chrome Empties: Fix bookmarks title bar bug

This addresses a bug where the title and search bar overlapped in the
NavigationItem in rare cases. Making the call to readd the search bar
before the call to update the title style fixes the behavior, though the
title style now won't change until the user interacts with the screen.

It's hard to say exactly *why* this fixes the issue without some more
knowledge of how UINavigationItem works internally, which we obviously
don't have, but my hunch (and the reason I tried this approach) is that
changing the title style triggers a re-layout of some kind.

Bug: 1116407
Change-Id: I774a1fc9a77a71c01626ddef4dc2586575318a5b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2367974Reviewed-by: Sergio Collazos <sczs@chromium.org>
Commit-Queue: Tommy Martino <tmartino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#800365}

ios/chrome/browser/ui/bookmarks/bookmark_home_view_controller.mm

@@ -1508,11 +1508,11 @@ std::vector<GURL> GetUrlsToOpen(const std::vector<const BookmarkNode*>& nodes) {
     if (self.sharedState.tableView.backgroundView == self.emptyViewBackground) {
       self.sharedState.tableView.backgroundView = nil;
     }
+    self.navigationItem.searchController = self.searchController;
     if ([self isDisplayingBookmarkRoot]) {
       self.navigationItem.largeTitleDisplayMode =
           UINavigationItemLargeTitleDisplayModeAutomatic;
     }
-    self.navigationItem.searchController = self.searchController;
   } else {
     self.sharedState.tableView.backgroundView = nil;
   }