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

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

Issue 4822002: Send malware reports when a user opts-in from the safe browsing interstitial ... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 10 years, 1 month 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.h
===================================================================
--- chrome/browser/safe_browsing/safe_browsing_blocking_page.h (revision 65873)
+++ chrome/browser/safe_browsing/safe_browsing_blocking_page.h (working copy)
@@ -39,6 +39,7 @@
class DictionaryValue;
class MessageLoop;
class SafeBrowsingBlockingPageFactory;
+class SafeBrowsingMalwareReport;
class TabContents;
class SafeBrowsingBlockingPage : public InterstitialPage {
@@ -102,6 +103,11 @@
// SBInterstitial[Phishing|Malware|Multiple][Show|Proceed|DontProceed].
void RecordUserAction(BlockingPageEvent event);
+ // Called when the insterstitial is going away. If there is a pending malware
+ // report, we look at the user's preferences, and if enabled, the
+ // report is scheduled to be sent on the |sb_service_|.
+ void FinishMalwareReport();
+
// A list of SafeBrowsingService::UnsafeResource for a tab that the user
// should be warned about. They are queued when displaying more than one
// interstitial at a time.
@@ -134,6 +140,11 @@
// The list of unsafe resources this page is warning about.
UnsafeResourceList unsafe_resources_;
lzheng 2010/11/16 00:18:12 Instead of the warning, do you mean the blocking p
panayiotis 2010/11/18 22:04:37 Done.
+ // A malware report that we start generating when the warning is
+ // shown. The report will be sent when the warning is gone
+ // (if the user enables the feature).
+ scoped_refptr<SafeBrowsingMalwareReport> malware_report_;
+
// The factory used to instanciate SafeBrowsingBlockingPage objects.
// Usefull for tests, so they can provide their own implementation of
// SafeBrowsingBlockingPage.

Powered by Google App Engine
This is Rietveld 408576698