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

Side by Side Diff: components/safe_browsing/public/interfaces/safe_browsing.mojom

Issue 2936773002: Move safe_browsing.mojom into components/safe_browsing. (Closed)
Patch Set: . Created 3 years, 6 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
« no previous file with comments | « components/safe_browsing/public/interfaces/OWNERS ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2017 The Chromium Authors. All rights reserved. 1 // Copyright 2017 The Chromium Authors. All rights reserved.
jam 2017/06/13 17:57:42 nit: since components/safe_browsing is not a servi
yzshen1 2017/06/13 18:12:14 I notice that in components/ there are others that
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 module chrome.mojom; 5 module safe_browsing.mojom;
6 6
7 import "content/public/common/resource_type.mojom"; 7 import "content/public/common/resource_type.mojom";
8 import "url/mojo/url.mojom"; 8 import "url/mojo/url.mojom";
9 9
10 interface SafeBrowsing { 10 interface SafeBrowsing {
11 // Queries SafeBrowsing whether |url| is safe to load, and creates a 11 // Queries SafeBrowsing whether |url| is safe to load, and creates a
12 // SafeBrowsingUrlChecker interface. 12 // SafeBrowsingUrlChecker interface.
13 // This checker interface should be used (and only used) for subsequent checks 13 // This checker interface should be used (and only used) for subsequent checks
14 // of redirects, so that the server side could keep track of the redirect 14 // of redirects, so that the server side could keep track of the redirect
15 // chain. Disconnecting the checker interface cancels on-going URL checks. 15 // chain. Disconnecting the checker interface cancels on-going URL checks.
16 // Please note that in that case if the check started by this method hasn't 16 // Please note that in that case if the check started by this method hasn't
17 // completed yet, it will also be canceled and return true as if the URL is 17 // completed yet, it will also be canceled and return true as if the URL is
18 // safe. 18 // safe.
19 // The check and (subsequent checks performed using SafeBrowsingUrlChecker) 19 // The check and (subsequent checks performed using SafeBrowsingUrlChecker)
20 // checks against SafeBrowsing's Malware, Phishing, and UwS blacklists. 20 // checks against SafeBrowsing's Malware, Phishing, and UwS blacklists.
21 CreateCheckerAndCheck( 21 CreateCheckerAndCheck(
22 int32 render_frame_id, 22 int32 render_frame_id,
23 SafeBrowsingUrlChecker& request, 23 SafeBrowsingUrlChecker& request,
24 url.mojom.Url url, 24 url.mojom.Url url,
25 int32 load_flags, 25 int32 load_flags,
26 content.mojom.ResourceType resource_type) => (bool safe); 26 content.mojom.ResourceType resource_type) => (bool safe);
27 }; 27 };
28 28
29 interface SafeBrowsingUrlChecker { 29 interface SafeBrowsingUrlChecker {
30 CheckUrl(url.mojom.Url url) => (bool safe); 30 CheckUrl(url.mojom.Url url) => (bool safe);
31 }; 31 };
OLDNEW
« no previous file with comments | « components/safe_browsing/public/interfaces/OWNERS ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698