Disable full URL lookups in OffTheRecord mode.

Bug: 963165
Change-Id: I13046027db2f8f8a8d9a4d2aaa0677401f67caf2
Fixed: 1028567
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1939958
Commit-Queue: Varun Khaneja <vakh@chromium.org>
Commit-Queue: Bettina Dea <bdea@chromium.org>
Auto-Submit: Varun Khaneja <vakh@chromium.org>
Reviewed-by: Bettina Dea <bdea@chromium.org>
Cr-Commit-Position: refs/heads/master@{#719683}

components/safe_browsing/realtime/policy_engine.cc

@@ -62,6 +62,9 @@ bool RealTimePolicyEngine::IsEnabledByPolicy(
 // static
 bool RealTimePolicyEngine::CanPerformFullURLLookup(
     content::BrowserContext* browser_context) {
+  if (browser_context->IsOffTheRecord())
+    return false;
+
   if (IsEnabledByPolicy(browser_context))
     return true;
 

components/safe_browsing/realtime/policy_engine_unittest.cc

@@ -104,6 +104,15 @@ TEST_F(RealTimePolicyEngineTest,
   EXPECT_FALSE(CanPerformFullURLLookup());
 }
 
+TEST_F(RealTimePolicyEngineTest,
+       TestCanPerformFullURLLookup_DisabledOffTheRecord) {
+  base::test::ScopedFeatureList feature_list;
+  pref_service_.SetManagedPref(prefs::kSafeBrowsingRealTimeLookupEnabled,
+                               std::make_unique<base::Value>(true));
+  test_context_.set_is_off_the_record(true);
+  EXPECT_FALSE(CanPerformFullURLLookup());
+}
+
 TEST_F(RealTimePolicyEngineTest,
        TestCanPerformFullURLLookup_DisabledUserOptin) {
   ASSERT_FALSE(IsUserOptedIn());