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

Unified Diff: chrome/browser/geolocation/geolocation_infobar_delegate.h

Issue 459953002: Migrate geolocation permissions to the new common permission class. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 3 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/geolocation/geolocation_infobar_delegate.h
diff --git a/chrome/browser/geolocation/geolocation_infobar_delegate.h b/chrome/browser/geolocation/geolocation_infobar_delegate.h
index 06b3801d6fb6825185630b1d00c78f8342d4d361..3515f063259d12521d2f132d17ef0fb0b94c3a86 100644
--- a/chrome/browser/geolocation/geolocation_infobar_delegate.h
+++ b/chrome/browser/geolocation/geolocation_infobar_delegate.h
@@ -7,17 +7,13 @@
#include <string>
-#include "components/content_settings/core/common/permission_request_id.h"
-#include "components/infobars/core/confirm_infobar_delegate.h"
-#include "url/gurl.h"
+#include "chrome/browser/content_settings/permission_infobar_delegate.h"
-class PermissionQueueController;
-class InfoBarService;
// GeolocationInfoBarDelegates are created by the
// GeolocationInfoBarQueueController to control the display
// and handling of geolocation permission infobars to the user.
-class GeolocationInfoBarDelegate : public ConfirmInfoBarDelegate {
+class GeolocationInfoBarDelegate : public PermissionInfobarDelegate {
public:
// Creates a geolocation infobar and delegate and adds the infobar to
// |infobar_service|. Returns the infobar if it was successfully added.
@@ -35,37 +31,13 @@ class GeolocationInfoBarDelegate : public ConfirmInfoBarDelegate {
const std::string& display_languages);
virtual ~GeolocationInfoBarDelegate();
- // ConfirmInfoBarDelegate:
- virtual bool Accept() OVERRIDE;
- virtual void InfoBarDismissed() OVERRIDE;
- virtual int GetIconID() const OVERRIDE;
- virtual Type GetInfoBarType() const OVERRIDE;
- virtual bool ShouldExpireInternal(
- const NavigationDetails& details) const OVERRIDE;
+ // PermissionInfoBarDelegate:
virtual base::string16 GetMessageText() const OVERRIDE;
- virtual base::string16 GetButtonLabel(InfoBarButton button) const OVERRIDE;
- virtual bool Cancel() OVERRIDE;
-
-
- // Call back to the controller, to inform of the user's decision.
- void SetPermission(bool update_content_setting, bool allowed);
-
- // Marks a flag internally to indicate that the user has interacted with the
- // bar. This makes it possible to log from the destructor when the bar has not
- // been used, i.e. it has been ignored by the user.
- void set_user_has_interacted() {
- user_has_interacted_ = true;
- }
+ virtual int GetIconID() const OVERRIDE;
- PermissionQueueController* controller_;
- const PermissionRequestID id_;
GURL requesting_frame_;
- int contents_unique_id_;
std::string display_languages_;
- // Whether the user has interacted with the geolocation infobar.
- bool user_has_interacted_;
-
DISALLOW_COPY_AND_ASSIGN(GeolocationInfoBarDelegate);
};

Powered by Google App Engine
This is Rietveld 408576698