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

Unified Diff: public/platform/Platform.h

Issue 283423005: Use Promises for screen.lockOrientation(). (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: review commenst Created 6 years, 7 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/platform/Platform.h
diff --git a/public/platform/Platform.h b/public/platform/Platform.h
index f2e79d66c03d00877a3b1b66682f1ed50092274a..8719bbd054a1fe29065cd74af81621532a6a02a1 100644
--- a/public/platform/Platform.h
+++ b/public/platform/Platform.h
@@ -42,7 +42,9 @@
#include "WebGamepads.h"
#include "WebGraphicsContext3D.h"
#include "WebLocalizedString.h"
+#include "WebLockOrientationCallback.h"
#include "WebScreenOrientationLockType.h"
+#include "WebScreenOrientationType.h"
#include "WebSpeechSynthesizer.h"
#include "WebStorageQuotaCallbacks.h"
#include "WebStorageQuotaType.h"
@@ -634,6 +636,13 @@ public:
virtual void setScreenOrientationListener(blink::WebScreenOrientationListener*) { }
virtual void lockOrientation(WebScreenOrientationLockType) { }
+ // Request a screen orientation lock and pass a |callback| object to be used
+ // to notify of success/failure. The |callback| parameter is expected to be
+ // owned by the implementation.
+ virtual void lockOrientation(WebScreenOrientationLockType, WebLockOrientationCallback* callback)
+ {
+ delete callback; // prevents memory leak if there is no implementation.
+ }
virtual void unlockOrientation() { }
« no previous file with comments | « Source/modules/screen_orientation/ScreenOrientation.idl ('k') | public/platform/WebLockOrientationCallback.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698