Skip to content

Commit c3d24de

Browse files
committed
[M74]: Check enabled_ before getting a SafeBrowsingApiHandler instance
R=drubery [email protected] (cherry picked from commit b11df6ef5e57d35cdf1707a30809e2d75613974a) Bug: 940345,907280 Change-Id: I56bcae95a16e9b4235ac3b25986965ac764af009 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1516042 Auto-Submit: Varun Khaneja <[email protected]> Reviewed-by: Daniel Rubery <[email protected]> Commit-Queue: Daniel Rubery <[email protected]> Commit-Queue: Varun Khaneja <[email protected]> Cr-Original-Commit-Position: refs/heads/master@{#639613} Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1518448 Reviewed-by: Varun Khaneja <[email protected]> Cr-Commit-Position: refs/branch-heads/3729@{#50} Cr-Branched-From: d4a8972-refs/heads/master@{#638880}
1 parent 842fb88 commit c3d24de

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

components/safe_browsing/android/remote_database_manager.cc

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -295,6 +295,10 @@ safe_browsing::ThreatSource RemoteSafeBrowsingDatabaseManager::GetThreatSource()
295295
}
296296

297297
std::string RemoteSafeBrowsingDatabaseManager::GetSafetyNetId() const {
298+
DCHECK_CURRENTLY_ON(BrowserThread::IO);
299+
if (!enabled_)
300+
return std::string();
301+
298302
SafeBrowsingApiHandler* api_handler = SafeBrowsingApiHandler::GetInstance();
299303
DCHECK(api_handler) << "SafeBrowsingApiHandler was never constructed";
300304
return api_handler->GetSafetyNetId();

0 commit comments

Comments
 (0)