Update TODO comment around regular vs incognito profiles.

No functional changes.

Added some if-change-then-change in another location.

Change-Id: I4a116bb506f5f8830905381e3b687b9d6ed13456
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2108637Reviewed-by: Jeffrey Cohen <jeffreycohen@chromium.org>
Reviewed-by: Tanya Gupta <tgupta@chromium.org>
Commit-Queue: Jeffrey Cohen <jeffreycohen@chromium.org>
Commit-Queue: Travis Skare <skare@chromium.org>
Cr-Commit-Position: refs/heads/master@{#751457}

chrome/android/java/src/org/chromium/chrome/browser/send_tab_to_self/NotificationManager.java

@@ -63,9 +63,9 @@ public class NotificationManager {
                     final String action = intent.getAction();
                     final String guid =
                             IntentUtils.safeGetStringExtra(intent, NOTIFICATION_GUID_EXTRA);
-                    // TODO(https://crbug.com/1041781): Use the current profile (i.e.,
-                    //  regular profile or incognito profile) instead of always using
-                    //  regular profile. It is wrong and need to be fixed.
+                    // If this feature ever supports incognito mode, we need to modify
+                    // this method to obtain the current profile, rather than the last-used
+                    // regular profile.
                     final Profile profile = Profile.getLastUsedRegularProfile();
                     switch (action) {
                         case NOTIFICATION_ACTION_TAP:

chrome/browser/send_tab_to_self/send_tab_to_self_util_unittest.cc

@@ -97,6 +97,9 @@ TEST_F(SendTabToSelfUtilTest, NativePage) {
 }
 
 TEST_F(SendTabToSelfUtilTest, IncognitoMode) {
+  // Note: if changing this, audit profile-finding logic in the feature.
+  // For example, NotificationManager.java in the Android code assumes
+  // incognito is not supported.
   EXPECT_FALSE(AreContentRequirementsMet(url_, incognito_profile_));
 }