Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1939)

Unified Diff: chrome/browser/safe_browsing/safe_browsing_blocking_page.cc

Issue 11275088: Remove implicit scoped_refptr operator T* Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 8 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/safe_browsing/safe_browsing_blocking_page.cc
diff --git a/chrome/browser/safe_browsing/safe_browsing_blocking_page.cc b/chrome/browser/safe_browsing/safe_browsing_blocking_page.cc
index a6c3ca8fe391d1ab529ff8954624a2a95105b3db..1da978b75ce61ffdbe735e7e5655d0d88bf23f8e 100644
--- a/chrome/browser/safe_browsing/safe_browsing_blocking_page.cc
+++ b/chrome/browser/safe_browsing/safe_browsing_blocking_page.cc
@@ -209,7 +209,7 @@ SafeBrowsingBlockingPage::SafeBrowsingBlockingPage(
// reports.
if (unsafe_resources.size() == 1 &&
unsafe_resources[0].threat_type == SafeBrowsingService::URL_MALWARE &&
- malware_details_ == NULL &&
+ malware_details_.get() == NULL &&
CanShowMalwareDetailsOption()) {
malware_details_ = MalwareDetails::NewMalwareDetails(
sb_service_, web_contents, unsafe_resources[0]);
@@ -691,7 +691,7 @@ void SafeBrowsingBlockingPage::RecordUserReactionTime(
}
void SafeBrowsingBlockingPage::FinishMalwareDetails(int64 delay_ms) {
- if (malware_details_ == NULL)
+ if (malware_details_.get() == NULL)
return; // Not all interstitials have malware details (eg phishing).
if (IsPrefEnabled(prefs::kSafeBrowsingReportingEnabled)) {

Powered by Google App Engine
This is Rietveld 408576698