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

Unified Diff: public/web/WebGeolocationPermissionRequest.h

Issue 319863003: Enable Oilpan for modules/geolocation by default. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 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 side-by-side diff with in-line comments
Download patch
Index: public/web/WebGeolocationPermissionRequest.h
diff --git a/public/web/WebGeolocationPermissionRequest.h b/public/web/WebGeolocationPermissionRequest.h
index 5236f6d2587e9108d4d4ab68eeb5e76249793fd8..38ed97a41e4707b2ef55ab026c7710bff342c21d 100644
--- a/public/web/WebGeolocationPermissionRequest.h
+++ b/public/web/WebGeolocationPermissionRequest.h
@@ -43,20 +43,19 @@ class WebSecurityOrigin;
// WebGeolocationClient::cancelPermissionRequest (request cancelled).
class WebGeolocationPermissionRequest {
public:
+ ~WebGeolocationPermissionRequest() { reset(); }
BLINK_EXPORT WebSecurityOrigin securityOrigin() const;
BLINK_EXPORT void setIsAllowed(bool);
#if BLINK_IMPLEMENTATION
- WebGeolocationPermissionRequest(WebCore::Geolocation* geolocation)
- : m_private(geolocation)
- {
- }
-
- WebCore::Geolocation* geolocation() const { return m_private; }
+ WebGeolocationPermissionRequest(WebCore::Geolocation*);
+ WebCore::Geolocation* geolocation() const { return m_private.get(); }
#endif
private:
- WebCore::Geolocation* m_private;
+ BLINK_EXPORT void reset();
+
+ WebPrivatePtr<WebCore::Geolocation> m_private;
};
}

Powered by Google App Engine
This is Rietveld 408576698