Replace IsIncognitoProfile with IsOffTheRecord in enterprise reporting.

ReportScheduler and ExtensionRequestObserverFactory use
IsIncognitoProfile() to prevent OffTheRecord profiles in two contexts.

IsIncognitoProfile() currently covers all non-guest off-the-record
profiles, but this will be soon changed to only incognito profiles.

So to keep the functionality as it is, IsIncognitoProfile() is replaced
with IsOffTheRecord().

This CL does not create any behavioral change now, and is to prevent
change when IsIncognitoProfile() is updated.

Bug: 1074201
Change-Id: I71dba6f52195df7b12a2ae7df984ce5dc374ed3c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2211343
Commit-Queue: Owen Min <zmin@chromium.org>
Auto-Submit: Ramin Halavati <rhalavati@chromium.org>
Reviewed-by: Owen Min <zmin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#771023}

chrome/browser/enterprise/reporting/notification/extension_request_observer_factory.cc

@@ -31,7 +31,7 @@ ExtensionRequestObserverFactory::~ExtensionRequestObserverFactory() {
 
 void ExtensionRequestObserverFactory::OnProfileAdded(Profile* profile) {
   if (profile->IsSystemProfile() || profile->IsGuestSession() ||
-      profile->IsIncognitoProfile()) {
+      profile->IsOffTheRecord()) {
     return;
   }
 

chrome/browser/enterprise/reporting/report_scheduler.cc

@@ -334,7 +334,7 @@ void ReportScheduler::RecordUploadTrigger(ReportTrigger trigger) {
 
 void ReportScheduler::OnProfileAdded(Profile* profile) {
   if (profile->IsSystemProfile() || profile->IsGuestSession() ||
-      profile->IsIncognitoProfile()) {
+      profile->IsOffTheRecord()) {
     return;
   }
   DCHECK(stale_profiles_);