Update ToolbarManager to use incognito profile when called incognito.

This CL update the profile passing to TrackerFactory from
ToolbarManager. Currently, it is always regular profile, however it is
not safe because of possible data leakage from incognito to regular
profile.

Therefore, this CL uses |isIncognito| boolean created from current
tab model to get correct profile object(i.e., regular or
off-the-record profile).

Bug: 1041781
Change-Id: Ie9942d02a345cd6a755da1d8fd0b34089b22425b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2075417Reviewed-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@{#745982}

chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarManager.java

@@ -1356,7 +1356,11 @@ public class ToolbarManager implements ScrimObserver, ToolbarTabController, UrlF
             // Assume data saver footer is shown only if data reduction proxy is enabled and
             // Chrome home is not
             if (DataReductionProxySettings.getInstance().isDataReductionProxyEnabled()) {
-                Tracker tracker = TrackerFactory.getTrackerForProfile(Profile.getLastUsedProfile());
+                boolean isIncognito = mTabModelSelector.getCurrentModel().isIncognito();
+                Profile profile = isIncognito
+                        ? Profile.getLastUsedRegularProfile().getOffTheRecordProfile()
+                        : Profile.getLastUsedRegularProfile();
+                Tracker tracker = TrackerFactory.getTrackerForProfile(profile);
                 tracker.notifyEvent(EventConstants.OVERFLOW_OPENED_WITH_DATA_SAVER_SHOWN);
             }
         });