Replace deprecated getLastUsedProfile function in TabListCoordinator.

|Profile#getLastUsedProfile| is depreciated and replaced with
|Profile#getLastUsedRegularProfile|. This CL updates TabListCoordinator
class.

Note: This change is only code clean-up, does not change any behavior.

Bug: 1041781
Change-Id: I387f85aede020ca88fc626c210fd6a4c9748ef62
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2049967Reviewed-by: Ramin Halavati <rhalavati@chromium.org>
Reviewed-by: Yusuf Ozuysal <yusufo@chromium.org>
Commit-Queue: Side YILMAZ <sideyilmaz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#740585}

chrome/android/features/tab_ui/java/src/org/chromium/chrome/browser/tasks/tab_management/TabListCoordinator.java

@@ -215,8 +215,11 @@ public class TabListCoordinator implements Destroyable {
             mRecyclerView.createDynamicView(dynamicResourceLoader);
         }
 
+        // TODO (https://crbug.com/1048632): Use the current profile (i.e., regular profile or
+        // incognito profile) instead of always using regular profile. It works correctly now, but
+        // it is not safe.
         TabListFaviconProvider tabListFaviconProvider =
-                new TabListFaviconProvider(context, Profile.getLastUsedProfile());
+                new TabListFaviconProvider(context, Profile.getLastUsedRegularProfile());
 
         mMediator = new TabListMediator(context, modelList, tabModelSelector, thumbnailProvider,
                 titleProvider, tabListFaviconProvider, actionOnRelatedTabs,