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

Side by Side Diff: chrome/browser/safe_browsing/ui_manager.h

Issue 1726403006: Switch Safe Browsing's metadata from string to struct. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix ParseJson in test Created 4 years, 9 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 // 4 //
5 // The Safe Browsing service is responsible for downloading anti-phishing and 5 // The Safe Browsing service is responsible for downloading anti-phishing and
6 // anti-malware tables and checking urls against them. 6 // anti-malware tables and checking urls against them.
7 7
8 #ifndef CHROME_BROWSER_SAFE_BROWSING_UI_MANAGER_H_ 8 #ifndef CHROME_BROWSER_SAFE_BROWSING_UI_MANAGER_H_
9 #define CHROME_BROWSER_SAFE_BROWSING_UI_MANAGER_H_ 9 #define CHROME_BROWSER_SAFE_BROWSING_UI_MANAGER_H_
10 10
(...skipping 59 matching lines...) Expand 10 before | Expand all | Expand 10 after
70 // other navigation has committed (whether by going back or unrelated 70 // other navigation has committed (whether by going back or unrelated
71 // navigations), though a pending navigation is okay. 71 // navigations), though a pending navigation is okay.
72 content::NavigationEntry* GetNavigationEntryForResource() const; 72 content::NavigationEntry* GetNavigationEntryForResource() const;
73 73
74 GURL url; 74 GURL url;
75 GURL original_url; 75 GURL original_url;
76 std::vector<GURL> redirect_urls; 76 std::vector<GURL> redirect_urls;
77 bool is_subresource; 77 bool is_subresource;
78 bool is_subframe; 78 bool is_subframe;
79 SBThreatType threat_type; 79 SBThreatType threat_type;
80 std::string threat_metadata; 80 ThreatMetadata threat_metadata;
81 UrlCheckCallback callback; // This is called back on |callback_thread|. 81 UrlCheckCallback callback; // This is called back on |callback_thread|.
82 scoped_refptr<base::SingleThreadTaskRunner> callback_thread; 82 scoped_refptr<base::SingleThreadTaskRunner> callback_thread;
83 int render_process_host_id; 83 int render_process_host_id;
84 int render_frame_id; 84 int render_frame_id;
85 safe_browsing::ThreatSource threat_source; 85 safe_browsing::ThreatSource threat_source;
86 }; 86 };
87 87
88 // Observer class can be used to get notified when a SafeBrowsing hit 88 // Observer class can be used to get notified when a SafeBrowsing hit
89 // was found. 89 // was found.
90 class Observer { 90 class Observer {
(...skipping 83 matching lines...) Expand 10 before | Expand all | Expand 10 after
174 scoped_refptr<SafeBrowsingService> sb_service_; 174 scoped_refptr<SafeBrowsingService> sb_service_;
175 175
176 base::ObserverList<Observer> observer_list_; 176 base::ObserverList<Observer> observer_list_;
177 177
178 DISALLOW_COPY_AND_ASSIGN(SafeBrowsingUIManager); 178 DISALLOW_COPY_AND_ASSIGN(SafeBrowsingUIManager);
179 }; 179 };
180 180
181 } // namespace safe_browsing 181 } // namespace safe_browsing
182 182
183 #endif // CHROME_BROWSER_SAFE_BROWSING_UI_MANAGER_H_ 183 #endif // CHROME_BROWSER_SAFE_BROWSING_UI_MANAGER_H_
OLDNEW
« no previous file with comments | « chrome/browser/safe_browsing/safe_browsing_test.cc ('k') | chrome/browser/safe_browsing/ui_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698