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

Side by Side Diff: chrome/browser/geolocation/geolocation_permission_context_extensions.h

Issue 955383003: ContentBrowserClient::RequestPermission replies with PermissionStatus instead of bool. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix android geolocation breakage Created 5 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 #ifndef CHROME_BROWSER_GEOLOCATION_GEOLOCATION_PERMISSION_CONTEXT_EXTENSIONS_H_ 5 #ifndef CHROME_BROWSER_GEOLOCATION_GEOLOCATION_PERMISSION_CONTEXT_EXTENSIONS_H_
6 #define CHROME_BROWSER_GEOLOCATION_GEOLOCATION_PERMISSION_CONTEXT_EXTENSIONS_H_ 6 #define CHROME_BROWSER_GEOLOCATION_GEOLOCATION_PERMISSION_CONTEXT_EXTENSIONS_H_
7 7
8 #include "base/callback_forward.h" 8 #include "base/callback_forward.h"
9 #include "base/macros.h" 9 #include "base/macros.h"
10 #include "components/content_settings/core/common/content_settings.h"
10 11
11 namespace content { 12 namespace content {
12 class WebContents; 13 class WebContents;
13 } 14 }
14 15
15 class GURL; 16 class GURL;
16 class PermissionRequestID; 17 class PermissionRequestID;
17 class Profile; 18 class Profile;
18 19
19 // Chrome extensions specific portions of GeolocationPermissionContext. 20 // Chrome extensions specific portions of GeolocationPermissionContext.
20 class GeolocationPermissionContextExtensions { 21 class GeolocationPermissionContextExtensions {
21 public: 22 public:
22 explicit GeolocationPermissionContextExtensions(Profile* profile); 23 explicit GeolocationPermissionContextExtensions(Profile* profile);
23 ~GeolocationPermissionContextExtensions(); 24 ~GeolocationPermissionContextExtensions();
24 25
25 // Returns true if the permission request was handled. In which case, 26 // Returns true if the permission request was handled. In which case,
26 // |permission_set| will be set to true if the permission changed, and the 27 // |permission_set| will be set to true if the permission changed, and the
27 // permission has been set to |new_permission|. 28 // permission has been set to |new_permission|.
28 bool RequestPermission(content::WebContents* web_contents, 29 bool RequestPermission(content::WebContents* web_contents,
29 const PermissionRequestID& request_id, 30 const PermissionRequestID& request_id,
30 int bridge_id, 31 int bridge_id,
31 const GURL& requesting_frame, 32 const GURL& requesting_frame,
32 bool user_gesture, 33 bool user_gesture,
33 base::Callback<void(bool)> callback, 34 const base::Callback<void(ContentSetting)>& callback,
34 bool* permission_set, 35 bool* permission_set,
35 bool* new_permission); 36 bool* new_permission);
36 37
37 // Returns true if the cancellation request was handled. 38 // Returns true if the cancellation request was handled.
38 bool CancelPermissionRequest(content::WebContents* web_contents, 39 bool CancelPermissionRequest(content::WebContents* web_contents,
39 int bridge_id); 40 int bridge_id);
40 41
41 private: 42 private:
42 Profile* profile_; 43 Profile* profile_;
43 44
44 DISALLOW_COPY_AND_ASSIGN(GeolocationPermissionContextExtensions); 45 DISALLOW_COPY_AND_ASSIGN(GeolocationPermissionContextExtensions);
45 }; 46 };
46 47
47 #endif // CHROME_BROWSER_GEOLOCATION_GEOLOCATION_PERMISSION_CONTEXT_EXTENSIONS_ H_ 48 #endif // CHROME_BROWSER_GEOLOCATION_GEOLOCATION_PERMISSION_CONTEXT_EXTENSIONS_ H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698